[Yaml] Removed the ability to parse a file in Yaml::parse() #13129

saro0h · 2014-12-26T18:17:32Z

Q	A
Bug fix?	no
New feature?	no
BC breaks?	yes
Deprecations?	no
Tests pass?	yes
Fixed tickets	~
License	MIT
Doc PR	~

timglabisch · 2014-12-26T21:49:35Z

src/Symfony/Component/Yaml/Yaml.php

-            $file = $input;
-            $input = file_get_contents($file);
-        }
-
        $yaml = new Parser();

        try {


return (new Parser())->parse ... should work?
no need for the try / catch block.

fabpot · 2014-12-29T11:31:23Z

Thank you @saro0h.

…rse() (saro0h) This PR was merged into the 3.0-dev branch. Discussion ---------- [Yaml] Removed the ability to parse a file in Yaml::parse() | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | BC breaks? | yes | Deprecations? | no | Tests pass? | yes | Fixed tickets | ~ | License | MIT | Doc PR | ~ Commits ------- e1c19f8 [Yaml] Removed the ability to parse a file in Yaml::parse()

…rse() (xabbuh) This PR was merged into the master branch. Discussion ---------- [Components][Yaml] don't describe removed usage of Yaml::parse() | Q | A | ------------- | --- | Doc fix? | yes | New docs? | no (but related to symfony/symfony#13129) | Applies to | 3.0+ | Fixed tickets | Commits ------- d33b78b don't describe removed usage of Yaml::parse()

timglabisch reviewed Dec 26, 2014
View reviewed changes

[Yaml] Removed the ability to parse a file in Yaml::parse()

e1c19f8

saro0h force-pushed the remove-yaml-parse branch from f7e80ce to e1c19f8 Compare December 28, 2014 17:25

fabpot merged commit e1c19f8 into symfony:master Dec 29, 2014

xabbuh mentioned this pull request Jan 4, 2015

[Components][Yaml] don't describe removed usage of Yaml::parse() symfony/symfony-docs#4758

Merged

fabpot mentioned this pull request Nov 16, 2015

Release v3.0.0-BETA1 #16560

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Yaml] Removed the ability to parse a file in Yaml::parse() #13129

[Yaml] Removed the ability to parse a file in Yaml::parse() #13129

[Yaml] Removed the ability to parse a file in Yaml::parse() #13129

[Yaml] Removed the ability to parse a file in Yaml::parse() #13129

Conversation

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment