-
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 Translation of error message on cart for deleted bundle option. #16777
Fix Translation of error message on cart for deleted bundle option. #16777
Conversation
…ted bundle option.
Hi @swnsma. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
@@ -43,6 +43,6 @@ | |||
<plugin name="multishipping_session_mapper" type="Magento\Multishipping\Model\Checkout\Type\Multishipping\Plugin" sortOrder="50" /> | |||
</type> | |||
<type name="Magento\Checkout\Controller\Cart"> | |||
<plugin name="multishipping_clear_addresses" type="Magento\Multishipping\Model\Cart\Controller\CartPlugin" /> | |||
<plugin name="multishipping_clear_addresses" type="Magento\Multishipping\Model\Cart\Controller\CartPlugin" sortOrder="50" /> |
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.
@swnsma, looks pretty unreliable to me. Can you describe what is going on and maybe we will try to elaborate a robust fix together?
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 @orlangur !
In two words, Inline Translate Phrase renderer loads later than error message Phrase become rendered.
The error message source.
__('The required options you selected are not available.') |
Rendered become loaded deep in Magento\Framework\App\Action\Plugin\Design
plugin call.
The plugin has sort order 30.
<plugin name="designLoader" type="Magento\Framework\App\Action\Plugin\Design" sortOrder="30"/> |
As alternative, sort order for this plugin could be set to 0, to prevent possible issue in future.
…ted bundle option. - change plugin sort order in case to prevent loading of not translatable messages for Localized exception in beforeDispatch plugin.
Hi @swnsma. Thank you for your contribution. Please, consider to port this solution to 2.3 release line. |
Description
Manual testing scenarios
Pre-conditions
Steps To Reproduce
Expected Result:
Actual Result:
Contribution checklist