Merge pull request zendframework/zendframework#5757 from ralphschindl…

…er/hotfix/config-writer-phparray-needs-to-respect-the-double-quote

Zend\Config\Writer\PhpArray needs to use var_export for strings, not addslahes()

src/Writer/PhpArray.php

@@ -76,12 +76,10 @@ protected function processIndented(array $config, array $arraySyntax, &$indentLe
                                   . $this->processIndented($value, $arraySyntax, $indentLevel)
                                   . str_repeat(self::INDENT_STRING, --$indentLevel) . $arraySyntax['close'] . ",\n";
                 }
-            } elseif (is_object($value)) {
+            } elseif (is_object($value) || is_string($value)) {
                 $arrayString .= var_export($value, true) . ",\n";
             } elseif (is_bool($value)) {
                 $arrayString .= ($value ? 'true' : 'false') . ",\n";
-            } elseif (is_string($value)) {
-                $arrayString .= "'" . addslashes($value) . "',\n";
             } elseif ($value === null) {
                 $arrayString .= "null,\n";
             } else {

test/Writer/PhpArrayTest.php

@@ -84,6 +84,21 @@ public function testRenderWithBracketArraySyntax()
         $this->assertEquals($expected, $configString);
     }
 
+    public function testRenderWithQuotesInString()
+    {
+        $config = new Config(array('one' => 'Test with "double" quotes', 'two' => 'Test with \'single\' quotes'));
+
+        $configString = $this->writer->toString($config);
+
+        $expected = "<?php\n";
+        $expected .= "return array(\n";
+        $expected .= "    'one' => 'Test with \"double\" quotes',\n";
+        $expected .= "    'two' => 'Test with \\'single\\' quotes',\n";
+        $expected .= ");\n";
+
+        $this->assertEquals($expected, $configString);
+    }
+
     public function testSetUseBracketArraySyntaxReturnsFluentInterface()
     {
         $this->assertSame($this->writer, $this->writer->setUseBracketArraySyntax(true));