blob: 9a3d82ee910ffa71a3879cea40bd258fda7825f4 [file] [log] [blame]
Anand Doshi885e0742015-03-03 14:55:30 +05301# Copyright (c) 2015, Frappe Technologies Pvt. Ltd. and Contributors
Rushabh Mehtae67d1fb2013-08-05 14:59:54 +05302# License: GNU General Public License v3. See license.txt
Nabin Haitc3afb252013-03-19 12:01:24 +05303
4from __future__ import unicode_literals
Rushabh Mehta793ba6b2014-02-14 15:47:51 +05305import frappe
6from frappe.utils import cint, flt, cstr
7from frappe import msgprint, _
8import frappe.defaults
Valmik Janglad89dcf62016-03-04 14:33:49 +05309from erpnext.accounts.utils import get_fiscal_year
Nabin Hait1f94fa22015-08-24 14:32:38 +053010from erpnext.accounts.general_ledger import make_gl_entries, delete_gl_entries, process_gl_map
Rushabh Mehta1f847992013-12-12 19:12:19 +053011from erpnext.controllers.accounts_controller import AccountsController
Rohit Waghchaure3899d742017-01-25 18:47:53 +053012from erpnext.stock.stock_ledger import get_valuation_rate
Nabin Haitc3afb252013-03-19 12:01:24 +053013
14class StockController(AccountsController):
Nabin Hait8af429d2016-11-16 17:21:59 +053015 def validate(self):
16 super(StockController, self).validate()
17 self.validate_inspection()
18
Nabin Haitbecf75d2014-03-27 17:18:29 +053019 def make_gl_entries(self, repost_future_gle=True):
Anand Doshif78d1ae2014-03-28 13:55:00 +053020 if self.docstatus == 2:
21 delete_gl_entries(voucher_type=self.doctype, voucher_no=self.name)
Anand Doshi2ce39cf2014-04-07 18:51:58 +053022
Rushabh Mehta793ba6b2014-02-14 15:47:51 +053023 if cint(frappe.defaults.get_global_default("auto_accounting_for_stock")):
Nabin Haitb9e04812014-09-26 14:22:18 +053024 warehouse_account = get_warehouse_account()
Anand Doshi2ce39cf2014-04-07 18:51:58 +053025
Anand Doshif78d1ae2014-03-28 13:55:00 +053026 if self.docstatus==1:
Nabin Haite83069c2013-11-14 18:40:08 +053027 gl_entries = self.get_gl_entries(warehouse_account)
Nabin Hait145e5e22013-10-22 23:51:41 +053028 make_gl_entries(gl_entries)
29
Nabin Haitbecf75d2014-03-27 17:18:29 +053030 if repost_future_gle:
Nabin Haitb9e04812014-09-26 14:22:18 +053031 items, warehouses = self.get_items_and_warehouses()
Nabin Haitadeb9762014-10-06 11:53:52 +053032 update_gl_entries_after(self.posting_date, self.posting_time, warehouses, items,
Nabin Haitf1a07ff2014-10-15 12:23:35 +053033 warehouse_account)
Anand Doshi2ce39cf2014-04-07 18:51:58 +053034
Nabin Haite83069c2013-11-14 18:40:08 +053035 def get_gl_entries(self, warehouse_account=None, default_expense_account=None,
Nabin Hait142007a2013-09-17 15:15:16 +053036 default_cost_center=None):
Nabin Haitadeb9762014-10-06 11:53:52 +053037
Nabin Hait142007a2013-09-17 15:15:16 +053038 if not warehouse_account:
Nabin Haitbecf75d2014-03-27 17:18:29 +053039 warehouse_account = get_warehouse_account()
Anand Doshi2ce39cf2014-04-07 18:51:58 +053040
Anand Doshide1a97d2014-04-17 11:37:46 +053041 sle_map = self.get_stock_ledger_details()
42 voucher_details = self.get_voucher_details(default_expense_account, default_cost_center, sle_map)
Anand Doshi2ce39cf2014-04-07 18:51:58 +053043
Nabin Hait2e296fa2013-08-28 18:53:11 +053044 gl_list = []
Nabin Hait7a75e102013-09-17 10:21:20 +053045 warehouse_with_no_account = []
Nabin Hait99d4ff42016-08-08 14:12:16 +053046
Nabin Hait8c61f342016-12-15 13:46:03 +053047 for item_row in voucher_details:
48 sle_list = sle_map.get(item_row.name)
Nabin Hait2e296fa2013-08-28 18:53:11 +053049 if sle_list:
50 for sle in sle_list:
51 if warehouse_account.get(sle.warehouse):
Nabin Hait2e296fa2013-08-28 18:53:11 +053052 # from warehouse account
Nabin Hait8c61f342016-12-15 13:46:03 +053053
54 self.check_expense_account(item_row)
Rohit Waghchaure3899d742017-01-25 18:47:53 +053055
Nabin Hait8c61f342016-12-15 13:46:03 +053056 if not sle.stock_value_difference:
Nabin Haitcb19f522017-01-25 19:20:35 +053057 self.update_stock_ledger_entries(sle)
Nabin Hait8c61f342016-12-15 13:46:03 +053058 self.validate_negative_stock(sle)
Rushabh Mehta14a908b2015-10-15 12:28:20 +053059
Nabin Hait2e296fa2013-08-28 18:53:11 +053060 gl_list.append(self.get_gl_dict({
Nabin Hait46bcbaf2015-08-19 13:49:10 +053061 "account": warehouse_account[sle.warehouse]["name"],
Nabin Hait8c61f342016-12-15 13:46:03 +053062 "against": item_row.expense_account,
63 "cost_center": item_row.cost_center,
Nabin Haitdc82d4f2014-04-07 12:02:57 +053064 "remarks": self.get("remarks") or "Accounting Entry for Stock",
Nabin Hait46bcbaf2015-08-19 13:49:10 +053065 "debit": flt(sle.stock_value_difference, 2),
Nabin Hait6e439a52015-08-28 19:24:22 +053066 }, warehouse_account[sle.warehouse]["account_currency"]))
Nabin Hait27994c22013-08-26 16:53:30 +053067
Rushabh Mehta14a908b2015-10-15 12:28:20 +053068 # to target warehouse / expense account
Nabin Hait2e296fa2013-08-28 18:53:11 +053069 gl_list.append(self.get_gl_dict({
Nabin Hait8c61f342016-12-15 13:46:03 +053070 "account": item_row.expense_account,
Nabin Hait46bcbaf2015-08-19 13:49:10 +053071 "against": warehouse_account[sle.warehouse]["name"],
Nabin Hait8c61f342016-12-15 13:46:03 +053072 "cost_center": item_row.cost_center,
Nabin Haitdc82d4f2014-04-07 12:02:57 +053073 "remarks": self.get("remarks") or "Accounting Entry for Stock",
Nabin Hait46bcbaf2015-08-19 13:49:10 +053074 "credit": flt(sle.stock_value_difference, 2),
Nabin Hait8c61f342016-12-15 13:46:03 +053075 "project": item_row.get("project") or self.get("project")
Nabin Hait2e296fa2013-08-28 18:53:11 +053076 }))
Nabin Hait7a75e102013-09-17 10:21:20 +053077 elif sle.warehouse not in warehouse_with_no_account:
78 warehouse_with_no_account.append(sle.warehouse)
Anand Doshi2ce39cf2014-04-07 18:51:58 +053079
80 if warehouse_with_no_account:
Nabin Haitd6625022016-10-24 18:17:57 +053081 for wh in warehouse_with_no_account:
82 if frappe.db.get_value("Warehouse", wh, "company"):
83 frappe.throw(_("Warehouse {0} is not linked to any account, please create/link the corresponding (Asset) account for the warehouse.").format(wh))
84
Rushabh Mehta9f0d6252014-04-14 19:20:45 +053085 msgprint(_("No accounting entries for the following warehouses") + ": \n" +
Nabin Hait7a75e102013-09-17 10:21:20 +053086 "\n".join(warehouse_with_no_account))
Rushabh Mehta14a908b2015-10-15 12:28:20 +053087
Nabin Hait2e296fa2013-08-28 18:53:11 +053088 return process_gl_map(gl_list)
Rohit Waghchaure3899d742017-01-25 18:47:53 +053089
90 def update_stock_ledger_entries(self, sle):
Nabin Haitcb19f522017-01-25 19:20:35 +053091 sle.valuation_rate = get_valuation_rate(sle.item_code, sle.warehouse,
92 sle.voucher_type, sle.voucher_no)
93 sle.stock_value = flt(sle.qty_after_transaction) * flt(sle.valuation_rate)
94 sle.stock_value_difference = sle.stock_value
Kanchan Chauhanb77eaf72017-01-28 13:18:49 +053095 if sle.name:
96 frappe.db.sql(""" update `tabStock Ledger Entry` set stock_value = %(stock_value)s,
97 valuation_rate = %(valuation_rate)s, stock_value_difference = %(stock_value_difference)s
98 where name = %(name)s""", (sle))
Rohit Waghchaure3899d742017-01-25 18:47:53 +053099
Nabin Hait8c61f342016-12-15 13:46:03 +0530100 def validate_negative_stock(self, sle):
101 if sle.qty_after_transaction < 0 and sle.actual_qty < 0:
Nabin Haita0b967f2017-01-18 18:35:58 +0530102 frappe.throw(_("Valuation rate not found for the Item {0}, which is required to do accounting entries (for booking expenses). Please create an incoming stock transaction or mention valuation rate in Item record, and then try submiting {1} {2}")
103 .format(sle.item_code, sle.voucher_type, sle.voucher_no))
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530104
Anand Doshide1a97d2014-04-17 11:37:46 +0530105 def get_voucher_details(self, default_expense_account, default_cost_center, sle_map):
106 if self.doctype == "Stock Reconciliation":
107 return [frappe._dict({ "name": voucher_detail_no, "expense_account": default_expense_account,
108 "cost_center": default_cost_center }) for voucher_detail_no, sle in sle_map.items()]
109 else:
Nabin Haitdd38a262014-12-26 13:15:21 +0530110 details = self.get("items")
Anand Doshi094610d2014-04-16 19:56:53 +0530111
Anand Doshide1a97d2014-04-17 11:37:46 +0530112 if default_expense_account or default_cost_center:
113 for d in details:
114 if default_expense_account and not d.get("expense_account"):
115 d.expense_account = default_expense_account
116 if default_cost_center and not d.get("cost_center"):
117 d.cost_center = default_cost_center
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530118
Anand Doshide1a97d2014-04-17 11:37:46 +0530119 return details
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530120
Nabin Haitb9e04812014-09-26 14:22:18 +0530121 def get_items_and_warehouses(self):
Nabin Haitbecf75d2014-03-27 17:18:29 +0530122 items, warehouses = [], []
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530123
Nabin Haitdd38a262014-12-26 13:15:21 +0530124 if hasattr(self, "items"):
125 item_doclist = self.get("items")
Anand Doshif78d1ae2014-03-28 13:55:00 +0530126 elif self.doctype == "Stock Reconciliation":
Nabin Haitbecf75d2014-03-27 17:18:29 +0530127 import json
128 item_doclist = []
Anand Doshif78d1ae2014-03-28 13:55:00 +0530129 data = json.loads(self.reconciliation_json)
Nabin Haitbecf75d2014-03-27 17:18:29 +0530130 for row in data[data.index(self.head_row)+1:]:
131 d = frappe._dict(zip(["item_code", "warehouse", "qty", "valuation_rate"], row))
132 item_doclist.append(d)
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530133
Nabin Haitbecf75d2014-03-27 17:18:29 +0530134 if item_doclist:
135 for d in item_doclist:
136 if d.item_code and d.item_code not in items:
137 items.append(d.item_code)
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530138
Nabin Haitdc82d4f2014-04-07 12:02:57 +0530139 if d.get("warehouse") and d.warehouse not in warehouses:
Nabin Haitbecf75d2014-03-27 17:18:29 +0530140 warehouses.append(d.warehouse)
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530141
Nabin Haitdc82d4f2014-04-07 12:02:57 +0530142 if self.doctype == "Stock Entry":
143 if d.get("s_warehouse") and d.s_warehouse not in warehouses:
144 warehouses.append(d.s_warehouse)
145 if d.get("t_warehouse") and d.t_warehouse not in warehouses:
146 warehouses.append(d.t_warehouse)
Nabin Haitbecf75d2014-03-27 17:18:29 +0530147
Nabin Haitb9e04812014-09-26 14:22:18 +0530148 return items, warehouses
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530149
Nabin Hait2e296fa2013-08-28 18:53:11 +0530150 def get_stock_ledger_details(self):
151 stock_ledger = {}
Kanchan Chauhanb77eaf72017-01-28 13:18:49 +0530152 for sle in frappe.db.sql("""select name, warehouse, stock_value_difference,
Nabin Hait8c61f342016-12-15 13:46:03 +0530153 voucher_detail_no, item_code, posting_date, posting_time, actual_qty, qty_after_transaction
Nabin Hait27994c22013-08-26 16:53:30 +0530154 from `tabStock Ledger Entry` where voucher_type=%s and voucher_no=%s""",
Anand Doshif78d1ae2014-03-28 13:55:00 +0530155 (self.doctype, self.name), as_dict=True):
Nabin Hait2e296fa2013-08-28 18:53:11 +0530156 stock_ledger.setdefault(sle.voucher_detail_no, []).append(sle)
157 return stock_ledger
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530158
Nabin Hait2e296fa2013-08-28 18:53:11 +0530159 def make_adjustment_entry(self, expected_gle, voucher_obj):
Rushabh Mehta1f847992013-12-12 19:12:19 +0530160 from erpnext.accounts.utils import get_stock_and_account_difference
Nabin Hait27994c22013-08-26 16:53:30 +0530161 account_list = [d.account for d in expected_gle]
162 acc_diff = get_stock_and_account_difference(account_list, expected_gle[0].posting_date)
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530163
Nabin Hait27994c22013-08-26 16:53:30 +0530164 cost_center = self.get_company_default("cost_center")
165 stock_adjustment_account = self.get_company_default("stock_adjustment_account")
166
Nabin Haitd4741942013-08-06 15:57:25 +0530167 gl_entries = []
168 for account, diff in acc_diff.items():
169 if diff:
Nabin Hait27994c22013-08-26 16:53:30 +0530170 gl_entries.append([
171 # stock in hand account
Nabin Hait2e296fa2013-08-28 18:53:11 +0530172 voucher_obj.get_gl_dict({
Nabin Hait27994c22013-08-26 16:53:30 +0530173 "account": account,
174 "against": stock_adjustment_account,
175 "debit": diff,
176 "remarks": "Adjustment Accounting Entry for Stock",
177 }),
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530178
Nabin Hait27994c22013-08-26 16:53:30 +0530179 # account against stock in hand
Nabin Hait2e296fa2013-08-28 18:53:11 +0530180 voucher_obj.get_gl_dict({
Nabin Hait27994c22013-08-26 16:53:30 +0530181 "account": stock_adjustment_account,
182 "against": account,
183 "credit": diff,
184 "cost_center": cost_center or None,
185 "remarks": "Adjustment Accounting Entry for Stock",
186 }),
187 ])
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530188
Nabin Haitd4741942013-08-06 15:57:25 +0530189 if gl_entries:
Rushabh Mehta1f847992013-12-12 19:12:19 +0530190 from erpnext.accounts.general_ledger import make_gl_entries
Nabin Haitd4741942013-08-06 15:57:25 +0530191 make_gl_entries(gl_entries)
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530192
Nabin Hait27994c22013-08-26 16:53:30 +0530193 def check_expense_account(self, item):
Rushabh Mehta052fe822014-04-16 19:20:11 +0530194 if not item.get("expense_account"):
Nabin Haitd3412542014-05-01 17:42:21 +0530195 frappe.throw(_("Expense or Difference account is mandatory for Item {0} as it impacts overall stock value").format(item.item_code))
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530196
Anand Doshi496123a2014-06-19 19:25:19 +0530197 else:
Nabin Hait00778c92014-06-25 19:12:24 +0530198 is_expense_account = frappe.db.get_value("Account",
199 item.get("expense_account"), "report_type")=="Profit and Loss"
jof2jc524ebbc2016-02-08 23:44:55 +0700200 if self.doctype not in ("Purchase Receipt", "Purchase Invoice", "Stock Reconciliation", "Stock Entry") and not is_expense_account:
Nabin Hait00778c92014-06-25 19:12:24 +0530201 frappe.throw(_("Expense / Difference account ({0}) must be a 'Profit or Loss' account")
202 .format(item.get("expense_account")))
Anand Doshi496123a2014-06-19 19:25:19 +0530203 if is_expense_account and not item.get("cost_center"):
204 frappe.throw(_("{0} {1}: Cost Center is mandatory for Item {2}").format(
205 _(self.doctype), self.name, item.get("item_code")))
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530206
207 def get_sl_entries(self, d, args):
Nabin Hait5288bde2014-11-03 15:08:21 +0530208 sl_dict = frappe._dict({
Nabin Hait296d6262014-05-02 17:36:13 +0530209 "item_code": d.get("item_code", None),
Nabin Haitdc82d4f2014-04-07 12:02:57 +0530210 "warehouse": d.get("warehouse", None),
Anand Doshif78d1ae2014-03-28 13:55:00 +0530211 "posting_date": self.posting_date,
212 "posting_time": self.posting_time,
Valmik Janglad89dcf62016-03-04 14:33:49 +0530213 'fiscal_year': get_fiscal_year(self.posting_date, company=self.company)[0],
Anand Doshif78d1ae2014-03-28 13:55:00 +0530214 "voucher_type": self.doctype,
215 "voucher_no": self.name,
Nabin Hait1e2f20a2013-08-02 11:42:11 +0530216 "voucher_detail_no": d.name,
Nabin Haitdc82d4f2014-04-07 12:02:57 +0530217 "actual_qty": (self.docstatus==1 and 1 or -1)*flt(d.get("stock_qty")),
Nabin Haitc865f222015-09-21 09:18:43 +0530218 "stock_uom": frappe.db.get_value("Item", args.get("item_code") or d.get("item_code"), "stock_uom"),
Nabin Hait1e2f20a2013-08-02 11:42:11 +0530219 "incoming_rate": 0,
Anand Doshif78d1ae2014-03-28 13:55:00 +0530220 "company": self.company,
Nabin Hait296d6262014-05-02 17:36:13 +0530221 "batch_no": cstr(d.get("batch_no")).strip(),
222 "serial_no": d.get("serial_no"),
Neil Trini Lasrado6e343e22016-03-09 17:02:59 +0530223 "project": d.get("project"),
Anand Doshif78d1ae2014-03-28 13:55:00 +0530224 "is_cancelled": self.docstatus==2 and "Yes" or "No"
Nabin Hait5288bde2014-11-03 15:08:21 +0530225 })
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530226
Nabin Hait1e2f20a2013-08-02 11:42:11 +0530227 sl_dict.update(args)
228 return sl_dict
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530229
Anand Doshic3997092015-04-06 19:29:16 +0530230 def make_sl_entries(self, sl_entries, is_amended=None, allow_negative_stock=False,
Nabin Hait54c865e2015-03-27 15:38:31 +0530231 via_landed_cost_voucher=False):
Rushabh Mehta1f847992013-12-12 19:12:19 +0530232 from erpnext.stock.stock_ledger import make_sl_entries
Nabin Hait54c865e2015-03-27 15:38:31 +0530233 make_sl_entries(sl_entries, is_amended, allow_negative_stock, via_landed_cost_voucher)
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530234
Nabin Hait3b6dc142016-08-19 16:39:33 +0530235 def make_gl_entries_on_cancel(self, repost_future_gle=True):
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530236 if frappe.db.sql("""select name from `tabGL Entry` where voucher_type=%s
Anand Doshif78d1ae2014-03-28 13:55:00 +0530237 and voucher_no=%s""", (self.doctype, self.name)):
Nabin Hait3b6dc142016-08-19 16:39:33 +0530238 self.make_gl_entries(repost_future_gle)
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530239
Anand Doshia740f752014-06-25 13:31:02 +0530240 def get_serialized_items(self):
241 serialized_items = []
Nabin Haitdd38a262014-12-26 13:15:21 +0530242 item_codes = list(set([d.item_code for d in self.get("items")]))
Anand Doshia740f752014-06-25 13:31:02 +0530243 if item_codes:
244 serialized_items = frappe.db.sql_list("""select name from `tabItem`
Rushabh Mehta1e8025b2015-07-24 15:16:25 +0530245 where has_serial_no=1 and name in ({})""".format(", ".join(["%s"]*len(item_codes))),
Anand Doshia740f752014-06-25 13:31:02 +0530246 tuple(item_codes))
247
248 return serialized_items
Rushabh Mehtab16b9cd2015-08-03 16:13:33 +0530249
Nabin Hait20105d82016-01-19 18:05:28 +0530250 def get_incoming_rate_for_sales_return(self, item_code, against_document):
Nabin Hait1d218422015-07-17 15:19:02 +0530251 incoming_rate = 0.0
252 if against_document and item_code:
Anand Doshi602e8252015-11-16 19:05:46 +0530253 incoming_rate = frappe.db.sql("""select abs(stock_value_difference / actual_qty)
Nabin Hait1d218422015-07-17 15:19:02 +0530254 from `tabStock Ledger Entry`
Rushabh Mehta14a908b2015-10-15 12:28:20 +0530255 where voucher_type = %s and voucher_no = %s
Nabin Hait20105d82016-01-19 18:05:28 +0530256 and item_code = %s limit 1""",
257 (self.doctype, against_document, item_code))
Nabin Hait1d218422015-07-17 15:19:02 +0530258 incoming_rate = incoming_rate[0][0] if incoming_rate else 0.0
259
260 return incoming_rate
Saurabh2e292062015-11-18 17:03:33 +0530261
262 def validate_warehouse(self):
263 from erpnext.stock.utils import validate_warehouse_company
264
265 warehouses = list(set([d.warehouse for d in
266 self.get("items") if getattr(d, "warehouse", None)]))
267
268 for w in warehouses:
269 validate_warehouse_company(w, self.company)
Nabin Haitbdab0ee2015-12-30 19:08:11 +0530270
Anand Doshi6b71ef52016-01-06 16:32:06 +0530271 def update_billing_percentage(self, update_modified=True):
Nabin Haitbdab0ee2015-12-30 19:08:11 +0530272 self._update_percent_field({
273 "target_dt": self.doctype + " Item",
274 "target_parent_dt": self.doctype,
275 "target_parent_field": "per_billed",
276 "target_ref_field": "amount",
277 "target_field": "billed_amt",
278 "name": self.name,
Anand Doshi6b71ef52016-01-06 16:32:06 +0530279 }, update_modified)
Anand Doshia740f752014-06-25 13:31:02 +0530280
Nabin Hait8af429d2016-11-16 17:21:59 +0530281 def validate_inspection(self):
Manas Solankicc902412016-11-10 19:15:11 +0530282 '''Checks if quality inspection is set for Items that require inspection.
283 On submit, throw an exception'''
Nabin Hait8af429d2016-11-16 17:21:59 +0530284
Manas Solanki60db98a2016-11-18 14:52:13 +0530285 inspection_required_fieldname = None
Nabin Hait8af429d2016-11-16 17:21:59 +0530286 if self.doctype in ["Purchase Receipt", "Purchase Invoice"]:
287 inspection_required_fieldname = "inspection_required_before_purchase"
288 elif self.doctype in ["Delivery Note", "Sales Invoice"]:
289 inspection_required_fieldname = "inspection_required_before_delivery"
290
291 if not inspection_required_fieldname or \
292 (self.doctype in ["Sales Invoice", "Purchase Invoice"] and not self.update_stock):
293 return
294
Manas Solankicc902412016-11-10 19:15:11 +0530295 for d in self.get('items'):
Nabin Hait8af429d2016-11-16 17:21:59 +0530296 if (frappe.db.get_value("Item", d.item_code, inspection_required_fieldname)
Manas Solankicc902412016-11-10 19:15:11 +0530297 and not d.quality_inspection):
Nabin Hait8af429d2016-11-16 17:21:59 +0530298
Manas Solankicc902412016-11-10 19:15:11 +0530299 frappe.msgprint(_("Quality Inspection required for Item {0}").format(d.item_code))
300 if self.docstatus==1:
301 raise frappe.ValidationError
302
Nabin Haitadeb9762014-10-06 11:53:52 +0530303def update_gl_entries_after(posting_date, posting_time, for_warehouses=None, for_items=None,
Nabin Haitf1a07ff2014-10-15 12:23:35 +0530304 warehouse_account=None):
Nabin Haitbecf75d2014-03-27 17:18:29 +0530305 def _delete_gl_entries(voucher_type, voucher_no):
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530306 frappe.db.sql("""delete from `tabGL Entry`
Nabin Haitbecf75d2014-03-27 17:18:29 +0530307 where voucher_type=%s and voucher_no=%s""", (voucher_type, voucher_no))
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530308
Nabin Haitbecf75d2014-03-27 17:18:29 +0530309 if not warehouse_account:
310 warehouse_account = get_warehouse_account()
Nabin Haitb9e04812014-09-26 14:22:18 +0530311
312 future_stock_vouchers = get_future_stock_vouchers(posting_date, posting_time, for_warehouses, for_items)
Nabin Haitbecf75d2014-03-27 17:18:29 +0530313 gle = get_voucherwise_gl_entries(future_stock_vouchers, posting_date)
Nabin Hait3b6dc142016-08-19 16:39:33 +0530314
Nabin Haitbecf75d2014-03-27 17:18:29 +0530315 for voucher_type, voucher_no in future_stock_vouchers:
316 existing_gle = gle.get((voucher_type, voucher_no), [])
Rushabh Mehta0a0f2492014-03-31 17:27:06 +0530317 voucher_obj = frappe.get_doc(voucher_type, voucher_no)
Nabin Haitbecf75d2014-03-27 17:18:29 +0530318 expected_gle = voucher_obj.get_gl_entries(warehouse_account)
319 if expected_gle:
Nabin Hait99d4ff42016-08-08 14:12:16 +0530320 if not existing_gle or not compare_existing_and_expected_gle(existing_gle, expected_gle):
321 _delete_gl_entries(voucher_type, voucher_no)
322 voucher_obj.make_gl_entries(repost_future_gle=False)
Nabin Haitbecf75d2014-03-27 17:18:29 +0530323 else:
324 _delete_gl_entries(voucher_type, voucher_no)
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530325
Nabin Haitbecf75d2014-03-27 17:18:29 +0530326def compare_existing_and_expected_gle(existing_gle, expected_gle):
327 matched = True
328 for entry in expected_gle:
robert schouten64657112016-09-12 11:17:14 +0800329 account_existed = False
Nabin Haitbecf75d2014-03-27 17:18:29 +0530330 for e in existing_gle:
robert schouten64657112016-09-12 11:17:14 +0800331 if entry.account == e.account:
332 account_existed = True
333 if entry.account == e.account and entry.against_account == e.against_account \
334 and (not entry.cost_center or not e.cost_center or entry.cost_center == e.cost_center) \
335 and (entry.debit != e.debit or entry.credit != e.credit):
336 matched = False
337 break
338 if not account_existed:
339 matched = False
340 break
Nabin Haitbecf75d2014-03-27 17:18:29 +0530341 return matched
342
Nabin Haitb9e04812014-09-26 14:22:18 +0530343def get_future_stock_vouchers(posting_date, posting_time, for_warehouses=None, for_items=None):
Nabin Haitbecf75d2014-03-27 17:18:29 +0530344 future_stock_vouchers = []
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530345
Anand Doshi913c51b2014-08-27 22:09:03 +0530346 values = []
Nabin Haitbecf75d2014-03-27 17:18:29 +0530347 condition = ""
348 if for_items:
Anand Doshi913c51b2014-08-27 22:09:03 +0530349 condition += " and item_code in ({})".format(", ".join(["%s"] * len(for_items)))
350 values += for_items
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530351
Nabin Haitb9e04812014-09-26 14:22:18 +0530352 if for_warehouses:
353 condition += " and warehouse in ({})".format(", ".join(["%s"] * len(for_warehouses)))
354 values += for_warehouses
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530355
356 for d in frappe.db.sql("""select distinct sle.voucher_type, sle.voucher_no
Nabin Haitbecf75d2014-03-27 17:18:29 +0530357 from `tabStock Ledger Entry` sle
Anand Doshi913c51b2014-08-27 22:09:03 +0530358 where timestamp(sle.posting_date, sle.posting_time) >= timestamp(%s, %s) {condition}
359 order by timestamp(sle.posting_date, sle.posting_time) asc, name asc""".format(condition=condition),
360 tuple([posting_date, posting_time] + values), as_dict=True):
Nabin Haitbecf75d2014-03-27 17:18:29 +0530361 future_stock_vouchers.append([d.voucher_type, d.voucher_no])
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530362
Nabin Haitbecf75d2014-03-27 17:18:29 +0530363 return future_stock_vouchers
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530364
Nabin Haitbecf75d2014-03-27 17:18:29 +0530365def get_voucherwise_gl_entries(future_stock_vouchers, posting_date):
366 gl_entries = {}
367 if future_stock_vouchers:
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530368 for d in frappe.db.sql("""select * from `tabGL Entry`
369 where posting_date >= %s and voucher_no in (%s)""" %
370 ('%s', ', '.join(['%s']*len(future_stock_vouchers))),
Nabin Haitbecf75d2014-03-27 17:18:29 +0530371 tuple([posting_date] + [d[1] for d in future_stock_vouchers]), as_dict=1):
372 gl_entries.setdefault((d.voucher_type, d.voucher_no), []).append(d)
Anand Doshi2ce39cf2014-04-07 18:51:58 +0530373
Nabin Haitbecf75d2014-03-27 17:18:29 +0530374 return gl_entries
375
376def get_warehouse_account():
Nabin Hait46bcbaf2015-08-19 13:49:10 +0530377 warehouse_account = frappe._dict()
Rushabh Mehta14a908b2015-10-15 12:28:20 +0530378
Nabin Hait6e439a52015-08-28 19:24:22 +0530379 for d in frappe.db.sql("""select warehouse, name, account_currency from tabAccount
Saurabh7fecb502016-06-24 13:11:29 +0530380 where account_type = 'Stock' and (warehouse is not null and warehouse != ''
Saurabh78333c72016-06-25 14:18:28 +0530381 and is_group != 1) and is_group=0 """, as_dict=1):
Nabin Hait46bcbaf2015-08-19 13:49:10 +0530382 warehouse_account.setdefault(d.warehouse, d)
Anand Doshi67d6a4e2014-03-28 13:41:59 +0530383 return warehouse_account