Merge pull request #30573 from frappe/mergify/bp/develop/pr-30382

fix: Added validation for single_threshold in Tax With Holding Category (backport #30382)
This commit is contained in:
Saqib Ansari 2022-04-05 15:52:28 +05:30 committed by GitHub
commit c4b5fa8ed1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -34,7 +34,9 @@ class TaxWithholdingCategory(Document):
def validate_thresholds(self):
for d in self.get("rates"):
if d.cumulative_threshold and d.cumulative_threshold < d.single_threshold:
if (
d.cumulative_threshold and d.single_threshold and d.cumulative_threshold < d.single_threshold
):
frappe.throw(
_("Row #{0}: Cumulative threshold cannot be less than Single Transaction threshold").format(
d.idx