Skip to content

[HttpKernel] Fix dealing with self/parent in ArgumentMetadataFactory #27281

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 1 commit into from
May 16, 2018

Conversation

nicolas-grekas
Copy link
Member

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

Applies CS fixes that where merged on 4.0 to 3.4, embeds a fix in ArgumentMetadataFactory, which couldn't deal with self/parent type hints.

@nicolas-grekas nicolas-grekas added this to the 3.4 milestone May 16, 2018
@nicolas-grekas nicolas-grekas changed the title fixed CS [HttpKernel] Fix dealing with self/parent in ArgumentMetadataFactory May 16, 2018
@nicolas-grekas
Copy link
Member Author

Thank you @fabpot.

@nicolas-grekas nicolas-grekas merged commit ba5cb1a into symfony:3.4 May 16, 2018
nicolas-grekas added a commit that referenced this pull request May 16, 2018
…ataFactory (fabpot)

This PR was merged into the 3.4 branch.

Discussion
----------

[HttpKernel] Fix dealing with self/parent in ArgumentMetadataFactory

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

Applies CS fixes that where merged on 4.0 to 3.4, embeds a fix in ArgumentMetadataFactory, which couldn't deal with self/parent type hints.

Commits
-------

ba5cb1a fixed CS
@nicolas-grekas nicolas-grekas deleted the cs branch May 16, 2018 14:51
This was referenced May 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants