Merge pull request #131 from wadmiraal/master

Fix folding mentioned in #95
jeroendesloovere · Jul 4, 2018 · 6f4627d · 6f4627d
2 parents 89f203e + f189848
commit 6f4627d
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/src/VCard.php b/src/VCard.php
@@ -637,7 +637,7 @@ protected function fold($text)
         }
 
         // split, wrap and trim trailing separator
-        return substr($this->chunk_split_unicode($text, 73, "\r\n "), 0, -3);
+        return substr($this->chunk_split_unicode($text, 75, "\r\n "), 0, -3);
     }
 
     /**

diff --git a/tests/VCardTest.php b/tests/VCardTest.php
@@ -77,7 +77,18 @@ public function tearDown()
 
     public function testAddAddress()
     {
-        $this->assertEquals($this->vcard, $this->vcard->addAddress());
+        $this->assertEquals($this->vcard, $this->vcard->addAddress(
+          '',
+          '88th Floor',
+          '555 East Flours Street',
+          'Los Angeles',
+          'CA',
+          '55555',
+          'USA'
+        ));
+      $this->assertContains('ADR;WORK;POSTAL;CHARSET=utf-8:;88th Floor;555 East Flours Street;Los Angele', $this->vcard->getOutput());
+      // Should fold on row 75, so we should not see the full address.
+      $this->assertNotContains('ADR;WORK;POSTAL;CHARSET=utf-8:;88th Floor;555 East Flours Street;Los Angeles;CA;55555;', $this->vcard->getOutput());
     }
 
     public function testAddBirthday()