blob: e200bf7923d6dd0f8273f1d6c26f131b493feb22 [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
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"):
Rushabh Mehtac567e8e2015-02-03 17:55:52 +053017 self.print_templates = {
18 "taxes": "templates/print_formats/includes/taxes.html"
Rushabh Mehta5b51cc82014-07-21 18:25:45 +053019 }
Rushabh Mehtaf84c2402014-07-18 16:39:31 +053020
Rushabh Mehtac0bb4532014-09-09 16:15:35 +053021 def get_feed(self):
22 return _("To {0} | {1} {2}").format(self.customer_name, self.currency,
Nabin Hait5690be12015-02-12 16:09:11 +053023 self.grand_total)
Rushabh Mehtac0bb4532014-09-09 16:15:35 +053024
Rushabh Mehtab5e76892014-07-29 16:07:43 +053025 def onload(self):
Rohit Waghchaure7d529892016-10-06 14:35:04 +053026 super(SellingController, self).onload()
Rushabh Mehtab5e76892014-07-29 16:07:43 +053027 if self.doctype in ("Sales Order", "Delivery Note", "Sales Invoice"):
Nabin Haitdd38a262014-12-26 13:15:21 +053028 for item in self.get("items"):
Saurabhd3135532016-02-25 18:59:20 +053029 item.update(get_bin_details(item.item_code,
Rushabh Mehtab5e76892014-07-29 16:07:43 +053030 item.warehouse))
31
Nabin Haitd1fd1e22013-10-18 12:29:11 +053032 def validate(self):
33 super(SellingController, self).validate()
34 self.validate_max_discount()
shreyas1cfe2d72016-10-19 18:36:12 +053035 self.validate_selling_price()
Nabin Haitd1fd1e22013-10-18 12:29:11 +053036 check_active_sales_items(self)
Anand Doshid2946502014-04-08 20:10:03 +053037
Anand Doshif3096132013-05-21 19:35:06 +053038 def set_missing_values(self, for_validate=False):
Anand Doshiabc10032013-06-14 17:44:03 +053039 super(SellingController, self).set_missing_values(for_validate)
Anand Doshid2946502014-04-08 20:10:03 +053040
Anand Doshif3096132013-05-21 19:35:06 +053041 # set contact and address details for customer, if they are not mentioned
Anand Doshiabc10032013-06-14 17:44:03 +053042 self.set_missing_lead_customer_details()
Nabin Hait096d3632013-10-17 17:01:14 +053043 self.set_price_list_and_item_details()
Anand Doshid2946502014-04-08 20:10:03 +053044
Anand Doshiabc10032013-06-14 17:44:03 +053045 def set_missing_lead_customer_details(self):
Rushabh Mehtaf14b8092014-04-03 14:30:42 +053046 if getattr(self, "customer", None):
Anand Doshi43624432014-02-25 15:42:16 +053047 from erpnext.accounts.party import _get_party_details
Rushabh Mehta2cc585f2015-02-26 15:01:23 +053048 party_details = _get_party_details(self.customer,
49 ignore_permissions=self.flags.ignore_permissions)
Nabin Hait6b039142014-05-02 15:45:10 +053050 if not self.meta.get_field("sales_team"):
51 party_details.pop("sales_team")
52
53 self.update_if_missing(party_details)
Anand Doshid2946502014-04-08 20:10:03 +053054
Rushabh Mehtaf14b8092014-04-03 14:30:42 +053055 elif getattr(self, "lead", None):
Rushabh Mehta60cfccb2015-02-17 10:36:54 +053056 from erpnext.crm.doctype.lead.lead import get_lead_details
Anand Doshi2f957f22016-04-08 17:36:10 +053057 self.update_if_missing(get_lead_details(
58 self.lead,
59 posting_date=self.get('transaction_date') or self.get('posting_date'),
60 company=self.company))
Anand Doshid2946502014-04-08 20:10:03 +053061
Nabin Hait096d3632013-10-17 17:01:14 +053062 def set_price_list_and_item_details(self):
63 self.set_price_list_currency("Selling")
Nabin Hait0aa71a52014-02-11 16:14:52 +053064 self.set_missing_item_details()
Anand Doshid2946502014-04-08 20:10:03 +053065
Anand Doshi99100a42013-07-04 17:13:53 +053066 def apply_shipping_rule(self):
Anand Doshif78d1ae2014-03-28 13:55:00 +053067 if self.shipping_rule:
Rushabh Mehtab385ecf2014-03-28 16:44:37 +053068 shipping_rule = frappe.get_doc("Shipping Rule", self.shipping_rule)
Nabin Hait5690be12015-02-12 16:09:11 +053069 value = self.base_net_total
Anand Doshid2946502014-04-08 20:10:03 +053070
Anand Doshi99100a42013-07-04 17:13:53 +053071 # TODO
72 # shipping rule calculation based on item's net weight
Anand Doshid2946502014-04-08 20:10:03 +053073
Anand Doshi99100a42013-07-04 17:13:53 +053074 shipping_amount = 0.0
Nabin Haite7d15362014-12-25 16:01:55 +053075 for condition in shipping_rule.get("conditions"):
Anand Doshi99100a42013-07-04 17:13:53 +053076 if not condition.to_value or (flt(condition.from_value) <= value <= flt(condition.to_value)):
77 shipping_amount = condition.shipping_amount
78 break
Anand Doshid2946502014-04-08 20:10:03 +053079
Anand Doshiac32bad2014-04-18 01:30:14 +053080 shipping_charge = {
Anand Doshi0b4943c2013-07-04 23:45:22 +053081 "doctype": "Sales Taxes and Charges",
Anand Doshi0b4943c2013-07-04 23:45:22 +053082 "charge_type": "Actual",
Anand Doshif78d1ae2014-03-28 13:55:00 +053083 "account_head": shipping_rule.account,
Anand Doshiac32bad2014-04-18 01:30:14 +053084 "cost_center": shipping_rule.cost_center
85 }
86
Nabin Haite7d15362014-12-25 16:01:55 +053087 existing_shipping_charge = self.get("taxes", filters=shipping_charge)
Anand Doshiac32bad2014-04-18 01:30:14 +053088 if existing_shipping_charge:
89 # take the last record found
Anand Doshiff8a8542015-05-26 18:06:09 -040090 existing_shipping_charge[-1].tax_amount = shipping_amount
Anand Doshiac32bad2014-04-18 01:30:14 +053091 else:
Anand Doshiff8a8542015-05-26 18:06:09 -040092 shipping_charge["tax_amount"] = shipping_amount
Anand Doshiac32bad2014-04-18 01:30:14 +053093 shipping_charge["description"] = shipping_rule.label
Nabin Haite7d15362014-12-25 16:01:55 +053094 self.append("taxes", shipping_charge)
Anand Doshiac32bad2014-04-18 01:30:14 +053095
96 self.calculate_taxes_and_totals()
97
98 def remove_shipping_charge(self):
99 if self.shipping_rule:
100 shipping_rule = frappe.get_doc("Shipping Rule", self.shipping_rule)
Nabin Haite7d15362014-12-25 16:01:55 +0530101 existing_shipping_charge = self.get("taxes", {
Anand Doshiac32bad2014-04-18 01:30:14 +0530102 "doctype": "Sales Taxes and Charges",
103 "charge_type": "Actual",
104 "account_head": shipping_rule.account,
105 "cost_center": shipping_rule.cost_center
Anand Doshi0b4943c2013-07-04 23:45:22 +0530106 })
Anand Doshiac32bad2014-04-18 01:30:14 +0530107 if existing_shipping_charge:
Nabin Haite7d15362014-12-25 16:01:55 +0530108 self.get("taxes").remove(existing_shipping_charge[-1])
Anand Doshiac32bad2014-04-18 01:30:14 +0530109 self.calculate_taxes_and_totals()
Anand Doshid2946502014-04-08 20:10:03 +0530110
Nabin Haitec52db82013-01-22 11:53:14 +0530111 def set_total_in_words(self):
Rushabh Mehta793ba6b2014-02-14 15:47:51 +0530112 from frappe.utils import money_in_words
Anand Doshif78d1ae2014-03-28 13:55:00 +0530113 company_currency = get_company_currency(self.company)
Anand Doshid2946502014-04-08 20:10:03 +0530114
Nabin Hait04d244a2015-07-22 17:47:49 +0530115 disable_rounded_total = cint(frappe.db.get_value("Global Defaults", None, "disable_rounded_total"))
Anand Doshid2946502014-04-08 20:10:03 +0530116
Nabin Hait5690be12015-02-12 16:09:11 +0530117 if self.meta.get_field("base_in_words"):
118 self.base_in_words = money_in_words(disable_rounded_total and
Nabin Hait04d244a2015-07-22 17:47:49 +0530119 abs(self.base_grand_total) or abs(self.base_rounded_total), company_currency)
Nabin Haitec52db82013-01-22 11:53:14 +0530120 if self.meta.get_field("in_words"):
Anand Doshid2946502014-04-08 20:10:03 +0530121 self.in_words = money_in_words(disable_rounded_total and
Nabin Hait04d244a2015-07-22 17:47:49 +0530122 abs(self.grand_total) or abs(self.rounded_total), self.currency)
Anand Doshid2946502014-04-08 20:10:03 +0530123
Anand Doshi923d41d2013-05-28 17:23:36 +0530124 def calculate_commission(self):
125 if self.meta.get_field("commission_rate"):
Nabin Hait5690be12015-02-12 16:09:11 +0530126 self.round_floats_in(self, ["base_net_total", "commission_rate"])
Anand Doshif78d1ae2014-03-28 13:55:00 +0530127 if self.commission_rate > 100.0:
Rushabh Mehta9f0d6252014-04-14 19:20:45 +0530128 throw(_("Commission rate cannot be greater than 100"))
Anand Doshid2946502014-04-08 20:10:03 +0530129
Nabin Hait5690be12015-02-12 16:09:11 +0530130 self.total_commission = flt(self.base_net_total * self.commission_rate / 100.0,
Anand Doshi923d41d2013-05-28 17:23:36 +0530131 self.precision("total_commission"))
Anand Doshif3096132013-05-21 19:35:06 +0530132
133 def calculate_contribution(self):
Anand Doshiac32bad2014-04-18 01:30:14 +0530134 if not self.meta.get_field("sales_team"):
135 return
136
Anand Doshif3096132013-05-21 19:35:06 +0530137 total = 0.0
Rushabh Mehtad2b34dc2014-03-27 16:12:56 +0530138 sales_team = self.get("sales_team")
Anand Doshif3096132013-05-21 19:35:06 +0530139 for sales_person in sales_team:
140 self.round_floats_in(sales_person)
141
142 sales_person.allocated_amount = flt(
Nabin Hait5690be12015-02-12 16:09:11 +0530143 self.base_net_total * sales_person.allocated_percentage / 100.0,
Anand Doshif3096132013-05-21 19:35:06 +0530144 self.precision("allocated_amount", sales_person))
Anand Doshid2946502014-04-08 20:10:03 +0530145
Anand Doshif3096132013-05-21 19:35:06 +0530146 total += sales_person.allocated_percentage
Anand Doshid2946502014-04-08 20:10:03 +0530147
Anand Doshif3096132013-05-21 19:35:06 +0530148 if sales_team and total != 100.0:
Rushabh Mehta9f0d6252014-04-14 19:20:45 +0530149 throw(_("Total allocated percentage for sales team should be 100"))
Anand Doshid2946502014-04-08 20:10:03 +0530150
Anand Doshi1dde46a2013-05-15 21:15:57 +0530151 def validate_order_type(self):
Saurabh5b7e9a12015-10-15 17:27:46 +0530152 valid_types = ["Sales", "Maintenance", "Shopping Cart"]
Anand Doshif78d1ae2014-03-28 13:55:00 +0530153 if not self.order_type:
154 self.order_type = "Sales"
155 elif self.order_type not in valid_types:
Anand Doshi80a988c2014-07-07 11:35:54 +0530156 throw(_("Order Type must be one of {0}").format(comma_or(valid_types)))
Anand Doshid2946502014-04-08 20:10:03 +0530157
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530158 def validate_max_discount(self):
Nabin Haitdd38a262014-12-26 13:15:21 +0530159 for d in self.get("items"):
Anand Doshie9baaa62014-02-26 12:35:33 +0530160 discount = flt(frappe.db.get_value("Item", d.item_code, "max_discount"))
Anand Doshid2946502014-04-08 20:10:03 +0530161
Nabin Haita7f757a2014-02-10 17:54:04 +0530162 if discount and flt(d.discount_percentage) > discount:
Rushabh Mehtac38fc712014-04-16 17:21:25 +0530163 frappe.throw(_("Maxiumm discount for Item {0} is {1}%").format(d.item_code, discount))
Anand Doshid2946502014-04-08 20:10:03 +0530164
shreyas1cfe2d72016-10-19 18:36:12 +0530165 def validate_selling_price(self):
166 selling_settings = frappe.get_single("Selling Settings")
shreyasedcba512016-10-20 11:50:00 +0530167 if not selling_settings.validate_selling_price:
shreyas1cfe2d72016-10-19 18:36:12 +0530168 return
169
170 for it in self.get("items"):
171 item = frappe.get_doc("Item", it.name)
172
shreyasedcba512016-10-20 11:50:00 +0530173 if flt(it.base_rate) < flt(item.last_purchase_rate) or flt(it.base_rate) < flt(item.valuation_rate):
174 frappe.throw(_("""Selling price for item {0} is lower than its Purchase rate or Valuation rate.
175 Selling price should be atleast {1}""").format(it.item_name, item.last_purchase_rate))
shreyas1cfe2d72016-10-19 18:36:12 +0530176
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530177 def get_item_list(self):
178 il = []
Nabin Haitdd38a262014-12-26 13:15:21 +0530179 for d in self.get("items"):
nathando247e9ff2014-09-03 15:03:31 +0800180 if d.qty is None:
Nabin Haitc41a9482014-08-18 15:40:15 +0530181 frappe.throw(_("Row {0}: Qty is mandatory").format(d.idx))
Anand Doshi01de9452016-01-26 16:22:50 +0530182
Neil Trini Lasradoed8cecb2015-07-07 13:59:23 +0530183 if self.has_product_bundle(d.item_code):
Nabin Haite7d15362014-12-25 16:01:55 +0530184 for p in self.get("packed_items"):
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530185 if p.parent_detail_docname == d.name and p.parent_item == d.item_code:
186 # the packing details table's qty is already multiplied with parent's qty
Rushabh Mehta793ba6b2014-02-14 15:47:51 +0530187 il.append(frappe._dict({
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530188 'warehouse': p.warehouse,
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530189 'item_code': p.item_code,
190 'qty': flt(p.qty),
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530191 'uom': p.uom,
192 'batch_no': cstr(p.batch_no).strip(),
193 'serial_no': cstr(p.serial_no).strip(),
Nabin Haitc865f222015-09-21 09:18:43 +0530194 'name': d.name,
195 'target_warehouse': p.target_warehouse
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530196 }))
197 else:
Rushabh Mehta793ba6b2014-02-14 15:47:51 +0530198 il.append(frappe._dict({
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530199 'warehouse': d.warehouse,
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530200 'item_code': d.item_code,
201 'qty': d.qty,
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530202 'uom': d.stock_uom,
Nabin Hait1d218422015-07-17 15:19:02 +0530203 'stock_uom': d.stock_uom,
Nabin Haitdc82d4f2014-04-07 12:02:57 +0530204 'batch_no': cstr(d.get("batch_no")).strip(),
205 'serial_no': cstr(d.get("serial_no")).strip(),
Nabin Haitc865f222015-09-21 09:18:43 +0530206 'name': d.name,
Nabin Haitb445be32015-09-21 10:56:02 +0530207 'target_warehouse': d.target_warehouse
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530208 }))
209 return il
Anand Doshid2946502014-04-08 20:10:03 +0530210
Neil Trini Lasradoed8cecb2015-07-07 13:59:23 +0530211 def has_product_bundle(self, item_code):
212 return frappe.db.sql("""select name from `tabProduct Bundle`
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530213 where new_item_code=%s and docstatus != 2""", item_code)
Anand Doshid2946502014-04-08 20:10:03 +0530214
Nabin Hait453cc372015-07-29 12:21:03 +0530215 def get_already_delivered_qty(self, current_docname, so, so_detail):
216 delivered_via_dn = frappe.db.sql("""select sum(qty) from `tabDelivery Note Item`
ankitjavalkarworke69a6112014-10-08 12:46:02 +0530217 where so_detail = %s and docstatus = 1
Anand Doshid2946502014-04-08 20:10:03 +0530218 and against_sales_order = %s
Nabin Hait453cc372015-07-29 12:21:03 +0530219 and parent != %s""", (so_detail, so, current_docname))
Anand Doshi01de9452016-01-26 16:22:50 +0530220
221 delivered_via_si = frappe.db.sql("""select sum(si_item.qty)
Nabin Hait18267912015-08-17 15:36:23 +0530222 from `tabSales Invoice Item` si_item, `tabSales Invoice` si
Anand Doshi602e8252015-11-16 19:05:46 +0530223 where si_item.parent = si.name and si.update_stock = 1
Anand Doshi01de9452016-01-26 16:22:50 +0530224 and si_item.so_detail = %s and si.docstatus = 1
Nabin Hait18267912015-08-17 15:36:23 +0530225 and si_item.sales_order = %s
226 and si.name != %s""", (so_detail, so, current_docname))
Anand Doshi01de9452016-01-26 16:22:50 +0530227
Nabin Hait453cc372015-07-29 12:21:03 +0530228 total_delivered_qty = (flt(delivered_via_dn[0][0]) if delivered_via_dn else 0) \
229 + (flt(delivered_via_si[0][0]) if delivered_via_si else 0)
Anand Doshi01de9452016-01-26 16:22:50 +0530230
Nabin Hait453cc372015-07-29 12:21:03 +0530231 return total_delivered_qty
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530232
233 def get_so_qty_and_warehouse(self, so_detail):
Anand Doshie9baaa62014-02-26 12:35:33 +0530234 so_item = frappe.db.sql("""select qty, warehouse from `tabSales Order Item`
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530235 where name = %s and docstatus = 1""", so_detail, as_dict=1)
236 so_qty = so_item and flt(so_item[0]["qty"]) or 0.0
Nabin Haita7f757a2014-02-10 17:54:04 +0530237 so_warehouse = so_item and so_item[0]["warehouse"] or ""
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530238 return so_qty, so_warehouse
Anand Doshid2946502014-04-08 20:10:03 +0530239
patilsangrambf2b5112016-02-22 16:24:23 +0530240 def check_close_sales_order(self, ref_fieldname):
Nabin Haitdd38a262014-12-26 13:15:21 +0530241 for d in self.get("items"):
Anand Doshif78d1ae2014-03-28 13:55:00 +0530242 if d.get(ref_fieldname):
Rushabh Mehtaf2227d02014-03-31 23:37:40 +0530243 status = frappe.db.get_value("Sales Order", d.get(ref_fieldname), "status")
patilsangrambf2b5112016-02-22 16:24:23 +0530244 if status == "Closed":
Saurabh8a8ef852015-10-21 19:28:53 +0530245 frappe.throw(_("Sales Order {0} is {1}").format(d.get(ref_fieldname), status))
shreyas1cfe2d72016-10-19 18:36:12 +0530246
Nabin Hait14aa9c52016-04-18 15:54:01 +0530247 def update_reserved_qty(self):
248 so_map = {}
249 for d in self.get("items"):
250 if d.so_detail:
251 if self.doctype == "Delivery Note" and d.against_sales_order:
252 so_map.setdefault(d.against_sales_order, []).append(d.so_detail)
253 elif self.doctype == "Sales Invoice" and d.sales_order and self.update_stock:
254 so_map.setdefault(d.sales_order, []).append(d.so_detail)
255
256 for so, so_item_rows in so_map.items():
257 if so and so_item_rows:
258 sales_order = frappe.get_doc("Sales Order", so)
259
260 if sales_order.status in ["Closed", "Cancelled"]:
261 frappe.throw(_("{0} {1} is cancelled or closed").format(_("Sales Order"), so),
262 frappe.InvalidStatusError)
263
264 sales_order.update_reserved_qty(so_item_rows)
265
266 def update_stock_ledger(self):
267 self.update_reserved_qty()
268
269 sl_entries = []
270 for d in self.get_item_list():
271 if frappe.db.get_value("Item", d.item_code, "is_stock_item") == 1 and flt(d.qty):
272 return_rate = 0
273 if cint(self.is_return) and self.return_against and self.docstatus==1:
274 return_rate = self.get_incoming_rate_for_sales_return(d.item_code, self.return_against)
275
276 # On cancellation or if return entry submission, make stock ledger entry for
277 # target warehouse first, to update serial no values properly
278
279 if d.warehouse and ((not cint(self.is_return) and self.docstatus==1)
280 or (cint(self.is_return) and self.docstatus==2)):
281 sl_entries.append(self.get_sl_entries(d, {
282 "actual_qty": -1*flt(d.qty),
283 "incoming_rate": return_rate
284 }))
285
286 if d.target_warehouse:
287 target_warehouse_sle = self.get_sl_entries(d, {
288 "actual_qty": flt(d.qty),
289 "warehouse": d.target_warehouse
290 })
291
292 if self.docstatus == 1:
293 if not cint(self.is_return):
294 args = frappe._dict({
295 "item_code": d.item_code,
296 "warehouse": d.warehouse,
297 "posting_date": self.posting_date,
298 "posting_time": self.posting_time,
299 "qty": -1*flt(d.qty),
300 "serial_no": d.serial_no
301 })
302 target_warehouse_sle.update({
303 "incoming_rate": get_incoming_rate(args)
304 })
305 else:
306 target_warehouse_sle.update({
307 "outgoing_rate": return_rate
308 })
309 sl_entries.append(target_warehouse_sle)
310
311 if d.warehouse and ((not cint(self.is_return) and self.docstatus==2)
312 or (cint(self.is_return) and self.docstatus==1)):
313 sl_entries.append(self.get_sl_entries(d, {
314 "actual_qty": -1*flt(d.qty),
315 "incoming_rate": return_rate
316 }))
317
318 self.make_sl_entries(sl_entries)
Anand Doshid2946502014-04-08 20:10:03 +0530319
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530320def check_active_sales_items(obj):
Nabin Haitdd38a262014-12-26 13:15:21 +0530321 for d in obj.get("items"):
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530322 if d.item_code:
Umair Sayyed72534de2016-04-15 12:52:12 +0530323 item = frappe.db.sql("""select docstatus,
Anand Doshi01de9452016-01-26 16:22:50 +0530324 income_account from tabItem where name = %s""",
Nabin Haitd1fd1e22013-10-18 12:29:11 +0530325 d.item_code, as_dict=True)[0]
shreyas1cfe2d72016-10-19 18:36:12 +0530326
Nabin Hait365ae272014-04-03 17:38:54 +0530327 if getattr(d, "income_account", None) and not item.income_account:
Anand Doshid2946502014-04-08 20:10:03 +0530328 frappe.db.set_value("Item", d.item_code, "income_account",
Nabin Hait89ad6f22013-10-18 12:55:59 +0530329 d.income_account)