blob: 5f50ae32a240fd4d74c600899a385cc6b998e78a [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 Mehta1f847992013-12-12 19:12:19 +05307from erpnext.setup.utils import get_company_currency
Rushabh Mehta9f0d6252014-04-14 19:20:45 +05308from frappe import _, throw
Saurabhd3135532016-02-25 18:59:20 +05309from erpnext.stock.get_item_details import get_bin_details
Nabin Hait14aa9c52016-04-18 15:54:01 +053010from erpnext.stock.utils import get_incoming_rate
shreyase8aaa922016-10-20 14:03:59 +053011from erpnext.stock.stock_ledger import get_valuation_rate
Kanchan Chauhan5a980ac2017-02-10 14:27:11 +053012from erpnext.stock.get_item_details import get_conversion_factor
Nabin Haitec52db82013-01-22 11:53:14 +053013
Rushabh Mehta1f847992013-12-12 19:12:19 +053014from erpnext.controllers.stock_controller import StockController
Nabin Haitbf495c92013-01-30 12:49:08 +053015
Nabin Haitc3afb252013-03-19 12:01:24 +053016class SellingController(StockController):
Rushabh Mehtaf84c2402014-07-18 16:39:31 +053017 def __setup__(self):
Nabin Haitde9c8a92015-02-23 01:06:00 +053018 if hasattr(self, "taxes"):
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"):
Saurabhd3135532016-02-25 18:59:20 +053031 item.update(get_bin_details(item.item_code,
Rushabh Mehtab5e76892014-07-29 16:07:43 +053032 item.warehouse))
33
Nabin Haitd1fd1e22013-10-18 12:29:11 +053034 def validate(self):
35 super(SellingController, self).validate()
36 self.validate_max_discount()
shreyas1cfe2d72016-10-19 18:36:12 +053037 self.validate_selling_price()
Kanchan Chauhan5a980ac2017-02-10 14:27:11 +053038 self.set_qty_as_per_stock_uom()
Nabin Haitd1fd1e22013-10-18 12:29:11 +053039 check_active_sales_items(self)
Anand Doshid2946502014-04-08 20:10:03 +053040
Anand Doshif3096132013-05-21 19:35:06 +053041 def set_missing_values(self, for_validate=False):
Anand Doshiabc10032013-06-14 17:44:03 +053042 super(SellingController, self).set_missing_values(for_validate)
Anand Doshid2946502014-04-08 20:10:03 +053043
Anand Doshif3096132013-05-21 19:35:06 +053044 # set contact and address details for customer, if they are not mentioned
Anand Doshiabc10032013-06-14 17:44:03 +053045 self.set_missing_lead_customer_details()
Nabin Hait096d3632013-10-17 17:01:14 +053046 self.set_price_list_and_item_details()
Anand Doshid2946502014-04-08 20:10:03 +053047
Anand Doshiabc10032013-06-14 17:44:03 +053048 def set_missing_lead_customer_details(self):
Rushabh Mehtaf14b8092014-04-03 14:30:42 +053049 if getattr(self, "customer", None):
Anand Doshi43624432014-02-25 15:42:16 +053050 from erpnext.accounts.party import _get_party_details
Rushabh Mehta2cc585f2015-02-26 15:01:23 +053051 party_details = _get_party_details(self.customer,
52 ignore_permissions=self.flags.ignore_permissions)
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
Nabin Hait096d3632013-10-17 17:01:14 +053065 def set_price_list_and_item_details(self):
66 self.set_price_list_currency("Selling")
Nabin Hait0aa71a52014-02-11 16:14:52 +053067 self.set_missing_item_details()
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 Doshif78d1ae2014-03-28 13:55:00 +0530116 company_currency = get_company_currency(self.company)
Anand Doshid2946502014-04-08 20:10:03 +0530117
Nabin Hait04d244a2015-07-22 17:47:49 +0530118 disable_rounded_total = cint(frappe.db.get_value("Global Defaults", None, "disable_rounded_total"))
Anand Doshid2946502014-04-08 20:10:03 +0530119
Nabin Hait5690be12015-02-12 16:09:11 +0530120 if self.meta.get_field("base_in_words"):
121 self.base_in_words = money_in_words(disable_rounded_total and
Nabin Hait04d244a2015-07-22 17:47:49 +0530122 abs(self.base_grand_total) or abs(self.base_rounded_total), company_currency)
Nabin Haitec52db82013-01-22 11:53:14 +0530123 if self.meta.get_field("in_words"):
Anand Doshid2946502014-04-08 20:10:03 +0530124 self.in_words = money_in_words(disable_rounded_total and
Nabin Hait04d244a2015-07-22 17:47:49 +0530125 abs(self.grand_total) or abs(self.rounded_total), self.currency)
Anand Doshid2946502014-04-08 20:10:03 +0530126
Anand Doshi923d41d2013-05-28 17:23:36 +0530127 def calculate_commission(self):
128 if self.meta.get_field("commission_rate"):
Nabin Hait5690be12015-02-12 16:09:11 +0530129 self.round_floats_in(self, ["base_net_total", "commission_rate"])
Anand Doshif78d1ae2014-03-28 13:55:00 +0530130 if self.commission_rate > 100.0:
Rushabh Mehta9f0d6252014-04-14 19:20:45 +0530131 throw(_("Commission rate cannot be greater than 100"))
Anand Doshid2946502014-04-08 20:10:03 +0530132
Nabin Hait5690be12015-02-12 16:09:11 +0530133 self.total_commission = flt(self.base_net_total * self.commission_rate / 100.0,
Anand Doshi923d41d2013-05-28 17:23:36 +0530134 self.precision("total_commission"))
Anand Doshif3096132013-05-21 19:35:06 +0530135
136 def calculate_contribution(self):
Anand Doshiac32bad2014-04-18 01:30:14 +0530137 if not self.meta.get_field("sales_team"):
138 return
139
Anand Doshif3096132013-05-21 19:35:06 +0530140 total = 0.0
Rushabh Mehtad2b34dc2014-03-27 16:12:56 +0530141 sales_team = self.get("sales_team")
Anand Doshif3096132013-05-21 19:35:06 +0530142 for sales_person in sales_team:
143 self.round_floats_in(sales_person)
144
145 sales_person.allocated_amount = flt(
Nabin Hait5690be12015-02-12 16:09:11 +0530146 self.base_net_total * sales_person.allocated_percentage / 100.0,
Anand Doshif3096132013-05-21 19:35:06 +0530147 self.precision("allocated_amount", sales_person))
Anand Doshid2946502014-04-08 20:10:03 +0530148
Anand Doshif3096132013-05-21 19:35:06 +0530149 total += sales_person.allocated_percentage
Anand Doshid2946502014-04-08 20:10:03 +0530150
Anand Doshif3096132013-05-21 19:35:06 +0530151 if sales_team and total != 100.0:
Rushabh Mehta9f0d6252014-04-14 19:20:45 +0530152 throw(_("Total allocated percentage for sales team should be 100"))
Anand Doshid2946502014-04-08 20:10:03 +0530153
Anand Doshi1dde46a2013-05-15 21:15:57 +0530154 def validate_order_type(self):
Saurabh5b7e9a12015-10-15 17:27:46 +0530155 valid_types = ["Sales", "Maintenance", "Shopping Cart"]
Anand Doshif78d1ae2014-03-28 13:55:00 +0530156 if not self.order_type:
157 self.order_type = "Sales"
158 elif self.order_type not in valid_types:
Anand Doshi80a988c2014-07-07 11:35:54 +0530159 throw(_("Order Type must be one of {0}").format(comma_or(valid_types)))
Anand Doshid2946502014-04-08 20:10:03 +0530160
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530161 def validate_max_discount(self):
Nabin Haitdd38a262014-12-26 13:15:21 +0530162 for d in self.get("items"):
Anand Doshie9baaa62014-02-26 12:35:33 +0530163 discount = flt(frappe.db.get_value("Item", d.item_code, "max_discount"))
Anand Doshid2946502014-04-08 20:10:03 +0530164
Nabin Haita7f757a2014-02-10 17:54:04 +0530165 if discount and flt(d.discount_percentage) > discount:
Rushabh Mehtac38fc712014-04-16 17:21:25 +0530166 frappe.throw(_("Maxiumm discount for Item {0} is {1}%").format(d.item_code, discount))
Anand Doshid2946502014-04-08 20:10:03 +0530167
Kanchan Chauhan5a980ac2017-02-10 14:27:11 +0530168 def set_qty_as_per_stock_uom(self):
169 for d in self.get("items"):
170 if d.meta.get_field("stock_qty"):
171 if not d.conversion_factor:
172 frappe.throw(_("Row {0}: Conversion Factor is mandatory").format(d.idx))
173 d.stock_qty = flt(d.qty) * flt(d.conversion_factor)
174
shreyas1cfe2d72016-10-19 18:36:12 +0530175 def validate_selling_price(self):
mbauskare9747a82016-11-16 17:19:47 +0530176 def throw_message(item_name, rate, ref_rate_field):
177 frappe.throw(_("""Selling price for item {0} is lower than its {1}. Selling price should be atleast {2}""")
178 .format(item_name, ref_rate_field, rate))
179
Nabin Hait9f26bc12016-10-20 16:45:53 +0530180 if not frappe.db.get_single_value("Selling Settings", "validate_selling_price"):
shreyas1cfe2d72016-10-19 18:36:12 +0530181 return
182
183 for it in self.get("items"):
mbauskare9747a82016-11-16 17:19:47 +0530184 last_purchase_rate, is_stock_item = frappe.db.get_value("Item", it.item_code, ["last_purchase_rate", "is_stock_item"])
shreyas1cfe2d72016-10-19 18:36:12 +0530185
shreyase8aaa922016-10-20 14:03:59 +0530186 if flt(it.base_rate) < flt(last_purchase_rate):
mbauskare9747a82016-11-16 17:19:47 +0530187 throw_message(it.item_name, last_purchase_rate, "last purchase rate")
shreyase8aaa922016-10-20 14:03:59 +0530188
189 last_valuation_rate = frappe.db.sql("""
190 SELECT valuation_rate FROM `tabStock Ledger Entry` WHERE item_code = %s
191 AND warehouse = %s AND valuation_rate > 0
192 ORDER BY posting_date DESC, posting_time DESC, name DESC LIMIT 1
193 """, (it.item_code, it.warehouse))
194
shreyase8aaa922016-10-20 14:03:59 +0530195 if is_stock_item and flt(it.base_rate) < flt(last_valuation_rate):
Nabin Hait9f26bc12016-10-20 16:45:53 +0530196 throw_message(it.name, last_valuation_rate, "valuation rate")
shreyase8aaa922016-10-20 14:03:59 +0530197
shreyas1cfe2d72016-10-19 18:36:12 +0530198
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530199 def get_item_list(self):
200 il = []
Nabin Haitdd38a262014-12-26 13:15:21 +0530201 for d in self.get("items"):
nathando247e9ff2014-09-03 15:03:31 +0800202 if d.qty is None:
Nabin Haitc41a9482014-08-18 15:40:15 +0530203 frappe.throw(_("Row {0}: Qty is mandatory").format(d.idx))
Anand Doshi01de9452016-01-26 16:22:50 +0530204
Neil Trini Lasradoed8cecb2015-07-07 13:59:23 +0530205 if self.has_product_bundle(d.item_code):
Nabin Haite7d15362014-12-25 16:01:55 +0530206 for p in self.get("packed_items"):
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530207 if p.parent_detail_docname == d.name and p.parent_item == d.item_code:
208 # the packing details table's qty is already multiplied with parent's qty
Rushabh Mehta793ba6b2014-02-14 15:47:51 +0530209 il.append(frappe._dict({
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530210 'warehouse': p.warehouse,
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530211 'item_code': p.item_code,
212 'qty': flt(p.qty),
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530213 'uom': p.uom,
214 'batch_no': cstr(p.batch_no).strip(),
215 'serial_no': cstr(p.serial_no).strip(),
Nabin Haitc865f222015-09-21 09:18:43 +0530216 'name': d.name,
217 'target_warehouse': p.target_warehouse
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530218 }))
219 else:
Rushabh Mehta793ba6b2014-02-14 15:47:51 +0530220 il.append(frappe._dict({
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530221 'warehouse': d.warehouse,
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530222 'item_code': d.item_code,
Nabin Haite68f32c2017-03-03 15:00:41 +0530223 'qty': d.stock_qty,
Kanchan Chauhan5a980ac2017-02-10 14:27:11 +0530224 'uom': d.uom,
Nabin Hait1d218422015-07-17 15:19:02 +0530225 'stock_uom': d.stock_uom,
Kanchan Chauhan5a980ac2017-02-10 14:27:11 +0530226 'conversion_factor': d.conversion_factor,
Nabin Haitdc82d4f2014-04-07 12:02:57 +0530227 'batch_no': cstr(d.get("batch_no")).strip(),
228 'serial_no': cstr(d.get("serial_no")).strip(),
Nabin Haitc865f222015-09-21 09:18:43 +0530229 'name': d.name,
Nabin Haitb445be32015-09-21 10:56:02 +0530230 'target_warehouse': d.target_warehouse
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530231 }))
232 return il
Anand Doshid2946502014-04-08 20:10:03 +0530233
Neil Trini Lasradoed8cecb2015-07-07 13:59:23 +0530234 def has_product_bundle(self, item_code):
235 return frappe.db.sql("""select name from `tabProduct Bundle`
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530236 where new_item_code=%s and docstatus != 2""", item_code)
Anand Doshid2946502014-04-08 20:10:03 +0530237
Nabin Hait453cc372015-07-29 12:21:03 +0530238 def get_already_delivered_qty(self, current_docname, so, so_detail):
239 delivered_via_dn = frappe.db.sql("""select sum(qty) from `tabDelivery Note Item`
ankitjavalkarworke69a6112014-10-08 12:46:02 +0530240 where so_detail = %s and docstatus = 1
Anand Doshid2946502014-04-08 20:10:03 +0530241 and against_sales_order = %s
Nabin Hait453cc372015-07-29 12:21:03 +0530242 and parent != %s""", (so_detail, so, current_docname))
Anand Doshi01de9452016-01-26 16:22:50 +0530243
244 delivered_via_si = frappe.db.sql("""select sum(si_item.qty)
Nabin Hait18267912015-08-17 15:36:23 +0530245 from `tabSales Invoice Item` si_item, `tabSales Invoice` si
Anand Doshi602e8252015-11-16 19:05:46 +0530246 where si_item.parent = si.name and si.update_stock = 1
Anand Doshi01de9452016-01-26 16:22:50 +0530247 and si_item.so_detail = %s and si.docstatus = 1
Nabin Hait18267912015-08-17 15:36:23 +0530248 and si_item.sales_order = %s
249 and si.name != %s""", (so_detail, so, current_docname))
Anand Doshi01de9452016-01-26 16:22:50 +0530250
Nabin Hait453cc372015-07-29 12:21:03 +0530251 total_delivered_qty = (flt(delivered_via_dn[0][0]) if delivered_via_dn else 0) \
252 + (flt(delivered_via_si[0][0]) if delivered_via_si else 0)
Anand Doshi01de9452016-01-26 16:22:50 +0530253
Nabin Hait453cc372015-07-29 12:21:03 +0530254 return total_delivered_qty
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530255
256 def get_so_qty_and_warehouse(self, so_detail):
Anand Doshie9baaa62014-02-26 12:35:33 +0530257 so_item = frappe.db.sql("""select qty, warehouse from `tabSales Order Item`
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530258 where name = %s and docstatus = 1""", so_detail, as_dict=1)
259 so_qty = so_item and flt(so_item[0]["qty"]) or 0.0
Nabin Haita7f757a2014-02-10 17:54:04 +0530260 so_warehouse = so_item and so_item[0]["warehouse"] or ""
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530261 return so_qty, so_warehouse
Anand Doshid2946502014-04-08 20:10:03 +0530262
patilsangrambf2b5112016-02-22 16:24:23 +0530263 def check_close_sales_order(self, ref_fieldname):
Nabin Haitdd38a262014-12-26 13:15:21 +0530264 for d in self.get("items"):
Anand Doshif78d1ae2014-03-28 13:55:00 +0530265 if d.get(ref_fieldname):
Rushabh Mehtaf2227d02014-03-31 23:37:40 +0530266 status = frappe.db.get_value("Sales Order", d.get(ref_fieldname), "status")
patilsangrambf2b5112016-02-22 16:24:23 +0530267 if status == "Closed":
Saurabh8a8ef852015-10-21 19:28:53 +0530268 frappe.throw(_("Sales Order {0} is {1}").format(d.get(ref_fieldname), status))
shreyas1cfe2d72016-10-19 18:36:12 +0530269
Nabin Hait14aa9c52016-04-18 15:54:01 +0530270 def update_reserved_qty(self):
271 so_map = {}
272 for d in self.get("items"):
273 if d.so_detail:
274 if self.doctype == "Delivery Note" and d.against_sales_order:
275 so_map.setdefault(d.against_sales_order, []).append(d.so_detail)
276 elif self.doctype == "Sales Invoice" and d.sales_order and self.update_stock:
277 so_map.setdefault(d.sales_order, []).append(d.so_detail)
278
279 for so, so_item_rows in so_map.items():
280 if so and so_item_rows:
281 sales_order = frappe.get_doc("Sales Order", so)
282
283 if sales_order.status in ["Closed", "Cancelled"]:
284 frappe.throw(_("{0} {1} is cancelled or closed").format(_("Sales Order"), so),
285 frappe.InvalidStatusError)
286
287 sales_order.update_reserved_qty(so_item_rows)
288
289 def update_stock_ledger(self):
290 self.update_reserved_qty()
291
292 sl_entries = []
293 for d in self.get_item_list():
294 if frappe.db.get_value("Item", d.item_code, "is_stock_item") == 1 and flt(d.qty):
Kanchan Chauhan5a980ac2017-02-10 14:27:11 +0530295 if flt(d.conversion_factor)==0.0:
296 d.conversion_factor = get_conversion_factor(d.item_code, d.uom).get("conversion_factor") or 1.0
Nabin Hait14aa9c52016-04-18 15:54:01 +0530297 return_rate = 0
298 if cint(self.is_return) and self.return_against and self.docstatus==1:
299 return_rate = self.get_incoming_rate_for_sales_return(d.item_code, self.return_against)
300
301 # On cancellation or if return entry submission, make stock ledger entry for
302 # target warehouse first, to update serial no values properly
303
304 if d.warehouse and ((not cint(self.is_return) and self.docstatus==1)
305 or (cint(self.is_return) and self.docstatus==2)):
306 sl_entries.append(self.get_sl_entries(d, {
Nabin Haite68f32c2017-03-03 15:00:41 +0530307 "actual_qty": -1*flt(d.qty),
Nabin Hait14aa9c52016-04-18 15:54:01 +0530308 "incoming_rate": return_rate
309 }))
310
311 if d.target_warehouse:
312 target_warehouse_sle = self.get_sl_entries(d, {
Nabin Haite68f32c2017-03-03 15:00:41 +0530313 "actual_qty": flt(d.qty),
Nabin Hait14aa9c52016-04-18 15:54:01 +0530314 "warehouse": d.target_warehouse
315 })
316
317 if self.docstatus == 1:
318 if not cint(self.is_return):
319 args = frappe._dict({
320 "item_code": d.item_code,
321 "warehouse": d.warehouse,
322 "posting_date": self.posting_date,
323 "posting_time": self.posting_time,
324 "qty": -1*flt(d.qty),
325 "serial_no": d.serial_no
326 })
327 target_warehouse_sle.update({
328 "incoming_rate": get_incoming_rate(args)
329 })
330 else:
331 target_warehouse_sle.update({
332 "outgoing_rate": return_rate
333 })
334 sl_entries.append(target_warehouse_sle)
335
336 if d.warehouse and ((not cint(self.is_return) and self.docstatus==2)
337 or (cint(self.is_return) and self.docstatus==1)):
338 sl_entries.append(self.get_sl_entries(d, {
Nabin Haite68f32c2017-03-03 15:00:41 +0530339 "actual_qty": -1*flt(d.qty),
Nabin Hait14aa9c52016-04-18 15:54:01 +0530340 "incoming_rate": return_rate
341 }))
342
343 self.make_sl_entries(sl_entries)
Anand Doshid2946502014-04-08 20:10:03 +0530344
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530345def check_active_sales_items(obj):
Nabin Haitdd38a262014-12-26 13:15:21 +0530346 for d in obj.get("items"):
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530347 if d.item_code:
Umair Sayyed72534de2016-04-15 12:52:12 +0530348 item = frappe.db.sql("""select docstatus,
Anand Doshi01de9452016-01-26 16:22:50 +0530349 income_account from tabItem where name = %s""",
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530350 d.item_code, as_dict=True)[0]
shreyas1cfe2d72016-10-19 18:36:12 +0530351
Nabin Hait365ae272014-04-03 17:38:54 +0530352 if getattr(d, "income_account", None) and not item.income_account:
Anand Doshid2946502014-04-08 20:10:03 +0530353 frappe.db.set_value("Item", d.item_code, "income_account",
Nabin Hait89ad6f22013-10-18 12:55:59 +0530354 d.income_account)