magento / magento2

Prior to making any Submission(s), you must sign an Adobe Contributor License Agreement, available here at: https://opensource.adobe.com/cla.html. All Submissions you make to Adobe Inc. and its affiliates, assigns and subsidiaries (collectively “Adobe”) are subject to the terms of the Adobe Contributor License Agreement.
http://www.magento.com
Open Software License 3.0
11.5k stars 9.3k forks source link

Wrong tax is displayed in checkout for Virtual products #17425

Closed navarr closed 5 months ago

navarr commented 6 years ago

Under certain scenarios the tax displayed in checkout for a virtual product does not match the address selected by the billing method

Preconditions

  1. Magento 2.2.6-dev installed -- Updated(09/19/2019): Still actual on 2.3-develop -- Updated(02/15/2021): Modified for similar issue on 2.4-develop
  2. Default Tax rates configured (typical of a clean installation)
  3. Addition of a Tax Rule that enables those rates virtual-product-setup
  4. A virtual product created with price point $10

Steps to reproduce

  1. Create a customer account
  2. Create a default billing & shipping address in New York, such as: 123 Main St Flushing, NY 11354
  3. Save
  4. Browse to category to add Virtual product to cart
  5. Proceed to checkout
  6. Note sales tax of $0.84
  7. Click "Edit" under the billing address
  8. Click "Update"
  9. Press back till you are no longer in checkout
  10. Go to customer account page
  11. Edit Address, set it to an address in California such as: 1 Main St San Jose, CA 95131
  12. Save
  13. Click Mini-cart and click "Proceed to Checkout"

Video Recreation

Expected result

  1. Sales tax of $0.83 (calculated off California address)

Actual result

  1. Sales tax of $0.84 (calculated off New York address)

Notice

magento-engcom-team commented 6 years ago

Hi @navarr. Thank you for your report. To help us process this issue please make sure that you provided the following information:

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:

@magento-engcom-team give me {$VERSION} instance

where {$VERSION} is version tags (starting from 2.2.0+) or develop branches (2.2-develop +). For more details, please, review the Magento Contributor Assistant documentation.

@navarr do you confirm that you was able to reproduce the issue on vanilla Magento instance following steps to reproduce?

navarr commented 6 years ago

@magento-engcom-team give me 2.2-develop instance

magento-engcom-team commented 6 years ago

Hi @navarr. Thank you for your request. I'm working on Magento 2.2-develop instance for you

magento-engcom-team commented 6 years ago

Hi @navarr, here is your Magento instance. Admin access: https://i-17425-2-2-develop.engcom.dev.magento.com/admin Login: admin Password: 123123q Instance will be terminated in up to 3 hours.

ghost commented 6 years ago

@navarr , thank you for your report. We've acknowledged the issue and added to our backlog.

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 14 days if no further activity occurs. Is this issue still relevant? If so, what is blocking it? Is there anything you can do to help move it forward? Thank you for your contributions!

navarr commented 3 years ago

@magento-engcom-team give me 2.4-develop instance

magento-deployment-service[bot] commented 3 years ago

Hi @navarr. Thank you for your request. I'm working on Magento instance for you.

magento-deployment-service[bot] commented 3 years ago

Hi @navarr, here is your Magento Instance: https://5161c12f2cbe4c20a233551fe0791c52-2-4-develop.instances.magento-community.engineering Admin access: https://5161c12f2cbe4c20a233551fe0791c52-2-4-develop.instances.magento-community.engineering/admin_d12c Login: d101717e Password: 823615cb0514

navarr commented 3 years ago

I've updated the issue with slightly different recreation steps for 2.4; as the original issue was fixed

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 14 days if no further activity occurs. Is this issue still relevant? If so, what is blocking it? Is there anything you can do to help move it forward? Thank you for your contributions!

navarr commented 3 years ago

There hasn't been a non-patch release since the last time I confirmed this issue. I don't believe it's qualified to be stale yet

Den4ik commented 3 years ago

@magento I'm working on it

m2-assistant[bot] commented 5 months ago

Hi @engcom-Bravo. Thank you for working on this issue. Looks like this issue is already verified and confirmed. But if you want to validate it one more time, please, go though the following instruction:


engcom-Bravo commented 5 months ago

@magento give me 2.4-develop instance

magento-deployment-service[bot] commented 5 months ago

Hi @engcom-Bravo. Thank you for your request. I'm working on Magento instance for you.

magento-deployment-service[bot] commented 5 months ago

Hi @engcom-Bravo, here is your Magento Instance: https://a580233b9d08ea9c20797b8817307472.instances-prod.magento-community.engineering Admin access: https://a580233b9d08ea9c20797b8817307472.instances-prod.magento-community.engineering/admin_152d Login: 03c8e4c0 Password: cbf0bacc4d06

engcom-Bravo commented 5 months ago

Hi @navarr,

Thanks for your reporting and collaboration.

We have verified the issue in Latest 2.4-develop instance and the issue is no more reproducible.Hence We are closing this issue.

Steps to reproduce

Screenshot 2024-05-03 at 12 15 20
navarr commented 5 months ago

@engcom-Bravo doesn't that screenshot show that you recreated the issue? If the tax rates are still the same, it's showing new York tax for a California address

The expected result is 83c of tax, but you've got 84