-
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
Checkout. Quote Address Street cloning issue #26276
Comments
Hi @yutv. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
For more details, please, review the Magento Contributor Assistant documentation. @yutv do you confirm that you were able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
Hi @sudheers-kensium. Thank you for working on this issue.
|
@magento give me 2.4-develop instance |
Hi @sudheers-kensium. Thank you for your request. I'm working on Magento 2.4-develop instance for you |
Hi @sudheers-kensium, here is your Magento instance. |
Hello @yutv , This is not an issue. It is working as expected because you marked both Shipping and as Billing address same. I am closing this issue. Feel free to reopen if you feel that this is Magento core issue. |
No, you are wrong. There is a bug. As you can see from the screenshot, changing the billing city doesn't lead to changing shipping city, but changing billing street does. An experienced front end developer may confirm that the issue exists. |
Hi @yutv. Thank you for your report.
The fix will be available with the upcoming 2.4.0 release. |
@yutv @sudheers-kensium @magento-engcom-team commit 2687221#diff-e96dae14765cbb77c452167706254377f17c2c2821f689958e306c9c4f81169b This is due to cloning that adds a bunch of function on top of data, see below: Another magento project on 2.4.2 Unfortunately i cannot confirm at this point steps to replicate as it's different from these 2 projects, but essentially reverting 2687221#diff-e96dae14765cbb77c452167706254377f17c2c2821f689958e306c9c4f81169b fixed it for me. Any advice on this matter? |
Hello @Ig0r-M-magic42, Most probably you have Google Tag Manager (GTM) enabled on Magento EE/Cloud version. It has the 'prototype' dependency in the As a hotfix the following patch may be used.
It resolves the issue on checkout but there are other occurrences of |
@yutv thanks for the feedback, it's quite helpful |
Preconditions (*)
Steps to reproduce (*)
Add a product to cart and go to the second step of checkout.
Open developer console and change billing address. e.g.
Expected result (*)
Actual result (*)
Cloning Issue
The issue is here app/code/Magento/Checkout/view/frontend/web/js/action/select-billing-address.js#L21
and can be fixed by replacing:
to
to clone address recursively including the street array.
The text was updated successfully, but these errors were encountered: