-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Fix #10687 - Product image roles disappearing #15606
Conversation
To fix magento#10687 - adds the 'types' key to the $images array to provide existing image roles (if not set), which are re-attributed after clearMediaAttribute() is called
Hi @sidolov, thank you for the review. |
@Scarraban thank you for contributing. Please accept Community Contributors team invitation here to gain extended permissions for this repository. |
Hi @Scarraban. Thank you for your contribution. Please, consider to port this solution to 2.3 release line. |
Hi @Scarraban. Any chance you have the fix for Magento version 2.2.3 ? Magento 2.2.3 has no file |
Hi @michel334 , Check my replication steps comment on the issue #10687 - this was a patch fix that references the ProductRepository.php file before I made the pull request. |
Thanks @Scarraban for the patch, it fixed our issue! |
To fix #10687 - adds the 'types' key to the images array to provide existing image roles, which are reset after clearMediaAttribute() is called.
Description
This PR is to fix the issue outlined in #10687 - image roles being removed when saving by the ProductRepository.
Fixed Issues (if relevant)
Manual testing scenarios
Contribution checklist