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

Make a clone of symfony 5.4 RegisterListenersPass for usage in DocumentManagerBundle #6579

Conversation

alexander-schranz
Copy link
Member

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets fixes #
Related issues/PRs #
License MIT
Documentation PR sulu/sulu-docs#

What's in this PR?

Make a clone of symfony 5.4 RegisterListenersPass for usage in DocumentManagerBundle.

Why?

For future symfony 6 compatibility. There the RegisterListenersPass was refractored and can not longer be used the way we using it here.

@alexander-schranz alexander-schranz added the DX Affecting the end developer label May 3, 2022
@alexander-schranz alexander-schranz force-pushed the enhancement/event-dispatcher-compatibility branch from 8e0477b to 6e2eb7c Compare May 3, 2022 12:00
@alexander-schranz alexander-schranz mentioned this pull request May 3, 2022
84 tasks
@alexander-schranz alexander-schranz force-pushed the enhancement/event-dispatcher-compatibility branch 3 times, most recently from 4397208 to 30cbd6c Compare May 3, 2022 12:35
@alexander-schranz alexander-schranz force-pushed the enhancement/event-dispatcher-compatibility branch from 30cbd6c to e38a2a3 Compare May 3, 2022 12:46
* This is a copy of the Symfony 5.4 EventDispatcher RegisterListenersPass as it is not available this way in Symfony 6.
* https://github.com/symfony/symfony/blob/5.4/src/Symfony/Component/EventDispatcher/DependencyInjection/RegisterListenersPass.php
*/
class RegisterListenersPass implements CompilerPassInterface
Copy link
Member Author

@alexander-schranz alexander-schranz May 3, 2022

Choose a reason for hiding this comment

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

This is just a copy of (v5.4.8): https://github.com/symfony/symfony/blob/5.4/src/Symfony/Component/EventDispatcher/DependencyInjection/RegisterListenersPass.php

Because they refractored it in Symfony 6 so we could not longer use it for our document event dispatcher as they hardcoded the service definitions.

@Prokyonn Prokyonn merged commit 11890c4 into sulu:2.5 May 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DX Affecting the end developer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants