Skip to content

Configurable product price changes affect all elements with the class 'priceBox' #5798

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

Closed
ntoombs19 opened this issue Jul 25, 2016 · 3 comments
Labels
bug report Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development

Comments

@ntoombs19
Copy link

Preconditions

Magento version 2.0.6 CE

Steps to reproduce

  1. Create a configurable product with options that, when selected, can change the final price.
  2. Set upsell/crossell/related products on this configurable product. The idea is to get several elements on the page that have the class 'priceBox'.
  3. Navigate to that product on the frontend and review all the prices on the page.
  4. Select the configurable option that will change the final price.
  5. Notice how not just the product price changed but the prices of all other products on the page as well.

Expected result

Only the price of the product being viewed should be changed.

Actual result

All prices contained in an element with the class 'priceBox' are changed.

Proposed solution

File: app/code/Magento/ConfigurableProduct/view/frontend/web/js/configurable.js
Line: 21
Change To: priceHolderSelector: '.product-info-main .price-box',

This should work because on line 415 the updatePrice event will only be triggered on the priceBox in the product-info-details element on the page rather than every priceBox on the page.

@veloraven veloraven self-assigned this Aug 8, 2016
@veloraven
Copy link
Contributor

@ntoombs19 thank you for your report.
However, this issue is duplicate for #4588 and we already have internal ticket MAGETWO-53135.
I have linked it with this issue as well.

@veloraven veloraven added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Aug 11, 2016
@veloraven veloraven removed their assignment Aug 11, 2016
@slavvka
Copy link
Member

slavvka commented Sep 13, 2016

The issue has been fixed and delivered to the mainline. It is also ported for 2.0 and 2.1 and will be delivered as a part of nearest patch releases.

@slavvka slavvka closed this as completed Sep 13, 2016
@elenleonova
Copy link

This issue has been delivered to 2.1.3 and 2.0.11 today.

magento-engcom-team pushed a commit that referenced this issue Jun 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Development

No branches or pull requests

5 participants