Skip to content

Update yaml component docs to reflect Yaml::parse(filename) deprecation #2623

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 10, 2013

Conversation

jbafford
Copy link
Contributor

@jbafford jbafford commented May 8, 2013

Q A
Doc fix? yes
New docs? no
Applies to 2.2+
Fixed tickets #2570

Using a filename as argument to Yaml::parse is deprecated as of 2.2 (according to the sources) and will be removed from sf 3.0. Updated the documentation to indicate this, and include the warning from the source about the need to validate the input first.

Obsoletes PR #2620.

jbafford added 2 commits May 8, 2013 10:22
…cation

Using a filename as argument to Yaml::parse is deprecated as of 2.3 and will be removed from sf 3.0. Updated the documentation to indicate this, and include the warning from the source about the need to validate the input first.

Conflicts:

	components/yaml/introduction.rst
Looking at the Symfony code, this was actually deprecated in 2.2. Move the warning text to the front because the validation warning is more important than the deprecation notice.
weaverryan added a commit that referenced this pull request May 10, 2013
Update yaml component docs to reflect Yaml::parse(filename) deprecation
@weaverryan weaverryan merged commit afce899 into symfony:2.2 May 10, 2013
@weaverryan
Copy link
Member

Perfect PR John - thanks! I've merged this in with no changes.

Cheers!

@jbafford jbafford deleted the i2570-22 branch May 10, 2013 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants