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

Storefront: Automatic shipping method / payment method changing does not work correctly if missing name translation in current language #2544

Open
AndreasA opened this issue Jun 20, 2022 · 0 comments
Labels

Comments

@AndreasA
Copy link
Contributor

AndreasA commented Jun 20, 2022

PHP Version

8.1

Shopware Version

6.4.12.0

Expected behaviour

Notice is correctly shown and page is reloaded to ensure all data is correctly updated.

Actual behaviour

Shipping method changed notice is not added to cart, if the shipping method name/payment method name is not translated in the current language as the language fallabck chain is not used.
Therefore, there is also no page refresh after the shipping/payment method has been changed which might also result in incorrect content.

See https://github.com/shopware/platform/blob/v6.4.12.0/src/Storefront/Checkout/Payment/BlockedPaymentMethodSwitcher.php#L91
https://github.com/shopware/platform/blob/6.4.12.0/src/Storefront/Checkout/Shipping/BlockedShippingMethodSwitcher.php#L92

How to reproduce

  • Setup two languages en-GB and de-DE with default being en-GB
  • Create two shipping methods and only store its content in en-GB.
  • Ensure that upon quantity changes one of the shipping methods is not available anymore with availability rules (or corresponding price matrix).
  • Open the page on de-DE and do all following steps on that page.
  • During checkout, select the shipping method that will disappear upon quantity changes.
  • On checkout confirm page, change the quantity so that the selected method is not available anymore.
  • There will be no changed shipping method notice and there also is no additional redirect which means that the although the shipping method was selected, it might not be shown as selected and/or some other changes due to the shipping method change might not be shown correctly.
@AndreasA AndreasA added the Bug label Jun 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant