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 | |
Nabin Hait | ec52db8 | 2013-01-22 11:53:14 +0530 | [diff] [blame] | 4 | |
Chillar Anand | 915b343 | 2021-09-02 16:44:59 +0530 | [diff] [blame] | 5 | import frappe |
| 6 | from frappe import _, bold, throw |
Rohit Waghchaure | 6c9b212 | 2022-12-05 14:48:18 +0530 | [diff] [blame] | 7 | from frappe.utils import cint, flt, get_link_to_form, nowtime |
Chillar Anand | 915b343 | 2021-09-02 16:44:59 +0530 | [diff] [blame] | 8 | |
Ankush Menat | f4d7499 | 2023-10-17 22:11:56 +0530 | [diff] [blame] | 9 | from erpnext.accounts.party import render_address |
Chillar Anand | 915b343 | 2021-09-02 16:44:59 +0530 | [diff] [blame] | 10 | from erpnext.controllers.accounts_controller import get_taxes_and_charges |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 11 | from erpnext.controllers.sales_and_purchase_return import get_rate_for_return |
Chillar Anand | 915b343 | 2021-09-02 16:44:59 +0530 | [diff] [blame] | 12 | from erpnext.controllers.stock_controller import StockController |
| 13 | from erpnext.stock.doctype.item.item import set_item_default |
| 14 | from erpnext.stock.get_item_details import get_bin_details, get_conversion_factor |
mergify[bot] | 4057682 | 2023-12-19 18:29:56 +0530 | [diff] [blame] | 15 | from erpnext.stock.utils import get_incoming_rate, get_valuation_method |
Chillar Anand | 915b343 | 2021-09-02 16:44:59 +0530 | [diff] [blame] | 16 | |
Nabin Hait | bf495c9 | 2013-01-30 12:49:08 +0530 | [diff] [blame] | 17 | |
Nabin Hait | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 18 | class SellingController(StockController): |
Nabin Hait | 993c6c0 | 2022-04-12 12:24:47 +0530 | [diff] [blame] | 19 | def __setup__(self): |
| 20 | self.flags.ignore_permlevel_for_fields = ["selling_price_list", "price_list_currency"] |
| 21 | |
Rushabh Mehta | b5e7689 | 2014-07-29 16:07:43 +0530 | [diff] [blame] | 22 | def onload(self): |
Akhil Narang | 3effaf2 | 2024-03-27 11:37:26 +0530 | [diff] [blame] | 23 | super().onload() |
Rushabh Mehta | b5e7689 | 2014-07-29 16:07:43 +0530 | [diff] [blame] | 24 | if self.doctype in ("Sales Order", "Delivery Note", "Sales Invoice"): |
Rohit Waghchaure | 02566a0 | 2023-01-20 22:38:56 +0530 | [diff] [blame] | 25 | for item in self.get("items") + (self.get("packed_items") or []): |
s-aga-r | c716dcc | 2022-12-29 12:34:18 +0530 | [diff] [blame] | 26 | item.update(get_bin_details(item.item_code, item.warehouse, include_child_warehouses=True)) |
Rushabh Mehta | b5e7689 | 2014-07-29 16:07:43 +0530 | [diff] [blame] | 27 | |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 28 | def validate(self): |
Akhil Narang | 3effaf2 | 2024-03-27 11:37:26 +0530 | [diff] [blame] | 29 | super().validate() |
Zarrar | fc03a04 | 2018-06-04 12:52:52 +0530 | [diff] [blame] | 30 | self.validate_items() |
Gursheen Anand | 5a3b133 | 2024-02-22 16:08:11 +0530 | [diff] [blame] | 31 | if not self.get("is_debit_note"): |
| 32 | self.validate_max_discount() |
shreyas | 1cfe2d7 | 2016-10-19 18:36:12 +0530 | [diff] [blame] | 33 | self.validate_selling_price() |
Kanchan Chauhan | 5a980ac | 2017-02-10 14:27:11 +0530 | [diff] [blame] | 34 | self.set_qty_as_per_stock_uom() |
Afshan | 01eca12 | 2020-12-08 09:27:56 +0530 | [diff] [blame] | 35 | self.set_po_nos(for_validate=True) |
Saif Ur Rehman | 5fdbc68 | 2018-11-26 19:32:16 +0500 | [diff] [blame] | 36 | self.set_gross_profit() |
Rushabh Mehta | 708e47a | 2018-08-08 16:37:31 +0530 | [diff] [blame] | 37 | set_default_income_account_for_item(self) |
Rohit Waghchaure | f4dc716 | 2018-11-15 17:04:02 +0530 | [diff] [blame] | 38 | self.set_customer_address() |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 39 | self.validate_for_duplicate_items() |
rohitwaghchaure | 6fa6caf | 2020-04-23 09:46:29 +0530 | [diff] [blame] | 40 | self.validate_target_warehouse() |
Deepesh Garg | fa15221 | 2022-11-28 19:22:35 +0530 | [diff] [blame] | 41 | self.validate_auto_repeat_subscription_dates() |
Rohit Waghchaure | 674bd3e | 2023-03-17 16:42:59 +0530 | [diff] [blame] | 42 | for table_field in ["items", "packed_items"]: |
| 43 | if self.get(table_field): |
| 44 | self.set_serial_and_batch_bundle(table_field) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 45 | |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 46 | def set_missing_values(self, for_validate=False): |
Akhil Narang | 3effaf2 | 2024-03-27 11:37:26 +0530 | [diff] [blame] | 47 | super().set_missing_values(for_validate) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 48 | |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 49 | # set contact and address details for customer, if they are not mentioned |
Deepesh Garg | 6f3d93e | 2020-09-18 11:23:35 +0530 | [diff] [blame] | 50 | self.set_missing_lead_customer_details(for_validate=for_validate) |
mbauskar | 9e9d242 | 2017-04-10 19:16:07 +0530 | [diff] [blame] | 51 | self.set_price_list_and_item_details(for_validate=for_validate) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 52 | |
Deepesh Garg | 6f3d93e | 2020-09-18 11:23:35 +0530 | [diff] [blame] | 53 | def set_missing_lead_customer_details(self, for_validate=False): |
Nabin Hait | 34c551d | 2019-07-03 10:34:31 +0530 | [diff] [blame] | 54 | customer, lead = None, None |
Rushabh Mehta | f14b809 | 2014-04-03 14:30:42 +0530 | [diff] [blame] | 55 | if getattr(self, "customer", None): |
Nabin Hait | 34c551d | 2019-07-03 10:34:31 +0530 | [diff] [blame] | 56 | customer = self.customer |
| 57 | elif self.doctype == "Opportunity" and self.party_name: |
| 58 | if self.opportunity_from == "Customer": |
| 59 | customer = self.party_name |
| 60 | else: |
| 61 | lead = self.party_name |
| 62 | elif self.doctype == "Quotation" and self.party_name: |
| 63 | if self.quotation_to == "Customer": |
| 64 | customer = self.party_name |
Nabin Hait | 5a0aacc | 2023-06-06 15:39:22 +0530 | [diff] [blame] | 65 | elif self.quotation_to == "Lead": |
Nabin Hait | 34c551d | 2019-07-03 10:34:31 +0530 | [diff] [blame] | 66 | lead = self.party_name |
| 67 | |
| 68 | if customer: |
Anand Doshi | 4362443 | 2014-02-25 15:42:16 +0530 | [diff] [blame] | 69 | from erpnext.accounts.party import _get_party_details |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 70 | |
rohitwaghchaure | 3ffe896 | 2018-07-13 17:40:48 +0530 | [diff] [blame] | 71 | fetch_payment_terms_template = False |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 72 | if self.get("__islocal") or self.company != frappe.db.get_value( |
| 73 | self.doctype, self.name, "company" |
| 74 | ): |
rohitwaghchaure | 3ffe896 | 2018-07-13 17:40:48 +0530 | [diff] [blame] | 75 | fetch_payment_terms_template = True |
| 76 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 77 | party_details = _get_party_details( |
| 78 | customer, |
Nabin Hait | 7eba1a3 | 2017-10-02 15:59:27 +0530 | [diff] [blame] | 79 | ignore_permissions=self.flags.ignore_permissions, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 80 | doctype=self.doctype, |
| 81 | company=self.company, |
| 82 | posting_date=self.get("posting_date"), |
Shreya Shah | 5615cb4 | 2018-10-07 11:42:07 +0530 | [diff] [blame] | 83 | fetch_payment_terms_template=fetch_payment_terms_template, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 84 | party_address=self.customer_address, |
| 85 | shipping_address=self.shipping_address_name, |
| 86 | company_address=self.get("company_address"), |
| 87 | ) |
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 |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 94 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 95 | self.update_if_missing( |
| 96 | get_lead_details( |
| 97 | lead, |
| 98 | posting_date=self.get("transaction_date") or self.get("posting_date"), |
| 99 | company=self.company, |
| 100 | ) |
| 101 | ) |
| 102 | |
| 103 | if self.get("taxes_and_charges") and not self.get("taxes") and not for_validate: |
| 104 | taxes = get_taxes_and_charges("Sales Taxes and Charges Template", self.taxes_and_charges) |
Deepesh Garg | 6f3d93e | 2020-09-18 11:23:35 +0530 | [diff] [blame] | 105 | for tax in taxes: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 106 | self.append("taxes", tax) |
Deepesh Garg | 6f3d93e | 2020-09-18 11:23:35 +0530 | [diff] [blame] | 107 | |
mbauskar | 9e9d242 | 2017-04-10 19:16:07 +0530 | [diff] [blame] | 108 | def set_price_list_and_item_details(self, for_validate=False): |
Nabin Hait | 096d363 | 2013-10-17 17:01:14 +0530 | [diff] [blame] | 109 | self.set_price_list_currency("Selling") |
mbauskar | 287fe81 | 2017-04-10 19:15:57 +0530 | [diff] [blame] | 110 | self.set_missing_item_details(for_validate=for_validate) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 111 | |
Anand Doshi | ac32bad | 2014-04-18 01:30:14 +0530 | [diff] [blame] | 112 | def remove_shipping_charge(self): |
| 113 | if self.shipping_rule: |
| 114 | shipping_rule = frappe.get_doc("Shipping Rule", self.shipping_rule) |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 115 | existing_shipping_charge = self.get( |
| 116 | "taxes", |
| 117 | { |
| 118 | "doctype": "Sales Taxes and Charges", |
| 119 | "charge_type": "Actual", |
| 120 | "account_head": shipping_rule.account, |
| 121 | "cost_center": shipping_rule.cost_center, |
| 122 | }, |
| 123 | ) |
Anand Doshi | ac32bad | 2014-04-18 01:30:14 +0530 | [diff] [blame] | 124 | if existing_shipping_charge: |
Nabin Hait | e7d1536 | 2014-12-25 16:01:55 +0530 | [diff] [blame] | 125 | self.get("taxes").remove(existing_shipping_charge[-1]) |
Anand Doshi | ac32bad | 2014-04-18 01:30:14 +0530 | [diff] [blame] | 126 | self.calculate_taxes_and_totals() |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 127 | |
Nabin Hait | ec52db8 | 2013-01-22 11:53:14 +0530 | [diff] [blame] | 128 | def set_total_in_words(self): |
Rushabh Mehta | 793ba6b | 2014-02-14 15:47:51 +0530 | [diff] [blame] | 129 | from frappe.utils import money_in_words |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 130 | |
Nabin Hait | 5690be1 | 2015-02-12 16:09:11 +0530 | [diff] [blame] | 131 | if self.meta.get_field("base_in_words"): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 132 | base_amount = abs( |
| 133 | self.base_grand_total if self.is_rounded_total_disabled() else self.base_rounded_total |
| 134 | ) |
Nabin Hait | 877e1bb | 2017-11-17 12:27:43 +0530 | [diff] [blame] | 135 | self.base_in_words = money_in_words(base_amount, self.company_currency) |
| 136 | |
Nabin Hait | ec52db8 | 2013-01-22 11:53:14 +0530 | [diff] [blame] | 137 | if self.meta.get_field("in_words"): |
Nabin Hait | 877e1bb | 2017-11-17 12:27:43 +0530 | [diff] [blame] | 138 | amount = abs(self.grand_total if self.is_rounded_total_disabled() else self.rounded_total) |
| 139 | self.in_words = money_in_words(amount, self.currency) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 140 | |
Anand Doshi | 923d41d | 2013-05-28 17:23:36 +0530 | [diff] [blame] | 141 | def calculate_commission(self): |
Deepesh Garg | f76da50 | 2024-03-29 17:57:58 +0530 | [diff] [blame] | 142 | if not self.meta.get_field("commission_rate"): |
Raffael Meyer | e10ab16 | 2021-11-30 13:24:18 +0100 | [diff] [blame] | 143 | return |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 144 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 145 | self.round_floats_in(self, ("amount_eligible_for_commission", "commission_rate")) |
Raffael Meyer | e10ab16 | 2021-11-30 13:24:18 +0100 | [diff] [blame] | 146 | |
| 147 | if not (0 <= self.commission_rate <= 100.0): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 148 | throw( |
| 149 | "{} {}".format( |
| 150 | _(self.meta.get_label("commission_rate")), |
| 151 | _("must be between 0 and 100"), |
| 152 | ) |
| 153 | ) |
Raffael Meyer | e10ab16 | 2021-11-30 13:24:18 +0100 | [diff] [blame] | 154 | |
| 155 | self.amount_eligible_for_commission = sum( |
| 156 | item.base_net_amount for item in self.items if item.grant_commission |
| 157 | ) |
| 158 | |
| 159 | self.total_commission = flt( |
| 160 | self.amount_eligible_for_commission * self.commission_rate / 100.0, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 161 | self.precision("total_commission"), |
Raffael Meyer | e10ab16 | 2021-11-30 13:24:18 +0100 | [diff] [blame] | 162 | ) |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 163 | |
| 164 | def calculate_contribution(self): |
Anand Doshi | ac32bad | 2014-04-18 01:30:14 +0530 | [diff] [blame] | 165 | if not self.meta.get_field("sales_team"): |
| 166 | return |
| 167 | |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 168 | total = 0.0 |
Rushabh Mehta | d2b34dc | 2014-03-27 16:12:56 +0530 | [diff] [blame] | 169 | sales_team = self.get("sales_team") |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 170 | for sales_person in sales_team: |
| 171 | self.round_floats_in(sales_person) |
| 172 | |
| 173 | sales_person.allocated_amount = flt( |
Indrajith.vs | 0c8276e | 2023-05-14 11:47:46 +0530 | [diff] [blame] | 174 | flt(self.amount_eligible_for_commission) * sales_person.allocated_percentage / 100.0, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 175 | self.precision("allocated_amount", sales_person), |
| 176 | ) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 177 | |
Anuja Pawar | 8755339 | 2021-03-01 10:44:13 +0530 | [diff] [blame] | 178 | if sales_person.commission_rate: |
| 179 | sales_person.incentives = flt( |
Sagar Vora | f4a6c78 | 2021-03-17 19:57:53 +0530 | [diff] [blame] | 180 | sales_person.allocated_amount * flt(sales_person.commission_rate) / 100.0, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 181 | self.precision("incentives", sales_person), |
| 182 | ) |
Anuja Pawar | 8755339 | 2021-03-01 10:44:13 +0530 | [diff] [blame] | 183 | |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 184 | total += sales_person.allocated_percentage |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 185 | |
Anand Doshi | f309613 | 2013-05-21 19:35:06 +0530 | [diff] [blame] | 186 | if sales_team and total != 100.0: |
Rushabh Mehta | 9f0d625 | 2014-04-14 19:20:45 +0530 | [diff] [blame] | 187 | throw(_("Total allocated percentage for sales team should be 100")) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 188 | |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 189 | def validate_max_discount(self): |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 190 | for d in self.get("items"): |
Nabin Hait | c35271a | 2018-08-10 19:01:04 +0530 | [diff] [blame] | 191 | if d.item_code: |
| 192 | discount = flt(frappe.get_cached_value("Item", d.item_code, "max_discount")) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 193 | |
Nabin Hait | c35271a | 2018-08-10 19:01:04 +0530 | [diff] [blame] | 194 | if discount and flt(d.discount_percentage) > discount: |
| 195 | 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] | 196 | |
Kanchan Chauhan | 5a980ac | 2017-02-10 14:27:11 +0530 | [diff] [blame] | 197 | def set_qty_as_per_stock_uom(self): |
rohitwaghchaure | dd4769e | 2023-09-20 18:09:03 +0530 | [diff] [blame] | 198 | allow_to_edit_stock_qty = frappe.db.get_single_value( |
| 199 | "Stock Settings", "allow_to_edit_stock_uom_qty_for_sales" |
| 200 | ) |
| 201 | |
Kanchan Chauhan | 5a980ac | 2017-02-10 14:27:11 +0530 | [diff] [blame] | 202 | for d in self.get("items"): |
| 203 | if d.meta.get_field("stock_qty"): |
| 204 | if not d.conversion_factor: |
| 205 | frappe.throw(_("Row {0}: Conversion Factor is mandatory").format(d.idx)) |
Rushabh Mehta | cc8b2b2 | 2017-03-31 12:44:29 +0530 | [diff] [blame] | 206 | d.stock_qty = flt(d.qty) * flt(d.conversion_factor) |
rohitwaghchaure | dd4769e | 2023-09-20 18:09:03 +0530 | [diff] [blame] | 207 | if allow_to_edit_stock_qty: |
| 208 | d.stock_qty = flt(d.stock_qty, d.precision("stock_qty")) |
Kanchan Chauhan | 5a980ac | 2017-02-10 14:27:11 +0530 | [diff] [blame] | 209 | |
shreyas | 1cfe2d7 | 2016-10-19 18:36:12 +0530 | [diff] [blame] | 210 | def validate_selling_price(self): |
Saqib | fff00ce | 2020-05-05 16:14:17 +0530 | [diff] [blame] | 211 | def throw_message(idx, item_name, rate, ref_rate_field): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 212 | throw( |
| 213 | _( |
| 214 | """Row #{0}: Selling rate for item {1} is lower than its {2}. |
Pruthvi Patel | 7c957d7 | 2021-08-24 21:24:25 +0530 | [diff] [blame] | 215 | Selling {3} should be atleast {4}.<br><br>Alternatively, |
| 216 | you can disable selling price validation in {5} to bypass |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 217 | this validation.""" |
| 218 | ).format( |
| 219 | idx, |
| 220 | bold(item_name), |
| 221 | bold(ref_rate_field), |
| 222 | bold("net rate"), |
| 223 | bold(rate), |
| 224 | get_link_to_form("Selling Settings", "Selling Settings"), |
| 225 | ), |
| 226 | title=_("Invalid Selling Price"), |
| 227 | ) |
mbauskar | e9747a8 | 2016-11-16 17:19:47 +0530 | [diff] [blame] | 228 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 229 | if self.get("is_return") or not frappe.db.get_single_value( |
| 230 | "Selling Settings", "validate_selling_price" |
Pruthvi Patel | 7c957d7 | 2021-08-24 21:24:25 +0530 | [diff] [blame] | 231 | ): |
rohitwaghchaure | 45ac31a | 2018-02-12 11:27:37 +0530 | [diff] [blame] | 232 | return |
| 233 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 234 | is_internal_customer = self.get("is_internal_customer") |
Pruthvi Patel | 7c957d7 | 2021-08-24 21:24:25 +0530 | [diff] [blame] | 235 | valuation_rate_map = {} |
| 236 | |
| 237 | for item in self.items: |
Krithi Ramani | 8c536ff | 2022-01-19 15:27:09 +0530 | [diff] [blame] | 238 | if not item.item_code or item.is_free_item: |
Saurabh | d7e1869 | 2017-09-18 15:43:35 +0530 | [diff] [blame] | 239 | continue |
prssanna | b57b3c9 | 2020-10-22 15:37:47 +0530 | [diff] [blame] | 240 | |
Pruthvi Patel | 7c957d7 | 2021-08-24 21:24:25 +0530 | [diff] [blame] | 241 | last_purchase_rate, is_stock_item = frappe.get_cached_value( |
| 242 | "Item", item.item_code, ("last_purchase_rate", "is_stock_item") |
| 243 | ) |
shreyas | e8aaa92 | 2016-10-20 14:03:59 +0530 | [diff] [blame] | 244 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 245 | last_purchase_rate_in_sales_uom = last_purchase_rate * (item.conversion_factor or 1) |
shreyas | e8aaa92 | 2016-10-20 14:03:59 +0530 | [diff] [blame] | 246 | |
Pruthvi Patel | 7c957d7 | 2021-08-24 21:24:25 +0530 | [diff] [blame] | 247 | if flt(item.base_net_rate) < flt(last_purchase_rate_in_sales_uom): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 248 | throw_message(item.idx, item.item_name, last_purchase_rate_in_sales_uom, "last purchase rate") |
Pruthvi Patel | 7c957d7 | 2021-08-24 21:24:25 +0530 | [diff] [blame] | 249 | |
| 250 | if is_internal_customer or not is_stock_item: |
| 251 | continue |
| 252 | |
| 253 | valuation_rate_map[(item.item_code, item.warehouse)] = None |
| 254 | |
| 255 | if not valuation_rate_map: |
| 256 | return |
| 257 | |
| 258 | or_conditions = ( |
| 259 | f"""(item_code = {frappe.db.escape(valuation_rate[0])} |
| 260 | and warehouse = {frappe.db.escape(valuation_rate[1])})""" |
| 261 | for valuation_rate in valuation_rate_map |
| 262 | ) |
| 263 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 264 | valuation_rates = frappe.db.sql( |
| 265 | f""" |
Pruthvi Patel | 7c957d7 | 2021-08-24 21:24:25 +0530 | [diff] [blame] | 266 | select |
| 267 | item_code, warehouse, valuation_rate |
| 268 | from |
| 269 | `tabBin` |
| 270 | where |
| 271 | ({" or ".join(or_conditions)}) |
| 272 | and valuation_rate > 0 |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 273 | """, |
| 274 | as_dict=True, |
| 275 | ) |
Pruthvi Patel | 7c957d7 | 2021-08-24 21:24:25 +0530 | [diff] [blame] | 276 | |
| 277 | for rate in valuation_rates: |
| 278 | valuation_rate_map[(rate.item_code, rate.warehouse)] = rate.valuation_rate |
| 279 | |
| 280 | for item in self.items: |
Krithi Ramani | 8c536ff | 2022-01-19 15:27:09 +0530 | [diff] [blame] | 281 | if not item.item_code or item.is_free_item: |
Pruthvi Patel | 7c957d7 | 2021-08-24 21:24:25 +0530 | [diff] [blame] | 282 | continue |
| 283 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 284 | last_valuation_rate = valuation_rate_map.get((item.item_code, item.warehouse)) |
Pruthvi Patel | 7c957d7 | 2021-08-24 21:24:25 +0530 | [diff] [blame] | 285 | |
| 286 | if not last_valuation_rate: |
| 287 | continue |
| 288 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 289 | last_valuation_rate_in_sales_uom = last_valuation_rate * (item.conversion_factor or 1) |
Pruthvi Patel | 7c957d7 | 2021-08-24 21:24:25 +0530 | [diff] [blame] | 290 | |
| 291 | if flt(item.base_net_rate) < flt(last_valuation_rate_in_sales_uom): |
mergify[bot] | d2aa4d5 | 2023-10-05 18:13:23 +0530 | [diff] [blame] | 292 | throw_message( |
Akhil Narang | 3effaf2 | 2024-03-27 11:37:26 +0530 | [diff] [blame] | 293 | item.idx, |
| 294 | item.item_name, |
| 295 | last_valuation_rate_in_sales_uom, |
| 296 | "valuation rate (Moving Average)", |
mergify[bot] | d2aa4d5 | 2023-10-05 18:13:23 +0530 | [diff] [blame] | 297 | ) |
shreyas | 1cfe2d7 | 2016-10-19 18:36:12 +0530 | [diff] [blame] | 298 | |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 299 | def get_item_list(self): |
| 300 | il = [] |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 301 | for d in self.get("items"): |
Neil Trini Lasrado | ed8cecb | 2015-07-07 13:59:23 +0530 | [diff] [blame] | 302 | if self.has_product_bundle(d.item_code): |
Nabin Hait | e7d1536 | 2014-12-25 16:01:55 +0530 | [diff] [blame] | 303 | for p in self.get("packed_items"): |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 304 | if p.parent_detail_docname == d.name and p.parent_item == d.item_code: |
| 305 | # the packing details table's qty is already multiplied with parent's qty |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 306 | il.append( |
| 307 | frappe._dict( |
| 308 | { |
| 309 | "warehouse": p.warehouse or d.warehouse, |
| 310 | "item_code": p.item_code, |
| 311 | "qty": flt(p.qty), |
rohitwaghchaure | 0743289 | 2023-12-17 12:42:07 +0530 | [diff] [blame] | 312 | "serial_no": p.serial_no if self.docstatus == 2 else None, |
| 313 | "batch_no": p.batch_no if self.docstatus == 2 else None, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 314 | "uom": p.uom, |
Rohit Waghchaure | d3ceb07 | 2023-03-31 09:03:54 +0530 | [diff] [blame] | 315 | "serial_and_batch_bundle": p.serial_and_batch_bundle |
| 316 | or get_serial_and_batch_bundle(p, self), |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 317 | "name": d.name, |
| 318 | "target_warehouse": p.target_warehouse, |
| 319 | "company": self.company, |
| 320 | "voucher_type": self.doctype, |
| 321 | "allow_zero_valuation": d.allow_zero_valuation_rate, |
| 322 | "sales_invoice_item": d.get("sales_invoice_item"), |
| 323 | "dn_detail": d.get("dn_detail"), |
| 324 | "incoming_rate": p.get("incoming_rate"), |
Rohit Waghchaure | 2372999 | 2022-09-02 18:43:55 +0530 | [diff] [blame] | 325 | "item_row": p, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 326 | } |
| 327 | ) |
| 328 | ) |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 329 | else: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 330 | il.append( |
| 331 | frappe._dict( |
| 332 | { |
| 333 | "warehouse": d.warehouse, |
| 334 | "item_code": d.item_code, |
| 335 | "qty": d.stock_qty, |
rohitwaghchaure | 0743289 | 2023-12-17 12:42:07 +0530 | [diff] [blame] | 336 | "serial_no": d.serial_no if self.docstatus == 2 else None, |
| 337 | "batch_no": d.batch_no if self.docstatus == 2 else None, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 338 | "uom": d.uom, |
| 339 | "stock_uom": d.stock_uom, |
| 340 | "conversion_factor": d.conversion_factor, |
Rohit Waghchaure | 6c9b212 | 2022-12-05 14:48:18 +0530 | [diff] [blame] | 341 | "serial_and_batch_bundle": d.serial_and_batch_bundle, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 342 | "name": d.name, |
| 343 | "target_warehouse": d.target_warehouse, |
| 344 | "company": self.company, |
| 345 | "voucher_type": self.doctype, |
| 346 | "allow_zero_valuation": d.allow_zero_valuation_rate, |
| 347 | "sales_invoice_item": d.get("sales_invoice_item"), |
| 348 | "dn_detail": d.get("dn_detail"), |
| 349 | "incoming_rate": d.get("incoming_rate"), |
Rohit Waghchaure | 2372999 | 2022-09-02 18:43:55 +0530 | [diff] [blame] | 350 | "item_row": d, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 351 | } |
| 352 | ) |
| 353 | ) |
Rohit Waghchaure | d3ceb07 | 2023-03-31 09:03:54 +0530 | [diff] [blame] | 354 | |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 355 | return il |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 356 | |
Neil Trini Lasrado | ed8cecb | 2015-07-07 13:59:23 +0530 | [diff] [blame] | 357 | def has_product_bundle(self, item_code): |
Gursheen Anand | 1657337 | 2023-11-22 16:45:14 +0530 | [diff] [blame] | 358 | product_bundle = frappe.qb.DocType("Product Bundle") |
| 359 | return ( |
| 360 | frappe.qb.from_(product_bundle) |
| 361 | .select(product_bundle.name) |
| 362 | .where((product_bundle.new_item_code == item_code) & (product_bundle.disabled == 0)) |
| 363 | ).run() |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 364 | |
Nabin Hait | 453cc37 | 2015-07-29 12:21:03 +0530 | [diff] [blame] | 365 | def get_already_delivered_qty(self, current_docname, so, so_detail): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 366 | delivered_via_dn = frappe.db.sql( |
| 367 | """select sum(qty) from `tabDelivery Note Item` |
ankitjavalkarwork | e69a611 | 2014-10-08 12:46:02 +0530 | [diff] [blame] | 368 | where so_detail = %s and docstatus = 1 |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 369 | and against_sales_order = %s |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 370 | and parent != %s""", |
| 371 | (so_detail, so, current_docname), |
| 372 | ) |
Anand Doshi | 01de945 | 2016-01-26 16:22:50 +0530 | [diff] [blame] | 373 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 374 | delivered_via_si = frappe.db.sql( |
| 375 | """select sum(si_item.qty) |
Nabin Hait | 1826791 | 2015-08-17 15:36:23 +0530 | [diff] [blame] | 376 | from `tabSales Invoice Item` si_item, `tabSales Invoice` si |
Anand Doshi | 602e825 | 2015-11-16 19:05:46 +0530 | [diff] [blame] | 377 | where si_item.parent = si.name and si.update_stock = 1 |
Anand Doshi | 01de945 | 2016-01-26 16:22:50 +0530 | [diff] [blame] | 378 | and si_item.so_detail = %s and si.docstatus = 1 |
Nabin Hait | 1826791 | 2015-08-17 15:36:23 +0530 | [diff] [blame] | 379 | and si_item.sales_order = %s |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 380 | and si.name != %s""", |
| 381 | (so_detail, so, current_docname), |
| 382 | ) |
Anand Doshi | 01de945 | 2016-01-26 16:22:50 +0530 | [diff] [blame] | 383 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 384 | total_delivered_qty = (flt(delivered_via_dn[0][0]) if delivered_via_dn else 0) + ( |
| 385 | flt(delivered_via_si[0][0]) if delivered_via_si else 0 |
| 386 | ) |
Anand Doshi | 01de945 | 2016-01-26 16:22:50 +0530 | [diff] [blame] | 387 | |
Nabin Hait | 453cc37 | 2015-07-29 12:21:03 +0530 | [diff] [blame] | 388 | return total_delivered_qty |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 389 | |
| 390 | def get_so_qty_and_warehouse(self, so_detail): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 391 | so_item = frappe.db.sql( |
| 392 | """select qty, warehouse from `tabSales Order Item` |
| 393 | where name = %s and docstatus = 1""", |
| 394 | so_detail, |
| 395 | as_dict=1, |
| 396 | ) |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 397 | 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] | 398 | so_warehouse = so_item and so_item[0]["warehouse"] or "" |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 399 | return so_qty, so_warehouse |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 400 | |
Mangesh-Khairnar | e17b709 | 2019-03-01 15:52:23 +0530 | [diff] [blame] | 401 | def check_sales_order_on_hold_or_close(self, ref_fieldname): |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 402 | for d in self.get("items"): |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 403 | if d.get(ref_fieldname): |
Rushabh Mehta | f2227d0 | 2014-03-31 23:37:40 +0530 | [diff] [blame] | 404 | status = frappe.db.get_value("Sales Order", d.get(ref_fieldname), "status") |
mergify[bot] | 35f72a1 | 2023-08-26 19:29:33 +0530 | [diff] [blame] | 405 | if status in ("Closed", "On Hold") and not self.is_return: |
Saurabh | 8a8ef85 | 2015-10-21 19:28:53 +0530 | [diff] [blame] | 406 | frappe.throw(_("Sales Order {0} is {1}").format(d.get(ref_fieldname), status)) |
shreyas | 1cfe2d7 | 2016-10-19 18:36:12 +0530 | [diff] [blame] | 407 | |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 408 | def update_reserved_qty(self): |
| 409 | so_map = {} |
| 410 | for d in self.get("items"): |
| 411 | if d.so_detail: |
| 412 | if self.doctype == "Delivery Note" and d.against_sales_order: |
| 413 | so_map.setdefault(d.against_sales_order, []).append(d.so_detail) |
| 414 | elif self.doctype == "Sales Invoice" and d.sales_order and self.update_stock: |
| 415 | so_map.setdefault(d.sales_order, []).append(d.so_detail) |
| 416 | |
| 417 | for so, so_item_rows in so_map.items(): |
| 418 | if so and so_item_rows: |
| 419 | sales_order = frappe.get_doc("Sales Order", so) |
| 420 | |
mergify[bot] | 35f72a1 | 2023-08-26 19:29:33 +0530 | [diff] [blame] | 421 | if (sales_order.status == "Closed" and not self.is_return) or sales_order.status in [ |
| 422 | "Cancelled" |
| 423 | ]: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 424 | frappe.throw( |
Akhil Narang | 3effaf2 | 2024-03-27 11:37:26 +0530 | [diff] [blame] | 425 | _("{0} {1} is cancelled or closed").format(_("Sales Order"), so), |
| 426 | frappe.InvalidStatusError, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 427 | ) |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 428 | |
| 429 | sales_order.update_reserved_qty(so_item_rows) |
| 430 | |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 431 | def set_incoming_rate(self): |
rohitwaghchaure | fe4540d | 2021-08-26 12:52:36 +0530 | [diff] [blame] | 432 | if self.doctype not in ("Delivery Note", "Sales Invoice"): |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 433 | return |
| 434 | |
| 435 | items = self.get("items") + (self.get("packed_items") or []) |
| 436 | for d in items: |
s-aga-r | e1b0fff | 2024-01-04 11:43:02 +0530 | [diff] [blame] | 437 | if not frappe.get_cached_value("Item", d.item_code, "is_stock_item"): |
| 438 | continue |
| 439 | |
mergify[bot] | 4057682 | 2023-12-19 18:29:56 +0530 | [diff] [blame] | 440 | if not self.get("return_against") or ( |
| 441 | get_valuation_method(d.item_code) == "Moving Average" and self.get("is_return") |
| 442 | ): |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 443 | # Get incoming rate based on original item cost based on valuation method |
Rohit Waghchaure | f9e230e | 2024-04-11 17:35:06 +0530 | [diff] [blame] | 444 | qty = flt(d.get("stock_qty") or d.get("actual_qty") or d.get("qty")) |
rohitwaghchaure | ce6c3b5 | 2021-04-13 20:55:52 +0530 | [diff] [blame] | 445 | |
rohitwaghchaure | 5922281 | 2024-03-15 17:55:41 +0530 | [diff] [blame] | 446 | if ( |
| 447 | not d.incoming_rate |
| 448 | or self.is_internal_transfer() |
| 449 | or (get_valuation_method(d.item_code) == "Moving Average" and self.get("is_return")) |
mergify[bot] | 0717536 | 2023-12-21 14:40:52 +0530 | [diff] [blame] | 450 | ): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 451 | d.incoming_rate = get_incoming_rate( |
| 452 | { |
| 453 | "item_code": d.item_code, |
| 454 | "warehouse": d.warehouse, |
| 455 | "posting_date": self.get("posting_date") or self.get("transaction_date"), |
| 456 | "posting_time": self.get("posting_time") or nowtime(), |
| 457 | "qty": qty if cint(self.get("is_return")) else (-1 * qty), |
Rohit Waghchaure | 674bd3e | 2023-03-17 16:42:59 +0530 | [diff] [blame] | 458 | "serial_and_batch_bundle": d.serial_and_batch_bundle, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 459 | "company": self.company, |
| 460 | "voucher_type": self.doctype, |
| 461 | "voucher_no": self.name, |
rohitwaghchaure | 5e9016f | 2023-12-01 21:42:22 +0530 | [diff] [blame] | 462 | "voucher_detail_no": d.name, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 463 | "allow_zero_valuation": d.get("allow_zero_valuation"), |
rohitwaghchaure | 5922281 | 2024-03-15 17:55:41 +0530 | [diff] [blame] | 464 | "batch_no": d.batch_no, |
| 465 | "serial_no": d.serial_no, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 466 | }, |
| 467 | raise_error_if_no_rate=False, |
| 468 | ) |
Deepesh Garg | b4be292 | 2021-01-28 13:09:56 +0530 | [diff] [blame] | 469 | |
| 470 | # For internal transfers use incoming rate as the valuation rate |
| 471 | if self.is_internal_transfer(): |
Deepesh Garg | ef9d126 | 2022-12-05 10:17:19 +0530 | [diff] [blame] | 472 | if self.doctype == "Delivery Note" or self.get("update_stock"): |
| 473 | if d.doctype == "Packed Item": |
| 474 | incoming_rate = flt( |
| 475 | flt(d.incoming_rate, d.precision("incoming_rate")) * d.conversion_factor, |
| 476 | d.precision("incoming_rate"), |
Ankush Menat | ee0a277 | 2022-05-12 18:17:09 +0530 | [diff] [blame] | 477 | ) |
Deepesh Garg | ef9d126 | 2022-12-05 10:17:19 +0530 | [diff] [blame] | 478 | if d.incoming_rate != incoming_rate: |
| 479 | d.incoming_rate = incoming_rate |
| 480 | else: |
| 481 | rate = flt( |
| 482 | flt(d.incoming_rate, d.precision("incoming_rate")) * d.conversion_factor, |
| 483 | d.precision("rate"), |
| 484 | ) |
| 485 | if d.rate != rate: |
| 486 | d.rate = rate |
| 487 | frappe.msgprint( |
| 488 | _( |
| 489 | "Row {0}: Item rate has been updated as per valuation rate since its an internal stock transfer" |
| 490 | ).format(d.idx), |
| 491 | alert=1, |
| 492 | ) |
Noah Jacob | ac0bc1f | 2021-06-24 19:36:17 +0530 | [diff] [blame] | 493 | |
Deepesh Garg | ef9d126 | 2022-12-05 10:17:19 +0530 | [diff] [blame] | 494 | d.discount_percentage = 0.0 |
| 495 | d.discount_amount = 0.0 |
| 496 | d.margin_rate_or_amount = 0.0 |
Deepesh Garg | b4be292 | 2021-01-28 13:09:56 +0530 | [diff] [blame] | 497 | |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 498 | elif self.get("return_against"): |
| 499 | # Get incoming rate of return entry from reference document |
| 500 | # based on original item cost as per valuation method |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 501 | d.incoming_rate = get_rate_for_return( |
| 502 | self.doctype, self.name, d.item_code, self.return_against, item_row=d |
| 503 | ) |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 504 | |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 505 | def update_stock_ledger(self): |
| 506 | self.update_reserved_qty() |
| 507 | |
| 508 | sl_entries = [] |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 509 | # Loop over items and packed items table |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 510 | for d in self.get_item_list(): |
Himanshu Mishra | 859d942 | 2018-08-08 18:33:50 +0530 | [diff] [blame] | 511 | if frappe.get_cached_value("Item", d.item_code, "is_stock_item") == 1 and flt(d.qty): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 512 | if flt(d.conversion_factor) == 0.0: |
| 513 | d.conversion_factor = ( |
| 514 | get_conversion_factor(d.item_code, d.uom).get("conversion_factor") or 1.0 |
| 515 | ) |
rohitwaghchaure | 019d4de | 2020-08-20 15:53:58 +0530 | [diff] [blame] | 516 | |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 517 | # On cancellation or return entry submission, make stock ledger entry for |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 518 | # target warehouse first, to update serial no values properly |
| 519 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 520 | if d.warehouse and ( |
| 521 | (not cint(self.is_return) and self.docstatus == 1) |
| 522 | or (cint(self.is_return) and self.docstatus == 2) |
| 523 | ): |
| 524 | sl_entries.append(self.get_sle_for_source_warehouse(d)) |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 525 | |
mergify[bot] | 233bf5d | 2021-10-05 16:16:21 +0530 | [diff] [blame] | 526 | if d.target_warehouse: |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 527 | sl_entries.append(self.get_sle_for_target_warehouse(d)) |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 528 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 529 | if d.warehouse and ( |
| 530 | (not cint(self.is_return) and self.docstatus == 2) |
| 531 | or (cint(self.is_return) and self.docstatus == 1) |
| 532 | ): |
| 533 | sl_entries.append(self.get_sle_for_source_warehouse(d)) |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 534 | |
Nabin Hait | 14aa9c5 | 2016-04-18 15:54:01 +0530 | [diff] [blame] | 535 | self.make_sl_entries(sl_entries) |
Anand Doshi | d294650 | 2014-04-08 20:10:03 +0530 | [diff] [blame] | 536 | |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 537 | def get_sle_for_source_warehouse(self, item_row): |
rohitwaghchaure | 5922281 | 2024-03-15 17:55:41 +0530 | [diff] [blame] | 538 | serial_and_batch_bundle = item_row.serial_and_batch_bundle |
| 539 | if serial_and_batch_bundle and self.is_internal_transfer() and self.is_return: |
| 540 | if self.docstatus == 1: |
| 541 | serial_and_batch_bundle = self.make_package_for_transfer( |
| 542 | serial_and_batch_bundle, item_row.warehouse, type_of_transaction="Inward" |
| 543 | ) |
| 544 | else: |
| 545 | serial_and_batch_bundle = frappe.db.get_value( |
| 546 | "Stock Ledger Entry", |
| 547 | {"voucher_detail_no": item_row.name, "warehouse": item_row.warehouse}, |
| 548 | "serial_and_batch_bundle", |
| 549 | ) |
| 550 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 551 | sle = self.get_sl_entries( |
| 552 | item_row, |
| 553 | { |
| 554 | "actual_qty": -1 * flt(item_row.qty), |
| 555 | "incoming_rate": item_row.incoming_rate, |
| 556 | "recalculate_rate": cint(self.is_return), |
rohitwaghchaure | 5922281 | 2024-03-15 17:55:41 +0530 | [diff] [blame] | 557 | "serial_and_batch_bundle": serial_and_batch_bundle, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 558 | }, |
| 559 | ) |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 560 | if item_row.target_warehouse and not cint(self.is_return): |
| 561 | sle.dependant_sle_voucher_detail_no = item_row.name |
| 562 | |
| 563 | return sle |
| 564 | |
| 565 | def get_sle_for_target_warehouse(self, item_row): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 566 | sle = self.get_sl_entries( |
| 567 | item_row, {"actual_qty": flt(item_row.qty), "warehouse": item_row.target_warehouse} |
| 568 | ) |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 569 | |
| 570 | if self.docstatus == 1: |
| 571 | if not cint(self.is_return): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 572 | sle.update({"incoming_rate": item_row.incoming_rate, "recalculate_rate": 1}) |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 573 | else: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 574 | sle.update({"outgoing_rate": item_row.incoming_rate}) |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 575 | if item_row.warehouse: |
| 576 | sle.dependant_sle_voucher_detail_no = item_row.name |
Deepesh Garg | b4be292 | 2021-01-28 13:09:56 +0530 | [diff] [blame] | 577 | |
rohitwaghchaure | 5922281 | 2024-03-15 17:55:41 +0530 | [diff] [blame] | 578 | if item_row.serial_and_batch_bundle and not cint(self.is_return): |
| 579 | type_of_transaction = "Inward" |
| 580 | if cint(self.is_return): |
| 581 | type_of_transaction = "Outward" |
| 582 | |
Rohit Waghchaure | 86da306 | 2023-03-20 14:15:34 +0530 | [diff] [blame] | 583 | sle["serial_and_batch_bundle"] = self.make_package_for_transfer( |
rohitwaghchaure | 5922281 | 2024-03-15 17:55:41 +0530 | [diff] [blame] | 584 | item_row.serial_and_batch_bundle, |
| 585 | item_row.target_warehouse, |
| 586 | type_of_transaction=type_of_transaction, |
Rohit Waghchaure | 86da306 | 2023-03-20 14:15:34 +0530 | [diff] [blame] | 587 | ) |
| 588 | |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 589 | return sle |
| 590 | |
Afshan | 01eca12 | 2020-12-08 09:27:56 +0530 | [diff] [blame] | 591 | def set_po_nos(self, for_validate=False): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 592 | if self.doctype == "Sales Invoice" and hasattr(self, "items"): |
Afshan | 01eca12 | 2020-12-08 09:27:56 +0530 | [diff] [blame] | 593 | if for_validate and self.po_no: |
| 594 | return |
Afshan | 3f971b2 | 2020-10-26 16:59:57 +0530 | [diff] [blame] | 595 | self.set_pos_for_sales_invoice() |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 596 | if self.doctype == "Delivery Note" and hasattr(self, "items"): |
Afshan | 01eca12 | 2020-12-08 09:27:56 +0530 | [diff] [blame] | 597 | if for_validate and self.po_no: |
| 598 | return |
Afshan | 3f971b2 | 2020-10-26 16:59:57 +0530 | [diff] [blame] | 599 | self.set_pos_for_delivery_note() |
| 600 | |
| 601 | def set_pos_for_sales_invoice(self): |
Afshan | 0a6e1b3 | 2020-10-26 21:23:31 +0530 | [diff] [blame] | 602 | po_nos = [] |
Afshan | 01eca12 | 2020-12-08 09:27:56 +0530 | [diff] [blame] | 603 | if self.po_no: |
| 604 | po_nos.append(self.po_no) |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 605 | self.get_po_nos("Sales Order", "sales_order", po_nos) |
| 606 | self.get_po_nos("Delivery Note", "delivery_note", po_nos) |
| 607 | self.po_no = ", ".join(list(set(x.strip() for x in ",".join(po_nos).split(",")))) |
Afshan | 3f971b2 | 2020-10-26 16:59:57 +0530 | [diff] [blame] | 608 | |
| 609 | def set_pos_for_delivery_note(self): |
Afshan | 0a6e1b3 | 2020-10-26 21:23:31 +0530 | [diff] [blame] | 610 | po_nos = [] |
Afshan | 01eca12 | 2020-12-08 09:27:56 +0530 | [diff] [blame] | 611 | if self.po_no: |
| 612 | po_nos.append(self.po_no) |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 613 | self.get_po_nos("Sales Order", "against_sales_order", po_nos) |
| 614 | self.get_po_nos("Sales Invoice", "against_sales_invoice", po_nos) |
| 615 | self.po_no = ", ".join(list(set(x.strip() for x in ",".join(po_nos).split(",")))) |
Afshan | 0a6e1b3 | 2020-10-26 21:23:31 +0530 | [diff] [blame] | 616 | |
| 617 | def get_po_nos(self, ref_doctype, ref_fieldname, po_nos): |
Ankush Menat | 9891780 | 2021-06-11 18:40:22 +0530 | [diff] [blame] | 618 | doc_list = list(set(d.get(ref_fieldname) for d in self.items if d.get(ref_fieldname))) |
Afshan | 0a6e1b3 | 2020-10-26 21:23:31 +0530 | [diff] [blame] | 619 | if doc_list: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 620 | po_nos += [ |
| 621 | d.po_no |
| 622 | for d in frappe.get_all(ref_doctype, "po_no", filters={"name": ("in", doc_list)}) |
| 623 | if d.get("po_no") |
| 624 | ] |
Nabin Hait | 96f4b22 | 2018-03-13 15:46:59 +0530 | [diff] [blame] | 625 | |
Saif Ur Rehman | 5fdbc68 | 2018-11-26 19:32:16 +0500 | [diff] [blame] | 626 | def set_gross_profit(self): |
Abhishek Balam | 18e7b60 | 2020-05-26 18:37:02 +0530 | [diff] [blame] | 627 | if self.doctype in ["Sales Order", "Quotation"]: |
Saif Ur Rehman | 5fdbc68 | 2018-11-26 19:32:16 +0500 | [diff] [blame] | 628 | for item in self.items: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 629 | item.gross_profit = flt( |
Akhil Narang | 3effaf2 | 2024-03-27 11:37:26 +0530 | [diff] [blame] | 630 | ((item.base_rate - flt(item.valuation_rate)) * item.stock_qty), |
| 631 | self.precision("amount", item), |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 632 | ) |
Saif Ur Rehman | 5fdbc68 | 2018-11-26 19:32:16 +0500 | [diff] [blame] | 633 | |
Rohit Waghchaure | f4dc716 | 2018-11-15 17:04:02 +0530 | [diff] [blame] | 634 | def set_customer_address(self): |
| 635 | address_dict = { |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 636 | "customer_address": "address_display", |
| 637 | "shipping_address_name": "shipping_address", |
| 638 | "company_address": "company_address_display", |
Deepesh Garg | 104fdcb | 2022-11-24 16:41:40 +0530 | [diff] [blame] | 639 | "dispatch_address_name": "dispatch_address", |
Rohit Waghchaure | f4dc716 | 2018-11-15 17:04:02 +0530 | [diff] [blame] | 640 | } |
| 641 | |
| 642 | for address_field, address_display_field in address_dict.items(): |
| 643 | if self.get(address_field): |
Ankush Menat | f4d7499 | 2023-10-17 22:11:56 +0530 | [diff] [blame] | 644 | self.set( |
| 645 | address_display_field, render_address(self.get(address_field), check_permissions=False) |
| 646 | ) |
Rohit Waghchaure | f4dc716 | 2018-11-15 17:04:02 +0530 | [diff] [blame] | 647 | |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 648 | def validate_for_duplicate_items(self): |
| 649 | check_list, chk_dupl_itm = [], [] |
| 650 | if cint(frappe.db.get_single_value("Selling Settings", "allow_multiple_items")): |
| 651 | return |
Saqib | 0681a48 | 2021-02-11 14:06:15 +0530 | [diff] [blame] | 652 | if self.doctype == "Sales Invoice" and self.is_consolidated: |
| 653 | return |
| 654 | if self.doctype == "POS Invoice": |
| 655 | return |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 656 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 657 | for d in self.get("items"): |
Saqib | 0681a48 | 2021-02-11 14:06:15 +0530 | [diff] [blame] | 658 | if self.doctype == "Sales Invoice": |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 659 | stock_items = [ |
| 660 | d.item_code, |
| 661 | d.description, |
| 662 | d.warehouse, |
| 663 | d.sales_order or d.delivery_note, |
| 664 | d.batch_no or "", |
| 665 | ] |
Jannat Patel | c1187bc | 2020-11-19 13:10:24 +0530 | [diff] [blame] | 666 | non_stock_items = [d.item_code, d.description, d.sales_order or d.delivery_note] |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 667 | elif self.doctype == "Delivery Note": |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 668 | stock_items = [ |
| 669 | d.item_code, |
| 670 | d.description, |
| 671 | d.warehouse, |
| 672 | d.against_sales_order or d.against_sales_invoice, |
| 673 | d.batch_no or "", |
| 674 | ] |
| 675 | non_stock_items = [ |
| 676 | d.item_code, |
| 677 | d.description, |
| 678 | d.against_sales_order or d.against_sales_invoice, |
| 679 | ] |
Rohit Waghchaure | 13c7e18 | 2019-08-29 19:45:19 +0530 | [diff] [blame] | 680 | elif self.doctype in ["Sales Order", "Quotation"]: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 681 | stock_items = [d.item_code, d.description, d.warehouse, ""] |
Jannat Patel | c1187bc | 2020-11-19 13:10:24 +0530 | [diff] [blame] | 682 | non_stock_items = [d.item_code, d.description] |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 683 | |
mergify[bot] | 89f516c | 2022-07-21 13:57:03 +0530 | [diff] [blame] | 684 | duplicate_items_msg = _("Item {0} entered multiple times.").format(frappe.bold(d.item_code)) |
| 685 | duplicate_items_msg += "<br><br>" |
| 686 | duplicate_items_msg += _("Please enable {} in {} to allow same item in multiple rows").format( |
| 687 | frappe.bold("Allow Item to Be Added Multiple Times in a Transaction"), |
| 688 | get_link_to_form("Selling Settings", "Selling Settings"), |
| 689 | ) |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 690 | if frappe.db.get_value("Item", d.item_code, "is_stock_item") == 1: |
Jannat Patel | c1187bc | 2020-11-19 13:10:24 +0530 | [diff] [blame] | 691 | if stock_items in check_list: |
Saqib | ac9e6ff | 2021-01-28 17:58:55 +0530 | [diff] [blame] | 692 | frappe.throw(duplicate_items_msg) |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 693 | else: |
Jannat Patel | c1187bc | 2020-11-19 13:10:24 +0530 | [diff] [blame] | 694 | check_list.append(stock_items) |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 695 | else: |
Jannat Patel | c1187bc | 2020-11-19 13:10:24 +0530 | [diff] [blame] | 696 | if non_stock_items in chk_dupl_itm: |
Saqib | ac9e6ff | 2021-01-28 17:58:55 +0530 | [diff] [blame] | 697 | frappe.throw(duplicate_items_msg) |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 698 | else: |
Jannat Patel | c1187bc | 2020-11-19 13:10:24 +0530 | [diff] [blame] | 699 | chk_dupl_itm.append(non_stock_items) |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 700 | |
rohitwaghchaure | 6fa6caf | 2020-04-23 09:46:29 +0530 | [diff] [blame] | 701 | def validate_target_warehouse(self): |
| 702 | items = self.get("items") + (self.get("packed_items") or []) |
| 703 | |
| 704 | for d in items: |
| 705 | if d.get("target_warehouse") and d.get("warehouse") == d.get("target_warehouse"): |
| 706 | warehouse = frappe.bold(d.get("target_warehouse")) |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 707 | frappe.throw( |
Akhil Narang | 3effaf2 | 2024-03-27 11:37:26 +0530 | [diff] [blame] | 708 | _( |
| 709 | "Row {0}: Delivery Warehouse ({1}) and Customer Warehouse ({2}) can not be same" |
| 710 | ).format(d.idx, warehouse, warehouse) |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 711 | ) |
Deepesh Garg | f3d78dd | 2019-08-23 11:42:35 +0530 | [diff] [blame] | 712 | |
mergify[bot] | 233bf5d | 2021-10-05 16:16:21 +0530 | [diff] [blame] | 713 | if not self.get("is_internal_customer") and any(d.get("target_warehouse") for d in items): |
| 714 | msg = _("Target Warehouse is set for some items but the customer is not an internal customer.") |
| 715 | msg += " " + _("This {} will be treated as material transfer.").format(_(self.doctype)) |
| 716 | frappe.msgprint(msg, title="Internal Transfer", alert=True) |
| 717 | |
Zarrar | fc03a04 | 2018-06-04 12:52:52 +0530 | [diff] [blame] | 718 | def validate_items(self): |
| 719 | # validate items to see if they have is_sales_item enabled |
| 720 | from erpnext.controllers.buying_controller import validate_item_type |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 721 | |
Zarrar | fc03a04 | 2018-06-04 12:52:52 +0530 | [diff] [blame] | 722 | validate_item_type(self, "is_sales_item", "sales") |
| 723 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 724 | |
Rushabh Mehta | 708e47a | 2018-08-08 16:37:31 +0530 | [diff] [blame] | 725 | def set_default_income_account_for_item(obj): |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 726 | for d in obj.get("items"): |
Nabin Hait | d1fd1e2 | 2013-10-18 12:29:11 +0530 | [diff] [blame] | 727 | if d.item_code: |
Manas Solanki | 5b93562 | 2018-05-16 17:44:51 +0530 | [diff] [blame] | 728 | if getattr(d, "income_account", None): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 729 | set_item_default(d.item_code, obj.company, "income_account", d.income_account) |
Rohit Waghchaure | d3ceb07 | 2023-03-31 09:03:54 +0530 | [diff] [blame] | 730 | |
| 731 | |
| 732 | def get_serial_and_batch_bundle(child, parent): |
| 733 | from erpnext.stock.serial_batch_bundle import SerialBatchCreation |
| 734 | |
rohitwaghchaure | bc9c480 | 2024-02-26 23:57:52 +0530 | [diff] [blame] | 735 | if child.get("use_serial_batch_fields"): |
| 736 | return |
| 737 | |
Akhil Narang | 3effaf2 | 2024-03-27 11:37:26 +0530 | [diff] [blame] | 738 | if not frappe.db.get_single_value("Stock Settings", "auto_create_serial_and_batch_bundle_for_outward"): |
Rohit Waghchaure | d3ceb07 | 2023-03-31 09:03:54 +0530 | [diff] [blame] | 739 | return |
| 740 | |
Akhil Narang | 3effaf2 | 2024-03-27 11:37:26 +0530 | [diff] [blame] | 741 | item_details = frappe.db.get_value("Item", child.item_code, ["has_serial_no", "has_batch_no"], as_dict=1) |
Rohit Waghchaure | d3ceb07 | 2023-03-31 09:03:54 +0530 | [diff] [blame] | 742 | |
| 743 | if not item_details.has_serial_no and not item_details.has_batch_no: |
| 744 | return |
| 745 | |
| 746 | sn_doc = SerialBatchCreation( |
| 747 | { |
| 748 | "item_code": child.item_code, |
| 749 | "warehouse": child.warehouse, |
| 750 | "voucher_type": parent.doctype, |
Rohit Waghchaure | d73b1f3 | 2024-03-30 10:46:56 +0530 | [diff] [blame] | 751 | "voucher_no": parent.name if parent.docstatus < 2 else None, |
Rohit Waghchaure | d3ceb07 | 2023-03-31 09:03:54 +0530 | [diff] [blame] | 752 | "voucher_detail_no": child.name, |
| 753 | "posting_date": parent.posting_date, |
| 754 | "posting_time": parent.posting_time, |
| 755 | "qty": child.qty, |
Rohit Waghchaure | d73b1f3 | 2024-03-30 10:46:56 +0530 | [diff] [blame] | 756 | "type_of_transaction": "Outward" if child.qty > 0 and parent.docstatus < 2 else "Inward", |
Rohit Waghchaure | d3ceb07 | 2023-03-31 09:03:54 +0530 | [diff] [blame] | 757 | "company": parent.company, |
| 758 | "do_not_submit": "True", |
| 759 | } |
| 760 | ) |
| 761 | |
| 762 | doc = sn_doc.make_serial_and_batch_bundle() |
| 763 | child.db_set("serial_and_batch_bundle", doc.name) |
| 764 | |
| 765 | return doc.name |