fix: Run income-tax-slab patch only if slab already exists in payroll period (#21684)
This commit is contained in:
parent
e684727923
commit
85a89812a4
@ -7,7 +7,7 @@ import frappe
|
|||||||
from frappe.model.utils.rename_field import rename_field
|
from frappe.model.utils.rename_field import rename_field
|
||||||
|
|
||||||
def execute():
|
def execute():
|
||||||
if not frappe.db.table_exists("Payroll Period"):
|
if not (frappe.db.table_exists("Payroll Period") and frappe.db.table_exists("Taxable Salary Slab")):
|
||||||
return
|
return
|
||||||
|
|
||||||
for doctype in ("income_tax_slab", "salary_structure_assignment", "employee_other_income", "income_tax_slab_other_charges"):
|
for doctype in ("income_tax_slab", "salary_structure_assignment", "employee_other_income", "income_tax_slab_other_charges"):
|
||||||
@ -60,6 +60,9 @@ def execute():
|
|||||||
""", (income_tax_slab.name, company.name, period.start_date))
|
""", (income_tax_slab.name, company.name, period.start_date))
|
||||||
|
|
||||||
# move other incomes to separate document
|
# move other incomes to separate document
|
||||||
|
if not frappe.db.table_exists("Employee Tax Exemption Proof Submission"):
|
||||||
|
return
|
||||||
|
|
||||||
migrated = []
|
migrated = []
|
||||||
proofs = frappe.get_all("Employee Tax Exemption Proof Submission",
|
proofs = frappe.get_all("Employee Tax Exemption Proof Submission",
|
||||||
filters = {'docstatus': 1},
|
filters = {'docstatus': 1},
|
||||||
@ -79,6 +82,9 @@ def execute():
|
|||||||
except:
|
except:
|
||||||
pass
|
pass
|
||||||
|
|
||||||
|
if not frappe.db.table_exists("Employee Tax Exemption Declaration"):
|
||||||
|
return
|
||||||
|
|
||||||
declerations = frappe.get_all("Employee Tax Exemption Declaration",
|
declerations = frappe.get_all("Employee Tax Exemption Declaration",
|
||||||
filters = {'docstatus': 1},
|
filters = {'docstatus': 1},
|
||||||
fields =['payroll_period', 'employee', 'company', 'income_from_other_sources']
|
fields =['payroll_period', 'employee', 'company', 'income_from_other_sources']
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user