Rushabh Mehta | ad45e31 | 2013-11-20 12:59:58 +0530 | [diff] [blame] | 1 | # Copyright (c) 2013, Web Notes 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 |
Nabin Hait | d474194 | 2013-08-06 15:57:25 +0530 | [diff] [blame] | 9 | |
Rushabh Mehta | 1f84799 | 2013-12-12 19:12:19 +0530 | [diff] [blame] | 10 | from erpnext.controllers.accounts_controller import AccountsController |
| 11 | from erpnext.accounts.general_ledger import make_gl_entries, delete_gl_entries |
Nabin Hait | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 12 | |
| 13 | class StockController(AccountsController): |
Nabin Hait | e83069c | 2013-11-14 18:40:08 +0530 | [diff] [blame] | 14 | def make_gl_entries(self, update_gl_entries_after=True): |
Nabin Hait | 145e5e2 | 2013-10-22 23:51:41 +0530 | [diff] [blame] | 15 | if self.doc.docstatus == 2: |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 16 | delete_gl_entries(voucher_type=self.doc.doctype, voucher_no=self.doc.name) |
Nabin Hait | 145e5e2 | 2013-10-22 23:51:41 +0530 | [diff] [blame] | 17 | |
Rushabh Mehta | 793ba6b | 2014-02-14 15:47:51 +0530 | [diff] [blame] | 18 | if cint(frappe.defaults.get_global_default("auto_accounting_for_stock")): |
Nabin Hait | 145e5e2 | 2013-10-22 23:51:41 +0530 | [diff] [blame] | 19 | warehouse_account = self.get_warehouse_account() |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 20 | |
Nabin Hait | 145e5e2 | 2013-10-22 23:51:41 +0530 | [diff] [blame] | 21 | if self.doc.docstatus==1: |
Nabin Hait | e83069c | 2013-11-14 18:40:08 +0530 | [diff] [blame] | 22 | gl_entries = self.get_gl_entries(warehouse_account) |
Nabin Hait | 145e5e2 | 2013-10-22 23:51:41 +0530 | [diff] [blame] | 23 | make_gl_entries(gl_entries) |
| 24 | |
Nabin Hait | e83069c | 2013-11-14 18:40:08 +0530 | [diff] [blame] | 25 | if update_gl_entries_after: |
| 26 | self.update_gl_entries_after(warehouse_account) |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 27 | |
Nabin Hait | e83069c | 2013-11-14 18:40:08 +0530 | [diff] [blame] | 28 | def get_gl_entries(self, warehouse_account=None, default_expense_account=None, |
Nabin Hait | 142007a | 2013-09-17 15:15:16 +0530 | [diff] [blame] | 29 | default_cost_center=None): |
Rushabh Mehta | 1f84799 | 2013-12-12 19:12:19 +0530 | [diff] [blame] | 30 | from erpnext.accounts.general_ledger import process_gl_map |
Nabin Hait | 142007a | 2013-09-17 15:15:16 +0530 | [diff] [blame] | 31 | if not warehouse_account: |
| 32 | warehouse_account = self.get_warehouse_account() |
| 33 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 34 | stock_ledger = self.get_stock_ledger_details() |
| 35 | voucher_details = self.get_voucher_details(stock_ledger, default_expense_account, |
| 36 | default_cost_center) |
| 37 | |
| 38 | gl_list = [] |
Nabin Hait | 7a75e10 | 2013-09-17 10:21:20 +0530 | [diff] [blame] | 39 | warehouse_with_no_account = [] |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 40 | for detail in voucher_details: |
| 41 | sle_list = stock_ledger.get(detail.name) |
| 42 | if sle_list: |
| 43 | for sle in sle_list: |
| 44 | if warehouse_account.get(sle.warehouse): |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 45 | # from warehouse account |
| 46 | gl_list.append(self.get_gl_dict({ |
| 47 | "account": warehouse_account[sle.warehouse], |
| 48 | "against": detail.expense_account, |
| 49 | "cost_center": detail.cost_center, |
| 50 | "remarks": self.doc.remarks or "Accounting Entry for Stock", |
Nabin Hait | 48bd4a1 | 2013-11-18 12:42:55 +0530 | [diff] [blame] | 51 | "debit": flt(sle.stock_value_difference, 2) |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 52 | })) |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 53 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 54 | # to target warehouse / expense account |
| 55 | gl_list.append(self.get_gl_dict({ |
| 56 | "account": detail.expense_account, |
| 57 | "against": warehouse_account[sle.warehouse], |
| 58 | "cost_center": detail.cost_center, |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 59 | "remarks": self.doc.remarks or "Accounting Entry for Stock", |
Nabin Hait | 48bd4a1 | 2013-11-18 12:42:55 +0530 | [diff] [blame] | 60 | "credit": flt(sle.stock_value_difference, 2) |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 61 | })) |
Nabin Hait | 7a75e10 | 2013-09-17 10:21:20 +0530 | [diff] [blame] | 62 | elif sle.warehouse not in warehouse_with_no_account: |
| 63 | warehouse_with_no_account.append(sle.warehouse) |
| 64 | |
| 65 | if warehouse_with_no_account: |
| 66 | msgprint(_("No accounting entries for following warehouses") + ": \n" + |
| 67 | "\n".join(warehouse_with_no_account)) |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 68 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 69 | return process_gl_map(gl_list) |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 70 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 71 | def get_voucher_details(self, stock_ledger, default_expense_account, default_cost_center): |
| 72 | if not default_expense_account: |
| 73 | details = self.doclist.get({"parentfield": self.fname}) |
| 74 | for d in details: |
| 75 | self.check_expense_account(d) |
| 76 | else: |
Rushabh Mehta | 793ba6b | 2014-02-14 15:47:51 +0530 | [diff] [blame] | 77 | details = [frappe._dict({ |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 78 | "name":d, |
| 79 | "expense_account": default_expense_account, |
| 80 | "cost_center": default_cost_center |
| 81 | }) for d in stock_ledger.keys()] |
| 82 | |
| 83 | return details |
| 84 | |
| 85 | def get_stock_ledger_details(self): |
| 86 | stock_ledger = {} |
Anand Doshi | e9baaa6 | 2014-02-26 12:35:33 +0530 | [diff] [blame] | 87 | for sle in frappe.db.sql("""select warehouse, stock_value_difference, voucher_detail_no |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 88 | from `tabStock Ledger Entry` where voucher_type=%s and voucher_no=%s""", |
| 89 | (self.doc.doctype, self.doc.name), as_dict=True): |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 90 | stock_ledger.setdefault(sle.voucher_detail_no, []).append(sle) |
| 91 | return stock_ledger |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 92 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 93 | def get_warehouse_account(self): |
Anand Doshi | e9baaa6 | 2014-02-26 12:35:33 +0530 | [diff] [blame] | 94 | warehouse_account = dict(frappe.db.sql("""select master_name, name from tabAccount |
Nabin Hait | 7a75e10 | 2013-09-17 10:21:20 +0530 | [diff] [blame] | 95 | where account_type = 'Warehouse' and ifnull(master_name, '') != ''""")) |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 96 | return warehouse_account |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 97 | |
Nabin Hait | 142007a | 2013-09-17 15:15:16 +0530 | [diff] [blame] | 98 | def update_gl_entries_after(self, warehouse_account=None): |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 99 | future_stock_vouchers = self.get_future_stock_vouchers() |
| 100 | gle = self.get_voucherwise_gl_entries(future_stock_vouchers) |
Nabin Hait | 142007a | 2013-09-17 15:15:16 +0530 | [diff] [blame] | 101 | if not warehouse_account: |
| 102 | warehouse_account = self.get_warehouse_account() |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 103 | for voucher_type, voucher_no in future_stock_vouchers: |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 104 | existing_gle = gle.get((voucher_type, voucher_no), []) |
Rushabh Mehta | 793ba6b | 2014-02-14 15:47:51 +0530 | [diff] [blame] | 105 | voucher_obj = frappe.get_obj(voucher_type, voucher_no) |
Nabin Hait | e83069c | 2013-11-14 18:40:08 +0530 | [diff] [blame] | 106 | expected_gle = voucher_obj.get_gl_entries(warehouse_account) |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 107 | if expected_gle: |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 108 | matched = True |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 109 | if existing_gle: |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 110 | for entry in expected_gle: |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 111 | for e in existing_gle: |
| 112 | if entry.account==e.account \ |
| 113 | and entry.against_account==e.against_account\ |
| 114 | and entry.cost_center==e.cost_center: |
| 115 | if entry.debit != e.debit or entry.credit != e.credit: |
| 116 | matched = False |
| 117 | break |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 118 | else: |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 119 | matched = False |
| 120 | |
| 121 | if not matched: |
| 122 | self.delete_gl_entries(voucher_type, voucher_no) |
Nabin Hait | e83069c | 2013-11-14 18:40:08 +0530 | [diff] [blame] | 123 | voucher_obj.make_gl_entries(update_gl_entries_after=False) |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 124 | else: |
| 125 | self.delete_gl_entries(voucher_type, voucher_no) |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 126 | |
| 127 | |
| 128 | def get_future_stock_vouchers(self): |
| 129 | future_stock_vouchers = [] |
Nabin Hait | bf4d27e | 2013-09-04 17:55:45 +0530 | [diff] [blame] | 130 | |
| 131 | if hasattr(self, "fname"): |
| 132 | item_list = [d.item_code for d in self.doclist.get({"parentfield": self.fname})] |
| 133 | condition = ''.join(['and item_code in (\'', '\', \''.join(item_list) ,'\')']) |
| 134 | else: |
| 135 | condition = "" |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 136 | |
Anand Doshi | e9baaa6 | 2014-02-26 12:35:33 +0530 | [diff] [blame] | 137 | for d in frappe.db.sql("""select distinct sle.voucher_type, sle.voucher_no |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 138 | from `tabStock Ledger Entry` sle |
Nabin Hait | bf4d27e | 2013-09-04 17:55:45 +0530 | [diff] [blame] | 139 | where timestamp(sle.posting_date, sle.posting_time) >= timestamp(%s, %s) %s |
| 140 | order by timestamp(sle.posting_date, sle.posting_time) asc, name asc""" % |
| 141 | ('%s', '%s', condition), (self.doc.posting_date, self.doc.posting_time), |
| 142 | as_dict=True): |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 143 | future_stock_vouchers.append([d.voucher_type, d.voucher_no]) |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 144 | |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 145 | return future_stock_vouchers |
| 146 | |
| 147 | def get_voucherwise_gl_entries(self, future_stock_vouchers): |
| 148 | gl_entries = {} |
| 149 | if future_stock_vouchers: |
Anand Doshi | e9baaa6 | 2014-02-26 12:35:33 +0530 | [diff] [blame] | 150 | for d in frappe.db.sql("""select * from `tabGL Entry` |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 151 | where posting_date >= %s and voucher_no in (%s)""" % |
| 152 | ('%s', ', '.join(['%s']*len(future_stock_vouchers))), |
| 153 | tuple([self.doc.posting_date] + [d[1] for d in future_stock_vouchers]), as_dict=1): |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 154 | gl_entries.setdefault((d.voucher_type, d.voucher_no), []).append(d) |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 155 | |
| 156 | return gl_entries |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 157 | |
| 158 | def delete_gl_entries(self, voucher_type, voucher_no): |
Anand Doshi | e9baaa6 | 2014-02-26 12:35:33 +0530 | [diff] [blame] | 159 | frappe.db.sql("""delete from `tabGL Entry` |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 160 | where voucher_type=%s and voucher_no=%s""", (voucher_type, voucher_no)) |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 161 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 162 | def make_adjustment_entry(self, expected_gle, voucher_obj): |
Rushabh Mehta | 1f84799 | 2013-12-12 19:12:19 +0530 | [diff] [blame] | 163 | from erpnext.accounts.utils import get_stock_and_account_difference |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 164 | account_list = [d.account for d in expected_gle] |
| 165 | acc_diff = get_stock_and_account_difference(account_list, expected_gle[0].posting_date) |
| 166 | |
| 167 | cost_center = self.get_company_default("cost_center") |
| 168 | stock_adjustment_account = self.get_company_default("stock_adjustment_account") |
| 169 | |
Nabin Hait | d474194 | 2013-08-06 15:57:25 +0530 | [diff] [blame] | 170 | gl_entries = [] |
| 171 | for account, diff in acc_diff.items(): |
| 172 | if diff: |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 173 | gl_entries.append([ |
| 174 | # stock in hand account |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 175 | voucher_obj.get_gl_dict({ |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 176 | "account": account, |
| 177 | "against": stock_adjustment_account, |
| 178 | "debit": diff, |
| 179 | "remarks": "Adjustment Accounting Entry for Stock", |
| 180 | }), |
| 181 | |
| 182 | # account against stock in hand |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 183 | voucher_obj.get_gl_dict({ |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 184 | "account": stock_adjustment_account, |
| 185 | "against": account, |
| 186 | "credit": diff, |
| 187 | "cost_center": cost_center or None, |
| 188 | "remarks": "Adjustment Accounting Entry for Stock", |
| 189 | }), |
| 190 | ]) |
| 191 | |
Nabin Hait | d474194 | 2013-08-06 15:57:25 +0530 | [diff] [blame] | 192 | if gl_entries: |
Rushabh Mehta | 1f84799 | 2013-12-12 19:12:19 +0530 | [diff] [blame] | 193 | from erpnext.accounts.general_ledger import make_gl_entries |
Nabin Hait | d474194 | 2013-08-06 15:57:25 +0530 | [diff] [blame] | 194 | make_gl_entries(gl_entries) |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 195 | |
| 196 | def check_expense_account(self, item): |
| 197 | if item.fields.has_key("expense_account") and not item.expense_account: |
Nabin Hait | 678130f | 2013-09-02 18:10:36 +0530 | [diff] [blame] | 198 | msgprint(_("""Expense/Difference account is mandatory for item: """) + item.item_code, |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 199 | raise_exception=1) |
| 200 | |
| 201 | if item.fields.has_key("expense_account") and not item.cost_center: |
| 202 | msgprint(_("""Cost Center is mandatory for item: """) + item.item_code, |
| 203 | raise_exception=1) |
Nabin Hait | d474194 | 2013-08-06 15:57:25 +0530 | [diff] [blame] | 204 | |
Nabin Hait | 1e2f20a | 2013-08-02 11:42:11 +0530 | [diff] [blame] | 205 | def get_sl_entries(self, d, args): |
| 206 | sl_dict = { |
| 207 | "item_code": d.item_code, |
| 208 | "warehouse": d.warehouse, |
| 209 | "posting_date": self.doc.posting_date, |
| 210 | "posting_time": self.doc.posting_time, |
| 211 | "voucher_type": self.doc.doctype, |
| 212 | "voucher_no": self.doc.name, |
| 213 | "voucher_detail_no": d.name, |
| 214 | "actual_qty": (self.doc.docstatus==1 and 1 or -1)*flt(d.stock_qty), |
| 215 | "stock_uom": d.stock_uom, |
| 216 | "incoming_rate": 0, |
| 217 | "company": self.doc.company, |
| 218 | "fiscal_year": self.doc.fiscal_year, |
Nabin Hait | 1e2f20a | 2013-08-02 11:42:11 +0530 | [diff] [blame] | 219 | "batch_no": cstr(d.batch_no).strip(), |
| 220 | "serial_no": d.serial_no, |
Nabin Hait | 74c281c | 2013-08-19 16:17:18 +0530 | [diff] [blame] | 221 | "project": d.project_name, |
Nabin Hait | 9653f60 | 2013-08-20 15:37:33 +0530 | [diff] [blame] | 222 | "is_cancelled": self.doc.docstatus==2 and "Yes" or "No" |
Nabin Hait | 1e2f20a | 2013-08-02 11:42:11 +0530 | [diff] [blame] | 223 | } |
| 224 | |
| 225 | sl_dict.update(args) |
| 226 | return sl_dict |
| 227 | |
| 228 | def make_sl_entries(self, sl_entries, is_amended=None): |
Rushabh Mehta | 1f84799 | 2013-12-12 19:12:19 +0530 | [diff] [blame] | 229 | from erpnext.stock.stock_ledger import make_sl_entries |
Nabin Hait | 74c281c | 2013-08-19 16:17:18 +0530 | [diff] [blame] | 230 | make_sl_entries(sl_entries, is_amended) |
| 231 | |
Nabin Hait | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 232 | def get_stock_ledger_entries(self, item_list=None, warehouse_list=None): |
Anand Doshi | 5dd6b1d | 2013-08-07 19:27:30 +0530 | [diff] [blame] | 233 | out = {} |
| 234 | |
Nabin Hait | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 235 | if not (item_list and warehouse_list): |
| 236 | item_list, warehouse_list = self.get_distinct_item_warehouse() |
| 237 | |
| 238 | if item_list and warehouse_list: |
Anand Doshi | e9baaa6 | 2014-02-26 12:35:33 +0530 | [diff] [blame] | 239 | res = frappe.db.sql("""select item_code, voucher_type, voucher_no, |
Nabin Hait | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 240 | voucher_detail_no, posting_date, posting_time, stock_value, |
| 241 | warehouse, actual_qty as qty from `tabStock Ledger Entry` |
Nabin Hait | 4ae729b | 2013-08-20 12:04:46 +0530 | [diff] [blame] | 242 | where company = %s and item_code in (%s) and warehouse in (%s) |
Nabin Hait | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 243 | order by item_code desc, warehouse desc, posting_date desc, |
| 244 | posting_time desc, name desc""" % |
| 245 | ('%s', ', '.join(['%s']*len(item_list)), ', '.join(['%s']*len(warehouse_list))), |
| 246 | tuple([self.doc.company] + item_list + warehouse_list), as_dict=1) |
Anand Doshi | 5dd6b1d | 2013-08-07 19:27:30 +0530 | [diff] [blame] | 247 | |
| 248 | for r in res: |
| 249 | if (r.item_code, r.warehouse) not in out: |
| 250 | out[(r.item_code, r.warehouse)] = [] |
| 251 | |
| 252 | out[(r.item_code, r.warehouse)].append(r) |
| 253 | |
| 254 | return out |
Nabin Hait | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 255 | |
| 256 | def get_distinct_item_warehouse(self): |
| 257 | item_list = [] |
| 258 | warehouse_list = [] |
| 259 | for item in self.doclist.get({"parentfield": self.fname}) \ |
| 260 | + self.doclist.get({"parentfield": "packing_details"}): |
| 261 | item_list.append(item.item_code) |
| 262 | warehouse_list.append(item.warehouse) |
| 263 | |
Nabin Hait | 787c02e | 2013-03-29 16:42:33 +0530 | [diff] [blame] | 264 | return list(set(item_list)), list(set(warehouse_list)) |
| 265 | |
| 266 | def make_cancel_gl_entries(self): |
Anand Doshi | e9baaa6 | 2014-02-26 12:35:33 +0530 | [diff] [blame] | 267 | if frappe.db.sql("""select name from `tabGL Entry` where voucher_type=%s |
Nabin Hait | 4ae729b | 2013-08-20 12:04:46 +0530 | [diff] [blame] | 268 | and voucher_no=%s""", (self.doc.doctype, self.doc.name)): |
Nabin Hait | 787c02e | 2013-03-29 16:42:33 +0530 | [diff] [blame] | 269 | self.make_gl_entries() |