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 |
Rushabh Mehta | 793ba6b | 2014-02-14 15:47:51 +0530 | [diff] [blame] | 5 | import frappe |
| 6 | from frappe.utils import cint, flt, cstr |
| 7 | from frappe import msgprint, _ |
| 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 |
Nabin Hait | 1f94fa2 | 2015-08-24 14:32:38 +0530 | [diff] [blame] | 10 | from erpnext.accounts.general_ledger import make_gl_entries, delete_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 | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 13 | |
| 14 | class StockController(AccountsController): |
Nabin Hait | 8af429d | 2016-11-16 17:21:59 +0530 | [diff] [blame] | 15 | def validate(self): |
| 16 | super(StockController, self).validate() |
| 17 | self.validate_inspection() |
| 18 | |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 19 | def make_gl_entries(self, repost_future_gle=True): |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 20 | if self.docstatus == 2: |
| 21 | delete_gl_entries(voucher_type=self.doctype, voucher_no=self.name) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 22 | |
Rushabh Mehta | 793ba6b | 2014-02-14 15:47:51 +0530 | [diff] [blame] | 23 | if cint(frappe.defaults.get_global_default("auto_accounting_for_stock")): |
Nabin Hait | b9e0481 | 2014-09-26 14:22:18 +0530 | [diff] [blame] | 24 | warehouse_account = get_warehouse_account() |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 25 | |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 26 | if self.docstatus==1: |
Nabin Hait | e83069c | 2013-11-14 18:40:08 +0530 | [diff] [blame] | 27 | gl_entries = self.get_gl_entries(warehouse_account) |
Nabin Hait | 145e5e2 | 2013-10-22 23:51:41 +0530 | [diff] [blame] | 28 | make_gl_entries(gl_entries) |
| 29 | |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 30 | if repost_future_gle: |
Nabin Hait | b9e0481 | 2014-09-26 14:22:18 +0530 | [diff] [blame] | 31 | items, warehouses = self.get_items_and_warehouses() |
Nabin Hait | adeb976 | 2014-10-06 11:53:52 +0530 | [diff] [blame] | 32 | update_gl_entries_after(self.posting_date, self.posting_time, warehouses, items, |
Nabin Hait | f1a07ff | 2014-10-15 12:23:35 +0530 | [diff] [blame] | 33 | warehouse_account) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 34 | |
Nabin Hait | e83069c | 2013-11-14 18:40:08 +0530 | [diff] [blame] | 35 | def get_gl_entries(self, warehouse_account=None, default_expense_account=None, |
Nabin Hait | 142007a | 2013-09-17 15:15:16 +0530 | [diff] [blame] | 36 | default_cost_center=None): |
Nabin Hait | adeb976 | 2014-10-06 11:53:52 +0530 | [diff] [blame] | 37 | |
Nabin Hait | 142007a | 2013-09-17 15:15:16 +0530 | [diff] [blame] | 38 | if not warehouse_account: |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 39 | warehouse_account = get_warehouse_account() |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 40 | |
Anand Doshi | de1a97d | 2014-04-17 11:37:46 +0530 | [diff] [blame] | 41 | sle_map = self.get_stock_ledger_details() |
| 42 | 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] | 43 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 44 | gl_list = [] |
Nabin Hait | 7a75e10 | 2013-09-17 10:21:20 +0530 | [diff] [blame] | 45 | warehouse_with_no_account = [] |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 46 | |
Nabin Hait | 8c61f34 | 2016-12-15 13:46:03 +0530 | [diff] [blame] | 47 | for item_row in voucher_details: |
| 48 | sle_list = sle_map.get(item_row.name) |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 49 | if sle_list: |
| 50 | for sle in sle_list: |
| 51 | if warehouse_account.get(sle.warehouse): |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 52 | # from warehouse account |
Nabin Hait | 8c61f34 | 2016-12-15 13:46:03 +0530 | [diff] [blame] | 53 | |
| 54 | self.check_expense_account(item_row) |
Rohit Waghchaure | 3899d74 | 2017-01-25 18:47:53 +0530 | [diff] [blame] | 55 | |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 56 | # If item is not a sample item |
| 57 | # and ( valuation rate not mentioned in an incoming entry |
| 58 | # or incoming entry not found while delivering the item), |
| 59 | # try to pick valuation rate from previous sle or Item master and update in SLE |
| 60 | # Otherwise, throw an exception |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 61 | |
| 62 | if not sle.stock_value_difference and self.doctype != "Stock Reconciliation" \ |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 63 | and not item_row.get("is_sample_item"): |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 64 | |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 65 | sle = self.update_stock_ledger_entries(sle) |
Rushabh Mehta | 14a908b | 2015-10-15 12:28:20 +0530 | [diff] [blame] | 66 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 67 | gl_list.append(self.get_gl_dict({ |
Nabin Hait | 46bcbaf | 2015-08-19 13:49:10 +0530 | [diff] [blame] | 68 | "account": warehouse_account[sle.warehouse]["name"], |
Nabin Hait | 8c61f34 | 2016-12-15 13:46:03 +0530 | [diff] [blame] | 69 | "against": item_row.expense_account, |
| 70 | "cost_center": item_row.cost_center, |
Nabin Hait | dc82d4f | 2014-04-07 12:02:57 +0530 | [diff] [blame] | 71 | "remarks": self.get("remarks") or "Accounting Entry for Stock", |
Nabin Hait | 46bcbaf | 2015-08-19 13:49:10 +0530 | [diff] [blame] | 72 | "debit": flt(sle.stock_value_difference, 2), |
Nabin Hait | 6e439a5 | 2015-08-28 19:24:22 +0530 | [diff] [blame] | 73 | }, warehouse_account[sle.warehouse]["account_currency"])) |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 74 | |
Rushabh Mehta | 14a908b | 2015-10-15 12:28:20 +0530 | [diff] [blame] | 75 | # to target warehouse / expense account |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 76 | gl_list.append(self.get_gl_dict({ |
Nabin Hait | 8c61f34 | 2016-12-15 13:46:03 +0530 | [diff] [blame] | 77 | "account": item_row.expense_account, |
Nabin Hait | 46bcbaf | 2015-08-19 13:49:10 +0530 | [diff] [blame] | 78 | "against": warehouse_account[sle.warehouse]["name"], |
Nabin Hait | 8c61f34 | 2016-12-15 13:46:03 +0530 | [diff] [blame] | 79 | "cost_center": item_row.cost_center, |
Nabin Hait | dc82d4f | 2014-04-07 12:02:57 +0530 | [diff] [blame] | 80 | "remarks": self.get("remarks") or "Accounting Entry for Stock", |
Nabin Hait | 46bcbaf | 2015-08-19 13:49:10 +0530 | [diff] [blame] | 81 | "credit": flt(sle.stock_value_difference, 2), |
Nabin Hait | 8c61f34 | 2016-12-15 13:46:03 +0530 | [diff] [blame] | 82 | "project": item_row.get("project") or self.get("project") |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 83 | })) |
Nabin Hait | 7a75e10 | 2013-09-17 10:21:20 +0530 | [diff] [blame] | 84 | elif sle.warehouse not in warehouse_with_no_account: |
| 85 | warehouse_with_no_account.append(sle.warehouse) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 86 | |
| 87 | if warehouse_with_no_account: |
Nabin Hait | d662502 | 2016-10-24 18:17:57 +0530 | [diff] [blame] | 88 | for wh in warehouse_with_no_account: |
| 89 | if frappe.db.get_value("Warehouse", wh, "company"): |
| 90 | frappe.throw(_("Warehouse {0} is not linked to any account, please create/link the corresponding (Asset) account for the warehouse.").format(wh)) |
| 91 | |
Rushabh Mehta | 9f0d625 | 2014-04-14 19:20:45 +0530 | [diff] [blame] | 92 | msgprint(_("No accounting entries for the following warehouses") + ": \n" + |
Nabin Hait | 7a75e10 | 2013-09-17 10:21:20 +0530 | [diff] [blame] | 93 | "\n".join(warehouse_with_no_account)) |
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 | return process_gl_map(gl_list) |
Rohit Waghchaure | 3899d74 | 2017-01-25 18:47:53 +0530 | [diff] [blame] | 96 | |
| 97 | def update_stock_ledger_entries(self, sle): |
Nabin Hait | cb19f52 | 2017-01-25 19:20:35 +0530 | [diff] [blame] | 98 | sle.valuation_rate = get_valuation_rate(sle.item_code, sle.warehouse, |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 99 | self.doctype, self.name) |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 100 | |
Nabin Hait | cb19f52 | 2017-01-25 19:20:35 +0530 | [diff] [blame] | 101 | sle.stock_value = flt(sle.qty_after_transaction) * flt(sle.valuation_rate) |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 102 | sle.stock_value_difference = flt(sle.actual_qty) * flt(sle.valuation_rate) |
| 103 | |
Kanchan Chauhan | b77eaf7 | 2017-01-28 13:18:49 +0530 | [diff] [blame] | 104 | if sle.name: |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 105 | frappe.db.sql(""" |
| 106 | update |
| 107 | `tabStock Ledger Entry` |
| 108 | set |
| 109 | stock_value = %(stock_value)s, |
| 110 | valuation_rate = %(valuation_rate)s, |
| 111 | stock_value_difference = %(stock_value_difference)s |
| 112 | where |
| 113 | name = %(name)s""", (sle)) |
| 114 | |
| 115 | return sle |
| 116 | |
Anand Doshi | de1a97d | 2014-04-17 11:37:46 +0530 | [diff] [blame] | 117 | def get_voucher_details(self, default_expense_account, default_cost_center, sle_map): |
| 118 | if self.doctype == "Stock Reconciliation": |
| 119 | return [frappe._dict({ "name": voucher_detail_no, "expense_account": default_expense_account, |
| 120 | "cost_center": default_cost_center }) for voucher_detail_no, sle in sle_map.items()] |
| 121 | else: |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 122 | details = self.get("items") |
Anand Doshi | 094610d | 2014-04-16 19:56:53 +0530 | [diff] [blame] | 123 | |
Anand Doshi | de1a97d | 2014-04-17 11:37:46 +0530 | [diff] [blame] | 124 | if default_expense_account or default_cost_center: |
| 125 | for d in details: |
| 126 | if default_expense_account and not d.get("expense_account"): |
| 127 | d.expense_account = default_expense_account |
| 128 | if default_cost_center and not d.get("cost_center"): |
| 129 | d.cost_center = default_cost_center |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 130 | |
Anand Doshi | de1a97d | 2014-04-17 11:37:46 +0530 | [diff] [blame] | 131 | return details |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 132 | |
Nabin Hait | b9e0481 | 2014-09-26 14:22:18 +0530 | [diff] [blame] | 133 | def get_items_and_warehouses(self): |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 134 | items, warehouses = [], [] |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 135 | |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 136 | if hasattr(self, "items"): |
| 137 | item_doclist = self.get("items") |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 138 | elif self.doctype == "Stock Reconciliation": |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 139 | import json |
| 140 | item_doclist = [] |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 141 | data = json.loads(self.reconciliation_json) |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 142 | for row in data[data.index(self.head_row)+1:]: |
| 143 | d = frappe._dict(zip(["item_code", "warehouse", "qty", "valuation_rate"], row)) |
| 144 | item_doclist.append(d) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 145 | |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 146 | if item_doclist: |
| 147 | for d in item_doclist: |
| 148 | if d.item_code and d.item_code not in items: |
| 149 | items.append(d.item_code) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 150 | |
Nabin Hait | dc82d4f | 2014-04-07 12:02:57 +0530 | [diff] [blame] | 151 | if d.get("warehouse") and d.warehouse not in warehouses: |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 152 | warehouses.append(d.warehouse) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 153 | |
Nabin Hait | dc82d4f | 2014-04-07 12:02:57 +0530 | [diff] [blame] | 154 | if self.doctype == "Stock Entry": |
| 155 | if d.get("s_warehouse") and d.s_warehouse not in warehouses: |
| 156 | warehouses.append(d.s_warehouse) |
| 157 | if d.get("t_warehouse") and d.t_warehouse not in warehouses: |
| 158 | warehouses.append(d.t_warehouse) |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 159 | |
Nabin Hait | b9e0481 | 2014-09-26 14:22:18 +0530 | [diff] [blame] | 160 | return items, warehouses |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 161 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 162 | def get_stock_ledger_details(self): |
| 163 | stock_ledger = {} |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 164 | stock_ledger_entries = frappe.db.sql(""" |
| 165 | select |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 166 | name, warehouse, stock_value_difference, valuation_rate, |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 167 | voucher_detail_no, item_code, posting_date, posting_time, |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 168 | actual_qty, qty_after_transaction |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 169 | from |
| 170 | `tabStock Ledger Entry` |
| 171 | where |
| 172 | voucher_type=%s and voucher_no=%s |
| 173 | """, (self.doctype, self.name), as_dict=True) |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 174 | |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 175 | for sle in stock_ledger_entries: |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 176 | stock_ledger.setdefault(sle.voucher_detail_no, []).append(sle) |
| 177 | return stock_ledger |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 178 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 179 | def make_adjustment_entry(self, expected_gle, voucher_obj): |
Rushabh Mehta | 1f84799 | 2013-12-12 19:12:19 +0530 | [diff] [blame] | 180 | from erpnext.accounts.utils import get_stock_and_account_difference |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 181 | account_list = [d.account for d in expected_gle] |
| 182 | acc_diff = get_stock_and_account_difference(account_list, expected_gle[0].posting_date) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 183 | |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 184 | cost_center = self.get_company_default("cost_center") |
| 185 | stock_adjustment_account = self.get_company_default("stock_adjustment_account") |
| 186 | |
Nabin Hait | d474194 | 2013-08-06 15:57:25 +0530 | [diff] [blame] | 187 | gl_entries = [] |
| 188 | for account, diff in acc_diff.items(): |
| 189 | if diff: |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 190 | gl_entries.append([ |
| 191 | # stock in hand account |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 192 | voucher_obj.get_gl_dict({ |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 193 | "account": account, |
| 194 | "against": stock_adjustment_account, |
| 195 | "debit": diff, |
| 196 | "remarks": "Adjustment Accounting Entry for Stock", |
| 197 | }), |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 198 | |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 199 | # account against stock in hand |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 200 | voucher_obj.get_gl_dict({ |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 201 | "account": stock_adjustment_account, |
| 202 | "against": account, |
| 203 | "credit": diff, |
| 204 | "cost_center": cost_center or None, |
| 205 | "remarks": "Adjustment Accounting Entry for Stock", |
| 206 | }), |
| 207 | ]) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 208 | |
Nabin Hait | d474194 | 2013-08-06 15:57:25 +0530 | [diff] [blame] | 209 | if gl_entries: |
Rushabh Mehta | 1f84799 | 2013-12-12 19:12:19 +0530 | [diff] [blame] | 210 | from erpnext.accounts.general_ledger import make_gl_entries |
Nabin Hait | d474194 | 2013-08-06 15:57:25 +0530 | [diff] [blame] | 211 | make_gl_entries(gl_entries) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 212 | |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 213 | def check_expense_account(self, item): |
Rushabh Mehta | 052fe82 | 2014-04-16 19:20:11 +0530 | [diff] [blame] | 214 | if not item.get("expense_account"): |
Nabin Hait | d341254 | 2014-05-01 17:42:21 +0530 | [diff] [blame] | 215 | frappe.throw(_("Expense or Difference account is mandatory for Item {0} as it impacts overall stock value").format(item.item_code)) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 216 | |
Anand Doshi | 496123a | 2014-06-19 19:25:19 +0530 | [diff] [blame] | 217 | else: |
Nabin Hait | 00778c9 | 2014-06-25 19:12:24 +0530 | [diff] [blame] | 218 | is_expense_account = frappe.db.get_value("Account", |
| 219 | item.get("expense_account"), "report_type")=="Profit and Loss" |
jof2jc | 524ebbc | 2016-02-08 23:44:55 +0700 | [diff] [blame] | 220 | 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] | 221 | frappe.throw(_("Expense / Difference account ({0}) must be a 'Profit or Loss' account") |
| 222 | .format(item.get("expense_account"))) |
Anand Doshi | 496123a | 2014-06-19 19:25:19 +0530 | [diff] [blame] | 223 | if is_expense_account and not item.get("cost_center"): |
| 224 | frappe.throw(_("{0} {1}: Cost Center is mandatory for Item {2}").format( |
| 225 | _(self.doctype), self.name, item.get("item_code"))) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 226 | |
| 227 | def get_sl_entries(self, d, args): |
Nabin Hait | 5288bde | 2014-11-03 15:08:21 +0530 | [diff] [blame] | 228 | sl_dict = frappe._dict({ |
Nabin Hait | 296d626 | 2014-05-02 17:36:13 +0530 | [diff] [blame] | 229 | "item_code": d.get("item_code", None), |
Nabin Hait | dc82d4f | 2014-04-07 12:02:57 +0530 | [diff] [blame] | 230 | "warehouse": d.get("warehouse", None), |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 231 | "posting_date": self.posting_date, |
| 232 | "posting_time": self.posting_time, |
Valmik Jangla | d89dcf6 | 2016-03-04 14:33:49 +0530 | [diff] [blame] | 233 | 'fiscal_year': get_fiscal_year(self.posting_date, company=self.company)[0], |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 234 | "voucher_type": self.doctype, |
| 235 | "voucher_no": self.name, |
Nabin Hait | 1e2f20a | 2013-08-02 11:42:11 +0530 | [diff] [blame] | 236 | "voucher_detail_no": d.name, |
Nabin Hait | dc82d4f | 2014-04-07 12:02:57 +0530 | [diff] [blame] | 237 | "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] | 238 | "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] | 239 | "incoming_rate": 0, |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 240 | "company": self.company, |
Nabin Hait | 296d626 | 2014-05-02 17:36:13 +0530 | [diff] [blame] | 241 | "batch_no": cstr(d.get("batch_no")).strip(), |
| 242 | "serial_no": d.get("serial_no"), |
Neil Trini Lasrado | 6e343e2 | 2016-03-09 17:02:59 +0530 | [diff] [blame] | 243 | "project": d.get("project"), |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 244 | "is_cancelled": self.docstatus==2 and "Yes" or "No" |
Nabin Hait | 5288bde | 2014-11-03 15:08:21 +0530 | [diff] [blame] | 245 | }) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 246 | |
Nabin Hait | 1e2f20a | 2013-08-02 11:42:11 +0530 | [diff] [blame] | 247 | sl_dict.update(args) |
| 248 | return sl_dict |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 249 | |
Anand Doshi | c399709 | 2015-04-06 19:29:16 +0530 | [diff] [blame] | 250 | def make_sl_entries(self, sl_entries, is_amended=None, allow_negative_stock=False, |
Nabin Hait | 54c865e | 2015-03-27 15:38:31 +0530 | [diff] [blame] | 251 | via_landed_cost_voucher=False): |
Rushabh Mehta | 1f84799 | 2013-12-12 19:12:19 +0530 | [diff] [blame] | 252 | from erpnext.stock.stock_ledger import make_sl_entries |
Nabin Hait | 54c865e | 2015-03-27 15:38:31 +0530 | [diff] [blame] | 253 | make_sl_entries(sl_entries, is_amended, allow_negative_stock, via_landed_cost_voucher) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 254 | |
Nabin Hait | 3b6dc14 | 2016-08-19 16:39:33 +0530 | [diff] [blame] | 255 | def make_gl_entries_on_cancel(self, repost_future_gle=True): |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 256 | 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] | 257 | and voucher_no=%s""", (self.doctype, self.name)): |
Nabin Hait | 3b6dc14 | 2016-08-19 16:39:33 +0530 | [diff] [blame] | 258 | self.make_gl_entries(repost_future_gle) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 259 | |
Anand Doshi | a740f75 | 2014-06-25 13:31:02 +0530 | [diff] [blame] | 260 | def get_serialized_items(self): |
| 261 | serialized_items = [] |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 262 | 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] | 263 | if item_codes: |
| 264 | serialized_items = frappe.db.sql_list("""select name from `tabItem` |
Rushabh Mehta | 1e8025b | 2015-07-24 15:16:25 +0530 | [diff] [blame] | 265 | 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] | 266 | tuple(item_codes)) |
| 267 | |
| 268 | return serialized_items |
Rushabh Mehta | b16b9cd | 2015-08-03 16:13:33 +0530 | [diff] [blame] | 269 | |
Nabin Hait | 20105d8 | 2016-01-19 18:05:28 +0530 | [diff] [blame] | 270 | def get_incoming_rate_for_sales_return(self, item_code, against_document): |
Nabin Hait | 1d21842 | 2015-07-17 15:19:02 +0530 | [diff] [blame] | 271 | incoming_rate = 0.0 |
| 272 | if against_document and item_code: |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 273 | incoming_rate = frappe.db.sql("""select abs(stock_value_difference / actual_qty) |
Nabin Hait | 1d21842 | 2015-07-17 15:19:02 +0530 | [diff] [blame] | 274 | from `tabStock Ledger Entry` |
Rushabh Mehta | 14a908b | 2015-10-15 12:28:20 +0530 | [diff] [blame] | 275 | where voucher_type = %s and voucher_no = %s |
Nabin Hait | 20105d8 | 2016-01-19 18:05:28 +0530 | [diff] [blame] | 276 | and item_code = %s limit 1""", |
| 277 | (self.doctype, against_document, item_code)) |
Nabin Hait | 1d21842 | 2015-07-17 15:19:02 +0530 | [diff] [blame] | 278 | incoming_rate = incoming_rate[0][0] if incoming_rate else 0.0 |
| 279 | |
| 280 | return incoming_rate |
Saurabh | 2e29206 | 2015-11-18 17:03:33 +0530 | [diff] [blame] | 281 | |
| 282 | def validate_warehouse(self): |
| 283 | from erpnext.stock.utils import validate_warehouse_company |
| 284 | |
| 285 | warehouses = list(set([d.warehouse for d in |
| 286 | self.get("items") if getattr(d, "warehouse", None)])) |
| 287 | |
| 288 | for w in warehouses: |
| 289 | validate_warehouse_company(w, self.company) |
Nabin Hait | bdab0ee | 2015-12-30 19:08:11 +0530 | [diff] [blame] | 290 | |
Anand Doshi | 6b71ef5 | 2016-01-06 16:32:06 +0530 | [diff] [blame] | 291 | def update_billing_percentage(self, update_modified=True): |
Nabin Hait | bdab0ee | 2015-12-30 19:08:11 +0530 | [diff] [blame] | 292 | self._update_percent_field({ |
| 293 | "target_dt": self.doctype + " Item", |
| 294 | "target_parent_dt": self.doctype, |
| 295 | "target_parent_field": "per_billed", |
| 296 | "target_ref_field": "amount", |
| 297 | "target_field": "billed_amt", |
| 298 | "name": self.name, |
Anand Doshi | 6b71ef5 | 2016-01-06 16:32:06 +0530 | [diff] [blame] | 299 | }, update_modified) |
Anand Doshi | a740f75 | 2014-06-25 13:31:02 +0530 | [diff] [blame] | 300 | |
Nabin Hait | 8af429d | 2016-11-16 17:21:59 +0530 | [diff] [blame] | 301 | def validate_inspection(self): |
Manas Solanki | cc90241 | 2016-11-10 19:15:11 +0530 | [diff] [blame] | 302 | '''Checks if quality inspection is set for Items that require inspection. |
| 303 | On submit, throw an exception''' |
Nabin Hait | 8af429d | 2016-11-16 17:21:59 +0530 | [diff] [blame] | 304 | |
Manas Solanki | 60db98a | 2016-11-18 14:52:13 +0530 | [diff] [blame] | 305 | inspection_required_fieldname = None |
Nabin Hait | 8af429d | 2016-11-16 17:21:59 +0530 | [diff] [blame] | 306 | if self.doctype in ["Purchase Receipt", "Purchase Invoice"]: |
| 307 | inspection_required_fieldname = "inspection_required_before_purchase" |
| 308 | elif self.doctype in ["Delivery Note", "Sales Invoice"]: |
| 309 | inspection_required_fieldname = "inspection_required_before_delivery" |
| 310 | |
| 311 | if not inspection_required_fieldname or \ |
| 312 | (self.doctype in ["Sales Invoice", "Purchase Invoice"] and not self.update_stock): |
| 313 | return |
| 314 | |
Manas Solanki | cc90241 | 2016-11-10 19:15:11 +0530 | [diff] [blame] | 315 | for d in self.get('items'): |
Nabin Hait | 8af429d | 2016-11-16 17:21:59 +0530 | [diff] [blame] | 316 | if (frappe.db.get_value("Item", d.item_code, inspection_required_fieldname) |
Manas Solanki | cc90241 | 2016-11-10 19:15:11 +0530 | [diff] [blame] | 317 | and not d.quality_inspection): |
Nabin Hait | 8af429d | 2016-11-16 17:21:59 +0530 | [diff] [blame] | 318 | |
Manas Solanki | cc90241 | 2016-11-10 19:15:11 +0530 | [diff] [blame] | 319 | frappe.msgprint(_("Quality Inspection required for Item {0}").format(d.item_code)) |
| 320 | if self.docstatus==1: |
| 321 | raise frappe.ValidationError |
| 322 | |
Nabin Hait | adeb976 | 2014-10-06 11:53:52 +0530 | [diff] [blame] | 323 | def update_gl_entries_after(posting_date, posting_time, for_warehouses=None, for_items=None, |
Nabin Hait | f1a07ff | 2014-10-15 12:23:35 +0530 | [diff] [blame] | 324 | warehouse_account=None): |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 325 | def _delete_gl_entries(voucher_type, voucher_no): |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 326 | frappe.db.sql("""delete from `tabGL Entry` |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 327 | where voucher_type=%s and voucher_no=%s""", (voucher_type, voucher_no)) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 328 | |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 329 | if not warehouse_account: |
| 330 | warehouse_account = get_warehouse_account() |
Nabin Hait | b9e0481 | 2014-09-26 14:22:18 +0530 | [diff] [blame] | 331 | |
| 332 | future_stock_vouchers = get_future_stock_vouchers(posting_date, posting_time, for_warehouses, for_items) |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 333 | gle = get_voucherwise_gl_entries(future_stock_vouchers, posting_date) |
Nabin Hait | 3b6dc14 | 2016-08-19 16:39:33 +0530 | [diff] [blame] | 334 | |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 335 | for voucher_type, voucher_no in future_stock_vouchers: |
| 336 | existing_gle = gle.get((voucher_type, voucher_no), []) |
Rushabh Mehta | 0a0f249 | 2014-03-31 17:27:06 +0530 | [diff] [blame] | 337 | voucher_obj = frappe.get_doc(voucher_type, voucher_no) |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 338 | expected_gle = voucher_obj.get_gl_entries(warehouse_account) |
| 339 | if expected_gle: |
Nabin Hait | 99d4ff4 | 2016-08-08 14:12:16 +0530 | [diff] [blame] | 340 | if not existing_gle or not compare_existing_and_expected_gle(existing_gle, expected_gle): |
| 341 | _delete_gl_entries(voucher_type, voucher_no) |
| 342 | voucher_obj.make_gl_entries(repost_future_gle=False) |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 343 | else: |
| 344 | _delete_gl_entries(voucher_type, voucher_no) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 345 | |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 346 | def compare_existing_and_expected_gle(existing_gle, expected_gle): |
| 347 | matched = True |
| 348 | for entry in expected_gle: |
robert schouten | 6465711 | 2016-09-12 11:17:14 +0800 | [diff] [blame] | 349 | account_existed = False |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 350 | for e in existing_gle: |
robert schouten | 6465711 | 2016-09-12 11:17:14 +0800 | [diff] [blame] | 351 | if entry.account == e.account: |
| 352 | account_existed = True |
| 353 | if entry.account == e.account and entry.against_account == e.against_account \ |
| 354 | and (not entry.cost_center or not e.cost_center or entry.cost_center == e.cost_center) \ |
| 355 | and (entry.debit != e.debit or entry.credit != e.credit): |
| 356 | matched = False |
| 357 | break |
| 358 | if not account_existed: |
| 359 | matched = False |
| 360 | break |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 361 | return matched |
| 362 | |
Nabin Hait | b9e0481 | 2014-09-26 14:22:18 +0530 | [diff] [blame] | 363 | def get_future_stock_vouchers(posting_date, posting_time, for_warehouses=None, for_items=None): |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 364 | future_stock_vouchers = [] |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 365 | |
Anand Doshi | 913c51b | 2014-08-27 22:09:03 +0530 | [diff] [blame] | 366 | values = [] |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 367 | condition = "" |
| 368 | if for_items: |
Anand Doshi | 913c51b | 2014-08-27 22:09:03 +0530 | [diff] [blame] | 369 | condition += " and item_code in ({})".format(", ".join(["%s"] * len(for_items))) |
| 370 | values += for_items |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 371 | |
Nabin Hait | b9e0481 | 2014-09-26 14:22:18 +0530 | [diff] [blame] | 372 | if for_warehouses: |
| 373 | condition += " and warehouse in ({})".format(", ".join(["%s"] * len(for_warehouses))) |
| 374 | values += for_warehouses |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 375 | |
| 376 | for d in frappe.db.sql("""select distinct sle.voucher_type, sle.voucher_no |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 377 | from `tabStock Ledger Entry` sle |
Anand Doshi | 913c51b | 2014-08-27 22:09:03 +0530 | [diff] [blame] | 378 | where timestamp(sle.posting_date, sle.posting_time) >= timestamp(%s, %s) {condition} |
| 379 | order by timestamp(sle.posting_date, sle.posting_time) asc, name asc""".format(condition=condition), |
| 380 | tuple([posting_date, posting_time] + values), as_dict=True): |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 381 | future_stock_vouchers.append([d.voucher_type, d.voucher_no]) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 382 | |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 383 | return future_stock_vouchers |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 384 | |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 385 | def get_voucherwise_gl_entries(future_stock_vouchers, posting_date): |
| 386 | gl_entries = {} |
| 387 | if future_stock_vouchers: |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 388 | for d in frappe.db.sql("""select * from `tabGL Entry` |
| 389 | where posting_date >= %s and voucher_no in (%s)""" % |
| 390 | ('%s', ', '.join(['%s']*len(future_stock_vouchers))), |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 391 | tuple([posting_date] + [d[1] for d in future_stock_vouchers]), as_dict=1): |
| 392 | gl_entries.setdefault((d.voucher_type, d.voucher_no), []).append(d) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 393 | |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 394 | return gl_entries |
| 395 | |
| 396 | def get_warehouse_account(): |
Nabin Hait | 46bcbaf | 2015-08-19 13:49:10 +0530 | [diff] [blame] | 397 | warehouse_account = frappe._dict() |
Rushabh Mehta | 14a908b | 2015-10-15 12:28:20 +0530 | [diff] [blame] | 398 | |
Nabin Hait | 6e439a5 | 2015-08-28 19:24:22 +0530 | [diff] [blame] | 399 | for d in frappe.db.sql("""select warehouse, name, account_currency from tabAccount |
Saurabh | 7fecb50 | 2016-06-24 13:11:29 +0530 | [diff] [blame] | 400 | where account_type = 'Stock' and (warehouse is not null and warehouse != '' |
Saurabh | 78333c7 | 2016-06-25 14:18:28 +0530 | [diff] [blame] | 401 | and is_group != 1) and is_group=0 """, as_dict=1): |
Nabin Hait | 46bcbaf | 2015-08-19 13:49:10 +0530 | [diff] [blame] | 402 | warehouse_account.setdefault(d.warehouse, d) |
Anand Doshi | 67d6a4e | 2014-03-28 13:41:59 +0530 | [diff] [blame] | 403 | return warehouse_account |