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.56k stars 9.32k forks source link

Price filter ignores inclusion of tax #28334

Open maderlock opened 4 years ago

maderlock commented 4 years ago

Preconditions

1.I have set Backend Catalog Price as a Excluding Tax And Front side Display Product Price in Catalog set as Including Tax.

  1. Magento 2.3.3p2
  2. Magento 2.4-develop

Steps to reproduce

Magento 2.3.3p2 Go to Backend and set this Tax Calculation configuration Stores -> Configuration -> Sales -> Tax 46853224-19609900-ce1b-11e8-9d9a-c7043df60132

Stores -> Taxes -> Tax Zones and Rates 46853275-3b5a1b80-ce1b-11e8-8b46-79b159ef5de4

Stores -> Taxes -> Tax Rules 46853330-562c9000-ce1b-11e8-8add-1537e0066fdd

Go to Front end listing page and filter it by Price Range 40 to 50

Expected result

In listing Page After Price filter Display result between 40 to 50

Actual result

In listing Page After Price filter Display result in some products are displayed greater than 50. Please check below screen shot . 46853576-2e89f780-ce1c-11e8-89ca-d7265d07baf3

m2-assistant[bot] commented 4 years ago

Hi @Sutharsan27. Thank you for working on this issue. In order to make sure that issue has enough information and ready for development, please read and check the following instruction: :point_down:

Sutharsan27 commented 4 years ago

@magento give me 2.4-develop instance

magento-engcom-team commented 4 years ago

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

magento-engcom-team commented 4 years ago

Hi @Sutharsan27, here is your Magento instance. Admin access: https://i-28334-2-4-develop.instances.magento-community.engineering/admin_7193 Login: 13e7c30c Password: bbfe6f6fbab3 Instance will be terminated in up to 3 hours.

Sutharsan27 commented 4 years ago

@magento give me 2.4-develop instance

magento-engcom-team commented 4 years ago

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

Sutharsan27 commented 4 years ago

@magento give me 2.4-develop instance

magento-engcom-team commented 4 years ago

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

m2-assistant[bot] commented 3 years ago

Hi @engcom-Bravo. Thank you for working on this issue. In order to make sure that issue has enough information and ready for development, please read and check the following instruction: :point_down:

engcom-Bravo commented 3 years ago

Hello @maderlock

Thank you for your report.

The reported behaviour is reproducible on the latest 2.4-develop. The main issue is that the products are filtered by the original price, not Price + Tax, which might be misleading from the user perspective.

magento-engcom-team commented 3 years ago

:white_check_mark: Confirmed by @engcom-Bravo Thank you for verifying the issue. Based on the provided information internal tickets MC-40293 were created

Issue Available: @engcom-Bravo, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself.

vlmed commented 3 years ago

@magento I am working on this

m2-assistant[bot] commented 3 years ago

Hi @engcom-Charlie. 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:


m2-assistant[bot] commented 3 years ago

Hi @engcom-Hotel. 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:


blmage commented 3 years ago

@magento I am working on this

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://71c4ff83b27b2b387c292b92210568d1.instances-prod.magento-community.engineering Admin access: https://71c4ff83b27b2b387c292b92210568d1.instances-prod.magento-community.engineering/admin_e7c2 Login: 20d5bd44 Password: 03b9e1027c79

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

Hi @maderlock,

Thanks for your reporting and collaboration.

We have verified the issue in Latest 2.4-develop instance and the issue is reproducible.Kindly refer the screenshots.

Screenshot 2024-06-04 at 14 24 54

In listing Page After Price filter Display result in some products are displayed greater than 50

Hence Confirming the issue.

Thanks.

maderlock commented 5 months ago

Thanks for confirming. It's worth noting that this is a particular issue in the UK and Europe where the expectation (and indeed in places legal requirement) is that tax is included in all prices, unlike some of the US where exclusive prices without tax are more common.

-- This email may contain confidential and/or privileged information. If you are not the intended recipient (or have received this email in error) please notify the sender immediately and destroy this email. Any unauthorised copying, disclosure or distribution of the material in this email is strictly forbidden.

github-jira-sync-bot commented 5 months ago

:white_check_mark: Jira issue https://jira.corp.adobe.com/browse/AC-12077 is successfully created for this GitHub issue.

m2-assistant[bot] commented 5 months ago

:white_check_mark: Confirmed by @engcom-Bravo. Thank you for verifying the issue.
Issue Available: @engcom-Bravo, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself.

danielraab commented 1 week ago

Is their already a possible solution/workaround/patch for that ?