calculate_tax_by_tax_slab has bug - adding 1 for nothing

[tinodj]

Information about bug

While it might working adding 1 for some currencies, with EUR this makes inconsistencies.

1 should be removed and is not necessary - it calculates wrongly.

Imagine the following brackets/slabs:

0-100 10%
100-200 20%

Someone getting 200 income tax should be: 30.

However, this function calculates: 30.3

In EUR (and many other currencies) very often these slabs are defined on two decimal places.

hrms/hrms/payroll/doctype/salary_slip/salary_slip.py

Line 2091 in 9d8c78d

def calculate_tax_by_tax_slab(

Module

HR, Payroll

Version

ERPNext: v15.14.4 (version-15)

Frappe Framework: v15.15.0 (version-15)

Frappe HR: v15.12.0 (version-15)

Installation method

None

Relevant log output / Stack trace / Full Error Message.

No response

Code of Conduct

• I agree to follow this project's Code of Conduct