Price tiers calculated on 2 decimals even when prices of 4 decimals are entered

[dverkade]

Description

Although a lot of currencies use only 2 decimals for displaying a price, products might have to be calculated on smaller prices especially in the B2B space. For instance the product price is just 0.5292 per piece for 5 pieces (tier price). Magento now first round the price per piece to 0,53 and then multiplies that by the number you are buying (500), which makes for a total price of 265,00 instead of 264,40.

Preconditions

1. Magento 2.2.4
2. PHP 7.1
3. Locale for the website is Netherlands in our case, but should be the same for US locale or any other locale where prices are displayed in 2 decimals.

Steps to reproduce

1. Add a product
2. Add a tier price and enter a 4 decimal price. For instance 0.5292 for 500 pieces.
3. Open the backoffice again. You see that the prices are already displayed rounded to 0.53 instead of the entered 0.5292
4. In de frontend, add 500 pieces of the product to the cart. The price in the cart is 265,00 instead of 264,40.

Expected result

1. Round the price after calculating the total price for the product.

Actual result

Prices showing in de backoffice. They are rounded to two decimals.

Prices as they are stored in the database.

[ghost]

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

[magento-engcom-team]

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

• 1. Add/Edit Component: XXXXX label(s) to the ticket, indicating the components it may be related to.

• 2. Verify that the issue is reproducible on 2.3-develop branch

  Details

  - Add the comment @magento-engcom-team give me 2.3-develop instance to deploy test instance on Magento infrastructure.
  - If the issue is reproducible on 2.3-develop branch, please, add the label Reproduced on 2.3.x.
  - If the issue is not reproducible, add your comment that issue is not reproducible and close the issue and stop verification process here!

• 3. Verify that the issue is reproducible on 2.2-develop branch.

  Details

  - Add the comment @magento-engcom-team give me 2.2-develop instance to deploy test instance on Magento infrastructure.
  - If the issue is reproducible on 2.2-develop branch, please add the label Reproduced on 2.2.x

• 4. If the issue is not relevant or is not reproducible any more, feel free to close it.

[orlangur]

Closing in favor of https://github.com/magento/magento2/issues/10532. Entering four decimals price is not a valid use case currently.

[dverkade]

I'm reopening this issue. Unlike @orlangur is mentioning this is a valid use case. First of all, you can enter prices with 4 decimals in the backoffice, and they will actually be saved. Second of all, @magento-engcom-team has confirmed and reproduced this issue.

[AleksLi]

@dverkade so what's now. Can it be fixed for particular use case as I've done in my PR #21554 or what? I mean in the scope of this ticket and then it could be fixed more generally.

[orlangur]

has confirmed and reproduced this issue

@dverkade does not mean issue is valid, just that it is reproduced.

Open the backoffice again. You see that the prices are already displayed rounded to 0.53 instead of the entered 0.5292

DIsplaying 0.53 is correct behavior. However, it would be even better to have proper validation to allow entering only 2 digits after comma.

[php4umagento]

This isisue is still reproducable with 2.3.5

[github-jira-sync-bot]

✅ Jira issue https://jira.corp.magento.com/browse/AC-1029 is successfully created for this GitHub issue.

[m2-assistant]

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

[github-jira-sync-bot]

❌ Cannot export the issue. This GitHub issue is already linked to Jira issue(s): https://jira.corp.magento.com/browse/AC-1029

[netdesign]

Hello to all, please check this Pull Request commit 27cd977

It should fix the issue

[JettyRohanAditya]

@magento I am working on this.

[m2-assistant]

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:

• 1. Add/Edit Area: XXXXX label to the ticket, indicating the functional areas it may be related to.
• 2. Verify that the issue is reproducible on 2.4-develop branch
     Details

     - Add the comment @magento give me 2.4-develop instance to deploy test instance on Magento infrastructure.
     - If the issue is reproducible on 2.4-develop branch, please, add the label Reproduced on 2.4.x.
     - If the issue is not reproducible, add your comment that issue is not reproducible and close the issue and stop verification process here!
• 3. If the issue is not relevant or is not reproducible any more, feel free to close it.

---

• Join Magento Community Engineering Slack and ask your questions in #github channel.

[engcom-Bravo]

@magento give me 2.4-develop instance

[magento-deployment-service]

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

[magento-deployment-service]

Hi @engcom-Bravo, here is your Magento Instance: https://218e98d9961f96cf70ba54e62c5fe351.instances-prod.magento-community.engineering
Admin access: https://218e98d9961f96cf70ba54e62c5fe351.instances-prod.magento-community.engineering/admin_4818
Login: ea1207c7
Password: b7b8b1029b5e