minor #22105 [Yaml] Fix error handling on ini file parsing (chalasr)

This PR was merged into the 3.3-dev branch. Discussion ---------- [Yaml] Fix error handling on ini file parsing | Q | A | ------------- | --- | Branch? | master | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | n/a | License | MIT | Doc PR | n/a Should make travis green again on master. /cc @xabbuh Commits ------- 71cec82 [Yaml] Fix error handling in parser
symfony · Mar 22, 2017 · daac6c2 · daac6c2
2 parents 3495b35 + 71cec82
commit daac6c2
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/src/Symfony/Component/Yaml/Parser.php b/src/Symfony/Component/Yaml/Parser.php
@@ -341,7 +341,7 @@ public function parse($value, $flags = 0)
                         try {
                             $parsedLine = Inline::parse($line, $flags, $this->refs);
 
-                            if (!is_string($value)) {
+                            if (!is_string($parsedLine)) {
                                 $parseError = true;
                                 break;
                             }

diff --git a/src/Symfony/Component/Yaml/Tests/ParserTest.php b/src/Symfony/Component/Yaml/Tests/ParserTest.php
@@ -1687,6 +1687,21 @@ public function testComplexMappingNestedInSequenceThrowsParseException()
         $this->parser->parse($yaml);
     }
 
+    /**
+     * @expectedException        \Symfony\Component\Yaml\Exception\ParseException
+     * @expectedExceptionMessage Unable to parse at line 1 (near "[parameters]").
+     */
+    public function testParsingIniThrowsException()
+    {
+        $ini = <<<INI
+[parameters]
+  foo = bar
+  bar = %foo%
+INI;
+
+        $this->parser->parse($ini);
+    }
+
     private function loadTestsFromFixtureFiles($testsFile)
     {
         $parser = new Parser();