-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[Bundle][Attribute]It will also listen on the product attribute objec… #14202
[Bundle][Attribute]It will also listen on the product attribute objec… #14202
Conversation
MrPogos
commented
Jul 29, 2022
Q | A |
---|---|
Branch? | 1.11 |
Bug fix? | yes |
New feature? | no |
BC breaks? | no |
Deprecations? | no |
Related tickets | fixes #14185 |
License | MIT |
…t for protection against deletion when it is used by the product object
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @MrPogos,
thanks a lot for your proposal! We appreciate it :) However, without reproducing scenario, I won't merge it. In addition, changes in previously generated are by default no-go for us. Can you alter your PR with:
- write behat scenario, that would cover your case
- revering changes to already generated migration
- adding new migration that will remove
CASCADE
on delete
src/Sylius/Bundle/CoreBundle/Migrations/Version20161202011555.php
Outdated
Show resolved
Hide resolved
…nd added a migration that disables cascading removal.
|
||
@ui | ||
Scenario: Try deleting a attribute from the registry when product use him | ||
And this product has a text attribute "Gun caliber" with value "11 mm" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And this product has a text attribute "Gun caliber" with value "11 mm" | |
Given this product has a text attribute "Gun caliber" with value "11 mm" |
@managing_product_attributes | ||
Feature:Removing a attribute | ||
As an administrator | ||
I want to be able to remove a attribute that is not assigned to any of the products |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I want to be able to remove a attribute that is not assigned to any of the products | |
I want to be able to remove an attribute that is not assigned to any of the products |
@@ -0,0 +1,21 @@ | |||
@managing_product_attributes | |||
Feature:Removing a attribute | |||
As an administrator |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As an administrator | |
In order to keep my collection of product attributes not cluttered | |
As an administrator |
Thanks @MrPogos and Patryk! 🎉 |
…lchrusciel) This PR was merged into the 1.11 branch. Discussion ---------- | Q | A | |-----------------|--------------------------------------------------------------| | Branch? | 1.11 | | Bug fix? | yes | | New feature? | no | | BC breaks? | no | | Deprecations? | no | | Related tickets | review from #14202 | | License | MIT | <!-- - Bug fixes must be submitted against the 1.11 branch - Features and deprecations must be submitted against the 1.12 branch - Make sure that the correct base branch is set To be sure you are not breaking any Backward Compatibilities, check the documentation: https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html --> Commits ------- 1a5529a [Minor][Behat] Attributes deletion scenario improvements