Merge pull request #64 from DominikTo/master

fixed binary converter issue

Author: evert

ChangeLog

@@ -19,6 +19,8 @@
 	* Added: --forgiving option to vobject utility.
 	* Fixed: Compound properties such as ADR were not correctly split up in
 	  vCard 2.1 quoted printable-encoded properties.
+	* Fixed: Issue 64: Encoding of binary properties of converted vCards.
+	  Thanks @DominikTo for the patch.
 
 3.1.2-stable (2013-08-13)
 	* Fixed: Setting correct property group on VCard conversion

lib/Sabre/VObject/Property/Binary.php

@@ -30,6 +30,32 @@ class Binary extends Property {
      */
     public $delimiter = null;
 
+    /**
+     * Updates the current value.
+     *
+     * This may be either a single, or multiple strings in an array.
+     *
+     * @param string|array $value
+     * @return void
+     */
+    public function setValue($value) {
+
+        if(is_array($value)) {
+
+            if(count($value) === 1) {
+                $this->value = $value[0];
+            } else {
+                throw new \InvalidArgumentException('The argument must either be a string or an array with only one child');
+            }
+
+        } else {
+
+            $this->value = $value;
+
+        }
+
+    }
+
     /**
      * Sets a raw value coming from a mimedir (iCalendar/vCard) file.
      *

tests/Sabre/VObject/Issue64Test.php

@@ -0,0 +1,19 @@
+<?php
+
+namespace Sabre\VObject;
+
+class Issue64Test extends \PHPUnit_Framework_TestCase {
+
+    function testRead() {
+
+        $vcard = Reader::read(file_get_contents(dirname(__FILE__) . '/issue64.vcf'));
+        $vcard = $vcard->convert(\Sabre\VObject\Document::VCARD30);
+        $vcard = $vcard->serialize();
+
+        $converted = Reader::read($vcard);
+
+        $this->assertInstanceOf('Sabre\\VObject\\Component\\VCard', $converted);
+
+    }
+
+}