Skip to content

[HttpFoundation] Added getter for status text/reason phrase #14421

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.

Sign up for GitHub

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

Jump to bottom
Closed
mnpenner wants to merge 2 commits into from
Closed

[HttpFoundation] Added getter for status text/reason phrase #14421

mnpenner wants to merge 2 commits into from

Conversation

@mnpenner
Copy link

@mnpenner mnpenner commented Apr 20, 2015

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #14388
License MIT
Doc PR n/a

@mnpenner mnpenner changed the title #14388 [HttpFoundation] Added getter for status text/reason phrase Apr 20, 2015
@fabpot
Copy link
Member

fabpot commented Apr 20, 2015

As I said on the PSR-7 PR, being able to set or retrieve the reason phrase should not be used. I won't add this in Symfony as there is no valid use cases.

@mnpenner
Copy link
Author

mnpenner commented Apr 20, 2015

There's a setter...you should at least add it for symmetry, but I guess there's no point arguing.

@mnpenner mnpenner closed this Apr 20, 2015
@Tobion Tobion mentioned this pull request May 28, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mnpenner @fabpot