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

Missing cascade into attribute set deletion #12110

Closed
asrenzo opened this issue Nov 8, 2017 · 3 comments
Closed

Missing cascade into attribute set deletion #12110

asrenzo opened this issue Nov 8, 2017 · 3 comments
Assignees
Labels
Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line Issue: Clear Description Gate 2 Passed. Manual verification of the issue description passed Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release

Comments

@asrenzo
Copy link

asrenzo commented Nov 8, 2017

Preconditions

  1. Create a custom attributes set
  2. Attach some products to this attributes set

Steps to reproduce

  1. Got to attributes set admin page
  2. Delete your custom attributes set
  3. Confirm you want to remove attached products also

Expected result

  1. Table url_rewrite should be cleaned from references to removed products
  2. A new product named like an old one should be insertable via API or admin interface

Actual result

  1. Impossible to add a new product named like an old one because a request_path with the same value as computed already exists in table url_rewrite;
@magento-engcom-team magento-engcom-team added Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Issue: Clear Description Gate 2 Passed. Manual verification of the issue description passed 2.1.x Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release labels Nov 8, 2017
@magento-engcom-team
Copy link
Contributor

@asrenzo, thank you for your report.
We've created internal ticket(s) MAGETWO-83507 to track progress on the issue.

@magento-engcom-team magento-engcom-team added Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release labels Nov 9, 2017
@nmalevanec nmalevanec self-assigned this Nov 9, 2017
@ishakhsuvarov
Copy link
Contributor

Hi @asrenzo
We are closing this ticket as fixes are already delivered with the linked PRs.

Thank you for collaboration.

@okorshenko okorshenko added Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line labels Dec 11, 2017
@asrenzo
Copy link
Author

asrenzo commented Dec 20, 2017

Hi,
Sounds like this was not fixed in 2.1.x version.
Don't you plan to do it ?
Regards,
Laurent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line Issue: Clear Description Gate 2 Passed. Manual verification of the issue description passed Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release
Projects
None yet
Development

No branches or pull requests

5 participants