Rushabh Mehta | b3c8f44 | 2017-06-21 17:22:38 +0530 | [diff] [blame] | 1 | import frappe, re |
| 2 | from frappe import _ |
Ranjith Kurungadam | a8e047a | 2018-06-14 17:56:16 +0530 | [diff] [blame] | 3 | from frappe.utils import cstr, flt, date_diff, getdate |
Rushabh Mehta | b3c8f44 | 2017-06-21 17:22:38 +0530 | [diff] [blame] | 4 | from erpnext.regional.india import states, state_numbers |
Nabin Hait | b962fc1 | 2017-07-17 18:02:31 +0530 | [diff] [blame] | 5 | from erpnext.controllers.taxes_and_totals import get_itemised_tax, get_itemised_taxable_amount |
Shreya Shah | 4fa600a | 2018-06-05 11:27:53 +0530 | [diff] [blame] | 6 | from erpnext.controllers.accounts_controller import get_taxes_and_charges |
Ranjith Kurungadam | a8e047a | 2018-06-14 17:56:16 +0530 | [diff] [blame] | 7 | from erpnext.hr.utils import get_salary_assignment |
| 8 | from erpnext.hr.doctype.salary_structure.salary_structure import make_salary_slip |
Rushabh Mehta | b3c8f44 | 2017-06-21 17:22:38 +0530 | [diff] [blame] | 9 | |
| 10 | def validate_gstin_for_india(doc, method): |
| 11 | if not hasattr(doc, 'gstin'): |
| 12 | return |
| 13 | |
| 14 | if doc.gstin: |
Nabin Hait | f3f0dfe | 2017-07-06 14:49:34 +0530 | [diff] [blame] | 15 | doc.gstin = doc.gstin.upper() |
rohitwaghchaure | 2c111b7 | 2018-04-11 15:50:06 +0530 | [diff] [blame] | 16 | if doc.gstin not in ["NA", "na"]: |
Aditya Duggal | f1bd39c | 2017-06-29 14:25:19 +0530 | [diff] [blame] | 17 | p = re.compile("[0-9]{2}[a-zA-Z]{5}[0-9]{4}[a-zA-Z]{1}[1-9A-Za-z]{1}[Z]{1}[0-9a-zA-Z]{1}") |
| 18 | if not p.match(doc.gstin): |
| 19 | frappe.throw(_("Invalid GSTIN or Enter NA for Unregistered")) |
Rushabh Mehta | b3c8f44 | 2017-06-21 17:22:38 +0530 | [diff] [blame] | 20 | |
Nabin Hait | 35cd1d3 | 2017-11-28 13:01:01 +0530 | [diff] [blame] | 21 | if not doc.gst_state: |
| 22 | if doc.state in states: |
| 23 | doc.gst_state = doc.state |
Rushabh Mehta | b3c8f44 | 2017-06-21 17:22:38 +0530 | [diff] [blame] | 24 | |
Nabin Hait | 35cd1d3 | 2017-11-28 13:01:01 +0530 | [diff] [blame] | 25 | if doc.gst_state: |
| 26 | doc.gst_state_number = state_numbers[doc.gst_state] |
| 27 | if doc.gstin and doc.gstin != "NA" and doc.gst_state_number != doc.gstin[:2]: |
| 28 | frappe.throw(_("First 2 digits of GSTIN should match with State number {0}") |
| 29 | .format(doc.gst_state_number)) |
Rushabh Mehta | 7231f29 | 2017-07-13 15:00:56 +0530 | [diff] [blame] | 30 | |
Nabin Hait | b962fc1 | 2017-07-17 18:02:31 +0530 | [diff] [blame] | 31 | def get_itemised_tax_breakup_header(item_doctype, tax_accounts): |
| 32 | if frappe.get_meta(item_doctype).has_field('gst_hsn_code'): |
| 33 | return [_("HSN/SAC"), _("Taxable Amount")] + tax_accounts |
| 34 | else: |
| 35 | return [_("Item"), _("Taxable Amount")] + tax_accounts |
Nabin Hait | b95ecd7 | 2018-02-16 13:19:04 +0530 | [diff] [blame] | 36 | |
Nabin Hait | b962fc1 | 2017-07-17 18:02:31 +0530 | [diff] [blame] | 37 | def get_itemised_tax_breakup_data(doc): |
| 38 | itemised_tax = get_itemised_tax(doc.taxes) |
| 39 | |
| 40 | itemised_taxable_amount = get_itemised_taxable_amount(doc.items) |
Nabin Hait | b95ecd7 | 2018-02-16 13:19:04 +0530 | [diff] [blame] | 41 | |
Nabin Hait | b962fc1 | 2017-07-17 18:02:31 +0530 | [diff] [blame] | 42 | if not frappe.get_meta(doc.doctype + " Item").has_field('gst_hsn_code'): |
| 43 | return itemised_tax, itemised_taxable_amount |
| 44 | |
| 45 | item_hsn_map = frappe._dict() |
| 46 | for d in doc.items: |
| 47 | item_hsn_map.setdefault(d.item_code or d.item_name, d.get("gst_hsn_code")) |
| 48 | |
| 49 | hsn_tax = {} |
| 50 | for item, taxes in itemised_tax.items(): |
| 51 | hsn_code = item_hsn_map.get(item) |
| 52 | hsn_tax.setdefault(hsn_code, frappe._dict()) |
| 53 | for tax_account, tax_detail in taxes.items(): |
| 54 | hsn_tax[hsn_code].setdefault(tax_account, {"tax_rate": 0, "tax_amount": 0}) |
| 55 | hsn_tax[hsn_code][tax_account]["tax_rate"] = tax_detail.get("tax_rate") |
| 56 | hsn_tax[hsn_code][tax_account]["tax_amount"] += tax_detail.get("tax_amount") |
| 57 | |
| 58 | # set taxable amount |
| 59 | hsn_taxable_amount = frappe._dict() |
| 60 | for item, taxable_amount in itemised_taxable_amount.items(): |
| 61 | hsn_code = item_hsn_map.get(item) |
| 62 | hsn_taxable_amount.setdefault(hsn_code, 0) |
| 63 | hsn_taxable_amount[hsn_code] += itemised_taxable_amount.get(item) |
| 64 | |
| 65 | return hsn_tax, hsn_taxable_amount |
| 66 | |
Shreya Shah | 4fa600a | 2018-06-05 11:27:53 +0530 | [diff] [blame] | 67 | def set_place_of_supply(doc, method=None): |
| 68 | doc.place_of_supply = get_place_of_supply(doc, doc.doctype) |
Nabin Hait | b95ecd7 | 2018-02-16 13:19:04 +0530 | [diff] [blame] | 69 | |
Rushabh Mehta | 7231f29 | 2017-07-13 15:00:56 +0530 | [diff] [blame] | 70 | # don't remove this function it is used in tests |
| 71 | def test_method(): |
| 72 | '''test function''' |
Nabin Hait | b95ecd7 | 2018-02-16 13:19:04 +0530 | [diff] [blame] | 73 | return 'overridden' |
Shreya Shah | 4fa600a | 2018-06-05 11:27:53 +0530 | [diff] [blame] | 74 | |
| 75 | def get_place_of_supply(out, doctype): |
| 76 | if not frappe.get_meta('Address').has_field('gst_state'): return |
| 77 | |
| 78 | if doctype in ("Sales Invoice", "Delivery Note"): |
| 79 | address_name = out.shipping_address_name or out.customer_address |
| 80 | elif doctype == "Purchase Invoice": |
| 81 | address_name = out.shipping_address or out.supplier_address |
| 82 | |
| 83 | if address_name: |
| 84 | address = frappe.db.get_value("Address", address_name, ["gst_state", "gst_state_number"], as_dict=1) |
Nabin Hait | 2390da6 | 2018-08-30 16:16:35 +0530 | [diff] [blame] | 85 | if address.gst_state and address.gst_state_number: |
| 86 | return cstr(address.gst_state_number) + "-" + cstr(address.gst_state) |
Shreya Shah | 4fa600a | 2018-06-05 11:27:53 +0530 | [diff] [blame] | 87 | |
| 88 | def get_regional_address_details(out, doctype, company): |
Shreya Shah | 4fa600a | 2018-06-05 11:27:53 +0530 | [diff] [blame] | 89 | out.place_of_supply = get_place_of_supply(out, doctype) |
| 90 | |
| 91 | if not out.place_of_supply: return |
| 92 | |
| 93 | if doctype in ("Sales Invoice", "Delivery Note"): |
| 94 | master_doctype = "Sales Taxes and Charges Template" |
Shreya | 2ad8172 | 2018-06-05 16:49:29 +0530 | [diff] [blame] | 95 | if not out.company_gstin: |
Shreya Shah | 4fa600a | 2018-06-05 11:27:53 +0530 | [diff] [blame] | 96 | return |
Shreya | 2ad8172 | 2018-06-05 16:49:29 +0530 | [diff] [blame] | 97 | elif doctype == "Purchase Invoice": |
Shreya Shah | 4fa600a | 2018-06-05 11:27:53 +0530 | [diff] [blame] | 98 | master_doctype = "Purchase Taxes and Charges Template" |
Shreya | 2ad8172 | 2018-06-05 16:49:29 +0530 | [diff] [blame] | 99 | if not out.supplier_gstin: |
Shreya Shah | 4fa600a | 2018-06-05 11:27:53 +0530 | [diff] [blame] | 100 | return |
| 101 | |
Rohit Waghchaure | de82ad1 | 2018-06-07 13:20:57 +0530 | [diff] [blame] | 102 | if ((doctype in ("Sales Invoice", "Delivery Note") and out.company_gstin |
| 103 | and out.company_gstin[:2] != out.place_of_supply[:2]) or (doctype == "Purchase Invoice" |
| 104 | and out.supplier_gstin and out.supplier_gstin[:2] != out.place_of_supply[:2])): |
Shreya Shah | 4fa600a | 2018-06-05 11:27:53 +0530 | [diff] [blame] | 105 | default_tax = frappe.db.get_value(master_doctype, {"company": company, "is_inter_state":1, "disabled":0}) |
| 106 | else: |
| 107 | default_tax = frappe.db.get_value(master_doctype, {"company": company, "disabled":0, "is_default": 1}) |
| 108 | |
| 109 | if not default_tax: |
| 110 | return |
| 111 | out["taxes_and_charges"] = default_tax |
| 112 | out.taxes = get_taxes_and_charges(master_doctype, default_tax) |
Ranjith Kurungadam | a8e047a | 2018-06-14 17:56:16 +0530 | [diff] [blame] | 113 | |
| 114 | def calculate_annual_eligible_hra_exemption(doc): |
Rushabh Mehta | 708e47a | 2018-08-08 16:37:31 +0530 | [diff] [blame] | 115 | basic_component = frappe.get_cached_value('Company', doc.company, "basic_component") |
| 116 | hra_component = frappe.get_cached_value('Company', doc.company, "hra_component") |
Ranjith Kurungadam | a8e047a | 2018-06-14 17:56:16 +0530 | [diff] [blame] | 117 | annual_exemption, monthly_exemption, hra_amount = 0, 0, 0 |
Ranjith Kurungadam | b1a756c | 2018-07-01 16:42:38 +0530 | [diff] [blame] | 118 | if hra_component and basic_component: |
Ranjith Kurungadam | a8e047a | 2018-06-14 17:56:16 +0530 | [diff] [blame] | 119 | assignment = get_salary_assignment(doc.employee, getdate()) |
| 120 | if assignment and frappe.db.exists("Salary Detail", { |
| 121 | "parent": assignment.salary_structure, |
| 122 | "salary_component": hra_component, "parentfield": "earnings"}): |
Ranjith Kurungadam | b1a756c | 2018-07-01 16:42:38 +0530 | [diff] [blame] | 123 | basic_amount, hra_amount = get_component_amt_from_salary_slip(doc.employee, |
| 124 | assignment.salary_structure, basic_component, hra_component) |
Ranjith Kurungadam | a8e047a | 2018-06-14 17:56:16 +0530 | [diff] [blame] | 125 | if hra_amount: |
| 126 | if doc.monthly_house_rent: |
| 127 | annual_exemption = calculate_hra_exemption(assignment.salary_structure, |
Ranjith Kurungadam | b1a756c | 2018-07-01 16:42:38 +0530 | [diff] [blame] | 128 | basic_amount, hra_amount, doc.monthly_house_rent, |
Ranjith Kurungadam | a8e047a | 2018-06-14 17:56:16 +0530 | [diff] [blame] | 129 | doc.rented_in_metro_city) |
| 130 | if annual_exemption > 0: |
| 131 | monthly_exemption = annual_exemption / 12 |
| 132 | else: |
| 133 | annual_exemption = 0 |
| 134 | return {"hra_amount": hra_amount, "annual_exemption": annual_exemption, "monthly_exemption": monthly_exemption} |
| 135 | |
Ranjith Kurungadam | b1a756c | 2018-07-01 16:42:38 +0530 | [diff] [blame] | 136 | def get_component_amt_from_salary_slip(employee, salary_structure, basic_component, hra_component): |
Ranjith Kurungadam | a8e047a | 2018-06-14 17:56:16 +0530 | [diff] [blame] | 137 | salary_slip = make_salary_slip(salary_structure, employee=employee) |
Ranjith Kurungadam | b1a756c | 2018-07-01 16:42:38 +0530 | [diff] [blame] | 138 | basic_amt, hra_amt = 0, 0 |
Ranjith Kurungadam | a8e047a | 2018-06-14 17:56:16 +0530 | [diff] [blame] | 139 | for earning in salary_slip.earnings: |
Ranjith Kurungadam | b1a756c | 2018-07-01 16:42:38 +0530 | [diff] [blame] | 140 | if earning.salary_component == basic_component: |
| 141 | basic_amt = earning.amount |
| 142 | elif earning.salary_component == hra_component: |
| 143 | hra_amt = earning.amount |
| 144 | if basic_amt and hra_amt: |
| 145 | return basic_amt, hra_amt |
Ranjith Kurungadam | 14e94f8 | 2018-07-16 16:12:46 +0530 | [diff] [blame] | 146 | return basic_amt, hra_amt |
Ranjith Kurungadam | a8e047a | 2018-06-14 17:56:16 +0530 | [diff] [blame] | 147 | |
Ranjith Kurungadam | b1a756c | 2018-07-01 16:42:38 +0530 | [diff] [blame] | 148 | def calculate_hra_exemption(salary_structure, basic, monthly_hra, monthly_house_rent, rented_in_metro_city): |
Ranjith Kurungadam | a8e047a | 2018-06-14 17:56:16 +0530 | [diff] [blame] | 149 | # TODO make this configurable |
| 150 | exemptions = [] |
| 151 | frequency = frappe.get_value("Salary Structure", salary_structure, "payroll_frequency") |
| 152 | # case 1: The actual amount allotted by the employer as the HRA. |
| 153 | exemptions.append(get_annual_component_pay(frequency, monthly_hra)) |
| 154 | actual_annual_rent = monthly_house_rent * 12 |
Ranjith Kurungadam | b1a756c | 2018-07-01 16:42:38 +0530 | [diff] [blame] | 155 | annual_basic = get_annual_component_pay(frequency, basic) |
Ranjith Kurungadam | a8e047a | 2018-06-14 17:56:16 +0530 | [diff] [blame] | 156 | # case 2: Actual rent paid less 10% of the basic salary. |
Ranjith Kurungadam | b1a756c | 2018-07-01 16:42:38 +0530 | [diff] [blame] | 157 | exemptions.append(flt(actual_annual_rent) - flt(annual_basic * 0.1)) |
Ranjith Kurungadam | a8e047a | 2018-06-14 17:56:16 +0530 | [diff] [blame] | 158 | # case 3: 50% of the basic salary, if the employee is staying in a metro city (40% for a non-metro city). |
Ranjith Kurungadam | b1a756c | 2018-07-01 16:42:38 +0530 | [diff] [blame] | 159 | exemptions.append(annual_basic * 0.5 if rented_in_metro_city else annual_basic * 0.4) |
Ranjith Kurungadam | a8e047a | 2018-06-14 17:56:16 +0530 | [diff] [blame] | 160 | # return minimum of 3 cases |
| 161 | return min(exemptions) |
| 162 | |
| 163 | def get_annual_component_pay(frequency, amount): |
| 164 | if frequency == "Daily": |
| 165 | return amount * 365 |
| 166 | elif frequency == "Weekly": |
| 167 | return amount * 52 |
| 168 | elif frequency == "Fortnightly": |
| 169 | return amount * 26 |
| 170 | elif frequency == "Monthly": |
| 171 | return amount * 12 |
| 172 | elif frequency == "Bimonthly": |
| 173 | return amount * 6 |
| 174 | |
| 175 | def validate_house_rent_dates(doc): |
| 176 | if not doc.rented_to_date or not doc.rented_from_date: |
| 177 | frappe.throw(_("House rented dates required for exemption calculation")) |
| 178 | if date_diff(doc.rented_to_date, doc.rented_from_date) < 14: |
| 179 | frappe.throw(_("House rented dates should be atleast 15 days apart")) |
| 180 | proofs = frappe.db.sql("""select name from `tabEmployee Tax Exemption Proof Submission` |
| 181 | where docstatus=1 and employee='{0}' and payroll_period='{1}' and |
| 182 | (rented_from_date between '{2}' and '{3}' or rented_to_date between |
| 183 | '{2}' and '{3}')""".format(doc.employee, doc.payroll_period, |
| 184 | doc.rented_from_date, doc.rented_to_date)) |
| 185 | if proofs: |
| 186 | frappe.throw(_("House rent paid days overlap with {0}").format(proofs[0][0])) |
| 187 | |
| 188 | def calculate_hra_exemption_for_period(doc): |
| 189 | monthly_rent, eligible_hra = 0, 0 |
| 190 | if doc.house_rent_payment_amount: |
| 191 | validate_house_rent_dates(doc) |
| 192 | # TODO receive rented months or validate dates are start and end of months? |
| 193 | # Calc monthly rent, round to nearest .5 |
| 194 | factor = flt(date_diff(doc.rented_to_date, doc.rented_from_date) + 1)/30 |
| 195 | factor = round(factor * 2)/2 |
| 196 | monthly_rent = doc.house_rent_payment_amount / factor |
| 197 | # update field used by calculate_annual_eligible_hra_exemption |
| 198 | doc.monthly_house_rent = monthly_rent |
| 199 | exemptions = calculate_annual_eligible_hra_exemption(doc) |
| 200 | |
| 201 | if exemptions["monthly_exemption"]: |
| 202 | # calc total exemption amount |
| 203 | eligible_hra = exemptions["monthly_exemption"] * factor |
Ranjith Kurungadam | 4f9744a | 2018-06-20 11:10:56 +0530 | [diff] [blame] | 204 | exemptions["monthly_house_rent"] = monthly_rent |
| 205 | exemptions["total_eligible_hra_exemption"] = eligible_hra |
| 206 | return exemptions |