merge pr to be uptodate

src/Formatter/Formatter.php

@@ -36,8 +36,8 @@ public function addVCard(VCard $vCard): self
 
     public function download(): void
     {
-        foreach ($this->getHeaders() as $header) {
-            header($header);
+        foreach ($this->getHeaders() as $key => $value) {
+            header(sprintf("%s: %s", $key, $value));
         }
 
         echo $this->getContent();

src/Formatter/Property/RoleFormatter.php

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+namespace JeroenDesloovere\VCard\Formatter\Property;
+
+final class RoleFormatter extends SimpleNodeFormatter
+{
+
+}

src/Parser/Property/RoleParser.php

@@ -0,0 +1,16 @@
+<?php
+
+declare(strict_types=1);
+
+namespace JeroenDesloovere\VCard\Parser\Property;
+
+use JeroenDesloovere\VCard\Property\Role;
+use JeroenDesloovere\VCard\Property\NodeInterface;
+
+final class RoleParser implements NodeParserInterface
+{
+    public function parseLine(string $value, array $parameters = []): NodeInterface
+    {
+        return new Role($value);
+    }
+}

src/Property/Role.php

@@ -0,0 +1,34 @@
+<?php
+
+declare(strict_types=1);
+
+namespace JeroenDesloovere\VCard\Property;
+
+use JeroenDesloovere\VCard\Formatter\Property\RoleFormatter;
+use JeroenDesloovere\VCard\Formatter\Property\NodeFormatterInterface;
+use JeroenDesloovere\VCard\Parser\Property\RoleParser;
+use JeroenDesloovere\VCard\Parser\Property\NodeParserInterface;
+use JeroenDesloovere\VCard\Property\Value\StringValue;
+
+final class Role extends StringValue implements PropertyInterface, SimpleNodeInterface
+{
+    public function getFormatter(): NodeFormatterInterface
+    {
+        return new RoleFormatter($this);
+    }
+
+    public static function getNode(): string
+    {
+        return 'ROLE';
+    }
+
+    public static function getParser(): NodeParserInterface
+    {
+        return new RoleParser();
+    }
+
+    public function isAllowedMultipleTimes(): bool
+    {
+        return false;
+    }
+}

src/VCard.php

@@ -25,6 +25,7 @@
 use JeroenDesloovere\VCard\Property\PropertyInterface;
 use JeroenDesloovere\VCard\Property\Telephone;
 use JeroenDesloovere\VCard\Property\Title;
+use JeroenDesloovere\VCard\Property\Role;
 
 final class VCard
 {
@@ -42,6 +43,7 @@ final class VCard
         Gender::class,
         Nickname::class,
         Title::class,
+        Role::class,
         Birthdate::class,
         Anniversary::class,
         Email::class,

tests/VCardTest.php

@@ -23,6 +23,7 @@
 use JeroenDesloovere\VCard\Property\Photo;
 use JeroenDesloovere\VCard\Property\Telephone;
 use JeroenDesloovere\VCard\Property\Title;
+use JeroenDesloovere\VCard\Property\Role;
 use JeroenDesloovere\VCard\VCard;
 use org\bovigo\vfs\vfsStream;
 use org\bovigo\vfs\vfsStreamDirectory;
@@ -80,6 +81,7 @@ private function setUpThirdVCard(): void
     {
         $this->thirdVCard = (new VCard(Kind::organization()))
             ->add(new Title('Apple'))
+            ->add(new Role('Fruit'))
             ->add(new Photo(__DIR__ . '/assets/landscape.jpeg'))
             ->add(new Logo(__DIR__ . '/assets/landscape.jpeg'))
             ->add(new Telephone('+32 486 00 00 00'));
@@ -186,8 +188,9 @@ public function testVCardGetProperties(): void
         $this->assertCount(1, $this->secondVCard->getProperties(Name::class));
         $this->assertCount(1, $this->secondVCard->getProperties(Address::class));
 
-        $this->assertCount(4, $this->thirdVCard->getProperties());
+        $this->assertCount(5, $this->thirdVCard->getProperties());
         $this->assertCount(1, $this->thirdVCard->getProperties(Title::class));
+        $this->assertCount(1, $this->thirdVCard->getProperties(Role::class));
         $this->assertCount(1, $this->thirdVCard->getProperties(Photo::class));
         $this->assertCount(1, $this->thirdVCard->getProperties(Logo::class));
         $this->assertCount(1, $this->thirdVCard->getProperties(Telephone::class));