Anand Doshi | 885e074 | 2015-03-03 14:55:30 +0530 | [diff] [blame] | 1 | # Copyright (c) 2015, Frappe Technologies Pvt. Ltd. and Contributors |
Rushabh Mehta | e67d1fb | 2013-08-05 14:59:54 +0530 | [diff] [blame] | 2 | # License: GNU General Public License v3. See license.txt |
Nabin Hait | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 3 | |
| 4 | from __future__ import unicode_literals |
Rohit Waghchaure | e9ff191 | 2017-06-19 12:54:59 +0530 | [diff] [blame] | 5 | import frappe, erpnext |
rohitwaghchaure | 28a4880 | 2020-04-28 13:01:43 +0530 | [diff] [blame] | 6 | from frappe.utils import cint, flt, cstr, get_link_to_form, today, getdate |
Aditya Hase | f79937d | 2019-01-23 00:28:37 +0530 | [diff] [blame] | 7 | from frappe import _ |
Rushabh Mehta | 793ba6b | 2014-02-14 15:47:51 +0530 | [diff] [blame] | 8 | import frappe.defaults |
Valmik Jangla | d89dcf6 | 2016-03-04 14:33:49 +0530 | [diff] [blame] | 9 | from erpnext.accounts.utils import get_fiscal_year |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 10 | from erpnext.accounts.general_ledger import make_gl_entries, make_reverse_gl_entries, process_gl_map |
Rushabh Mehta | 1f84799 | 2013-12-12 19:12:19 +0530 | [diff] [blame] | 11 | from erpnext.controllers.accounts_controller import AccountsController |
Rohit Waghchaure | 3899d74 | 2017-01-25 18:47:53 +0530 | [diff] [blame] | 12 | from erpnext.stock.stock_ledger import get_valuation_rate |
Nabin Hait | 6d7b0ce | 2017-06-15 11:09:27 +0530 | [diff] [blame] | 13 | from erpnext.stock import get_warehouse_account_map |
Nabin Hait | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 14 | |
Nabin Hait | 5a9579b | 2018-12-24 14:54:42 +0530 | [diff] [blame] | 15 | class QualityInspectionRequiredError(frappe.ValidationError): pass |
| 16 | class QualityInspectionRejectedError(frappe.ValidationError): pass |
Anurag Mishra | 0c8411d | 2018-12-28 16:53:00 +0530 | [diff] [blame] | 17 | class QualityInspectionNotSubmittedError(frappe.ValidationError): pass |
Nabin Hait | 5a9579b | 2018-12-24 14:54:42 +0530 | [diff] [blame] | 18 | |
Nabin Hait | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 19 | class StockController(AccountsController): |
Nabin Hait | 8af429d | 2016-11-16 17:21:59 +0530 | [diff] [blame] | 20 | def validate(self): |
| 21 | super(StockController, self).validate() |
Marica | fd3d976 | 2020-06-12 12:30:59 +0530 | [diff] [blame] | 22 | if not self.get('is_return'): |
marination | 596560c | 2020-06-11 16:39:03 +0530 | [diff] [blame] | 23 | self.validate_inspection() |
rohitwaghchaure | 9af557f | 2019-12-30 13:26:47 +0530 | [diff] [blame] | 24 | self.validate_serialized_batch() |
marination | fd04e96 | 2020-04-03 15:46:48 +0530 | [diff] [blame] | 25 | self.validate_customer_provided_item() |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 26 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 27 | def make_gl_entries(self, gl_entries=None): |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 28 | if self.docstatus == 2: |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 29 | make_reverse_gl_entries(voucher_type=self.doctype, voucher_no=self.name) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 30 | |
Rohit Waghchaure | e9ff191 | 2017-06-19 12:54:59 +0530 | [diff] [blame] | 31 | if cint(erpnext.is_perpetual_inventory_enabled(self.company)): |
Rohit Waghchaure | 6b33c9b | 2019-03-08 11:13:35 +0530 | [diff] [blame] | 32 | warehouse_account = get_warehouse_account_map(self.company) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 33 | |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 34 | if self.docstatus==1: |
Nabin Hait | 9784d27 | 2016-12-30 16:21:35 +0530 | [diff] [blame] | 35 | if not gl_entries: |
| 36 | gl_entries = self.get_gl_entries(warehouse_account) |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 37 | make_gl_entries(gl_entries) |
Nabin Hait | 145e5e2 | 2013-10-22 23:51:41 +0530 | [diff] [blame] | 38 | |
Rohit Waghchaure | b6381c6 | 2018-06-12 13:54:40 +0530 | [diff] [blame] | 39 | elif self.doctype in ['Purchase Receipt', 'Purchase Invoice'] and self.docstatus == 1: |
Rohit Waghchaure | 4216972 | 2018-05-16 18:16:08 +0530 | [diff] [blame] | 40 | gl_entries = [] |
| 41 | gl_entries = self.get_asset_gl_entry(gl_entries) |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 42 | make_gl_entries(gl_entries) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 43 | |
rohitwaghchaure | 9af557f | 2019-12-30 13:26:47 +0530 | [diff] [blame] | 44 | def validate_serialized_batch(self): |
| 45 | from erpnext.stock.doctype.serial_no.serial_no import get_serial_nos |
| 46 | for d in self.get("items"): |
| 47 | if hasattr(d, 'serial_no') and hasattr(d, 'batch_no') and d.serial_no and d.batch_no: |
| 48 | serial_nos = get_serial_nos(d.serial_no) |
| 49 | for serial_no_data in frappe.get_all("Serial No", |
| 50 | filters={"name": ("in", serial_nos)}, fields=["batch_no", "name"]): |
| 51 | if serial_no_data.batch_no != d.batch_no: |
| 52 | frappe.throw(_("Row #{0}: Serial No {1} does not belong to Batch {2}") |
| 53 | .format(d.idx, serial_no_data.name, d.batch_no)) |
| 54 | |
rohitwaghchaure | 28a4880 | 2020-04-28 13:01:43 +0530 | [diff] [blame] | 55 | if d.qty > 0 and d.get("batch_no") and self.get("posting_date") and self.docstatus < 2: |
| 56 | expiry_date = frappe.get_cached_value("Batch", d.get("batch_no"), "expiry_date") |
| 57 | |
| 58 | if expiry_date and getdate(expiry_date) < getdate(self.posting_date): |
| 59 | frappe.throw(_("Row #{0}: The batch {1} has already expired.") |
| 60 | .format(d.idx, get_link_to_form("Batch", d.get("batch_no")))) |
| 61 | |
Nabin Hait | e83069c | 2013-11-14 18:40:08 +0530 | [diff] [blame] | 62 | def get_gl_entries(self, warehouse_account=None, default_expense_account=None, |
Nabin Hait | 142007a | 2013-09-17 15:15:16 +0530 | [diff] [blame] | 63 | default_cost_center=None): |
Nabin Hait | adeb976 | 2014-10-06 11:53:52 +0530 | [diff] [blame] | 64 | |
Nabin Hait | 142007a | 2013-09-17 15:15:16 +0530 | [diff] [blame] | 65 | if not warehouse_account: |
Rohit Waghchaure | 6b33c9b | 2019-03-08 11:13:35 +0530 | [diff] [blame] | 66 | warehouse_account = get_warehouse_account_map(self.company) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 67 | |
Anand Doshi | de1a97d | 2014-04-17 11:37:46 +0530 | [diff] [blame] | 68 | sle_map = self.get_stock_ledger_details() |
| 69 | voucher_details = self.get_voucher_details(default_expense_account, default_cost_center, sle_map) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 70 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 71 | gl_list = [] |
Nabin Hait | 7a75e10 | 2013-09-17 10:21:20 +0530 | [diff] [blame] | 72 | warehouse_with_no_account = [] |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 73 | |
rohitwaghchaure | 61ad629 | 2020-01-15 16:35:31 +0530 | [diff] [blame] | 74 | precision = frappe.get_precision("GL Entry", "debit_in_account_currency") |
Nabin Hait | 8c61f34 | 2016-12-15 13:46:03 +0530 | [diff] [blame] | 75 | for item_row in voucher_details: |
| 76 | sle_list = sle_map.get(item_row.name) |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 77 | if sle_list: |
| 78 | for sle in sle_list: |
| 79 | if warehouse_account.get(sle.warehouse): |
Deepesh Garg | 15ff6a5 | 2020-02-18 12:28:41 +0530 | [diff] [blame] | 80 | # from warehouse account/ target warehouse account |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 81 | |
Nabin Hait | 8c61f34 | 2016-12-15 13:46:03 +0530 | [diff] [blame] | 82 | self.check_expense_account(item_row) |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 83 | |
Javier Wong | 9b11d9b | 2017-04-14 18:24:04 +0800 | [diff] [blame] | 84 | # If the item does not have the allow zero valuation rate flag set |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 85 | # and ( valuation rate not mentioned in an incoming entry |
Rushabh Mehta | cc8b2b2 | 2017-03-31 12:44:29 +0530 | [diff] [blame] | 86 | # or incoming entry not found while delivering the item), |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 87 | # try to pick valuation rate from previous sle or Item master and update in SLE |
| 88 | # Otherwise, throw an exception |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 89 | |
| 90 | if not sle.stock_value_difference and self.doctype != "Stock Reconciliation" \ |
Javier Wong | 9b11d9b | 2017-04-14 18:24:04 +0800 | [diff] [blame] | 91 | and not item_row.get("allow_zero_valuation_rate"): |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 92 | |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 93 | sle = self.update_stock_ledger_entries(sle) |
Rushabh Mehta | 14a908b | 2015-10-15 12:28:20 +0530 | [diff] [blame] | 94 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 95 | gl_list.append(self.get_gl_dict({ |
Nabin Hait | 6d7b0ce | 2017-06-15 11:09:27 +0530 | [diff] [blame] | 96 | "account": warehouse_account[sle.warehouse]["account"], |
Nabin Hait | 8c61f34 | 2016-12-15 13:46:03 +0530 | [diff] [blame] | 97 | "against": item_row.expense_account, |
| 98 | "cost_center": item_row.cost_center, |
Deepesh Garg | 1b30ca6 | 2020-06-19 12:12:08 +0530 | [diff] [blame] | 99 | "project": item_row.project or self.get('project'), |
Nabin Hait | dc82d4f | 2014-04-07 12:02:57 +0530 | [diff] [blame] | 100 | "remarks": self.get("remarks") or "Accounting Entry for Stock", |
rohitwaghchaure | 61ad629 | 2020-01-15 16:35:31 +0530 | [diff] [blame] | 101 | "debit": flt(sle.stock_value_difference, precision), |
Nabin Hait | 8770ea2 | 2019-05-22 18:03:09 +0530 | [diff] [blame] | 102 | "is_opening": item_row.get("is_opening") or self.get("is_opening") or "No", |
deepeshgarg007 | 3d11ac0 | 2019-05-19 00:02:01 +0530 | [diff] [blame] | 103 | }, warehouse_account[sle.warehouse]["account_currency"], item=item_row)) |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 104 | |
Deepesh Garg | 15ff6a5 | 2020-02-18 12:28:41 +0530 | [diff] [blame] | 105 | # expense account |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 106 | gl_list.append(self.get_gl_dict({ |
Nabin Hait | 8c61f34 | 2016-12-15 13:46:03 +0530 | [diff] [blame] | 107 | "account": item_row.expense_account, |
Nabin Hait | 6d7b0ce | 2017-06-15 11:09:27 +0530 | [diff] [blame] | 108 | "against": warehouse_account[sle.warehouse]["account"], |
Nabin Hait | 8c61f34 | 2016-12-15 13:46:03 +0530 | [diff] [blame] | 109 | "cost_center": item_row.cost_center, |
Deepesh Garg | 1b30ca6 | 2020-06-19 12:12:08 +0530 | [diff] [blame] | 110 | "project": item_row.project or self.get('project'), |
Nabin Hait | dc82d4f | 2014-04-07 12:02:57 +0530 | [diff] [blame] | 111 | "remarks": self.get("remarks") or "Accounting Entry for Stock", |
rohitwaghchaure | 61ad629 | 2020-01-15 16:35:31 +0530 | [diff] [blame] | 112 | "credit": flt(sle.stock_value_difference, precision), |
Nabin Hait | 3f119ec | 2019-05-16 17:28:39 +0530 | [diff] [blame] | 113 | "project": item_row.get("project") or self.get("project"), |
Nabin Hait | 8770ea2 | 2019-05-22 18:03:09 +0530 | [diff] [blame] | 114 | "is_opening": item_row.get("is_opening") or self.get("is_opening") or "No" |
deepeshgarg007 | 3d11ac0 | 2019-05-19 00:02:01 +0530 | [diff] [blame] | 115 | }, item=item_row)) |
Nabin Hait | 7a75e10 | 2013-09-17 10:21:20 +0530 | [diff] [blame] | 116 | elif sle.warehouse not in warehouse_with_no_account: |
| 117 | warehouse_with_no_account.append(sle.warehouse) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 118 | |
| 119 | if warehouse_with_no_account: |
Nabin Hait | d662502 | 2016-10-24 18:17:57 +0530 | [diff] [blame] | 120 | for wh in warehouse_with_no_account: |
| 121 | if frappe.db.get_value("Warehouse", wh, "company"): |
Nabin Hait | 6d7b0ce | 2017-06-15 11:09:27 +0530 | [diff] [blame] | 122 | frappe.throw(_("Warehouse {0} is not linked to any account, please mention the account in the warehouse record or set default inventory account in company {1}.").format(wh, self.company)) |
Rushabh Mehta | 14a908b | 2015-10-15 12:28:20 +0530 | [diff] [blame] | 123 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 124 | return process_gl_map(gl_list) |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 125 | |
Rohit Waghchaure | 3899d74 | 2017-01-25 18:47:53 +0530 | [diff] [blame] | 126 | def update_stock_ledger_entries(self, sle): |
Rushabh Mehta | cc8b2b2 | 2017-03-31 12:44:29 +0530 | [diff] [blame] | 127 | sle.valuation_rate = get_valuation_rate(sle.item_code, sle.warehouse, |
Rohit Waghchaure | a5f4094 | 2017-06-16 15:21:36 +0530 | [diff] [blame] | 128 | self.doctype, self.name, currency=self.company_currency, company=self.company) |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 129 | |
Nabin Hait | cb19f52 | 2017-01-25 19:20:35 +0530 | [diff] [blame] | 130 | sle.stock_value = flt(sle.qty_after_transaction) * flt(sle.valuation_rate) |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 131 | sle.stock_value_difference = flt(sle.actual_qty) * flt(sle.valuation_rate) |
Rushabh Mehta | cc8b2b2 | 2017-03-31 12:44:29 +0530 | [diff] [blame] | 132 | |
Kanchan Chauhan | b77eaf7 | 2017-01-28 13:18:49 +0530 | [diff] [blame] | 133 | if sle.name: |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 134 | frappe.db.sql(""" |
Rushabh Mehta | cc8b2b2 | 2017-03-31 12:44:29 +0530 | [diff] [blame] | 135 | update |
| 136 | `tabStock Ledger Entry` |
| 137 | set |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 138 | stock_value = %(stock_value)s, |
Rushabh Mehta | cc8b2b2 | 2017-03-31 12:44:29 +0530 | [diff] [blame] | 139 | valuation_rate = %(valuation_rate)s, |
| 140 | stock_value_difference = %(stock_value_difference)s |
| 141 | where |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 142 | name = %(name)s""", (sle)) |
Rushabh Mehta | cc8b2b2 | 2017-03-31 12:44:29 +0530 | [diff] [blame] | 143 | |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 144 | return sle |
Rushabh Mehta | cc8b2b2 | 2017-03-31 12:44:29 +0530 | [diff] [blame] | 145 | |
Anand Doshi | de1a97d | 2014-04-17 11:37:46 +0530 | [diff] [blame] | 146 | def get_voucher_details(self, default_expense_account, default_cost_center, sle_map): |
| 147 | if self.doctype == "Stock Reconciliation": |
Nabin Hait | 3f119ec | 2019-05-16 17:28:39 +0530 | [diff] [blame] | 148 | reconciliation_purpose = frappe.db.get_value(self.doctype, self.name, "purpose") |
| 149 | is_opening = "Yes" if reconciliation_purpose == "Opening Stock" else "No" |
| 150 | details = [] |
Nabin Hait | 34c551d | 2019-07-03 10:34:31 +0530 | [diff] [blame] | 151 | for voucher_detail_no in sle_map: |
Nabin Hait | 3f119ec | 2019-05-16 17:28:39 +0530 | [diff] [blame] | 152 | details.append(frappe._dict({ |
| 153 | "name": voucher_detail_no, |
| 154 | "expense_account": default_expense_account, |
| 155 | "cost_center": default_cost_center, |
| 156 | "is_opening": is_opening |
| 157 | })) |
| 158 | return details |
Anand Doshi | de1a97d | 2014-04-17 11:37:46 +0530 | [diff] [blame] | 159 | else: |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 160 | details = self.get("items") |
Anand Doshi | 094610d | 2014-04-16 19:56:53 +0530 | [diff] [blame] | 161 | |
Anand Doshi | de1a97d | 2014-04-17 11:37:46 +0530 | [diff] [blame] | 162 | if default_expense_account or default_cost_center: |
| 163 | for d in details: |
| 164 | if default_expense_account and not d.get("expense_account"): |
| 165 | d.expense_account = default_expense_account |
| 166 | if default_cost_center and not d.get("cost_center"): |
| 167 | d.cost_center = default_cost_center |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 168 | |
Anand Doshi | de1a97d | 2014-04-17 11:37:46 +0530 | [diff] [blame] | 169 | return details |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 170 | |
Nabin Hait | b9e0481 | 2014-09-26 14:22:18 +0530 | [diff] [blame] | 171 | def get_items_and_warehouses(self): |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 172 | items, warehouses = [], [] |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 173 | |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 174 | if hasattr(self, "items"): |
| 175 | item_doclist = self.get("items") |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 176 | elif self.doctype == "Stock Reconciliation": |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 177 | import json |
| 178 | item_doclist = [] |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 179 | data = json.loads(self.reconciliation_json) |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 180 | for row in data[data.index(self.head_row)+1:]: |
| 181 | d = frappe._dict(zip(["item_code", "warehouse", "qty", "valuation_rate"], row)) |
| 182 | item_doclist.append(d) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 183 | |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 184 | if item_doclist: |
| 185 | for d in item_doclist: |
| 186 | if d.item_code and d.item_code not in items: |
| 187 | items.append(d.item_code) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 188 | |
Nabin Hait | dc82d4f | 2014-04-07 12:02:57 +0530 | [diff] [blame] | 189 | if d.get("warehouse") and d.warehouse not in warehouses: |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 190 | warehouses.append(d.warehouse) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 191 | |
Nabin Hait | dc82d4f | 2014-04-07 12:02:57 +0530 | [diff] [blame] | 192 | if self.doctype == "Stock Entry": |
| 193 | if d.get("s_warehouse") and d.s_warehouse not in warehouses: |
| 194 | warehouses.append(d.s_warehouse) |
| 195 | if d.get("t_warehouse") and d.t_warehouse not in warehouses: |
| 196 | warehouses.append(d.t_warehouse) |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 197 | |
Nabin Hait | b9e0481 | 2014-09-26 14:22:18 +0530 | [diff] [blame] | 198 | return items, warehouses |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 199 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 200 | def get_stock_ledger_details(self): |
| 201 | stock_ledger = {} |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 202 | stock_ledger_entries = frappe.db.sql(""" |
Rushabh Mehta | cc8b2b2 | 2017-03-31 12:44:29 +0530 | [diff] [blame] | 203 | select |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 204 | name, warehouse, stock_value_difference, valuation_rate, |
Rushabh Mehta | cc8b2b2 | 2017-03-31 12:44:29 +0530 | [diff] [blame] | 205 | voucher_detail_no, item_code, posting_date, posting_time, |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 206 | actual_qty, qty_after_transaction |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 207 | from |
| 208 | `tabStock Ledger Entry` |
| 209 | where |
| 210 | voucher_type=%s and voucher_no=%s |
| 211 | """, (self.doctype, self.name), as_dict=True) |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 212 | |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 213 | for sle in stock_ledger_entries: |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 214 | stock_ledger.setdefault(sle.voucher_detail_no, []).append(sle) |
| 215 | return stock_ledger |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 216 | |
Rushabh Mehta | 551406a | 2017-04-21 12:40:19 +0530 | [diff] [blame] | 217 | def make_batches(self, warehouse_field): |
Rushabh Mehta | e385b5b | 2017-04-20 15:21:01 +0530 | [diff] [blame] | 218 | '''Create batches if required. Called before submit''' |
| 219 | for d in self.items: |
Rushabh Mehta | 551406a | 2017-04-21 12:40:19 +0530 | [diff] [blame] | 220 | if d.get(warehouse_field) and not d.batch_no: |
| 221 | has_batch_no, create_new_batch = frappe.db.get_value('Item', d.item_code, ['has_batch_no', 'create_new_batch']) |
| 222 | if has_batch_no and create_new_batch: |
| 223 | d.batch_no = frappe.get_doc(dict( |
| 224 | doctype='Batch', |
| 225 | item=d.item_code, |
| 226 | supplier=getattr(self, 'supplier', None), |
| 227 | reference_doctype=self.doctype, |
| 228 | reference_name=self.name)).insert().name |
Rushabh Mehta | e385b5b | 2017-04-20 15:21:01 +0530 | [diff] [blame] | 229 | |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 230 | def check_expense_account(self, item): |
Rushabh Mehta | 052fe82 | 2014-04-16 19:20:11 +0530 | [diff] [blame] | 231 | if not item.get("expense_account"): |
marination | 17422a3 | 2020-06-17 13:46:21 +0530 | [diff] [blame] | 232 | frappe.throw(_("Row #{0}: Expense Account not set for Item {1}. Please set an Expense \ |
| 233 | Account in the Items table").format(item.idx, frappe.bold(item.item_code)), |
| 234 | title=_("Expense Account Missing")) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 235 | |
Anand Doshi | 496123a | 2014-06-19 19:25:19 +0530 | [diff] [blame] | 236 | else: |
Nabin Hait | 00778c9 | 2014-06-25 19:12:24 +0530 | [diff] [blame] | 237 | is_expense_account = frappe.db.get_value("Account", |
| 238 | item.get("expense_account"), "report_type")=="Profit and Loss" |
jof2jc | 524ebbc | 2016-02-08 23:44:55 +0700 | [diff] [blame] | 239 | if self.doctype not in ("Purchase Receipt", "Purchase Invoice", "Stock Reconciliation", "Stock Entry") and not is_expense_account: |
Nabin Hait | 00778c9 | 2014-06-25 19:12:24 +0530 | [diff] [blame] | 240 | frappe.throw(_("Expense / Difference account ({0}) must be a 'Profit or Loss' account") |
| 241 | .format(item.get("expense_account"))) |
Anand Doshi | 496123a | 2014-06-19 19:25:19 +0530 | [diff] [blame] | 242 | if is_expense_account and not item.get("cost_center"): |
| 243 | frappe.throw(_("{0} {1}: Cost Center is mandatory for Item {2}").format( |
| 244 | _(self.doctype), self.name, item.get("item_code"))) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 245 | |
Rohit Waghchaure | 8996b7d | 2020-01-23 17:36:52 +0530 | [diff] [blame] | 246 | def delete_auto_created_batches(self): |
| 247 | for d in self.items: |
| 248 | if not d.batch_no: continue |
| 249 | |
Nabin Hait | ddc3bb2 | 2020-03-17 17:04:18 +0530 | [diff] [blame] | 250 | serial_nos = [sr.name for sr in frappe.get_all("Serial No", {'batch_no': d.batch_no})] |
Saqib | e9ac3e0 | 2020-03-02 15:02:58 +0530 | [diff] [blame] | 251 | if serial_nos: |
| 252 | frappe.db.set_value("Serial No", { 'name': ['in', serial_nos] }, "batch_no", None) |
| 253 | |
Rohit Waghchaure | 8996b7d | 2020-01-23 17:36:52 +0530 | [diff] [blame] | 254 | d.batch_no = None |
| 255 | d.db_set("batch_no", None) |
| 256 | |
| 257 | for data in frappe.get_all("Batch", |
| 258 | {'reference_name': self.name, 'reference_doctype': self.doctype}): |
| 259 | frappe.delete_doc("Batch", data.name) |
Rohit Waghchaure | 4f4dbf1 | 2020-01-23 12:42:42 +0530 | [diff] [blame] | 260 | |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 261 | def get_sl_entries(self, d, args): |
Nabin Hait | 5288bde | 2014-11-03 15:08:21 +0530 | [diff] [blame] | 262 | sl_dict = frappe._dict({ |
Nabin Hait | 296d626 | 2014-05-02 17:36:13 +0530 | [diff] [blame] | 263 | "item_code": d.get("item_code", None), |
Nabin Hait | dc82d4f | 2014-04-07 12:02:57 +0530 | [diff] [blame] | 264 | "warehouse": d.get("warehouse", None), |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 265 | "posting_date": self.posting_date, |
| 266 | "posting_time": self.posting_time, |
Valmik Jangla | d89dcf6 | 2016-03-04 14:33:49 +0530 | [diff] [blame] | 267 | 'fiscal_year': get_fiscal_year(self.posting_date, company=self.company)[0], |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 268 | "voucher_type": self.doctype, |
| 269 | "voucher_no": self.name, |
Nabin Hait | 1e2f20a | 2013-08-02 11:42:11 +0530 | [diff] [blame] | 270 | "voucher_detail_no": d.name, |
Nabin Hait | dc82d4f | 2014-04-07 12:02:57 +0530 | [diff] [blame] | 271 | "actual_qty": (self.docstatus==1 and 1 or -1)*flt(d.get("stock_qty")), |
Nabin Hait | c865f22 | 2015-09-21 09:18:43 +0530 | [diff] [blame] | 272 | "stock_uom": frappe.db.get_value("Item", args.get("item_code") or d.get("item_code"), "stock_uom"), |
Nabin Hait | 1e2f20a | 2013-08-02 11:42:11 +0530 | [diff] [blame] | 273 | "incoming_rate": 0, |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 274 | "company": self.company, |
Nabin Hait | 296d626 | 2014-05-02 17:36:13 +0530 | [diff] [blame] | 275 | "batch_no": cstr(d.get("batch_no")).strip(), |
| 276 | "serial_no": d.get("serial_no"), |
rohitwaghchaure | 93779c2 | 2018-03-21 17:52:41 +0530 | [diff] [blame] | 277 | "project": d.get("project") or self.get('project'), |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 278 | "is_cancelled": 1 if self.docstatus==2 else 0 |
Nabin Hait | 5288bde | 2014-11-03 15:08:21 +0530 | [diff] [blame] | 279 | }) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 280 | |
Nabin Hait | 1e2f20a | 2013-08-02 11:42:11 +0530 | [diff] [blame] | 281 | sl_dict.update(args) |
| 282 | return sl_dict |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 283 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 284 | def make_sl_entries(self, sl_entries, allow_negative_stock=False, |
Nabin Hait | 54c865e | 2015-03-27 15:38:31 +0530 | [diff] [blame] | 285 | via_landed_cost_voucher=False): |
Rushabh Mehta | 1f84799 | 2013-12-12 19:12:19 +0530 | [diff] [blame] | 286 | from erpnext.stock.stock_ledger import make_sl_entries |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 287 | make_sl_entries(sl_entries, allow_negative_stock, via_landed_cost_voucher) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 288 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 289 | def make_gl_entries_on_cancel(self): |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 290 | if frappe.db.sql("""select name from `tabGL Entry` where voucher_type=%s |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 291 | and voucher_no=%s""", (self.doctype, self.name)): |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 292 | self.make_gl_entries() |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 293 | |
Anand Doshi | a740f75 | 2014-06-25 13:31:02 +0530 | [diff] [blame] | 294 | def get_serialized_items(self): |
| 295 | serialized_items = [] |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 296 | item_codes = list(set([d.item_code for d in self.get("items")])) |
Anand Doshi | a740f75 | 2014-06-25 13:31:02 +0530 | [diff] [blame] | 297 | if item_codes: |
| 298 | serialized_items = frappe.db.sql_list("""select name from `tabItem` |
Rushabh Mehta | 1e8025b | 2015-07-24 15:16:25 +0530 | [diff] [blame] | 299 | where has_serial_no=1 and name in ({})""".format(", ".join(["%s"]*len(item_codes))), |
Anand Doshi | a740f75 | 2014-06-25 13:31:02 +0530 | [diff] [blame] | 300 | tuple(item_codes)) |
| 301 | |
| 302 | return serialized_items |
Rushabh Mehta | b16b9cd | 2015-08-03 16:13:33 +0530 | [diff] [blame] | 303 | |
rohitwaghchaure | 019d4de | 2020-08-20 15:53:58 +0530 | [diff] [blame] | 304 | def get_incoming_rate_for_return(self, item_code, against_document, against_document_no=None): |
Nabin Hait | 1d21842 | 2015-07-17 15:19:02 +0530 | [diff] [blame] | 305 | incoming_rate = 0.0 |
rohitwaghchaure | 019d4de | 2020-08-20 15:53:58 +0530 | [diff] [blame] | 306 | cond = '' |
Nabin Hait | 1d21842 | 2015-07-17 15:19:02 +0530 | [diff] [blame] | 307 | if against_document and item_code: |
rohitwaghchaure | 019d4de | 2020-08-20 15:53:58 +0530 | [diff] [blame] | 308 | if against_document_no: |
| 309 | cond = " and voucher_detail_no = %s" %(frappe.db.escape(against_document_no)) |
| 310 | |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 311 | incoming_rate = frappe.db.sql("""select abs(stock_value_difference / actual_qty) |
Nabin Hait | 1d21842 | 2015-07-17 15:19:02 +0530 | [diff] [blame] | 312 | from `tabStock Ledger Entry` |
Rushabh Mehta | 14a908b | 2015-10-15 12:28:20 +0530 | [diff] [blame] | 313 | where voucher_type = %s and voucher_no = %s |
rohitwaghchaure | 019d4de | 2020-08-20 15:53:58 +0530 | [diff] [blame] | 314 | and item_code = %s {0} limit 1""".format(cond), |
Nabin Hait | 20105d8 | 2016-01-19 18:05:28 +0530 | [diff] [blame] | 315 | (self.doctype, against_document, item_code)) |
rohitwaghchaure | 019d4de | 2020-08-20 15:53:58 +0530 | [diff] [blame] | 316 | |
Nabin Hait | 1d21842 | 2015-07-17 15:19:02 +0530 | [diff] [blame] | 317 | incoming_rate = incoming_rate[0][0] if incoming_rate else 0.0 |
| 318 | |
| 319 | return incoming_rate |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 320 | |
Saurabh | 2e29206 | 2015-11-18 17:03:33 +0530 | [diff] [blame] | 321 | def validate_warehouse(self): |
| 322 | from erpnext.stock.utils import validate_warehouse_company |
| 323 | |
| 324 | warehouses = list(set([d.warehouse for d in |
| 325 | self.get("items") if getattr(d, "warehouse", None)])) |
| 326 | |
Deepesh Garg | 15ff6a5 | 2020-02-18 12:28:41 +0530 | [diff] [blame] | 327 | target_warehouses = list(set([d.target_warehouse for d in |
| 328 | self.get("items") if getattr(d, "target_warehouse", None)])) |
| 329 | |
| 330 | warehouses.extend(target_warehouses) |
| 331 | |
| 332 | from_warehouse = list(set([d.from_warehouse for d in |
| 333 | self.get("items") if getattr(d, "from_warehouse", None)])) |
| 334 | |
| 335 | warehouses.extend(from_warehouse) |
| 336 | |
Saurabh | 2e29206 | 2015-11-18 17:03:33 +0530 | [diff] [blame] | 337 | for w in warehouses: |
| 338 | validate_warehouse_company(w, self.company) |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 339 | |
Anand Doshi | 6b71ef5 | 2016-01-06 16:32:06 +0530 | [diff] [blame] | 340 | def update_billing_percentage(self, update_modified=True): |
Nabin Hait | bdab0ee | 2015-12-30 19:08:11 +0530 | [diff] [blame] | 341 | self._update_percent_field({ |
| 342 | "target_dt": self.doctype + " Item", |
| 343 | "target_parent_dt": self.doctype, |
| 344 | "target_parent_field": "per_billed", |
| 345 | "target_ref_field": "amount", |
| 346 | "target_field": "billed_amt", |
| 347 | "name": self.name, |
Anand Doshi | 6b71ef5 | 2016-01-06 16:32:06 +0530 | [diff] [blame] | 348 | }, update_modified) |
Anand Doshi | a740f75 | 2014-06-25 13:31:02 +0530 | [diff] [blame] | 349 | |
Nabin Hait | 8af429d | 2016-11-16 17:21:59 +0530 | [diff] [blame] | 350 | def validate_inspection(self): |
Manas Solanki | cc90241 | 2016-11-10 19:15:11 +0530 | [diff] [blame] | 351 | '''Checks if quality inspection is set for Items that require inspection. |
| 352 | On submit, throw an exception''' |
Manas Solanki | 60db98a | 2016-11-18 14:52:13 +0530 | [diff] [blame] | 353 | inspection_required_fieldname = None |
Nabin Hait | 8af429d | 2016-11-16 17:21:59 +0530 | [diff] [blame] | 354 | if self.doctype in ["Purchase Receipt", "Purchase Invoice"]: |
| 355 | inspection_required_fieldname = "inspection_required_before_purchase" |
| 356 | elif self.doctype in ["Delivery Note", "Sales Invoice"]: |
| 357 | inspection_required_fieldname = "inspection_required_before_delivery" |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 358 | |
rohitwaghchaure | 4e8fdf7 | 2018-02-22 11:03:48 +0530 | [diff] [blame] | 359 | if ((not inspection_required_fieldname and self.doctype != "Stock Entry") or |
| 360 | (self.doctype == "Stock Entry" and not self.inspection_required) or |
| 361 | (self.doctype in ["Sales Invoice", "Purchase Invoice"] and not self.update_stock)): |
Nabin Hait | 8af429d | 2016-11-16 17:21:59 +0530 | [diff] [blame] | 362 | return |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 363 | |
Manas Solanki | cc90241 | 2016-11-10 19:15:11 +0530 | [diff] [blame] | 364 | for d in self.get('items'): |
Nabin Hait | 5a9579b | 2018-12-24 14:54:42 +0530 | [diff] [blame] | 365 | qa_required = False |
rohitwaghchaure | 4e8fdf7 | 2018-02-22 11:03:48 +0530 | [diff] [blame] | 366 | if (inspection_required_fieldname and not d.quality_inspection and |
| 367 | frappe.db.get_value("Item", d.item_code, inspection_required_fieldname)): |
Nabin Hait | 5a9579b | 2018-12-24 14:54:42 +0530 | [diff] [blame] | 368 | qa_required = True |
rohitwaghchaure | 4e8fdf7 | 2018-02-22 11:03:48 +0530 | [diff] [blame] | 369 | elif self.doctype == "Stock Entry" and not d.quality_inspection and d.t_warehouse: |
Nabin Hait | 5a9579b | 2018-12-24 14:54:42 +0530 | [diff] [blame] | 370 | qa_required = True |
Anurag Mishra | 0c8411d | 2018-12-28 16:53:00 +0530 | [diff] [blame] | 371 | if self.docstatus == 1 and d.quality_inspection: |
| 372 | qa_doc = frappe.get_doc("Quality Inspection", d.quality_inspection) |
| 373 | if qa_doc.docstatus == 0: |
| 374 | link = frappe.utils.get_link_to_form('Quality Inspection', d.quality_inspection) |
Anurag Mishra | 0885485 | 2019-01-01 15:00:16 +0530 | [diff] [blame] | 375 | frappe.throw(_("Quality Inspection: {0} is not submitted for the item: {1} in row {2}").format(link, d.item_code, d.idx), QualityInspectionNotSubmittedError) |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 376 | |
Anurag Mishra | 0c8411d | 2018-12-28 16:53:00 +0530 | [diff] [blame] | 377 | qa_failed = any([r.status=="Rejected" for r in qa_doc.readings]) |
| 378 | if qa_failed: |
| 379 | frappe.throw(_("Row {0}: Quality Inspection rejected for item {1}") |
| 380 | .format(d.idx, d.item_code), QualityInspectionRejectedError) |
| 381 | elif qa_required : |
Nabin Hait | 34c551d | 2019-07-03 10:34:31 +0530 | [diff] [blame] | 382 | action = frappe.get_doc('Stock Settings').action_if_quality_inspection_is_not_submitted |
| 383 | if self.docstatus==1 and action == 'Stop': |
| 384 | frappe.throw(_("Quality Inspection required for Item {0} to submit").format(frappe.bold(d.item_code)), |
| 385 | exc=QualityInspectionRequiredError) |
| 386 | else: |
| 387 | frappe.msgprint(_("Create Quality Inspection for Item {0}").format(frappe.bold(d.item_code))) |
Manas Solanki | cc90241 | 2016-11-10 19:15:11 +0530 | [diff] [blame] | 388 | |
Nabin Hait | b2d3c0f | 2018-06-14 15:54:34 +0530 | [diff] [blame] | 389 | def update_blanket_order(self): |
Nabin Hait | d1f40ad | 2018-06-14 17:09:55 +0530 | [diff] [blame] | 390 | blanket_orders = list(set([d.blanket_order for d in self.items if d.blanket_order])) |
Nabin Hait | b2d3c0f | 2018-06-14 15:54:34 +0530 | [diff] [blame] | 391 | for blanket_order in blanket_orders: |
| 392 | frappe.get_doc("Blanket Order", blanket_order).update_ordered_qty() |
Manas Solanki | e5e87f7 | 2018-05-28 20:07:08 +0530 | [diff] [blame] | 393 | |
marination | fd04e96 | 2020-04-03 15:46:48 +0530 | [diff] [blame] | 394 | def validate_customer_provided_item(self): |
| 395 | for d in self.get('items'): |
| 396 | # Customer Provided parts will have zero valuation rate |
| 397 | if frappe.db.get_value('Item', d.item_code, 'is_customer_provided_item'): |
| 398 | d.allow_zero_valuation_rate = 1 |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 399 | |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 400 | def compare_existing_and_expected_gle(existing_gle, expected_gle): |
| 401 | matched = True |
| 402 | for entry in expected_gle: |
robert schouten | 6465711 | 2016-09-12 11:17:14 +0800 | [diff] [blame] | 403 | account_existed = False |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 404 | for e in existing_gle: |
robert schouten | 6465711 | 2016-09-12 11:17:14 +0800 | [diff] [blame] | 405 | if entry.account == e.account: |
| 406 | account_existed = True |
| 407 | if entry.account == e.account and entry.against_account == e.against_account \ |
| 408 | and (not entry.cost_center or not e.cost_center or entry.cost_center == e.cost_center) \ |
| 409 | and (entry.debit != e.debit or entry.credit != e.credit): |
| 410 | matched = False |
| 411 | break |
| 412 | if not account_existed: |
| 413 | matched = False |
| 414 | break |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 415 | return matched |