Skip to content

[Workflow] Fix dispatch of entered event when the subject is already in this marking #60194

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 14, 2025
Merged

[Workflow] Fix dispatch of entered event when the subject is already in this marking #60194

merged 1 commit into from
Apr 14, 2025
+46 −1

Conversation

lyrixx
Copy link
Member

@lyrixx lyrixx commented Apr 10, 2025

Q A
Branch? 6.4
Bug fix? yes
New feature? no
Deprecations? no
Issues Fix #57924
License MIT

This was wrong: #37813

@carsonbot carsonbot added this to the 6.4 milestone Apr 10, 2025
@lyrixx lyrixx requested a review from stof April 11, 2025 10:14
@lyrixx lyrixx linked an issue Apr 11, 2025 that may be closed by this pull request
Symfony Workflow Component - in some cases event "workflow.[workflow name].entered.[place name]" is being dispatched multiple times for a place even though marking was already in that place #57924
Closed
@nicolas-grekas
Copy link
Member

Thank you @lyrixx.

@nicolas-grekas nicolas-grekas merged commit ddc2787 into symfony:6.4 Apr 14, 2025
10 of 11 checks passed
@lyrixx lyrixx deleted the workflow-event-entered branch April 14, 2025 12:57
This was referenced May 2, 2025
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexandre-daubois alexandre-daubois alexandre-daubois approved these changes

@stof stof Awaiting requested review from stof

Assignees
No one assigned
Labels
Bug Status: Needs Review Workflow
Projects
None yet
Milestone
6.4
4 participants
@lyrixx @nicolas-grekas @alexandre-daubois @carsonbot