-
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 the error that is wrong link title of a downloadable product when enabling "Use Default Value" #27295
Fix the error that is wrong link title of a downloadable product when enabling "Use Default Value" #27295
Conversation
… enabling "Use Default Value"
Hi @tna274. Thank you for your contribution
For more details, please, review the Magento Contributor Guide documentation. |
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.
Hi @tna274, thank you for your contribution, could you please review the minor recommendation?
Could you please also check and fix the failing Static Tests?
Thank you.
… enabling "Use Default Value" - Fix unit test for the function that was fixed
Update app/code/Magento/Downloadable/Model/Link/Builder.php Co-Authored-By: Eduard Chitoraga <e.chitoraga@atwix.com>
… enabling "Use Default Value" - Revert incorrect file
… enabling "Use Default Value" - Update unit test
… enabling "Use Default Value" - Fix static code
… enabling "Use Default Value" - Fix static code
… enabling "Use Default Value" - Fix static code
Hi @eduard13 |
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.
Indeed, the failing tests aren't related to these changes.
Hi @eduard13, thank you for the review. |
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.
Hi, @tna274.
During testing, we faced the issue.
Problem: Not able to update the Samples Title by "Use Default Value" checkbox
Manual testing scenario:
- Create a new downloadable product with a link and sample;
- Save the product;
- Choose scope at any store view;
- In "Downloadable Information" section, uncheck "Use Default Value" boxes under link title and sample title, and change the values.
- Save the product;
- Now, check "Use Default Value" boxes under the link and sample titles;
- Save the product;
Actual Result:
✖️ The default value for the link title was saved but for the sample title was not.
@tna274 Could you take a look?
Thanks!
Pull Request state was updated. Re-review required.
Hi @engcom-Alfa |
Hi,@eduard13. Could you review the latest changes? Thanks! |
Hi @eduard13, thank you for the review. |
✔️ QA Passed |
@magento run all tests |
… magento2.4/fixbug_default_link_title
… product when enabling "Use Default Value" #27295
Hi @tna274, thank you for your contribution! |
Description (*)
Fix the error that is wrong link title of a downloadable product when enabling "Use Default Value"
Related Pull Requests
N/A
Fixed Issues (if relevant)
#27169 - Not able to update value with "use default checkbox" for Downloadable Product
Manual testing scenarios (*)
Questions or comments
Contribution checklist (*)