Skip to content

Skip classes with Symfony 7.1 Workflow listener attributes#923

Merged
TomasVotruba merged 3 commits intorectorphp:mainfrom
dionisvl:fix/skip-workflow-listener-attributes
Mar 6, 2026
Merged

Skip classes with Symfony 7.1 Workflow listener attributes#923
TomasVotruba merged 3 commits intorectorphp:mainfrom
dionisvl:fix/skip-workflow-listener-attributes

Conversation

@dionisvl
Copy link
Contributor

@dionisvl dionisvl commented Mar 4, 2026

Extends hasAsListenerAttribute() to also skip classes using Symfony 7.1 workflow attributes (AsTransitionListener, AsGuardListener, AsCompletedListener, etc.) — not just AsEventListener.

Fixes rectorphp/rector#9685

EventListenerToEventSubscriberRector now skips classes that use any of the
7 workflow listener attributes introduced in Symfony 7.1, not just AsEventListener.

Fixes rectorphp/rector#9685
@TomasVotruba
Copy link
Member

TomasVotruba commented Mar 5, 2026

Thank you💪

Can you fix CI as well?

@dionisvl dionisvl requested a review from TomasVotruba March 5, 2026 17:35
@dionisvl
Copy link
Contributor Author

dionisvl commented Mar 6, 2026

@TomasVotruba Hi, CI fixed

@TomasVotruba
Copy link
Member

Awesome, thank you for contribution 👍

@TomasVotruba TomasVotruba merged commit 6f80310 into rectorphp:main Mar 6, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

EventListenerToEventSubscriberRector should skip classes with Symfony 7.1 Workflow attributes

2 participants