Merge pull request #2 from clue-labs/legacy

Fix legacy PHP < 5.5.4 by omitting default escape character

Author: clue

src/Encoder.php

@@ -14,9 +14,9 @@ class Encoder extends EventEmitter implements WritableStreamInterface
     private $temp = false;
     private $closed = false;
 
-    private $delimiter = ',';
-    private $enclosure = '"';
-    private $escapeChar = '\\';
+    private $delimiter;
+    private $enclosure;
+    private $escapeChar;
 
     public function __construct(WritableStreamInterface $output, $delimiter = ',', $enclosure = '"', $escapeChar = '\\')
     {
@@ -48,7 +48,19 @@ public function write($data)
             return false;
         }
 
-        if (!is_array($data) || fputcsv($this->temp, $data, $this->delimiter, $this->enclosure, $this->escapeChar) === false) {
+        $written = false;
+        if (is_array($data)) {
+            // custom escape character requires PHP 5.5.4+ (see constructor check)
+            // @codeCoverageIgnoreStart
+            if ($this->escapeChar === '\\') {
+                $written = fputcsv($this->temp, $data, $this->delimiter, $this->enclosure);
+            } else {
+                $written = fputcsv($this->temp, $data, $this->delimiter, $this->enclosure, $this->escapeChar);
+            }
+            // @codeCoverageIgnoreEnd
+        }
+
+        if ($written === false) {
             $this->handleError(new \RuntimeException('Unable to encode CSV'));
             return false;
         }