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
[DoctrineBundle] remove doctrine.event_subscriber tag and replace by doctrine.event_listener tag #1650
[DoctrineBundle] remove doctrine.event_subscriber tag and replace by doctrine.event_listener tag #1650
Conversation
77b10b9
to
b5c5053
Compare
What about those defined in xml config files? 🤔 https://github.com/search?q=repo%3Adoctrine%2FDoctrineBundle%20doctrine.event_subscriber&type=code |
All doctrine schema subscribers are deprecated since 6.3 and are going to be removed and replaced by their respective doctrine schema listeners already present in xml config files if I'm not wrong. |
Not sure if we should do this in patch release |
True. This should target 2.10.x. |
ok thanks ! i rebase and update the targeted branch then |
…doctrine.event_listener tag
b5c5053
to
89ac05c
Compare
Thanks @alli83 |
…ed (alli83) This PR was merged into the 6.3 branch. Discussion ---------- [DoctrineBridge] skip subscriber if listener already defined | Q | A | ------------- | --- | Branch? | 6.3 | Bug fix? | no | New feature? | no | Deprecations? | | Tickets | | License | MIT | Doc PR | Following #49918 and doctrine/DoctrineBundle#1650 skip doctrine event subscriber if a doctrine event listener is already defined for the same definition. Commits ------- 5a867c5 [DoctrineBridge] skip subscriber if listener already defined
Remove doctrine.event_subscribers tag in bundle and replace them with doctrine.event_listener tag.
following symfony/symfony#49918