Anand Doshi | 885e074 | 2015-03-03 14:55:30 +0530 | [diff] [blame] | 1 | # Copyright (c) 2015, Frappe 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 | ec52db8 | 2013-01-22 11:53:14 +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 |
Saqib | 3daad22 | 2020-10-12 18:31:23 +0530 | [diff] [blame] | 6 | from frappe.utils import cint, flt, cstr, comma_or, get_link_to_form |
Rushabh Mehta | 9f0d625 | 2014-04-14 19:20:45 +0530 | [diff] [blame] | 7 | from frappe import _, throw |
Saurabh | d313553 | 2016-02-25 18:59:20 +0530 | [diff] [blame] | 8 | from erpnext.stock.get_item_details import get_bin_details |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 9 | from erpnext.stock.utils import get_incoming_rate |
Kanchan Chauhan | 5a980ac | 2017-02-10 14:27:11 +0530 | [diff] [blame] | 10 | from erpnext.stock.get_item_details import get_conversion_factor |
Aditya Hase | f79937d | 2019-01-23 00:28:37 +0530 | [diff] [blame] | 11 | from erpnext.stock.doctype.item.item import set_item_default |
Rohit Waghchaure | f4dc716 | 2018-11-15 17:04:02 +0530 | [diff] [blame] | 12 | from frappe.contacts.doctype.address.address import get_address_display |
Deepesh Garg | 6f3d93e | 2020-09-18 11:23:35 +0530 | [diff] [blame] | 13 | from erpnext.controllers.accounts_controller import get_taxes_and_charges |
Nabin Hait | ec52db8 | 2013-01-22 11:53:14 +0530 | [diff] [blame] | 14 | |
Rushabh Mehta | 1f84799 | 2013-12-12 19:12:19 +0530 | [diff] [blame] | 15 | from erpnext.controllers.stock_controller import StockController |
Nabin Hait | bf495c9 | 2013-01-30 12:49:08 +0530 | [diff] [blame] | 16 | |
Nabin Hait | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 17 | class SellingController(StockController): |
Rushabh Mehta | f84c240 | 2014-07-18 16:39:31 +0530 | [diff] [blame] | 18 | def __setup__(self): |
Nabin Hait | de9c8a9 | 2015-02-23 01:06:00 +0530 | [diff] [blame] | 19 | if hasattr(self, "taxes"): |
Nabin Hait | 45dce89 | 2017-09-14 15:17:38 +0530 | [diff] [blame] | 20 | self.flags.print_taxes_with_zero_amount = cint(frappe.db.get_single_value("Print Settings", |
Ameya Shenoy | 2806d83 | 2018-11-28 08:12:48 +0000 | [diff] [blame] | 21 | "print_taxes_with_zero_amount")) |
rohitwaghchaure | bf4c114 | 2018-01-08 15:20:15 +0530 | [diff] [blame] | 22 | self.flags.show_inclusive_tax_in_print = self.is_inclusive_tax() |
| 23 | |
Rushabh Mehta | c567e8e | 2015-02-03 17:55:52 +0530 | [diff] [blame] | 24 | self.print_templates = { |
rohitwaghchaure | bf4c114 | 2018-01-08 15:20:15 +0530 | [diff] [blame] | 25 | "total": "templates/print_formats/includes/total.html", |
Rushabh Mehta | c567e8e | 2015-02-03 17:55:52 +0530 | [diff] [blame] | 26 | "taxes": "templates/print_formats/includes/taxes.html" |
Rushabh Mehta | 5b51cc8 | 2014-07-21 18:25:45 +0530 | [diff] [blame] | 27 | } |
Rushabh Mehta | f84c240 | 2014-07-18 16:39:31 +0530 | [diff] [blame] | 28 | |
Rushabh Mehta | c0bb453 | 2014-09-09 16:15:35 +0530 | [diff] [blame] | 29 | def get_feed(self): |
| 30 | return _("To {0} | {1} {2}").format(self.customer_name, self.currency, |
Nabin Hait | 5690be1 | 2015-02-12 16:09:11 +0530 | [diff] [blame] | 31 | self.grand_total) |
Rushabh Mehta | c0bb453 | 2014-09-09 16:15:35 +0530 | [diff] [blame] | 32 | |
Rushabh Mehta | b5e7689 | 2014-07-29 16:07:43 +0530 | [diff] [blame] | 33 | def onload(self): |
Rohit Waghchaure | 7d52989 | 2016-10-06 14:35:04 +0530 | [diff] [blame] | 34 | super(SellingController, self).onload() |
Rushabh Mehta | b5e7689 | 2014-07-29 16:07:43 +0530 | [diff] [blame] | 35 | if self.doctype in ("Sales Order", "Delivery Note", "Sales Invoice"): |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 36 | for item in self.get("items"): |
Nabin Hait | 877e1bb | 2017-11-17 12:27:43 +0530 | [diff] [blame] | 37 | item.update(get_bin_details(item.item_code, item.warehouse)) |
Rushabh Mehta | b5e7689 | 2014-07-29 16:07:43 +0530 | [diff] [blame] | 38 | |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 39 | def validate(self): |
| 40 | super(SellingController, self).validate() |
Zarrar | fc03a04 | 2018-06-04 12:52:52 +0530 | [diff] [blame] | 41 | self.validate_items() |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 42 | self.validate_max_discount() |
shreyas | 1cfe2d7 | 2016-10-19 18:36:12 +0530 | [diff] [blame] | 43 | self.validate_selling_price() |
Kanchan Chauhan | 5a980ac | 2017-02-10 14:27:11 +0530 | [diff] [blame] | 44 | self.set_qty_as_per_stock_uom() |
Nabin Hait | 96f4b22 | 2018-03-13 15:46:59 +0530 | [diff] [blame] | 45 | self.set_po_nos() |
Saif Ur Rehman | 5fdbc68 | 2018-11-26 19:32:16 +0500 | [diff] [blame] | 46 | self.set_gross_profit() |
Rushabh Mehta | 708e47a | 2018-08-08 16:37:31 +0530 | [diff] [blame] | 47 | set_default_income_account_for_item(self) |
Rohit Waghchaure | f4dc716 | 2018-11-15 17:04:02 +0530 | [diff] [blame] | 48 | self.set_customer_address() |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 49 | self.validate_for_duplicate_items() |
rohitwaghchaure | 6fa6caf | 2020-04-23 09:46:29 +0530 | [diff] [blame] | 50 | self.validate_target_warehouse() |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 51 | |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 52 | def set_missing_values(self, for_validate=False): |
Shreya Shah | 5615cb4 | 2018-10-07 11:42:07 +0530 | [diff] [blame] | 53 | |
Anand Doshi | abc1003 | 2013-06-14 17:44:03 +0530 | [diff] [blame] | 54 | super(SellingController, self).set_missing_values(for_validate) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 55 | |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 56 | # set contact and address details for customer, if they are not mentioned |
Deepesh Garg | 6f3d93e | 2020-09-18 11:23:35 +0530 | [diff] [blame] | 57 | self.set_missing_lead_customer_details(for_validate=for_validate) |
mbauskar | 9e9d242 | 2017-04-10 19:16:07 +0530 | [diff] [blame] | 58 | self.set_price_list_and_item_details(for_validate=for_validate) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 59 | |
Deepesh Garg | 6f3d93e | 2020-09-18 11:23:35 +0530 | [diff] [blame] | 60 | def set_missing_lead_customer_details(self, for_validate=False): |
Nabin Hait | 34c551d | 2019-07-03 10:34:31 +0530 | [diff] [blame] | 61 | customer, lead = None, None |
Rushabh Mehta | f14b809 | 2014-04-03 14:30:42 +0530 | [diff] [blame] | 62 | if getattr(self, "customer", None): |
Nabin Hait | 34c551d | 2019-07-03 10:34:31 +0530 | [diff] [blame] | 63 | customer = self.customer |
| 64 | elif self.doctype == "Opportunity" and self.party_name: |
| 65 | if self.opportunity_from == "Customer": |
| 66 | customer = self.party_name |
| 67 | else: |
| 68 | lead = self.party_name |
| 69 | elif self.doctype == "Quotation" and self.party_name: |
| 70 | if self.quotation_to == "Customer": |
| 71 | customer = self.party_name |
| 72 | else: |
| 73 | lead = self.party_name |
| 74 | |
| 75 | if customer: |
Anand Doshi | 4362443 | 2014-02-25 15:42:16 +0530 | [diff] [blame] | 76 | from erpnext.accounts.party import _get_party_details |
rohitwaghchaure | 3ffe896 | 2018-07-13 17:40:48 +0530 | [diff] [blame] | 77 | fetch_payment_terms_template = False |
| 78 | if (self.get("__islocal") or |
| 79 | self.company != frappe.db.get_value(self.doctype, self.name, 'company')): |
| 80 | fetch_payment_terms_template = True |
| 81 | |
Nabin Hait | 34c551d | 2019-07-03 10:34:31 +0530 | [diff] [blame] | 82 | party_details = _get_party_details(customer, |
Nabin Hait | 7eba1a3 | 2017-10-02 15:59:27 +0530 | [diff] [blame] | 83 | ignore_permissions=self.flags.ignore_permissions, |
rohitwaghchaure | 3ffe896 | 2018-07-13 17:40:48 +0530 | [diff] [blame] | 84 | doctype=self.doctype, company=self.company, |
Deepesh Garg | c152b92 | 2020-09-20 20:04:10 +0530 | [diff] [blame] | 85 | posting_date=self.get('posting_date'), |
Shreya Shah | 5615cb4 | 2018-10-07 11:42:07 +0530 | [diff] [blame] | 86 | fetch_payment_terms_template=fetch_payment_terms_template, |
Shreya Shah | 2291323 | 2018-10-11 10:13:27 +0530 | [diff] [blame] | 87 | party_address=self.customer_address, shipping_address=self.shipping_address_name) |
Nabin Hait | 6b03914 | 2014-05-02 15:45:10 +0530 | [diff] [blame] | 88 | if not self.meta.get_field("sales_team"): |
| 89 | party_details.pop("sales_team") |
Nabin Hait | 6b03914 | 2014-05-02 15:45:10 +0530 | [diff] [blame] | 90 | self.update_if_missing(party_details) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 91 | |
Nabin Hait | 34c551d | 2019-07-03 10:34:31 +0530 | [diff] [blame] | 92 | elif lead: |
Rushabh Mehta | 60cfccb | 2015-02-17 10:36:54 +0530 | [diff] [blame] | 93 | from erpnext.crm.doctype.lead.lead import get_lead_details |
Nabin Hait | 34c551d | 2019-07-03 10:34:31 +0530 | [diff] [blame] | 94 | self.update_if_missing(get_lead_details(lead, |
Anand Doshi | 2f957f2 | 2016-04-08 17:36:10 +0530 | [diff] [blame] | 95 | posting_date=self.get('transaction_date') or self.get('posting_date'), |
| 96 | company=self.company)) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 97 | |
Deepesh Garg | 6f3d93e | 2020-09-18 11:23:35 +0530 | [diff] [blame] | 98 | if self.get('taxes_and_charges') and not self.get('taxes') and not for_validate: |
| 99 | taxes = get_taxes_and_charges('Sales Taxes and Charges Template', self.taxes_and_charges) |
| 100 | for tax in taxes: |
| 101 | self.append('taxes', tax) |
| 102 | |
mbauskar | 9e9d242 | 2017-04-10 19:16:07 +0530 | [diff] [blame] | 103 | def set_price_list_and_item_details(self, for_validate=False): |
Nabin Hait | 096d363 | 2013-10-17 17:01:14 +0530 | [diff] [blame] | 104 | self.set_price_list_currency("Selling") |
mbauskar | 287fe81 | 2017-04-10 19:15:57 +0530 | [diff] [blame] | 105 | self.set_missing_item_details(for_validate=for_validate) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 106 | |
Anand Doshi | ac32bad | 2014-04-18 01:30:14 +0530 | [diff] [blame] | 107 | def remove_shipping_charge(self): |
| 108 | if self.shipping_rule: |
| 109 | shipping_rule = frappe.get_doc("Shipping Rule", self.shipping_rule) |
Nabin Hait | e7d1536 | 2014-12-25 16:01:55 +0530 | [diff] [blame] | 110 | existing_shipping_charge = self.get("taxes", { |
Anand Doshi | ac32bad | 2014-04-18 01:30:14 +0530 | [diff] [blame] | 111 | "doctype": "Sales Taxes and Charges", |
| 112 | "charge_type": "Actual", |
| 113 | "account_head": shipping_rule.account, |
| 114 | "cost_center": shipping_rule.cost_center |
Anand Doshi | 0b4943c | 2013-07-04 23:45:22 +0530 | [diff] [blame] | 115 | }) |
Anand Doshi | ac32bad | 2014-04-18 01:30:14 +0530 | [diff] [blame] | 116 | if existing_shipping_charge: |
Nabin Hait | e7d1536 | 2014-12-25 16:01:55 +0530 | [diff] [blame] | 117 | self.get("taxes").remove(existing_shipping_charge[-1]) |
Anand Doshi | ac32bad | 2014-04-18 01:30:14 +0530 | [diff] [blame] | 118 | self.calculate_taxes_and_totals() |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 119 | |
Nabin Hait | ec52db8 | 2013-01-22 11:53:14 +0530 | [diff] [blame] | 120 | def set_total_in_words(self): |
Rushabh Mehta | 793ba6b | 2014-02-14 15:47:51 +0530 | [diff] [blame] | 121 | from frappe.utils import money_in_words |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 122 | |
Nabin Hait | 5690be1 | 2015-02-12 16:09:11 +0530 | [diff] [blame] | 123 | if self.meta.get_field("base_in_words"): |
Nabin Hait | 877e1bb | 2017-11-17 12:27:43 +0530 | [diff] [blame] | 124 | base_amount = abs(self.base_grand_total |
| 125 | if self.is_rounded_total_disabled() else self.base_rounded_total) |
| 126 | self.base_in_words = money_in_words(base_amount, self.company_currency) |
| 127 | |
Nabin Hait | ec52db8 | 2013-01-22 11:53:14 +0530 | [diff] [blame] | 128 | if self.meta.get_field("in_words"): |
Nabin Hait | 877e1bb | 2017-11-17 12:27:43 +0530 | [diff] [blame] | 129 | amount = abs(self.grand_total if self.is_rounded_total_disabled() else self.rounded_total) |
| 130 | self.in_words = money_in_words(amount, self.currency) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 131 | |
Anand Doshi | 923d41d | 2013-05-28 17:23:36 +0530 | [diff] [blame] | 132 | def calculate_commission(self): |
| 133 | if self.meta.get_field("commission_rate"): |
Nabin Hait | 5690be1 | 2015-02-12 16:09:11 +0530 | [diff] [blame] | 134 | self.round_floats_in(self, ["base_net_total", "commission_rate"]) |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 135 | if self.commission_rate > 100.0: |
Rushabh Mehta | 9f0d625 | 2014-04-14 19:20:45 +0530 | [diff] [blame] | 136 | throw(_("Commission rate cannot be greater than 100")) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 137 | |
Nabin Hait | 5690be1 | 2015-02-12 16:09:11 +0530 | [diff] [blame] | 138 | self.total_commission = flt(self.base_net_total * self.commission_rate / 100.0, |
Anand Doshi | 923d41d | 2013-05-28 17:23:36 +0530 | [diff] [blame] | 139 | self.precision("total_commission")) |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 140 | |
| 141 | def calculate_contribution(self): |
Anand Doshi | ac32bad | 2014-04-18 01:30:14 +0530 | [diff] [blame] | 142 | if not self.meta.get_field("sales_team"): |
| 143 | return |
| 144 | |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 145 | total = 0.0 |
Rushabh Mehta | d2b34dc | 2014-03-27 16:12:56 +0530 | [diff] [blame] | 146 | sales_team = self.get("sales_team") |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 147 | for sales_person in sales_team: |
| 148 | self.round_floats_in(sales_person) |
| 149 | |
| 150 | sales_person.allocated_amount = flt( |
Nabin Hait | 5690be1 | 2015-02-12 16:09:11 +0530 | [diff] [blame] | 151 | self.base_net_total * sales_person.allocated_percentage / 100.0, |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 152 | self.precision("allocated_amount", sales_person)) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 153 | |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 154 | total += sales_person.allocated_percentage |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 155 | |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 156 | if sales_team and total != 100.0: |
Rushabh Mehta | 9f0d625 | 2014-04-14 19:20:45 +0530 | [diff] [blame] | 157 | throw(_("Total allocated percentage for sales team should be 100")) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 158 | |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 159 | def validate_max_discount(self): |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 160 | for d in self.get("items"): |
Nabin Hait | c35271a | 2018-08-10 19:01:04 +0530 | [diff] [blame] | 161 | if d.item_code: |
| 162 | discount = flt(frappe.get_cached_value("Item", d.item_code, "max_discount")) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 163 | |
Nabin Hait | c35271a | 2018-08-10 19:01:04 +0530 | [diff] [blame] | 164 | if discount and flt(d.discount_percentage) > discount: |
| 165 | frappe.throw(_("Maximum discount for Item {0} is {1}%").format(d.item_code, discount)) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 166 | |
Kanchan Chauhan | 5a980ac | 2017-02-10 14:27:11 +0530 | [diff] [blame] | 167 | 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 Mehta | cc8b2b2 | 2017-03-31 12:44:29 +0530 | [diff] [blame] | 172 | d.stock_qty = flt(d.qty) * flt(d.conversion_factor) |
Kanchan Chauhan | 5a980ac | 2017-02-10 14:27:11 +0530 | [diff] [blame] | 173 | |
shreyas | 1cfe2d7 | 2016-10-19 18:36:12 +0530 | [diff] [blame] | 174 | def validate_selling_price(self): |
Saqib | fff00ce | 2020-05-05 16:14:17 +0530 | [diff] [blame] | 175 | def throw_message(idx, item_name, rate, ref_rate_field): |
Saqib | 3daad22 | 2020-10-12 18:31:23 +0530 | [diff] [blame] | 176 | bold_net_rate = frappe.bold("net rate") |
| 177 | msg = (_("""Row #{}: Selling rate for item {} is lower than its {}. Selling {} should be atleast {}""") |
| 178 | .format(idx, frappe.bold(item_name), frappe.bold(ref_rate_field), bold_net_rate, frappe.bold(rate))) |
| 179 | msg += "<br><br>" |
| 180 | msg += (_("""You can alternatively disable selling price validation in {} to bypass this validation.""") |
| 181 | .format(get_link_to_form("Selling Settings", "Selling Settings"))) |
| 182 | frappe.throw(msg, title=_("Invalid Selling Price")) |
mbauskar | e9747a8 | 2016-11-16 17:19:47 +0530 | [diff] [blame] | 183 | |
Nabin Hait | 9f26bc1 | 2016-10-20 16:45:53 +0530 | [diff] [blame] | 184 | if not frappe.db.get_single_value("Selling Settings", "validate_selling_price"): |
shreyas | 1cfe2d7 | 2016-10-19 18:36:12 +0530 | [diff] [blame] | 185 | return |
rohitwaghchaure | 45ac31a | 2018-02-12 11:27:37 +0530 | [diff] [blame] | 186 | if hasattr(self, "is_return") and self.is_return: |
| 187 | return |
| 188 | |
shreyas | 1cfe2d7 | 2016-10-19 18:36:12 +0530 | [diff] [blame] | 189 | for it in self.get("items"): |
Saurabh | d7e1869 | 2017-09-18 15:43:35 +0530 | [diff] [blame] | 190 | if not it.item_code: |
| 191 | continue |
Saqib | 3daad22 | 2020-10-12 18:31:23 +0530 | [diff] [blame] | 192 | |
Himanshu Mishra | 859d942 | 2018-08-08 18:33:50 +0530 | [diff] [blame] | 193 | last_purchase_rate, is_stock_item = frappe.get_cached_value("Item", it.item_code, ["last_purchase_rate", "is_stock_item"]) |
Saqib | 3daad22 | 2020-10-12 18:31:23 +0530 | [diff] [blame] | 194 | last_purchase_rate_in_sales_uom = last_purchase_rate * (it.conversion_factor or 1) |
| 195 | if flt(it.base_net_rate) < flt(last_purchase_rate_in_sales_uom): |
Saqib | fff00ce | 2020-05-05 16:14:17 +0530 | [diff] [blame] | 196 | throw_message(it.idx, frappe.bold(it.item_name), last_purchase_rate_in_sales_uom, "last purchase rate") |
shreyas | e8aaa92 | 2016-10-20 14:03:59 +0530 | [diff] [blame] | 197 | |
| 198 | last_valuation_rate = frappe.db.sql(""" |
| 199 | SELECT valuation_rate FROM `tabStock Ledger Entry` WHERE item_code = %s |
| 200 | AND warehouse = %s AND valuation_rate > 0 |
Aditya Hase | 0c16424 | 2019-01-07 22:07:13 +0530 | [diff] [blame] | 201 | ORDER BY posting_date DESC, posting_time DESC, creation DESC LIMIT 1 |
shreyas | e8aaa92 | 2016-10-20 14:03:59 +0530 | [diff] [blame] | 202 | """, (it.item_code, it.warehouse)) |
Nabin Hait | 42343bb | 2017-05-05 17:23:17 +0530 | [diff] [blame] | 203 | if last_valuation_rate: |
Saqib | 3daad22 | 2020-10-12 18:31:23 +0530 | [diff] [blame] | 204 | last_valuation_rate_in_sales_uom = last_valuation_rate[0][0] * (it.conversion_factor or 1) |
| 205 | if is_stock_item and flt(it.base_net_rate) < flt(last_valuation_rate_in_sales_uom) \ |
Deepesh Garg | 15ff6a5 | 2020-02-18 12:28:41 +0530 | [diff] [blame] | 206 | and not self.get('is_internal_customer'): |
Saqib | fff00ce | 2020-05-05 16:14:17 +0530 | [diff] [blame] | 207 | throw_message(it.idx, frappe.bold(it.item_name), last_valuation_rate_in_sales_uom, "valuation rate") |
shreyas | e8aaa92 | 2016-10-20 14:03:59 +0530 | [diff] [blame] | 208 | |
shreyas | 1cfe2d7 | 2016-10-19 18:36:12 +0530 | [diff] [blame] | 209 | |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 210 | def get_item_list(self): |
| 211 | il = [] |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 212 | for d in self.get("items"): |
nathando | 247e9ff | 2014-09-03 15:03:31 +0800 | [diff] [blame] | 213 | if d.qty is None: |
Nabin Hait | c41a948 | 2014-08-18 15:40:15 +0530 | [diff] [blame] | 214 | frappe.throw(_("Row {0}: Qty is mandatory").format(d.idx)) |
Anand Doshi | 01de945 | 2016-01-26 16:22:50 +0530 | [diff] [blame] | 215 | |
Neil Trini Lasrado | ed8cecb | 2015-07-07 13:59:23 +0530 | [diff] [blame] | 216 | if self.has_product_bundle(d.item_code): |
Nabin Hait | e7d1536 | 2014-12-25 16:01:55 +0530 | [diff] [blame] | 217 | for p in self.get("packed_items"): |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 218 | if p.parent_detail_docname == d.name and p.parent_item == d.item_code: |
| 219 | # the packing details table's qty is already multiplied with parent's qty |
Rushabh Mehta | 793ba6b | 2014-02-14 15:47:51 +0530 | [diff] [blame] | 220 | il.append(frappe._dict({ |
mbauskar | e471b02 | 2017-04-12 16:12:10 +0530 | [diff] [blame] | 221 | 'warehouse': p.warehouse or d.warehouse, |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 222 | 'item_code': p.item_code, |
| 223 | 'qty': flt(p.qty), |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 224 | 'uom': p.uom, |
| 225 | 'batch_no': cstr(p.batch_no).strip(), |
| 226 | 'serial_no': cstr(p.serial_no).strip(), |
Nabin Hait | c865f22 | 2015-09-21 09:18:43 +0530 | [diff] [blame] | 227 | 'name': d.name, |
rohitwaghchaure | ce8adec | 2017-12-15 12:13:50 +0530 | [diff] [blame] | 228 | 'target_warehouse': p.target_warehouse, |
| 229 | 'company': self.company, |
| 230 | 'voucher_type': self.doctype, |
rohitwaghchaure | 019d4de | 2020-08-20 15:53:58 +0530 | [diff] [blame] | 231 | 'allow_zero_valuation': d.allow_zero_valuation_rate, |
| 232 | 'sales_invoice_item': d.get("sales_invoice_item"), |
| 233 | 'delivery_note_item': d.get("dn_detail") |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 234 | })) |
| 235 | else: |
Rushabh Mehta | 793ba6b | 2014-02-14 15:47:51 +0530 | [diff] [blame] | 236 | il.append(frappe._dict({ |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 237 | 'warehouse': d.warehouse, |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 238 | 'item_code': d.item_code, |
Nabin Hait | e68f32c | 2017-03-03 15:00:41 +0530 | [diff] [blame] | 239 | 'qty': d.stock_qty, |
Kanchan Chauhan | 5a980ac | 2017-02-10 14:27:11 +0530 | [diff] [blame] | 240 | 'uom': d.uom, |
Nabin Hait | 1d21842 | 2015-07-17 15:19:02 +0530 | [diff] [blame] | 241 | 'stock_uom': d.stock_uom, |
Kanchan Chauhan | 5a980ac | 2017-02-10 14:27:11 +0530 | [diff] [blame] | 242 | 'conversion_factor': d.conversion_factor, |
Nabin Hait | dc82d4f | 2014-04-07 12:02:57 +0530 | [diff] [blame] | 243 | 'batch_no': cstr(d.get("batch_no")).strip(), |
| 244 | 'serial_no': cstr(d.get("serial_no")).strip(), |
Nabin Hait | c865f22 | 2015-09-21 09:18:43 +0530 | [diff] [blame] | 245 | 'name': d.name, |
rohitwaghchaure | ce8adec | 2017-12-15 12:13:50 +0530 | [diff] [blame] | 246 | 'target_warehouse': d.target_warehouse, |
| 247 | 'company': self.company, |
| 248 | 'voucher_type': self.doctype, |
rohitwaghchaure | 019d4de | 2020-08-20 15:53:58 +0530 | [diff] [blame] | 249 | 'allow_zero_valuation': d.allow_zero_valuation_rate, |
| 250 | 'sales_invoice_item': d.get("sales_invoice_item"), |
| 251 | 'delivery_note_item': d.get("dn_detail") |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 252 | })) |
| 253 | return il |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 254 | |
Neil Trini Lasrado | ed8cecb | 2015-07-07 13:59:23 +0530 | [diff] [blame] | 255 | def has_product_bundle(self, item_code): |
| 256 | return frappe.db.sql("""select name from `tabProduct Bundle` |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 257 | where new_item_code=%s and docstatus != 2""", item_code) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 258 | |
Nabin Hait | 453cc37 | 2015-07-29 12:21:03 +0530 | [diff] [blame] | 259 | def get_already_delivered_qty(self, current_docname, so, so_detail): |
| 260 | delivered_via_dn = frappe.db.sql("""select sum(qty) from `tabDelivery Note Item` |
ankitjavalkarwork | e69a611 | 2014-10-08 12:46:02 +0530 | [diff] [blame] | 261 | where so_detail = %s and docstatus = 1 |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 262 | and against_sales_order = %s |
Nabin Hait | 453cc37 | 2015-07-29 12:21:03 +0530 | [diff] [blame] | 263 | and parent != %s""", (so_detail, so, current_docname)) |
Anand Doshi | 01de945 | 2016-01-26 16:22:50 +0530 | [diff] [blame] | 264 | |
| 265 | delivered_via_si = frappe.db.sql("""select sum(si_item.qty) |
Nabin Hait | 1826791 | 2015-08-17 15:36:23 +0530 | [diff] [blame] | 266 | from `tabSales Invoice Item` si_item, `tabSales Invoice` si |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 267 | where si_item.parent = si.name and si.update_stock = 1 |
Anand Doshi | 01de945 | 2016-01-26 16:22:50 +0530 | [diff] [blame] | 268 | and si_item.so_detail = %s and si.docstatus = 1 |
Nabin Hait | 1826791 | 2015-08-17 15:36:23 +0530 | [diff] [blame] | 269 | and si_item.sales_order = %s |
| 270 | and si.name != %s""", (so_detail, so, current_docname)) |
Anand Doshi | 01de945 | 2016-01-26 16:22:50 +0530 | [diff] [blame] | 271 | |
Nabin Hait | 453cc37 | 2015-07-29 12:21:03 +0530 | [diff] [blame] | 272 | total_delivered_qty = (flt(delivered_via_dn[0][0]) if delivered_via_dn else 0) \ |
| 273 | + (flt(delivered_via_si[0][0]) if delivered_via_si else 0) |
Anand Doshi | 01de945 | 2016-01-26 16:22:50 +0530 | [diff] [blame] | 274 | |
Nabin Hait | 453cc37 | 2015-07-29 12:21:03 +0530 | [diff] [blame] | 275 | return total_delivered_qty |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 276 | |
| 277 | def get_so_qty_and_warehouse(self, so_detail): |
Anand Doshi | e9baaa6 | 2014-02-26 12:35:33 +0530 | [diff] [blame] | 278 | so_item = frappe.db.sql("""select qty, warehouse from `tabSales Order Item` |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 279 | where name = %s and docstatus = 1""", so_detail, as_dict=1) |
| 280 | so_qty = so_item and flt(so_item[0]["qty"]) or 0.0 |
Nabin Hait | a7f757a | 2014-02-10 17:54:04 +0530 | [diff] [blame] | 281 | so_warehouse = so_item and so_item[0]["warehouse"] or "" |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 282 | return so_qty, so_warehouse |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 283 | |
Mangesh-Khairnar | e17b709 | 2019-03-01 15:52:23 +0530 | [diff] [blame] | 284 | def check_sales_order_on_hold_or_close(self, ref_fieldname): |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 285 | for d in self.get("items"): |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 286 | if d.get(ref_fieldname): |
Rushabh Mehta | f2227d0 | 2014-03-31 23:37:40 +0530 | [diff] [blame] | 287 | status = frappe.db.get_value("Sales Order", d.get(ref_fieldname), "status") |
Mangesh-Khairnar | 86e650c | 2019-03-02 20:30:21 +0530 | [diff] [blame] | 288 | if status in ("Closed", "On Hold"): |
Saurabh | 8a8ef85 | 2015-10-21 19:28:53 +0530 | [diff] [blame] | 289 | frappe.throw(_("Sales Order {0} is {1}").format(d.get(ref_fieldname), status)) |
shreyas | 1cfe2d7 | 2016-10-19 18:36:12 +0530 | [diff] [blame] | 290 | |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 291 | def update_reserved_qty(self): |
| 292 | so_map = {} |
| 293 | for d in self.get("items"): |
| 294 | if d.so_detail: |
| 295 | if self.doctype == "Delivery Note" and d.against_sales_order: |
| 296 | so_map.setdefault(d.against_sales_order, []).append(d.so_detail) |
| 297 | elif self.doctype == "Sales Invoice" and d.sales_order and self.update_stock: |
| 298 | so_map.setdefault(d.sales_order, []).append(d.so_detail) |
| 299 | |
| 300 | for so, so_item_rows in so_map.items(): |
| 301 | if so and so_item_rows: |
| 302 | sales_order = frappe.get_doc("Sales Order", so) |
| 303 | |
| 304 | if sales_order.status in ["Closed", "Cancelled"]: |
| 305 | frappe.throw(_("{0} {1} is cancelled or closed").format(_("Sales Order"), so), |
| 306 | frappe.InvalidStatusError) |
| 307 | |
| 308 | sales_order.update_reserved_qty(so_item_rows) |
| 309 | |
| 310 | def update_stock_ledger(self): |
| 311 | self.update_reserved_qty() |
| 312 | |
| 313 | sl_entries = [] |
| 314 | for d in self.get_item_list(): |
Himanshu Mishra | 859d942 | 2018-08-08 18:33:50 +0530 | [diff] [blame] | 315 | if frappe.get_cached_value("Item", d.item_code, "is_stock_item") == 1 and flt(d.qty): |
Kanchan Chauhan | 5a980ac | 2017-02-10 14:27:11 +0530 | [diff] [blame] | 316 | if flt(d.conversion_factor)==0.0: |
| 317 | d.conversion_factor = get_conversion_factor(d.item_code, d.uom).get("conversion_factor") or 1.0 |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 318 | return_rate = 0 |
| 319 | if cint(self.is_return) and self.return_against and self.docstatus==1: |
rohitwaghchaure | 019d4de | 2020-08-20 15:53:58 +0530 | [diff] [blame] | 320 | against_document_no = (d.get("sales_invoice_item") |
| 321 | if self.doctype == "Sales Invoice" else d.get("delivery_note_item")) |
| 322 | |
| 323 | return_rate = self.get_incoming_rate_for_return(d.item_code, |
| 324 | self.return_against, against_document_no) |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 325 | |
| 326 | # On cancellation or if return entry submission, make stock ledger entry for |
| 327 | # target warehouse first, to update serial no values properly |
| 328 | |
| 329 | if d.warehouse and ((not cint(self.is_return) and self.docstatus==1) |
| 330 | or (cint(self.is_return) and self.docstatus==2)): |
| 331 | sl_entries.append(self.get_sl_entries(d, { |
Nabin Hait | e68f32c | 2017-03-03 15:00:41 +0530 | [diff] [blame] | 332 | "actual_qty": -1*flt(d.qty), |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 333 | "incoming_rate": return_rate |
| 334 | })) |
| 335 | |
| 336 | if d.target_warehouse: |
| 337 | target_warehouse_sle = self.get_sl_entries(d, { |
Nabin Hait | e68f32c | 2017-03-03 15:00:41 +0530 | [diff] [blame] | 338 | "actual_qty": flt(d.qty), |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 339 | "warehouse": d.target_warehouse |
| 340 | }) |
| 341 | |
| 342 | if self.docstatus == 1: |
| 343 | if not cint(self.is_return): |
| 344 | args = frappe._dict({ |
| 345 | "item_code": d.item_code, |
| 346 | "warehouse": d.warehouse, |
| 347 | "posting_date": self.posting_date, |
| 348 | "posting_time": self.posting_time, |
| 349 | "qty": -1*flt(d.qty), |
rohitwaghchaure | ce8adec | 2017-12-15 12:13:50 +0530 | [diff] [blame] | 350 | "serial_no": d.serial_no, |
| 351 | "company": d.company, |
| 352 | "voucher_type": d.voucher_type, |
| 353 | "voucher_no": d.name, |
| 354 | "allow_zero_valuation": d.allow_zero_valuation |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 355 | }) |
| 356 | target_warehouse_sle.update({ |
| 357 | "incoming_rate": get_incoming_rate(args) |
| 358 | }) |
| 359 | else: |
| 360 | target_warehouse_sle.update({ |
| 361 | "outgoing_rate": return_rate |
| 362 | }) |
| 363 | sl_entries.append(target_warehouse_sle) |
| 364 | |
| 365 | if d.warehouse and ((not cint(self.is_return) and self.docstatus==2) |
| 366 | or (cint(self.is_return) and self.docstatus==1)): |
| 367 | sl_entries.append(self.get_sl_entries(d, { |
Nabin Hait | e68f32c | 2017-03-03 15:00:41 +0530 | [diff] [blame] | 368 | "actual_qty": -1*flt(d.qty), |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 369 | "incoming_rate": return_rate |
| 370 | })) |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 371 | self.make_sl_entries(sl_entries) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 372 | |
Nabin Hait | 96f4b22 | 2018-03-13 15:46:59 +0530 | [diff] [blame] | 373 | def set_po_nos(self): |
Shreya | f7b3943 | 2018-03-19 11:32:39 +0530 | [diff] [blame] | 374 | if self.doctype in ("Delivery Note", "Sales Invoice") and hasattr(self, "items"): |
Nabin Hait | 96f4b22 | 2018-03-13 15:46:59 +0530 | [diff] [blame] | 375 | ref_fieldname = "against_sales_order" if self.doctype == "Delivery Note" else "sales_order" |
Nabin Hait | 1e4cfb4 | 2018-03-15 12:27:24 +0530 | [diff] [blame] | 376 | sales_orders = list(set([d.get(ref_fieldname) for d in self.items if d.get(ref_fieldname)])) |
Nabin Hait | 96f4b22 | 2018-03-13 15:46:59 +0530 | [diff] [blame] | 377 | if sales_orders: |
| 378 | po_nos = frappe.get_all('Sales Order', 'po_no', filters = {'name': ('in', sales_orders)}) |
patilsangram | 667f0fb | 2018-11-08 12:51:41 +0530 | [diff] [blame] | 379 | if po_nos and po_nos[0].get('po_no'): |
| 380 | self.po_no = ', '.join(list(set([d.po_no for d in po_nos if d.po_no]))) |
Nabin Hait | 96f4b22 | 2018-03-13 15:46:59 +0530 | [diff] [blame] | 381 | |
Saif Ur Rehman | 5fdbc68 | 2018-11-26 19:32:16 +0500 | [diff] [blame] | 382 | def set_gross_profit(self): |
Abhishek Balam | 18e7b60 | 2020-05-26 18:37:02 +0530 | [diff] [blame] | 383 | if self.doctype in ["Sales Order", "Quotation"]: |
Saif Ur Rehman | 5fdbc68 | 2018-11-26 19:32:16 +0500 | [diff] [blame] | 384 | for item in self.items: |
| 385 | item.gross_profit = flt(((item.base_rate - item.valuation_rate) * item.stock_qty), self.precision("amount", item)) |
| 386 | |
| 387 | |
Rohit Waghchaure | f4dc716 | 2018-11-15 17:04:02 +0530 | [diff] [blame] | 388 | def set_customer_address(self): |
| 389 | address_dict = { |
| 390 | 'customer_address': 'address_display', |
| 391 | 'shipping_address_name': 'shipping_address', |
| 392 | 'company_address': 'company_address_display' |
| 393 | } |
| 394 | |
| 395 | for address_field, address_display_field in address_dict.items(): |
| 396 | if self.get(address_field): |
| 397 | self.set(address_display_field, get_address_display(self.get(address_field))) |
| 398 | |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 399 | def validate_for_duplicate_items(self): |
| 400 | check_list, chk_dupl_itm = [], [] |
| 401 | if cint(frappe.db.get_single_value("Selling Settings", "allow_multiple_items")): |
| 402 | return |
| 403 | |
| 404 | for d in self.get('items'): |
| 405 | if self.doctype == "Sales Invoice": |
| 406 | e = [d.item_code, d.description, d.warehouse, d.sales_order or d.delivery_note, d.batch_no or ''] |
| 407 | f = [d.item_code, d.description, d.sales_order or d.delivery_note] |
| 408 | elif self.doctype == "Delivery Note": |
| 409 | e = [d.item_code, d.description, d.warehouse, d.against_sales_order or d.against_sales_invoice, d.batch_no or ''] |
| 410 | f = [d.item_code, d.description, d.against_sales_order or d.against_sales_invoice] |
Rohit Waghchaure | 13c7e18 | 2019-08-29 19:45:19 +0530 | [diff] [blame] | 411 | elif self.doctype in ["Sales Order", "Quotation"]: |
| 412 | e = [d.item_code, d.description, d.warehouse, ''] |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 413 | f = [d.item_code, d.description] |
| 414 | |
| 415 | if frappe.db.get_value("Item", d.item_code, "is_stock_item") == 1: |
| 416 | if e in check_list: |
| 417 | frappe.throw(_("Note: Item {0} entered multiple times").format(d.item_code)) |
| 418 | else: |
| 419 | check_list.append(e) |
| 420 | else: |
| 421 | if f in chk_dupl_itm: |
| 422 | frappe.throw(_("Note: Item {0} entered multiple times").format(d.item_code)) |
| 423 | else: |
| 424 | chk_dupl_itm.append(f) |
| 425 | |
rohitwaghchaure | 6fa6caf | 2020-04-23 09:46:29 +0530 | [diff] [blame] | 426 | def validate_target_warehouse(self): |
| 427 | items = self.get("items") + (self.get("packed_items") or []) |
| 428 | |
| 429 | for d in items: |
| 430 | if d.get("target_warehouse") and d.get("warehouse") == d.get("target_warehouse"): |
| 431 | warehouse = frappe.bold(d.get("target_warehouse")) |
| 432 | frappe.throw(_("Row {0}: Delivery Warehouse ({1}) and Customer Warehouse ({2}) can not be same") |
| 433 | .format(d.idx, warehouse, warehouse)) |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 434 | |
Zarrar | fc03a04 | 2018-06-04 12:52:52 +0530 | [diff] [blame] | 435 | def validate_items(self): |
| 436 | # validate items to see if they have is_sales_item enabled |
| 437 | from erpnext.controllers.buying_controller import validate_item_type |
| 438 | validate_item_type(self, "is_sales_item", "sales") |
| 439 | |
Rushabh Mehta | 708e47a | 2018-08-08 16:37:31 +0530 | [diff] [blame] | 440 | def set_default_income_account_for_item(obj): |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 441 | for d in obj.get("items"): |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 442 | if d.item_code: |
Manas Solanki | 5b93562 | 2018-05-16 17:44:51 +0530 | [diff] [blame] | 443 | if getattr(d, "income_account", None): |
Rushabh Mehta | 708e47a | 2018-08-08 16:37:31 +0530 | [diff] [blame] | 444 | set_item_default(d.item_code, obj.company, 'income_account', d.income_account) |