Skip to content
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

[SecurityBundle] Fix referencing aliases from RegisterEntryPointsPass #39982

Merged
merged 1 commit into from Jan 26, 2021

Conversation

chalasr
Copy link
Member

@chalasr chalasr commented Jan 26, 2021

Q A
Branch? 5.2
Bug fix? yes
New feature? no
Deprecations? no
Tickets #39881
License MIT
Doc PR -

Resolving child definitions' classes in this pass unlocks registering it as TYPE_BEFORE_OPTIMIZATION, and get rid of the issue reported in #39881.
The parent class resolution logic was borrowed from an existing pass (RegisterEventListenersAndSubscribersPass), that makes it the best solution I can think of.

@chalasr chalasr changed the title [SecurityBundle] Fix referencing aliases from RegisterEntryPointPass [SecurityBundle] Fix referencing aliases from RegisterEntryPointsPass Jan 26, 2021
@nicolas-grekas
Copy link
Member

nicolas-grekas commented Jan 26, 2021

Thank you @chalasr.

@nicolas-grekas nicolas-grekas merged commit f7e6196 into symfony:5.2 Jan 26, 2021
5 checks passed
@chalasr chalasr deleted the entrypoint-pass branch Jan 26, 2021
@fabpot fabpot mentioned this pull request Jan 27, 2021
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants