Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Added a note about request accept-* headers parsing. #1868

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
Nov 16, 2012

Conversation

jfsimon
Copy link
Contributor

@jfsimon jfsimon commented Oct 30, 2012

A note about changes introduced here: symfony/symfony#5841

@@ -215,6 +215,23 @@ the
method tells you if the request contains a Session which was started in one of
the previous requests.

Accessing `Accept-*` Headers Data
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you need to mention that the Request class has methods that do the work for you and list them.

@jfsimon
Copy link
Contributor Author

jfsimon commented Nov 5, 2012

@fabpot is it ok?

* :method:`Symfony\\Component\\HttpFoundation\\Request::getCharsets`:
Returns the list of accepted languages ordered by descending quality;

If you need to egt full access to parsed data from ``Accept``, ``Accept-Language``,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo here: get

weaverryan added a commit that referenced this pull request Nov 16, 2012
Added a note about request accept-* headers parsing.
@weaverryan weaverryan merged commit 6374c39 into symfony:master Nov 16, 2012
weaverryan added a commit that referenced this pull request Nov 16, 2012
@weaverryan
Copy link
Member

Hi Jean-François!

Nice PR - I've merged this in! I also backported the first parts - that talk about the methods on Request - back to the 2.0 branch so that we can enjoy those added details for all versions.

Thanks!

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.

5 participants