blob: 0e15eee521cfdd660cc088b6f842e8a0327bf1c6 [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 Haitec52db82013-01-22 11:53:14 +05303
4from __future__ import unicode_literals
Rushabh Mehta793ba6b2014-02-14 15:47:51 +05305import frappe
Nabin Hait3237c752015-02-17 11:11:11 +05306from frappe.utils import cint, flt, cstr, comma_or
Rushabh Mehta9f0d6252014-04-14 19:20:45 +05307from frappe import _, throw
Saurabhd3135532016-02-25 18:59:20 +05308from erpnext.stock.get_item_details import get_bin_details
Nabin Hait14aa9c52016-04-18 15:54:01 +05309from erpnext.stock.utils import get_incoming_rate
Kanchan Chauhan5a980ac2017-02-10 14:27:11 +053010from erpnext.stock.get_item_details import get_conversion_factor
Nabin Haitec52db82013-01-22 11:53:14 +053011
Rushabh Mehta1f847992013-12-12 19:12:19 +053012from erpnext.controllers.stock_controller import StockController
Nabin Haitbf495c92013-01-30 12:49:08 +053013
Nabin Haitc3afb252013-03-19 12:01:24 +053014class SellingController(StockController):
Rushabh Mehtaf84c2402014-07-18 16:39:31 +053015 def __setup__(self):
Nabin Haitde9c8a92015-02-23 01:06:00 +053016 if hasattr(self, "taxes"):
Nabin Hait45dce892017-09-14 15:17:38 +053017 self.flags.print_taxes_with_zero_amount = cint(frappe.db.get_single_value("Print Settings",
18 "print_taxes_with_zero_amount"))
Rushabh Mehtac567e8e2015-02-03 17:55:52 +053019 self.print_templates = {
20 "taxes": "templates/print_formats/includes/taxes.html"
Rushabh Mehta5b51cc82014-07-21 18:25:45 +053021 }
Rushabh Mehtaf84c2402014-07-18 16:39:31 +053022
Rushabh Mehtac0bb4532014-09-09 16:15:35 +053023 def get_feed(self):
24 return _("To {0} | {1} {2}").format(self.customer_name, self.currency,
Nabin Hait5690be12015-02-12 16:09:11 +053025 self.grand_total)
Rushabh Mehtac0bb4532014-09-09 16:15:35 +053026
Rushabh Mehtab5e76892014-07-29 16:07:43 +053027 def onload(self):
Rohit Waghchaure7d529892016-10-06 14:35:04 +053028 super(SellingController, self).onload()
Rushabh Mehtab5e76892014-07-29 16:07:43 +053029 if self.doctype in ("Sales Order", "Delivery Note", "Sales Invoice"):
Nabin Haitdd38a262014-12-26 13:15:21 +053030 for item in self.get("items"):
Nabin Hait877e1bb2017-11-17 12:27:43 +053031 item.update(get_bin_details(item.item_code, item.warehouse))
Rushabh Mehtab5e76892014-07-29 16:07:43 +053032
Nabin Haitd1fd1e22013-10-18 12:29:11 +053033 def validate(self):
34 super(SellingController, self).validate()
35 self.validate_max_discount()
shreyas1cfe2d72016-10-19 18:36:12 +053036 self.validate_selling_price()
Kanchan Chauhan5a980ac2017-02-10 14:27:11 +053037 self.set_qty_as_per_stock_uom()
Nabin Haitd1fd1e22013-10-18 12:29:11 +053038 check_active_sales_items(self)
Anand Doshid2946502014-04-08 20:10:03 +053039
Anand Doshif3096132013-05-21 19:35:06 +053040 def set_missing_values(self, for_validate=False):
Anand Doshiabc10032013-06-14 17:44:03 +053041 super(SellingController, self).set_missing_values(for_validate)
Anand Doshid2946502014-04-08 20:10:03 +053042
Anand Doshif3096132013-05-21 19:35:06 +053043 # set contact and address details for customer, if they are not mentioned
Anand Doshiabc10032013-06-14 17:44:03 +053044 self.set_missing_lead_customer_details()
mbauskar9e9d2422017-04-10 19:16:07 +053045 self.set_price_list_and_item_details(for_validate=for_validate)
Anand Doshid2946502014-04-08 20:10:03 +053046
Anand Doshiabc10032013-06-14 17:44:03 +053047 def set_missing_lead_customer_details(self):
Rushabh Mehtaf14b8092014-04-03 14:30:42 +053048 if getattr(self, "customer", None):
Anand Doshi43624432014-02-25 15:42:16 +053049 from erpnext.accounts.party import _get_party_details
Rushabh Mehta2cc585f2015-02-26 15:01:23 +053050 party_details = _get_party_details(self.customer,
Nabin Hait7eba1a32017-10-02 15:59:27 +053051 ignore_permissions=self.flags.ignore_permissions,
52 doctype=self.doctype, company=self.company)
Nabin Hait6b039142014-05-02 15:45:10 +053053 if not self.meta.get_field("sales_team"):
54 party_details.pop("sales_team")
55
56 self.update_if_missing(party_details)
Anand Doshid2946502014-04-08 20:10:03 +053057
Rushabh Mehtaf14b8092014-04-03 14:30:42 +053058 elif getattr(self, "lead", None):
Rushabh Mehta60cfccb2015-02-17 10:36:54 +053059 from erpnext.crm.doctype.lead.lead import get_lead_details
Anand Doshi2f957f22016-04-08 17:36:10 +053060 self.update_if_missing(get_lead_details(
61 self.lead,
62 posting_date=self.get('transaction_date') or self.get('posting_date'),
63 company=self.company))
Anand Doshid2946502014-04-08 20:10:03 +053064
mbauskar9e9d2422017-04-10 19:16:07 +053065 def set_price_list_and_item_details(self, for_validate=False):
Nabin Hait096d3632013-10-17 17:01:14 +053066 self.set_price_list_currency("Selling")
mbauskar287fe812017-04-10 19:15:57 +053067 self.set_missing_item_details(for_validate=for_validate)
Anand Doshid2946502014-04-08 20:10:03 +053068
Anand Doshi99100a42013-07-04 17:13:53 +053069 def apply_shipping_rule(self):
Anand Doshif78d1ae2014-03-28 13:55:00 +053070 if self.shipping_rule:
Rushabh Mehtab385ecf2014-03-28 16:44:37 +053071 shipping_rule = frappe.get_doc("Shipping Rule", self.shipping_rule)
Nabin Hait5690be12015-02-12 16:09:11 +053072 value = self.base_net_total
Anand Doshid2946502014-04-08 20:10:03 +053073
Anand Doshi99100a42013-07-04 17:13:53 +053074 # TODO
75 # shipping rule calculation based on item's net weight
Anand Doshid2946502014-04-08 20:10:03 +053076
Anand Doshi99100a42013-07-04 17:13:53 +053077 shipping_amount = 0.0
Nabin Haite7d15362014-12-25 16:01:55 +053078 for condition in shipping_rule.get("conditions"):
Anand Doshi99100a42013-07-04 17:13:53 +053079 if not condition.to_value or (flt(condition.from_value) <= value <= flt(condition.to_value)):
80 shipping_amount = condition.shipping_amount
81 break
Anand Doshid2946502014-04-08 20:10:03 +053082
Anand Doshiac32bad2014-04-18 01:30:14 +053083 shipping_charge = {
Anand Doshi0b4943c2013-07-04 23:45:22 +053084 "doctype": "Sales Taxes and Charges",
Anand Doshi0b4943c2013-07-04 23:45:22 +053085 "charge_type": "Actual",
Anand Doshif78d1ae2014-03-28 13:55:00 +053086 "account_head": shipping_rule.account,
Anand Doshiac32bad2014-04-18 01:30:14 +053087 "cost_center": shipping_rule.cost_center
88 }
89
Nabin Haite7d15362014-12-25 16:01:55 +053090 existing_shipping_charge = self.get("taxes", filters=shipping_charge)
Anand Doshiac32bad2014-04-18 01:30:14 +053091 if existing_shipping_charge:
92 # take the last record found
Anand Doshiff8a8542015-05-26 18:06:09 -040093 existing_shipping_charge[-1].tax_amount = shipping_amount
Anand Doshiac32bad2014-04-18 01:30:14 +053094 else:
Anand Doshiff8a8542015-05-26 18:06:09 -040095 shipping_charge["tax_amount"] = shipping_amount
Anand Doshiac32bad2014-04-18 01:30:14 +053096 shipping_charge["description"] = shipping_rule.label
Nabin Haite7d15362014-12-25 16:01:55 +053097 self.append("taxes", shipping_charge)
Anand Doshiac32bad2014-04-18 01:30:14 +053098
99 self.calculate_taxes_and_totals()
100
101 def remove_shipping_charge(self):
102 if self.shipping_rule:
103 shipping_rule = frappe.get_doc("Shipping Rule", self.shipping_rule)
Nabin Haite7d15362014-12-25 16:01:55 +0530104 existing_shipping_charge = self.get("taxes", {
Anand Doshiac32bad2014-04-18 01:30:14 +0530105 "doctype": "Sales Taxes and Charges",
106 "charge_type": "Actual",
107 "account_head": shipping_rule.account,
108 "cost_center": shipping_rule.cost_center
Anand Doshi0b4943c2013-07-04 23:45:22 +0530109 })
Anand Doshiac32bad2014-04-18 01:30:14 +0530110 if existing_shipping_charge:
Nabin Haite7d15362014-12-25 16:01:55 +0530111 self.get("taxes").remove(existing_shipping_charge[-1])
Anand Doshiac32bad2014-04-18 01:30:14 +0530112 self.calculate_taxes_and_totals()
Anand Doshid2946502014-04-08 20:10:03 +0530113
Nabin Haitec52db82013-01-22 11:53:14 +0530114 def set_total_in_words(self):
Rushabh Mehta793ba6b2014-02-14 15:47:51 +0530115 from frappe.utils import money_in_words
Anand Doshid2946502014-04-08 20:10:03 +0530116
Nabin Hait5690be12015-02-12 16:09:11 +0530117 if self.meta.get_field("base_in_words"):
Nabin Hait877e1bb2017-11-17 12:27:43 +0530118 base_amount = abs(self.base_grand_total
119 if self.is_rounded_total_disabled() else self.base_rounded_total)
120 self.base_in_words = money_in_words(base_amount, self.company_currency)
121
Nabin Haitec52db82013-01-22 11:53:14 +0530122 if self.meta.get_field("in_words"):
Nabin Hait877e1bb2017-11-17 12:27:43 +0530123 amount = abs(self.grand_total if self.is_rounded_total_disabled() else self.rounded_total)
124 self.in_words = money_in_words(amount, self.currency)
Anand Doshid2946502014-04-08 20:10:03 +0530125
Anand Doshi923d41d2013-05-28 17:23:36 +0530126 def calculate_commission(self):
127 if self.meta.get_field("commission_rate"):
Nabin Hait5690be12015-02-12 16:09:11 +0530128 self.round_floats_in(self, ["base_net_total", "commission_rate"])
Anand Doshif78d1ae2014-03-28 13:55:00 +0530129 if self.commission_rate > 100.0:
Rushabh Mehta9f0d6252014-04-14 19:20:45 +0530130 throw(_("Commission rate cannot be greater than 100"))
Anand Doshid2946502014-04-08 20:10:03 +0530131
Nabin Hait5690be12015-02-12 16:09:11 +0530132 self.total_commission = flt(self.base_net_total * self.commission_rate / 100.0,
Anand Doshi923d41d2013-05-28 17:23:36 +0530133 self.precision("total_commission"))
Anand Doshif3096132013-05-21 19:35:06 +0530134
135 def calculate_contribution(self):
Anand Doshiac32bad2014-04-18 01:30:14 +0530136 if not self.meta.get_field("sales_team"):
137 return
138
Anand Doshif3096132013-05-21 19:35:06 +0530139 total = 0.0
Rushabh Mehtad2b34dc2014-03-27 16:12:56 +0530140 sales_team = self.get("sales_team")
Anand Doshif3096132013-05-21 19:35:06 +0530141 for sales_person in sales_team:
142 self.round_floats_in(sales_person)
143
144 sales_person.allocated_amount = flt(
Nabin Hait5690be12015-02-12 16:09:11 +0530145 self.base_net_total * sales_person.allocated_percentage / 100.0,
Anand Doshif3096132013-05-21 19:35:06 +0530146 self.precision("allocated_amount", sales_person))
Anand Doshid2946502014-04-08 20:10:03 +0530147
Anand Doshif3096132013-05-21 19:35:06 +0530148 total += sales_person.allocated_percentage
Anand Doshid2946502014-04-08 20:10:03 +0530149
Anand Doshif3096132013-05-21 19:35:06 +0530150 if sales_team and total != 100.0:
Rushabh Mehta9f0d6252014-04-14 19:20:45 +0530151 throw(_("Total allocated percentage for sales team should be 100"))
Anand Doshid2946502014-04-08 20:10:03 +0530152
Anand Doshi1dde46a2013-05-15 21:15:57 +0530153 def validate_order_type(self):
Saurabh5b7e9a12015-10-15 17:27:46 +0530154 valid_types = ["Sales", "Maintenance", "Shopping Cart"]
Anand Doshif78d1ae2014-03-28 13:55:00 +0530155 if not self.order_type:
156 self.order_type = "Sales"
157 elif self.order_type not in valid_types:
Anand Doshi80a988c2014-07-07 11:35:54 +0530158 throw(_("Order Type must be one of {0}").format(comma_or(valid_types)))
Anand Doshid2946502014-04-08 20:10:03 +0530159
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530160 def validate_max_discount(self):
Nabin Haitdd38a262014-12-26 13:15:21 +0530161 for d in self.get("items"):
Anand Doshie9baaa62014-02-26 12:35:33 +0530162 discount = flt(frappe.db.get_value("Item", d.item_code, "max_discount"))
Anand Doshid2946502014-04-08 20:10:03 +0530163
Nabin Haita7f757a2014-02-10 17:54:04 +0530164 if discount and flt(d.discount_percentage) > discount:
Rushabh Mehtac38fc712014-04-16 17:21:25 +0530165 frappe.throw(_("Maxiumm discount for Item {0} is {1}%").format(d.item_code, discount))
Anand Doshid2946502014-04-08 20:10:03 +0530166
Kanchan Chauhan5a980ac2017-02-10 14:27:11 +0530167 def set_qty_as_per_stock_uom(self):
168 for d in self.get("items"):
169 if d.meta.get_field("stock_qty"):
170 if not d.conversion_factor:
171 frappe.throw(_("Row {0}: Conversion Factor is mandatory").format(d.idx))
Rushabh Mehtacc8b2b22017-03-31 12:44:29 +0530172 d.stock_qty = flt(d.qty) * flt(d.conversion_factor)
Kanchan Chauhan5a980ac2017-02-10 14:27:11 +0530173
shreyas1cfe2d72016-10-19 18:36:12 +0530174 def validate_selling_price(self):
mbauskare9747a82016-11-16 17:19:47 +0530175 def throw_message(item_name, rate, ref_rate_field):
Nabin Hait42343bb2017-05-05 17:23:17 +0530176 frappe.throw(_("""Selling rate for item {0} is lower than its {1}. Selling rate should be atleast {2}""")
mbauskare9747a82016-11-16 17:19:47 +0530177 .format(item_name, ref_rate_field, rate))
178
Nabin Hait9f26bc12016-10-20 16:45:53 +0530179 if not frappe.db.get_single_value("Selling Settings", "validate_selling_price"):
shreyas1cfe2d72016-10-19 18:36:12 +0530180 return
181
182 for it in self.get("items"):
Saurabhd7e18692017-09-18 15:43:35 +0530183 if not it.item_code:
184 continue
185
mbauskare9747a82016-11-16 17:19:47 +0530186 last_purchase_rate, is_stock_item = frappe.db.get_value("Item", it.item_code, ["last_purchase_rate", "is_stock_item"])
Nabin Hait42343bb2017-05-05 17:23:17 +0530187 last_purchase_rate_in_sales_uom = last_purchase_rate / (it.conversion_factor or 1)
188 if flt(it.base_rate) < flt(last_purchase_rate_in_sales_uom):
189 throw_message(it.item_name, last_purchase_rate_in_sales_uom, "last purchase rate")
shreyase8aaa922016-10-20 14:03:59 +0530190
191 last_valuation_rate = frappe.db.sql("""
192 SELECT valuation_rate FROM `tabStock Ledger Entry` WHERE item_code = %s
193 AND warehouse = %s AND valuation_rate > 0
194 ORDER BY posting_date DESC, posting_time DESC, name DESC LIMIT 1
195 """, (it.item_code, it.warehouse))
Nabin Hait42343bb2017-05-05 17:23:17 +0530196 if last_valuation_rate:
197 last_valuation_rate_in_sales_uom = last_valuation_rate[0][0] / (it.conversion_factor or 1)
198 if is_stock_item and flt(it.base_rate) < flt(last_valuation_rate_in_sales_uom):
199 throw_message(it.name, last_valuation_rate_in_sales_uom, "valuation rate")
shreyase8aaa922016-10-20 14:03:59 +0530200
shreyas1cfe2d72016-10-19 18:36:12 +0530201
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530202 def get_item_list(self):
203 il = []
Nabin Haitdd38a262014-12-26 13:15:21 +0530204 for d in self.get("items"):
nathando247e9ff2014-09-03 15:03:31 +0800205 if d.qty is None:
Nabin Haitc41a9482014-08-18 15:40:15 +0530206 frappe.throw(_("Row {0}: Qty is mandatory").format(d.idx))
Anand Doshi01de9452016-01-26 16:22:50 +0530207
Neil Trini Lasradoed8cecb2015-07-07 13:59:23 +0530208 if self.has_product_bundle(d.item_code):
Nabin Haite7d15362014-12-25 16:01:55 +0530209 for p in self.get("packed_items"):
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530210 if p.parent_detail_docname == d.name and p.parent_item == d.item_code:
211 # the packing details table's qty is already multiplied with parent's qty
Rushabh Mehta793ba6b2014-02-14 15:47:51 +0530212 il.append(frappe._dict({
mbauskare471b022017-04-12 16:12:10 +0530213 'warehouse': p.warehouse or d.warehouse,
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530214 'item_code': p.item_code,
215 'qty': flt(p.qty),
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530216 'uom': p.uom,
217 'batch_no': cstr(p.batch_no).strip(),
218 'serial_no': cstr(p.serial_no).strip(),
Nabin Haitc865f222015-09-21 09:18:43 +0530219 'name': d.name,
220 'target_warehouse': p.target_warehouse
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530221 }))
222 else:
Rushabh Mehta793ba6b2014-02-14 15:47:51 +0530223 il.append(frappe._dict({
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530224 'warehouse': d.warehouse,
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530225 'item_code': d.item_code,
Nabin Haite68f32c2017-03-03 15:00:41 +0530226 'qty': d.stock_qty,
Kanchan Chauhan5a980ac2017-02-10 14:27:11 +0530227 'uom': d.uom,
Nabin Hait1d218422015-07-17 15:19:02 +0530228 'stock_uom': d.stock_uom,
Kanchan Chauhan5a980ac2017-02-10 14:27:11 +0530229 'conversion_factor': d.conversion_factor,
Nabin Haitdc82d4f2014-04-07 12:02:57 +0530230 'batch_no': cstr(d.get("batch_no")).strip(),
231 'serial_no': cstr(d.get("serial_no")).strip(),
Nabin Haitc865f222015-09-21 09:18:43 +0530232 'name': d.name,
Nabin Haitb445be32015-09-21 10:56:02 +0530233 'target_warehouse': d.target_warehouse
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530234 }))
235 return il
Anand Doshid2946502014-04-08 20:10:03 +0530236
Neil Trini Lasradoed8cecb2015-07-07 13:59:23 +0530237 def has_product_bundle(self, item_code):
238 return frappe.db.sql("""select name from `tabProduct Bundle`
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530239 where new_item_code=%s and docstatus != 2""", item_code)
Anand Doshid2946502014-04-08 20:10:03 +0530240
Nabin Hait453cc372015-07-29 12:21:03 +0530241 def get_already_delivered_qty(self, current_docname, so, so_detail):
242 delivered_via_dn = frappe.db.sql("""select sum(qty) from `tabDelivery Note Item`
ankitjavalkarworke69a6112014-10-08 12:46:02 +0530243 where so_detail = %s and docstatus = 1
Anand Doshid2946502014-04-08 20:10:03 +0530244 and against_sales_order = %s
Nabin Hait453cc372015-07-29 12:21:03 +0530245 and parent != %s""", (so_detail, so, current_docname))
Anand Doshi01de9452016-01-26 16:22:50 +0530246
247 delivered_via_si = frappe.db.sql("""select sum(si_item.qty)
Nabin Hait18267912015-08-17 15:36:23 +0530248 from `tabSales Invoice Item` si_item, `tabSales Invoice` si
Anand Doshi602e8252015-11-16 19:05:46 +0530249 where si_item.parent = si.name and si.update_stock = 1
Anand Doshi01de9452016-01-26 16:22:50 +0530250 and si_item.so_detail = %s and si.docstatus = 1
Nabin Hait18267912015-08-17 15:36:23 +0530251 and si_item.sales_order = %s
252 and si.name != %s""", (so_detail, so, current_docname))
Anand Doshi01de9452016-01-26 16:22:50 +0530253
Nabin Hait453cc372015-07-29 12:21:03 +0530254 total_delivered_qty = (flt(delivered_via_dn[0][0]) if delivered_via_dn else 0) \
255 + (flt(delivered_via_si[0][0]) if delivered_via_si else 0)
Anand Doshi01de9452016-01-26 16:22:50 +0530256
Nabin Hait453cc372015-07-29 12:21:03 +0530257 return total_delivered_qty
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530258
259 def get_so_qty_and_warehouse(self, so_detail):
Anand Doshie9baaa62014-02-26 12:35:33 +0530260 so_item = frappe.db.sql("""select qty, warehouse from `tabSales Order Item`
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530261 where name = %s and docstatus = 1""", so_detail, as_dict=1)
262 so_qty = so_item and flt(so_item[0]["qty"]) or 0.0
Nabin Haita7f757a2014-02-10 17:54:04 +0530263 so_warehouse = so_item and so_item[0]["warehouse"] or ""
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530264 return so_qty, so_warehouse
Anand Doshid2946502014-04-08 20:10:03 +0530265
patilsangrambf2b5112016-02-22 16:24:23 +0530266 def check_close_sales_order(self, ref_fieldname):
Nabin Haitdd38a262014-12-26 13:15:21 +0530267 for d in self.get("items"):
Anand Doshif78d1ae2014-03-28 13:55:00 +0530268 if d.get(ref_fieldname):
Rushabh Mehtaf2227d02014-03-31 23:37:40 +0530269 status = frappe.db.get_value("Sales Order", d.get(ref_fieldname), "status")
patilsangrambf2b5112016-02-22 16:24:23 +0530270 if status == "Closed":
Saurabh8a8ef852015-10-21 19:28:53 +0530271 frappe.throw(_("Sales Order {0} is {1}").format(d.get(ref_fieldname), status))
shreyas1cfe2d72016-10-19 18:36:12 +0530272
Nabin Hait14aa9c52016-04-18 15:54:01 +0530273 def update_reserved_qty(self):
274 so_map = {}
275 for d in self.get("items"):
276 if d.so_detail:
277 if self.doctype == "Delivery Note" and d.against_sales_order:
278 so_map.setdefault(d.against_sales_order, []).append(d.so_detail)
279 elif self.doctype == "Sales Invoice" and d.sales_order and self.update_stock:
280 so_map.setdefault(d.sales_order, []).append(d.so_detail)
281
282 for so, so_item_rows in so_map.items():
283 if so and so_item_rows:
284 sales_order = frappe.get_doc("Sales Order", so)
285
286 if sales_order.status in ["Closed", "Cancelled"]:
287 frappe.throw(_("{0} {1} is cancelled or closed").format(_("Sales Order"), so),
288 frappe.InvalidStatusError)
289
290 sales_order.update_reserved_qty(so_item_rows)
291
292 def update_stock_ledger(self):
293 self.update_reserved_qty()
294
295 sl_entries = []
296 for d in self.get_item_list():
297 if frappe.db.get_value("Item", d.item_code, "is_stock_item") == 1 and flt(d.qty):
Kanchan Chauhan5a980ac2017-02-10 14:27:11 +0530298 if flt(d.conversion_factor)==0.0:
299 d.conversion_factor = get_conversion_factor(d.item_code, d.uom).get("conversion_factor") or 1.0
Nabin Hait14aa9c52016-04-18 15:54:01 +0530300 return_rate = 0
301 if cint(self.is_return) and self.return_against and self.docstatus==1:
302 return_rate = self.get_incoming_rate_for_sales_return(d.item_code, self.return_against)
303
304 # On cancellation or if return entry submission, make stock ledger entry for
305 # target warehouse first, to update serial no values properly
306
307 if d.warehouse and ((not cint(self.is_return) and self.docstatus==1)
308 or (cint(self.is_return) and self.docstatus==2)):
309 sl_entries.append(self.get_sl_entries(d, {
Nabin Haite68f32c2017-03-03 15:00:41 +0530310 "actual_qty": -1*flt(d.qty),
Nabin Hait14aa9c52016-04-18 15:54:01 +0530311 "incoming_rate": return_rate
312 }))
313
314 if d.target_warehouse:
315 target_warehouse_sle = self.get_sl_entries(d, {
Nabin Haite68f32c2017-03-03 15:00:41 +0530316 "actual_qty": flt(d.qty),
Nabin Hait14aa9c52016-04-18 15:54:01 +0530317 "warehouse": d.target_warehouse
318 })
319
320 if self.docstatus == 1:
321 if not cint(self.is_return):
322 args = frappe._dict({
323 "item_code": d.item_code,
324 "warehouse": d.warehouse,
325 "posting_date": self.posting_date,
326 "posting_time": self.posting_time,
327 "qty": -1*flt(d.qty),
328 "serial_no": d.serial_no
329 })
330 target_warehouse_sle.update({
331 "incoming_rate": get_incoming_rate(args)
332 })
333 else:
334 target_warehouse_sle.update({
335 "outgoing_rate": return_rate
336 })
337 sl_entries.append(target_warehouse_sle)
338
339 if d.warehouse and ((not cint(self.is_return) and self.docstatus==2)
340 or (cint(self.is_return) and self.docstatus==1)):
341 sl_entries.append(self.get_sl_entries(d, {
Nabin Haite68f32c2017-03-03 15:00:41 +0530342 "actual_qty": -1*flt(d.qty),
Nabin Hait14aa9c52016-04-18 15:54:01 +0530343 "incoming_rate": return_rate
344 }))
345
346 self.make_sl_entries(sl_entries)
Anand Doshid2946502014-04-08 20:10:03 +0530347
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530348def check_active_sales_items(obj):
Nabin Haitdd38a262014-12-26 13:15:21 +0530349 for d in obj.get("items"):
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530350 if d.item_code:
Umair Sayyed72534de2016-04-15 12:52:12 +0530351 item = frappe.db.sql("""select docstatus,
Anand Doshi01de9452016-01-26 16:22:50 +0530352 income_account from tabItem where name = %s""",
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530353 d.item_code, as_dict=True)[0]
shreyas1cfe2d72016-10-19 18:36:12 +0530354
Nabin Hait365ae272014-04-03 17:38:54 +0530355 if getattr(d, "income_account", None) and not item.income_account:
Anand Doshid2946502014-04-08 20:10:03 +0530356 frappe.db.set_value("Item", d.item_code, "income_account",
Nabin Hait89ad6f22013-10-18 12:55:59 +0530357 d.income_account)