schuetz3/schuetz3-erpnext
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: use get_all instead of get_value as get_value api dont supports between condition

Browse Source
This commit is contained in:
Saurabh
2022-12-26 12:08:44 +05:30
parent c3b9059c1b
commit bc04e05b46
1 changed files with 16 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
erpnext/payroll/doctype/salary_slip/salary_slip.py
View File

@@ -1063,27 +1063,26 @@ class SalarySlip(TransactionBase):
)
exempted_amount = flt(exempted_amount[0][0]) if exempted_amount else 0
opening_taxable_earning = self.get_opening_for(
opening_taxable_earning = self.get_opening_balance_for(
"taxable_earnings_till_date", start_date, end_date
)
return (taxable_earnings + opening_taxable_earning) - exempted_amount
def get_opening_for(self, field_to_select, start_date, end_date):
return (
frappe.db.get_value(
"Salary Structure Assignment",
{
"employee": self.employee,
"salary_structure": self.salary_structure,
"from_date": ["between", [start_date, end_date]],
"docstatus": 1,
},
field_to_select,
)
or 0
def get_opening_balance_for(self, field_to_select, start_date, end_date):
opening_balance = frappe.db.get_all(
"Salary Structure Assignment",
{
"employee": self.employee,
"salary_structure": self.salary_structure,
"from_date": ["between", (start_date, end_date)],
"docstatus": 1,
},
field_to_select,
)
return opening_balance[0].get(field_to_select) if opening_balance else 0.0
def get_tax_paid_in_period(self, start_date, end_date, tax_component):
# find total_tax_paid, tax paid for benefit, additional_salary
total_tax_paid = flt(
@@ -1111,7 +1110,9 @@ class SalarySlip(TransactionBase):
)[0][0]
)
tax_deducted_till_date = self.get_opening_for("tax_deducted_till_date", start_date, end_date)
tax_deducted_till_date = self.get_opening_balance_for(
"tax_deducted_till_date", start_date, end_date
)
return total_tax_paid + tax_deducted_till_date