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

Removing the promotion rule when deleting a product affects promotion trigger rules #15201

Closed
laurentdabbb opened this issue Aug 2, 2023 · 1 comment
Assignees
Labels
Bug Confirmed bugs or bugfixes.

Comments

@laurentdabbb
Copy link

Sylius version affected: 1.12.8+

Description
I think the fix #14932 merged in 1.12.8 cause a bigger issue.
A listener has been added to remove promotions rules of type "contains_product" when the targeted product is removed. But deleting a promotion rule changes the application's promotion rules...

Consider a promotion "PLOP" with an action "free delivery" (100% discount on shipping fees) and a rule "cart contains product with code ABC". This promotion is active.
Deleting ABC productresults in the deletion of the promotion rule on "PLOP" promotion.
From now, "PLOP" promotion is still active and no longer has rules. This promotion is applicable for all orders. All orders on the store will be shipped for free...

Possible Solution
Do not remove promotion rule.

Thanks.

@jakubtobiasz jakubtobiasz added the Potential Bug Potential bugs or bugfixes, that needs to be reproduced. label Aug 29, 2023
@jakubtobiasz
Copy link
Contributor

Hello @laurentdabbb 👋🏼,
we'll check this in the upcoming sprint (Weeks 36-37).

@jakubtobiasz jakubtobiasz self-assigned this Aug 29, 2023
@jakubtobiasz jakubtobiasz added Bug Confirmed bugs or bugfixes. and removed Potential Bug Potential bugs or bugfixes, that needs to be reproduced. labels Aug 31, 2023
@GSadee GSadee closed this as completed in c8fdef8 Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Confirmed bugs or bugfixes.
Projects
None yet
Development

No branches or pull requests

2 participants