mailalerts icon indicating copy to clipboard operation
mailalerts copied to clipboard

Fix rounding error in new order email

Open mcdado opened this issue 10 years ago • 2 comments

The previous values are rounded to 2 decimals in the following method:

  1. Order::getOrders()
  2. Order::getProductsDetail()
  3. Order::setProductPrices()

This is due to backwards compatibility with 1.4, but it can lead to rounding errors in the product lines in the email. Instead, here we are using other values provided by the Core.

See example screenshot, where the total is not aligned with the product total:

Calculation error Please note how the total in the product line is not the same as the product total below. The minus-minus on the coupon line is another issue, precisely: https://github.com/PrestaShop/mailalerts/pull/25

mcdado avatar Dec 01 '15 21:12 mcdado

:up:

mcdado avatar Jan 07 '16 17:01 mcdado

Anybody?

mcdado avatar Sep 22 '16 15:09 mcdado