-
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
[TASK] Remove translation of attribute store label in getAdditionalData #20482
Conversation
Attribute labels should be translated in magento admin setting correct translation for stores
Hi @c-walter. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
Hi @torhoehn, thank you for the review. |
Hi @c-walter, thank you for your contribution! |
Attribute labels should be translated in magento admin setting correct translation for stores
Magento offers the possibility to set different labels for attributes in magento admin.
This commit solves the issue that some attribute-labels might get translated even if they are already translated, by setting correct value in magento admin.
Description (*)
In \Magento\Catalog\Block\Product\View\Attributes::getAdditionalData an array of attributes will be returned.
Each array element contains 'code', 'value' and 'label'.
Following the admin field in "stores -> attribtues -> products" select any attribute and there in "manage labels" tab
you are supposed to translate the attribute label for the specific languages / store-views in there.
But since the attribute-label will be translated again you might not get the result you set in magento admin, depending if you have a translation for the label you have chosen
Manual testing scenarios (*)
Steps to reproduce
Contribution checklist (*)