0033933: Docuemnt level taxes calculation fails when tax rate has decimals

Openbravo Issue Tracking System - Retail Modules

View Issue Details

Project

Category

View Status

Date Submitted

Last Update

0033933

Retail Modules

Web POS

public

2016-09-08 23:22

2018-07-27 15:33

Reporter

guilleaer

Assigned To

guilleaer

Priority

high

Severity

major

Reproducibility

always

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

Target Version

RR16Q4

Fixed in Version

Merge Request Status

Review Assigned To

marvintm

OBNetwork customer

Support ticket

Regression level

Regression date

Regression introduced in release

Regression introduced by commit

Triggers an Emergency Pack

Summary

0033933: Docuemnt level taxes calculation fails when tax rate has decimals

Description

Document level tax calculations done by web POS are wrong when the tax rate has decimals.

It happens because the algorithm is rounding earlier than expected.

Steps To Reproduce

Create a tax with a rate of 21.5 (you can modify ENTREGAS IVA 21% rate's)
Open web POS in a terminal using price not including taxes
Add a product
- Avalanche Transceiver (150.50)
Realize that taxes calculations are wrong
- Tax amount is 27.14 but it should be 26.63
- Net should be 123.87 but it is 123.36

This happens because when executing the algorithm, we are rounding.
In this case, the algorithm first calculates the total gross 150.5
Then the net is recalculated with the following expression:

newNet = round(totalGross / (1+rate))

The problem is that wrongly we are doing the following

newNet = round(totalGross / round(1+rate))

So in this particular case, newNet calculation is wrong:

wrong -> newNet = round(150.5 / round(1+0.215))
wrong -> newNet = round(150.5 / 1+1.22)
wrong -> newNet = round(123.3606557) -> 123.36
correct -> newNet = round(150.5 / 1+1.215)
correct -> newNet = round(123.8683128) -> 123.87

Because tax amount is based on net, it is also wrong
taxamount = total - newNet

Wrong -> 150.5 - 123.36 = 27.14
Correct -> 150.5 - 123.87 = 26.63

Proposed Solution

round in the correct place when net is being calculated

do
newNet = round(totalGross / (1+rate))
instead of
newNet = round(totalGross / round(1+rate))

Additional Information

Tags

No tags attached.

Relationships

depends on	backport	0033985	RR16Q3.2	closed	Retail	Docuemnt level taxes calculation fails when tax rate has decimals
depends on	backport	0033986	RR16Q2.3	closed	Retail	Docuemnt level taxes calculation fails when tax rate has decimals
related to	defect	0034084		new	Retail	Pending test to be developed