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 | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 3 | |
Rohan Bansal | 7f8b95e | 2021-04-14 14:12:03 +0530 | [diff] [blame] | 4 | import json |
marination | fac4035 | 2020-12-07 21:35:49 +0530 | [diff] [blame] | 5 | from collections import defaultdict |
Ankush Menat | e6ab8df | 2022-02-06 13:02:34 +0530 | [diff] [blame] | 6 | from typing import List, Tuple |
Rohan Bansal | 7f8b95e | 2021-04-14 14:12:03 +0530 | [diff] [blame] | 7 | |
| 8 | import frappe |
rohitwaghchaure | 8fdc244 | 2024-01-27 21:37:58 +0530 | [diff] [blame] | 9 | from frappe import _, bold |
rohitwaghchaure | 4b24fcd | 2024-02-20 23:45:07 +0530 | [diff] [blame] | 10 | from frappe.utils import cint, cstr, flt, get_link_to_form, getdate |
Rohan Bansal | 7f8b95e | 2021-04-14 14:12:03 +0530 | [diff] [blame] | 11 | |
| 12 | import erpnext |
Chillar Anand | 915b343 | 2021-09-02 16:44:59 +0530 | [diff] [blame] | 13 | from erpnext.accounts.general_ledger import ( |
| 14 | make_gl_entries, |
| 15 | make_reverse_gl_entries, |
| 16 | process_gl_map, |
| 17 | ) |
ruthra kumar | 46ea814 | 2023-07-28 08:29:19 +0530 | [diff] [blame] | 18 | from erpnext.accounts.utils import cancel_exchange_gain_loss_journal, get_fiscal_year |
Rushabh Mehta | 1f84799 | 2013-12-12 19:12:19 +0530 | [diff] [blame] | 19 | from erpnext.controllers.accounts_controller import AccountsController |
Nabin Hait | 6d7b0ce | 2017-06-15 11:09:27 +0530 | [diff] [blame] | 20 | from erpnext.stock import get_warehouse_account_map |
Rohit Waghchaure | dbec5cf | 2022-06-22 12:24:08 +0530 | [diff] [blame] | 21 | from erpnext.stock.doctype.inventory_dimension.inventory_dimension import ( |
| 22 | get_evaluated_inventory_dimension, |
| 23 | ) |
Rohit Waghchaure | 0165012 | 2024-02-06 13:31:36 +0530 | [diff] [blame] | 24 | from erpnext.stock.doctype.serial_and_batch_bundle.serial_and_batch_bundle import ( |
| 25 | get_type_of_transaction, |
| 26 | ) |
Ankush Menat | 8f57724 | 2022-01-07 11:10:23 +0530 | [diff] [blame] | 27 | from erpnext.stock.stock_ledger import get_items_to_be_repost |
Rohan Bansal | 7f8b95e | 2021-04-14 14:12:03 +0530 | [diff] [blame] | 28 | |
Nabin Hait | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 29 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 30 | class QualityInspectionRequiredError(frappe.ValidationError): |
| 31 | pass |
| 32 | |
| 33 | |
| 34 | class QualityInspectionRejectedError(frappe.ValidationError): |
| 35 | pass |
| 36 | |
| 37 | |
| 38 | class QualityInspectionNotSubmittedError(frappe.ValidationError): |
| 39 | pass |
| 40 | |
Nabin Hait | 5a9579b | 2018-12-24 14:54:42 +0530 | [diff] [blame] | 41 | |
Rohit Waghchaure | 795c943 | 2022-08-17 13:48:56 +0530 | [diff] [blame] | 42 | class BatchExpiredError(frappe.ValidationError): |
| 43 | pass |
| 44 | |
| 45 | |
Nabin Hait | c3afb25 | 2013-03-19 12:01:24 +0530 | [diff] [blame] | 46 | class StockController(AccountsController): |
Nabin Hait | 8af429d | 2016-11-16 17:21:59 +0530 | [diff] [blame] | 47 | def validate(self): |
| 48 | super(StockController, self).validate() |
s-aga-r | 094ecc1 | 2024-02-12 17:36:14 +0530 | [diff] [blame] | 49 | |
| 50 | if self.docstatus == 0: |
| 51 | self.validate_duplicate_serial_and_batch_bundle() |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 52 | if not self.get("is_return"): |
marination | 596560c | 2020-06-11 16:39:03 +0530 | [diff] [blame] | 53 | self.validate_inspection() |
rohitwaghchaure | 9af557f | 2019-12-30 13:26:47 +0530 | [diff] [blame] | 54 | self.validate_serialized_batch() |
marination | eecfc4c | 2021-07-22 13:23:54 +0530 | [diff] [blame] | 55 | self.clean_serial_nos() |
marination | fd04e96 | 2020-04-03 15:46:48 +0530 | [diff] [blame] | 56 | self.validate_customer_provided_item() |
Anupam Kumar | 7e1dcf9 | 2021-02-11 20:19:30 +0530 | [diff] [blame] | 57 | self.set_rate_of_stock_uom() |
Deepesh Garg | b4be292 | 2021-01-28 13:09:56 +0530 | [diff] [blame] | 58 | self.validate_internal_transfer() |
marination | fac4035 | 2020-12-07 21:35:49 +0530 | [diff] [blame] | 59 | self.validate_putaway_capacity() |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 60 | |
s-aga-r | 094ecc1 | 2024-02-12 17:36:14 +0530 | [diff] [blame] | 61 | def validate_duplicate_serial_and_batch_bundle(self): |
| 62 | if sbb_list := [ |
| 63 | item.get("serial_and_batch_bundle") |
| 64 | for item in self.items |
| 65 | if item.get("serial_and_batch_bundle") |
| 66 | ]: |
| 67 | SLE = frappe.qb.DocType("Stock Ledger Entry") |
| 68 | data = ( |
| 69 | frappe.qb.from_(SLE) |
| 70 | .select(SLE.voucher_type, SLE.voucher_no, SLE.serial_and_batch_bundle) |
| 71 | .where( |
| 72 | (SLE.docstatus == 1) |
| 73 | & (SLE.serial_and_batch_bundle.notnull()) |
| 74 | & (SLE.serial_and_batch_bundle.isin(sbb_list)) |
| 75 | ) |
| 76 | .limit(1) |
| 77 | ).run(as_dict=True) |
| 78 | |
| 79 | if data: |
| 80 | data = data[0] |
| 81 | frappe.throw( |
| 82 | _("Serial and Batch Bundle {0} is already used in {1} {2}.").format( |
| 83 | frappe.bold(data.serial_and_batch_bundle), data.voucher_type, data.voucher_no |
| 84 | ) |
| 85 | ) |
| 86 | |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 87 | def make_gl_entries(self, gl_entries=None, from_repost=False): |
Anand Doshi | f78d1ae | 2014-03-28 13:55:00 +0530 | [diff] [blame] | 88 | if self.docstatus == 2: |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 89 | make_reverse_gl_entries(voucher_type=self.doctype, voucher_no=self.name) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 90 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 91 | provisional_accounting_for_non_stock_items = cint( |
Daizy Modi | 4efc947 | 2022-11-07 09:21:03 +0530 | [diff] [blame] | 92 | frappe.get_cached_value( |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 93 | "Company", self.company, "enable_provisional_accounting_for_non_stock_items" |
| 94 | ) |
| 95 | ) |
Deepesh Garg | 528c713 | 2022-02-01 14:42:55 +0530 | [diff] [blame] | 96 | |
Deepesh Garg | d1ec0a6 | 2023-10-23 00:16:40 +0530 | [diff] [blame] | 97 | is_asset_pr = any(d.get("is_fixed_asset") for d in self.get("items")) |
| 98 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 99 | if ( |
| 100 | cint(erpnext.is_perpetual_inventory_enabled(self.company)) |
| 101 | or provisional_accounting_for_non_stock_items |
Deepesh Garg | d1ec0a6 | 2023-10-23 00:16:40 +0530 | [diff] [blame] | 102 | or is_asset_pr |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 103 | ): |
Rohit Waghchaure | 6b33c9b | 2019-03-08 11:13:35 +0530 | [diff] [blame] | 104 | warehouse_account = get_warehouse_account_map(self.company) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 105 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 106 | if self.docstatus == 1: |
Nabin Hait | 9784d27 | 2016-12-30 16:21:35 +0530 | [diff] [blame] | 107 | if not gl_entries: |
| 108 | gl_entries = self.get_gl_entries(warehouse_account) |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 109 | make_gl_entries(gl_entries, from_repost=from_repost) |
Nabin Hait | 145e5e2 | 2013-10-22 23:51:41 +0530 | [diff] [blame] | 110 | |
rohitwaghchaure | 9af557f | 2019-12-30 13:26:47 +0530 | [diff] [blame] | 111 | def validate_serialized_batch(self): |
| 112 | from erpnext.stock.doctype.serial_no.serial_no import get_serial_nos |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 113 | |
Rohit Waghchaure | 795c943 | 2022-08-17 13:48:56 +0530 | [diff] [blame] | 114 | is_material_issue = False |
| 115 | if self.doctype == "Stock Entry" and self.purpose == "Material Issue": |
| 116 | is_material_issue = True |
| 117 | |
rohitwaghchaure | 9af557f | 2019-12-30 13:26:47 +0530 | [diff] [blame] | 118 | for d in self.get("items"): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 119 | if hasattr(d, "serial_no") and hasattr(d, "batch_no") and d.serial_no and d.batch_no: |
| 120 | serial_nos = frappe.get_all( |
| 121 | "Serial No", |
Rohit Waghchaure | 6b482eb | 2021-07-23 16:40:45 +0530 | [diff] [blame] | 122 | fields=["batch_no", "name", "warehouse"], |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 123 | filters={"name": ("in", get_serial_nos(d.serial_no))}, |
Rohit Waghchaure | 6b482eb | 2021-07-23 16:40:45 +0530 | [diff] [blame] | 124 | ) |
| 125 | |
| 126 | for row in serial_nos: |
| 127 | if row.warehouse and row.batch_no != d.batch_no: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 128 | frappe.throw( |
| 129 | _("Row #{0}: Serial No {1} does not belong to Batch {2}").format( |
| 130 | d.idx, row.name, d.batch_no |
| 131 | ) |
| 132 | ) |
rohitwaghchaure | 9af557f | 2019-12-30 13:26:47 +0530 | [diff] [blame] | 133 | |
Rohit Waghchaure | 795c943 | 2022-08-17 13:48:56 +0530 | [diff] [blame] | 134 | if is_material_issue: |
| 135 | continue |
| 136 | |
Saqib Ansari | 903055b | 2020-10-20 11:59:06 +0530 | [diff] [blame] | 137 | if flt(d.qty) > 0.0 and d.get("batch_no") and self.get("posting_date") and self.docstatus < 2: |
rohitwaghchaure | 28a4880 | 2020-04-28 13:01:43 +0530 | [diff] [blame] | 138 | expiry_date = frappe.get_cached_value("Batch", d.get("batch_no"), "expiry_date") |
| 139 | |
| 140 | if expiry_date and getdate(expiry_date) < getdate(self.posting_date): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 141 | frappe.throw( |
| 142 | _("Row #{0}: The batch {1} has already expired.").format( |
| 143 | d.idx, get_link_to_form("Batch", d.get("batch_no")) |
Rohit Waghchaure | 795c943 | 2022-08-17 13:48:56 +0530 | [diff] [blame] | 144 | ), |
| 145 | BatchExpiredError, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 146 | ) |
rohitwaghchaure | 28a4880 | 2020-04-28 13:01:43 +0530 | [diff] [blame] | 147 | |
marination | eecfc4c | 2021-07-22 13:23:54 +0530 | [diff] [blame] | 148 | def clean_serial_nos(self): |
Ankush Menat | b20df37 | 2022-01-24 19:19:58 +0530 | [diff] [blame] | 149 | from erpnext.stock.doctype.serial_no.serial_no import clean_serial_no_string |
| 150 | |
marination | eecfc4c | 2021-07-22 13:23:54 +0530 | [diff] [blame] | 151 | for row in self.get("items"): |
| 152 | if hasattr(row, "serial_no") and row.serial_no: |
Ankush Menat | b20df37 | 2022-01-24 19:19:58 +0530 | [diff] [blame] | 153 | # remove extra whitespace and store one serial no on each line |
| 154 | row.serial_no = clean_serial_no_string(row.serial_no) |
marination | eecfc4c | 2021-07-22 13:23:54 +0530 | [diff] [blame] | 155 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 156 | for row in self.get("packed_items") or []: |
Ankush Menat | e177c52 | 2022-01-24 19:28:26 +0530 | [diff] [blame] | 157 | if hasattr(row, "serial_no") and row.serial_no: |
| 158 | # remove extra whitespace and store one serial no on each line |
| 159 | row.serial_no = clean_serial_no_string(row.serial_no) |
| 160 | |
rohitwaghchaure | bc9c480 | 2024-02-26 23:57:52 +0530 | [diff] [blame] | 161 | def make_bundle_using_old_serial_batch_fields(self, table_name=None): |
rohitwaghchaure | a4cbfab | 2024-02-19 10:25:36 +0530 | [diff] [blame] | 162 | if self.get("_action") == "update_after_submit": |
| 163 | return |
| 164 | |
Rohit Waghchaure | c1e869f | 2024-02-05 12:40:26 +0530 | [diff] [blame] | 165 | # To handle test cases |
| 166 | if frappe.flags.in_test and frappe.flags.use_serial_and_batch_fields: |
| 167 | return |
| 168 | |
rohitwaghchaure | bc9c480 | 2024-02-26 23:57:52 +0530 | [diff] [blame] | 169 | if not table_name: |
| 170 | table_name = "items" |
| 171 | |
Rohit Waghchaure | c1e869f | 2024-02-05 12:40:26 +0530 | [diff] [blame] | 172 | if self.doctype == "Asset Capitalization": |
| 173 | table_name = "stock_items" |
| 174 | |
| 175 | for row in self.get(table_name): |
rohitwaghchaure | 4b24fcd | 2024-02-20 23:45:07 +0530 | [diff] [blame] | 176 | if row.serial_and_batch_bundle and (row.serial_no or row.batch_no): |
| 177 | self.validate_serial_nos_and_batches_with_bundle(row) |
| 178 | |
Rohit Waghchaure | 9fafc83 | 2024-02-04 10:42:31 +0530 | [diff] [blame] | 179 | if not row.serial_no and not row.batch_no and not row.get("rejected_serial_no"): |
| 180 | continue |
| 181 | |
| 182 | if not row.use_serial_batch_fields and ( |
| 183 | row.serial_no or row.batch_no or row.get("rejected_serial_no") |
| 184 | ): |
rohitwaghchaure | 4b24fcd | 2024-02-20 23:45:07 +0530 | [diff] [blame] | 185 | row.use_serial_batch_fields = 1 |
Rohit Waghchaure | 9fafc83 | 2024-02-04 10:42:31 +0530 | [diff] [blame] | 186 | |
| 187 | if row.use_serial_batch_fields and ( |
Rohit Waghchaure | c1e869f | 2024-02-05 12:40:26 +0530 | [diff] [blame] | 188 | not row.serial_and_batch_bundle and not row.get("rejected_serial_and_batch_bundle") |
Rohit Waghchaure | 9fafc83 | 2024-02-04 10:42:31 +0530 | [diff] [blame] | 189 | ): |
rohitwaghchaure | cef6291 | 2024-03-06 19:43:36 +0530 | [diff] [blame] | 190 | bundle_details = { |
| 191 | "item_code": row.item_code, |
| 192 | "posting_date": self.posting_date, |
| 193 | "posting_time": self.posting_time, |
| 194 | "voucher_type": self.doctype, |
| 195 | "voucher_no": self.name, |
| 196 | "voucher_detail_no": row.name, |
| 197 | "company": self.company, |
| 198 | "is_rejected": 1 if row.get("rejected_warehouse") else 0, |
rohitwaghchaure | cef6291 | 2024-03-06 19:43:36 +0530 | [diff] [blame] | 199 | "use_serial_batch_fields": row.use_serial_batch_fields, |
| 200 | "do_not_submit": True, |
| 201 | } |
Rohit Waghchaure | 0165012 | 2024-02-06 13:31:36 +0530 | [diff] [blame] | 202 | |
rohitwaghchaure | 01856a6 | 2024-03-07 14:14:19 +0530 | [diff] [blame] | 203 | if row.qty: |
| 204 | self.update_bundle_details(bundle_details, table_name, row) |
| 205 | self.create_serial_batch_bundle(bundle_details, row) |
Rohit Waghchaure | 9fafc83 | 2024-02-04 10:42:31 +0530 | [diff] [blame] | 206 | |
rohitwaghchaure | 01856a6 | 2024-03-07 14:14:19 +0530 | [diff] [blame] | 207 | if row.get("rejected_qty"): |
| 208 | self.update_bundle_details(bundle_details, table_name, row, is_rejected=True) |
| 209 | self.create_serial_batch_bundle(bundle_details, row) |
Rohit Waghchaure | 9fafc83 | 2024-02-04 10:42:31 +0530 | [diff] [blame] | 210 | |
rohitwaghchaure | 01856a6 | 2024-03-07 14:14:19 +0530 | [diff] [blame] | 211 | def update_bundle_details(self, bundle_details, table_name, row, is_rejected=False): |
| 212 | from erpnext.stock.doctype.serial_no.serial_no import get_serial_nos |
| 213 | |
rohitwaghchaure | cef6291 | 2024-03-06 19:43:36 +0530 | [diff] [blame] | 214 | # Since qty field is different for different doctypes |
| 215 | qty = row.get("qty") |
| 216 | warehouse = row.get("warehouse") |
| 217 | |
| 218 | if table_name == "packed_items": |
| 219 | type_of_transaction = "Inward" |
| 220 | if not self.is_return: |
| 221 | type_of_transaction = "Outward" |
| 222 | else: |
| 223 | type_of_transaction = get_type_of_transaction(self, row) |
| 224 | |
| 225 | if hasattr(row, "stock_qty"): |
| 226 | qty = row.stock_qty |
| 227 | |
| 228 | if self.doctype == "Stock Entry": |
| 229 | qty = row.transfer_qty |
| 230 | warehouse = row.s_warehouse or row.t_warehouse |
| 231 | |
rohitwaghchaure | 01856a6 | 2024-03-07 14:14:19 +0530 | [diff] [blame] | 232 | serial_nos = row.serial_no |
| 233 | if is_rejected: |
| 234 | serial_nos = row.get("rejected_serial_no") |
| 235 | type_of_transaction = "Inward" if not self.is_return else "Outward" |
| 236 | qty = row.get("rejected_qty") |
| 237 | warehouse = row.get("rejected_warehouse") |
| 238 | |
rohitwaghchaure | cef6291 | 2024-03-06 19:43:36 +0530 | [diff] [blame] | 239 | bundle_details.update( |
| 240 | { |
| 241 | "qty": qty, |
rohitwaghchaure | 01856a6 | 2024-03-07 14:14:19 +0530 | [diff] [blame] | 242 | "is_rejected": is_rejected, |
rohitwaghchaure | cef6291 | 2024-03-06 19:43:36 +0530 | [diff] [blame] | 243 | "type_of_transaction": type_of_transaction, |
| 244 | "warehouse": warehouse, |
| 245 | "batches": frappe._dict({row.batch_no: qty}) if row.batch_no else None, |
rohitwaghchaure | 01856a6 | 2024-03-07 14:14:19 +0530 | [diff] [blame] | 246 | "serial_nos": get_serial_nos(serial_nos) if serial_nos else None, |
| 247 | "batch_no": row.batch_no, |
rohitwaghchaure | cef6291 | 2024-03-06 19:43:36 +0530 | [diff] [blame] | 248 | } |
| 249 | ) |
| 250 | |
rohitwaghchaure | 01856a6 | 2024-03-07 14:14:19 +0530 | [diff] [blame] | 251 | def create_serial_batch_bundle(self, bundle_details, row): |
| 252 | from erpnext.stock.serial_batch_bundle import SerialBatchCreation |
| 253 | |
| 254 | sn_doc = SerialBatchCreation(bundle_details).make_serial_and_batch_bundle() |
| 255 | |
| 256 | field = "serial_and_batch_bundle" |
| 257 | if bundle_details.get("is_rejected"): |
| 258 | field = "rejected_serial_and_batch_bundle" |
| 259 | |
| 260 | row.set(field, sn_doc.name) |
| 261 | row.db_set({field: sn_doc.name}) |
| 262 | |
rohitwaghchaure | 4b24fcd | 2024-02-20 23:45:07 +0530 | [diff] [blame] | 263 | def validate_serial_nos_and_batches_with_bundle(self, row): |
| 264 | from erpnext.stock.doctype.serial_no.serial_no import get_serial_nos |
| 265 | |
| 266 | throw_error = False |
| 267 | if row.serial_no: |
| 268 | serial_nos = frappe.get_all( |
| 269 | "Serial and Batch Entry", fields=["serial_no"], filters={"parent": row.serial_and_batch_bundle} |
| 270 | ) |
| 271 | serial_nos = sorted([cstr(d.serial_no) for d in serial_nos]) |
| 272 | parsed_serial_nos = get_serial_nos(row.serial_no) |
| 273 | |
| 274 | if len(serial_nos) != len(parsed_serial_nos): |
| 275 | throw_error = True |
| 276 | elif serial_nos != parsed_serial_nos: |
| 277 | for serial_no in serial_nos: |
| 278 | if serial_no not in parsed_serial_nos: |
| 279 | throw_error = True |
| 280 | break |
| 281 | |
| 282 | elif row.batch_no: |
| 283 | batches = frappe.get_all( |
| 284 | "Serial and Batch Entry", fields=["batch_no"], filters={"parent": row.serial_and_batch_bundle} |
| 285 | ) |
| 286 | batches = sorted([d.batch_no for d in batches]) |
| 287 | |
| 288 | if batches != [row.batch_no]: |
| 289 | throw_error = True |
| 290 | |
| 291 | if throw_error: |
| 292 | frappe.throw( |
| 293 | _( |
| 294 | "At row {0}: Serial and Batch Bundle {1} has already created. Please remove the values from the serial no or batch no fields." |
| 295 | ).format(row.idx, row.serial_and_batch_bundle) |
| 296 | ) |
| 297 | |
Rohit Waghchaure | 9fafc83 | 2024-02-04 10:42:31 +0530 | [diff] [blame] | 298 | def set_use_serial_batch_fields(self): |
| 299 | if frappe.db.get_single_value("Stock Settings", "use_serial_batch_fields"): |
| 300 | for row in self.items: |
| 301 | row.use_serial_batch_fields = 1 |
| 302 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 303 | def get_gl_entries( |
| 304 | self, warehouse_account=None, default_expense_account=None, default_cost_center=None |
| 305 | ): |
Nabin Hait | adeb976 | 2014-10-06 11:53:52 +0530 | [diff] [blame] | 306 | |
Nabin Hait | 142007a | 2013-09-17 15:15:16 +0530 | [diff] [blame] | 307 | if not warehouse_account: |
Rohit Waghchaure | 6b33c9b | 2019-03-08 11:13:35 +0530 | [diff] [blame] | 308 | warehouse_account = get_warehouse_account_map(self.company) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 309 | |
Anand Doshi | de1a97d | 2014-04-17 11:37:46 +0530 | [diff] [blame] | 310 | sle_map = self.get_stock_ledger_details() |
| 311 | voucher_details = self.get_voucher_details(default_expense_account, default_cost_center, sle_map) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 312 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 313 | gl_list = [] |
Nabin Hait | 7a75e10 | 2013-09-17 10:21:20 +0530 | [diff] [blame] | 314 | warehouse_with_no_account = [] |
Nabin Hait | 19f8fa5 | 2021-02-22 22:27:22 +0530 | [diff] [blame] | 315 | precision = self.get_debit_field_precision() |
Nabin Hait | 8c61f34 | 2016-12-15 13:46:03 +0530 | [diff] [blame] | 316 | for item_row in voucher_details: |
| 317 | sle_list = sle_map.get(item_row.name) |
Deepesh Garg | 6e47fd5 | 2022-09-26 21:15:57 +0530 | [diff] [blame] | 318 | sle_rounding_diff = 0.0 |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 319 | if sle_list: |
| 320 | for sle in sle_list: |
| 321 | if warehouse_account.get(sle.warehouse): |
Deepesh Garg | f17ea2c | 2020-12-11 21:30:39 +0530 | [diff] [blame] | 322 | # from warehouse account |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 323 | |
Deepesh Garg | df2a0e2 | 2022-10-11 14:55:09 +0530 | [diff] [blame] | 324 | sle_rounding_diff += flt(sle.stock_value_difference) |
Deepesh Garg | 6e47fd5 | 2022-09-26 21:15:57 +0530 | [diff] [blame] | 325 | |
Nabin Hait | 8c61f34 | 2016-12-15 13:46:03 +0530 | [diff] [blame] | 326 | self.check_expense_account(item_row) |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 327 | |
Deepesh Garg | f17ea2c | 2020-12-11 21:30:39 +0530 | [diff] [blame] | 328 | # expense account/ target_warehouse / source_warehouse |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 329 | if item_row.get("target_warehouse"): |
| 330 | warehouse = item_row.get("target_warehouse") |
Deepesh Garg | f17ea2c | 2020-12-11 21:30:39 +0530 | [diff] [blame] | 331 | expense_account = warehouse_account[warehouse]["account"] |
| 332 | else: |
| 333 | expense_account = item_row.expense_account |
| 334 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 335 | gl_list.append( |
| 336 | self.get_gl_dict( |
| 337 | { |
| 338 | "account": warehouse_account[sle.warehouse]["account"], |
| 339 | "against": expense_account, |
| 340 | "cost_center": item_row.cost_center, |
| 341 | "project": item_row.project or self.get("project"), |
| 342 | "remarks": self.get("remarks") or _("Accounting Entry for Stock"), |
| 343 | "debit": flt(sle.stock_value_difference, precision), |
| 344 | "is_opening": item_row.get("is_opening") or self.get("is_opening") or "No", |
| 345 | }, |
| 346 | warehouse_account[sle.warehouse]["account_currency"], |
| 347 | item=item_row, |
| 348 | ) |
| 349 | ) |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 350 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 351 | gl_list.append( |
| 352 | self.get_gl_dict( |
| 353 | { |
| 354 | "account": expense_account, |
| 355 | "against": warehouse_account[sle.warehouse]["account"], |
| 356 | "cost_center": item_row.cost_center, |
| 357 | "remarks": self.get("remarks") or _("Accounting Entry for Stock"), |
Ankush Menat | 65b21ee | 2022-06-07 14:49:24 +0530 | [diff] [blame] | 358 | "debit": -1 * flt(sle.stock_value_difference, precision), |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 359 | "project": item_row.get("project") or self.get("project"), |
| 360 | "is_opening": item_row.get("is_opening") or self.get("is_opening") or "No", |
| 361 | }, |
| 362 | item=item_row, |
| 363 | ) |
| 364 | ) |
Nabin Hait | 7a75e10 | 2013-09-17 10:21:20 +0530 | [diff] [blame] | 365 | elif sle.warehouse not in warehouse_with_no_account: |
| 366 | warehouse_with_no_account.append(sle.warehouse) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 367 | |
Deepesh Garg | 9aa5e20 | 2022-10-12 15:53:28 +0530 | [diff] [blame] | 368 | if abs(sle_rounding_diff) > (1.0 / (10**precision)) and self.is_internal_transfer(): |
Deepesh Garg | 1c05c00 | 2022-10-12 14:19:09 +0530 | [diff] [blame] | 369 | warehouse_asset_account = "" |
Deepesh Garg | df2a0e2 | 2022-10-11 14:55:09 +0530 | [diff] [blame] | 370 | if self.get("is_internal_customer"): |
Deepesh Garg | 1c05c00 | 2022-10-12 14:19:09 +0530 | [diff] [blame] | 371 | warehouse_asset_account = warehouse_account[item_row.get("target_warehouse")]["account"] |
Deepesh Garg | df2a0e2 | 2022-10-11 14:55:09 +0530 | [diff] [blame] | 372 | elif self.get("is_internal_supplier"): |
Deepesh Garg | 1c05c00 | 2022-10-12 14:19:09 +0530 | [diff] [blame] | 373 | warehouse_asset_account = warehouse_account[item_row.get("warehouse")]["account"] |
Deepesh Garg | 6e47fd5 | 2022-09-26 21:15:57 +0530 | [diff] [blame] | 374 | |
Daizy Modi | 4efc947 | 2022-11-07 09:21:03 +0530 | [diff] [blame] | 375 | expense_account = frappe.get_cached_value("Company", self.company, "default_expense_account") |
Deepesh Garg | ce9164e | 2023-07-11 12:03:38 +0530 | [diff] [blame] | 376 | if not expense_account: |
| 377 | frappe.throw( |
| 378 | _( |
| 379 | "Please set default cost of goods sold account in company {0} for booking rounding gain and loss during stock transfer" |
| 380 | ).format(frappe.bold(self.company)) |
| 381 | ) |
Deepesh Garg | 1c05c00 | 2022-10-12 14:19:09 +0530 | [diff] [blame] | 382 | |
Deepesh Garg | 6e47fd5 | 2022-09-26 21:15:57 +0530 | [diff] [blame] | 383 | gl_list.append( |
| 384 | self.get_gl_dict( |
| 385 | { |
Deepesh Garg | df2a0e2 | 2022-10-11 14:55:09 +0530 | [diff] [blame] | 386 | "account": expense_account, |
Deepesh Garg | 1c05c00 | 2022-10-12 14:19:09 +0530 | [diff] [blame] | 387 | "against": warehouse_asset_account, |
Deepesh Garg | 6e47fd5 | 2022-09-26 21:15:57 +0530 | [diff] [blame] | 388 | "cost_center": item_row.cost_center, |
| 389 | "project": item_row.project or self.get("project"), |
Deepesh Garg | df2a0e2 | 2022-10-11 14:55:09 +0530 | [diff] [blame] | 390 | "remarks": _("Rounding gain/loss Entry for Stock Transfer"), |
Deepesh Garg | 6e47fd5 | 2022-09-26 21:15:57 +0530 | [diff] [blame] | 391 | "debit": sle_rounding_diff, |
| 392 | "is_opening": item_row.get("is_opening") or self.get("is_opening") or "No", |
| 393 | }, |
| 394 | warehouse_account[sle.warehouse]["account_currency"], |
| 395 | item=item_row, |
| 396 | ) |
| 397 | ) |
| 398 | |
| 399 | gl_list.append( |
| 400 | self.get_gl_dict( |
| 401 | { |
Deepesh Garg | 1c05c00 | 2022-10-12 14:19:09 +0530 | [diff] [blame] | 402 | "account": warehouse_asset_account, |
Deepesh Garg | df2a0e2 | 2022-10-11 14:55:09 +0530 | [diff] [blame] | 403 | "against": expense_account, |
Deepesh Garg | 6e47fd5 | 2022-09-26 21:15:57 +0530 | [diff] [blame] | 404 | "cost_center": item_row.cost_center, |
Deepesh Garg | df2a0e2 | 2022-10-11 14:55:09 +0530 | [diff] [blame] | 405 | "remarks": _("Rounding gain/loss Entry for Stock Transfer"), |
| 406 | "credit": sle_rounding_diff, |
Deepesh Garg | 6e47fd5 | 2022-09-26 21:15:57 +0530 | [diff] [blame] | 407 | "project": item_row.get("project") or self.get("project"), |
| 408 | "is_opening": item_row.get("is_opening") or self.get("is_opening") or "No", |
| 409 | }, |
| 410 | item=item_row, |
| 411 | ) |
| 412 | ) |
| 413 | |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 414 | if warehouse_with_no_account: |
Nabin Hait | d662502 | 2016-10-24 18:17:57 +0530 | [diff] [blame] | 415 | for wh in warehouse_with_no_account: |
Daizy Modi | 4efc947 | 2022-11-07 09:21:03 +0530 | [diff] [blame] | 416 | if frappe.get_cached_value("Warehouse", wh, "company"): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 417 | frappe.throw( |
| 418 | _( |
| 419 | "Warehouse {0} is not linked to any account, please mention the account in the warehouse record or set default inventory account in company {1}." |
| 420 | ).format(wh, self.company) |
| 421 | ) |
Rushabh Mehta | 14a908b | 2015-10-15 12:28:20 +0530 | [diff] [blame] | 422 | |
Nabin Hait | 19f8fa5 | 2021-02-22 22:27:22 +0530 | [diff] [blame] | 423 | return process_gl_map(gl_list, precision=precision) |
| 424 | |
| 425 | def get_debit_field_precision(self): |
| 426 | if not frappe.flags.debit_field_precision: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 427 | frappe.flags.debit_field_precision = frappe.get_precision( |
| 428 | "GL Entry", "debit_in_account_currency" |
| 429 | ) |
Nabin Hait | 19f8fa5 | 2021-02-22 22:27:22 +0530 | [diff] [blame] | 430 | |
| 431 | return frappe.flags.debit_field_precision |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 432 | |
Anand Doshi | de1a97d | 2014-04-17 11:37:46 +0530 | [diff] [blame] | 433 | def get_voucher_details(self, default_expense_account, default_cost_center, sle_map): |
| 434 | if self.doctype == "Stock Reconciliation": |
Nabin Hait | 3f119ec | 2019-05-16 17:28:39 +0530 | [diff] [blame] | 435 | reconciliation_purpose = frappe.db.get_value(self.doctype, self.name, "purpose") |
| 436 | is_opening = "Yes" if reconciliation_purpose == "Opening Stock" else "No" |
| 437 | details = [] |
Nabin Hait | 34c551d | 2019-07-03 10:34:31 +0530 | [diff] [blame] | 438 | for voucher_detail_no in sle_map: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 439 | details.append( |
| 440 | frappe._dict( |
| 441 | { |
| 442 | "name": voucher_detail_no, |
| 443 | "expense_account": default_expense_account, |
| 444 | "cost_center": default_cost_center, |
| 445 | "is_opening": is_opening, |
| 446 | } |
| 447 | ) |
| 448 | ) |
Nabin Hait | 3f119ec | 2019-05-16 17:28:39 +0530 | [diff] [blame] | 449 | return details |
Anand Doshi | de1a97d | 2014-04-17 11:37:46 +0530 | [diff] [blame] | 450 | else: |
Nabin Hait | dd38a26 | 2014-12-26 13:15:21 +0530 | [diff] [blame] | 451 | details = self.get("items") |
Anand Doshi | 094610d | 2014-04-16 19:56:53 +0530 | [diff] [blame] | 452 | |
Anand Doshi | de1a97d | 2014-04-17 11:37:46 +0530 | [diff] [blame] | 453 | if default_expense_account or default_cost_center: |
| 454 | for d in details: |
| 455 | if default_expense_account and not d.get("expense_account"): |
| 456 | d.expense_account = default_expense_account |
| 457 | if default_cost_center and not d.get("cost_center"): |
| 458 | d.cost_center = default_cost_center |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 459 | |
Anand Doshi | de1a97d | 2014-04-17 11:37:46 +0530 | [diff] [blame] | 460 | return details |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 461 | |
Ankush Menat | e6ab8df | 2022-02-06 13:02:34 +0530 | [diff] [blame] | 462 | def get_items_and_warehouses(self) -> Tuple[List[str], List[str]]: |
| 463 | """Get list of items and warehouses affected by a transaction""" |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 464 | |
Ankush Menat | e6ab8df | 2022-02-06 13:02:34 +0530 | [diff] [blame] | 465 | if not (hasattr(self, "items") or hasattr(self, "packed_items")): |
| 466 | return [], [] |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 467 | |
Ankush Menat | e6ab8df | 2022-02-06 13:02:34 +0530 | [diff] [blame] | 468 | item_rows = (self.get("items") or []) + (self.get("packed_items") or []) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 469 | |
Ankush Menat | e6ab8df | 2022-02-06 13:02:34 +0530 | [diff] [blame] | 470 | items = {d.item_code for d in item_rows if d.item_code} |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 471 | |
Ankush Menat | e6ab8df | 2022-02-06 13:02:34 +0530 | [diff] [blame] | 472 | warehouses = set() |
| 473 | for d in item_rows: |
| 474 | if d.get("warehouse"): |
| 475 | warehouses.add(d.warehouse) |
Nabin Hait | becf75d | 2014-03-27 17:18:29 +0530 | [diff] [blame] | 476 | |
Ankush Menat | e6ab8df | 2022-02-06 13:02:34 +0530 | [diff] [blame] | 477 | if self.doctype == "Stock Entry": |
| 478 | if d.get("s_warehouse"): |
| 479 | warehouses.add(d.s_warehouse) |
| 480 | if d.get("t_warehouse"): |
| 481 | warehouses.add(d.t_warehouse) |
| 482 | |
| 483 | return list(items), list(warehouses) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 484 | |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 485 | def get_stock_ledger_details(self): |
| 486 | stock_ledger = {} |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 487 | stock_ledger_entries = frappe.db.sql( |
| 488 | """ |
Rushabh Mehta | cc8b2b2 | 2017-03-31 12:44:29 +0530 | [diff] [blame] | 489 | select |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 490 | name, warehouse, stock_value_difference, valuation_rate, |
Rushabh Mehta | cc8b2b2 | 2017-03-31 12:44:29 +0530 | [diff] [blame] | 491 | voucher_detail_no, item_code, posting_date, posting_time, |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 492 | actual_qty, qty_after_transaction |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 493 | from |
| 494 | `tabStock Ledger Entry` |
| 495 | where |
Ankush Menat | 0ca60af | 2022-02-08 10:24:19 +0530 | [diff] [blame] | 496 | voucher_type=%s and voucher_no=%s and is_cancelled = 0 |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 497 | """, |
| 498 | (self.doctype, self.name), |
| 499 | as_dict=True, |
| 500 | ) |
Nabin Hait | 0a6aaf4 | 2017-02-07 01:23:26 +0530 | [diff] [blame] | 501 | |
Nabin Hait | ea8fab5 | 2017-02-06 17:13:39 +0530 | [diff] [blame] | 502 | for sle in stock_ledger_entries: |
Deepesh Garg | b4be292 | 2021-01-28 13:09:56 +0530 | [diff] [blame] | 503 | stock_ledger.setdefault(sle.voucher_detail_no, []).append(sle) |
Nabin Hait | 2e296fa | 2013-08-28 18:53:11 +0530 | [diff] [blame] | 504 | return stock_ledger |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 505 | |
Nabin Hait | 27994c2 | 2013-08-26 16:53:30 +0530 | [diff] [blame] | 506 | def check_expense_account(self, item): |
Rushabh Mehta | 052fe82 | 2014-04-16 19:20:11 +0530 | [diff] [blame] | 507 | if not item.get("expense_account"): |
Rohit Waghchaure | ceab692 | 2020-11-18 17:57:35 +0530 | [diff] [blame] | 508 | msg = _("Please set an Expense Account in the Items table") |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 509 | frappe.throw( |
| 510 | _("Row #{0}: Expense Account not set for the Item {1}. {2}").format( |
| 511 | item.idx, frappe.bold(item.item_code), msg |
| 512 | ), |
| 513 | title=_("Expense Account Missing"), |
| 514 | ) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 515 | |
Anand Doshi | 496123a | 2014-06-19 19:25:19 +0530 | [diff] [blame] | 516 | else: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 517 | is_expense_account = ( |
| 518 | frappe.get_cached_value("Account", item.get("expense_account"), "report_type") |
| 519 | == "Profit and Loss" |
| 520 | ) |
| 521 | if ( |
| 522 | self.doctype |
Sagar Sharma | 2d04e71 | 2022-08-17 15:57:41 +0530 | [diff] [blame] | 523 | not in ( |
| 524 | "Purchase Receipt", |
| 525 | "Purchase Invoice", |
| 526 | "Stock Reconciliation", |
| 527 | "Stock Entry", |
| 528 | "Subcontracting Receipt", |
| 529 | ) |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 530 | and not is_expense_account |
| 531 | ): |
| 532 | frappe.throw( |
| 533 | _("Expense / Difference account ({0}) must be a 'Profit or Loss' account").format( |
| 534 | item.get("expense_account") |
| 535 | ) |
| 536 | ) |
Anand Doshi | 496123a | 2014-06-19 19:25:19 +0530 | [diff] [blame] | 537 | if is_expense_account and not item.get("cost_center"): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 538 | frappe.throw( |
| 539 | _("{0} {1}: Cost Center is mandatory for Item {2}").format( |
| 540 | _(self.doctype), self.name, item.get("item_code") |
| 541 | ) |
| 542 | ) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 543 | |
Rohit Waghchaure | 8996b7d | 2020-01-23 17:36:52 +0530 | [diff] [blame] | 544 | def delete_auto_created_batches(self): |
Rohit Waghchaure | bc75a7e | 2022-10-10 13:28:19 +0530 | [diff] [blame] | 545 | for row in self.items: |
| 546 | if row.serial_and_batch_bundle: |
| 547 | frappe.db.set_value( |
| 548 | "Serial and Batch Bundle", row.serial_and_batch_bundle, {"is_cancelled": 1} |
| 549 | ) |
Rohit Waghchaure | 8996b7d | 2020-01-23 17:36:52 +0530 | [diff] [blame] | 550 | |
Rohit Waghchaure | bc75a7e | 2022-10-10 13:28:19 +0530 | [diff] [blame] | 551 | row.db_set("serial_and_batch_bundle", None) |
Saqib | e9ac3e0 | 2020-03-02 15:02:58 +0530 | [diff] [blame] | 552 | |
Rohit Waghchaure | d3ceb07 | 2023-03-31 09:03:54 +0530 | [diff] [blame] | 553 | def set_serial_and_batch_bundle(self, table_name=None, ignore_validate=False): |
Rohit Waghchaure | 648efca | 2023-03-28 12:16:27 +0530 | [diff] [blame] | 554 | if not table_name: |
| 555 | table_name = "items" |
Rohit Waghchaure | 8996b7d | 2020-01-23 17:36:52 +0530 | [diff] [blame] | 556 | |
Rohit Waghchaure | d3ceb07 | 2023-03-31 09:03:54 +0530 | [diff] [blame] | 557 | QTY_FIELD = { |
| 558 | "serial_and_batch_bundle": "qty", |
| 559 | "current_serial_and_batch_bundle": "current_qty", |
| 560 | "rejected_serial_and_batch_bundle": "rejected_qty", |
| 561 | } |
| 562 | |
Rohit Waghchaure | 648efca | 2023-03-28 12:16:27 +0530 | [diff] [blame] | 563 | for row in self.get(table_name): |
s-aga-r | c20241f | 2024-01-12 15:26:35 +0530 | [diff] [blame] | 564 | for field in QTY_FIELD.keys(): |
Rohit Waghchaure | d3ceb07 | 2023-03-31 09:03:54 +0530 | [diff] [blame] | 565 | if row.get(field): |
| 566 | frappe.get_doc("Serial and Batch Bundle", row.get(field)).set_serial_and_batch_values( |
| 567 | self, row, qty_field=QTY_FIELD[field] |
| 568 | ) |
Rohit Waghchaure | 648efca | 2023-03-28 12:16:27 +0530 | [diff] [blame] | 569 | |
Rohit Waghchaure | 86da306 | 2023-03-20 14:15:34 +0530 | [diff] [blame] | 570 | def make_package_for_transfer( |
| 571 | self, serial_and_batch_bundle, warehouse, type_of_transaction=None, do_not_submit=None |
| 572 | ): |
| 573 | bundle_doc = frappe.get_doc("Serial and Batch Bundle", serial_and_batch_bundle) |
| 574 | |
| 575 | if not type_of_transaction: |
| 576 | type_of_transaction = "Inward" |
| 577 | |
| 578 | bundle_doc = frappe.copy_doc(bundle_doc) |
| 579 | bundle_doc.warehouse = warehouse |
| 580 | bundle_doc.type_of_transaction = type_of_transaction |
| 581 | bundle_doc.voucher_type = self.doctype |
| 582 | bundle_doc.voucher_no = self.name |
| 583 | bundle_doc.is_cancelled = 0 |
| 584 | |
Rohit Waghchaure | 5bb3173 | 2023-03-21 10:54:41 +0530 | [diff] [blame] | 585 | for row in bundle_doc.entries: |
Rohit Waghchaure | 86da306 | 2023-03-20 14:15:34 +0530 | [diff] [blame] | 586 | row.is_outward = 0 |
| 587 | row.qty = abs(row.qty) |
| 588 | row.stock_value_difference = abs(row.stock_value_difference) |
| 589 | if type_of_transaction == "Outward": |
| 590 | row.qty *= -1 |
| 591 | row.stock_value_difference *= row.stock_value_difference |
| 592 | row.is_outward = 1 |
| 593 | |
| 594 | row.warehouse = warehouse |
| 595 | |
Rohit Waghchaure | 5bb3173 | 2023-03-21 10:54:41 +0530 | [diff] [blame] | 596 | bundle_doc.calculate_qty_and_amount() |
Rohit Waghchaure | 86da306 | 2023-03-20 14:15:34 +0530 | [diff] [blame] | 597 | bundle_doc.flags.ignore_permissions = True |
Rohit Waghchaure | d3ceb07 | 2023-03-31 09:03:54 +0530 | [diff] [blame] | 598 | bundle_doc.save(ignore_permissions=True) |
Rohit Waghchaure | 86da306 | 2023-03-20 14:15:34 +0530 | [diff] [blame] | 599 | |
Rohit Waghchaure | 86da306 | 2023-03-20 14:15:34 +0530 | [diff] [blame] | 600 | return bundle_doc.name |
Rohit Waghchaure | 4f4dbf1 | 2020-01-23 12:42:42 +0530 | [diff] [blame] | 601 | |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 602 | def get_sl_entries(self, d, args): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 603 | sl_dict = frappe._dict( |
| 604 | { |
| 605 | "item_code": d.get("item_code", None), |
| 606 | "warehouse": d.get("warehouse", None), |
Rohit Waghchaure | bc75a7e | 2022-10-10 13:28:19 +0530 | [diff] [blame] | 607 | "serial_and_batch_bundle": d.get("serial_and_batch_bundle"), |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 608 | "posting_date": self.posting_date, |
| 609 | "posting_time": self.posting_time, |
| 610 | "fiscal_year": get_fiscal_year(self.posting_date, company=self.company)[0], |
| 611 | "voucher_type": self.doctype, |
| 612 | "voucher_no": self.name, |
| 613 | "voucher_detail_no": d.name, |
| 614 | "actual_qty": (self.docstatus == 1 and 1 or -1) * flt(d.get("stock_qty")), |
Rohit Waghchaure | 2d5ccc0 | 2023-05-01 21:17:18 +0530 | [diff] [blame] | 615 | "stock_uom": frappe.get_cached_value( |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 616 | "Item", args.get("item_code") or d.get("item_code"), "stock_uom" |
| 617 | ), |
| 618 | "incoming_rate": 0, |
| 619 | "company": self.company, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 620 | "project": d.get("project") or self.get("project"), |
| 621 | "is_cancelled": 1 if self.docstatus == 2 else 0, |
| 622 | } |
| 623 | ) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 624 | |
Nabin Hait | 1e2f20a | 2013-08-02 11:42:11 +0530 | [diff] [blame] | 625 | sl_dict.update(args) |
Rohit Waghchaure | e576f7f | 2022-06-30 19:12:06 +0530 | [diff] [blame] | 626 | self.update_inventory_dimensions(d, sl_dict) |
Rohit Waghchaure | dbec5cf | 2022-06-22 12:24:08 +0530 | [diff] [blame] | 627 | |
rohitwaghchaure | 0743289 | 2023-12-17 12:42:07 +0530 | [diff] [blame] | 628 | if self.docstatus == 2: |
| 629 | # To handle denormalized serial no records, will br deprecated in v16 |
| 630 | for field in ["serial_no", "batch_no"]: |
| 631 | if d.get(field): |
| 632 | sl_dict[field] = d.get(field) |
| 633 | |
Nabin Hait | 1e2f20a | 2013-08-02 11:42:11 +0530 | [diff] [blame] | 634 | return sl_dict |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 635 | |
Rohit Waghchaure | dbec5cf | 2022-06-22 12:24:08 +0530 | [diff] [blame] | 636 | def update_inventory_dimensions(self, row, sl_dict) -> None: |
Rohit Waghchaure | 2372999 | 2022-09-02 18:43:55 +0530 | [diff] [blame] | 637 | # To handle delivery note and sales invoice |
| 638 | if row.get("item_row"): |
| 639 | row = row.get("item_row") |
| 640 | |
Rohit Waghchaure | 289e6cd | 2022-07-15 15:43:38 +0530 | [diff] [blame] | 641 | dimensions = get_evaluated_inventory_dimension(row, sl_dict, parent_doc=self) |
| 642 | for dimension in dimensions: |
Rohit Waghchaure | 0b39a01 | 2022-08-17 11:56:13 +0530 | [diff] [blame] | 643 | if not dimension: |
| 644 | continue |
| 645 | |
Rohit Waghchaure | 6798b90 | 2023-05-09 16:07:14 +0530 | [diff] [blame] | 646 | if self.doctype in [ |
| 647 | "Purchase Invoice", |
| 648 | "Purchase Receipt", |
| 649 | "Sales Invoice", |
| 650 | "Delivery Note", |
| 651 | "Stock Entry", |
| 652 | ]: |
Rohit Waghchaure | 38aaba5 | 2023-05-13 13:00:05 +0530 | [diff] [blame] | 653 | if ( |
| 654 | ( |
| 655 | sl_dict.actual_qty > 0 |
| 656 | and not self.get("is_return") |
| 657 | or sl_dict.actual_qty < 0 |
| 658 | and self.get("is_return") |
| 659 | ) |
| 660 | and self.doctype in ["Purchase Invoice", "Purchase Receipt"] |
| 661 | ) or ( |
| 662 | ( |
| 663 | sl_dict.actual_qty < 0 |
| 664 | and not self.get("is_return") |
| 665 | or sl_dict.actual_qty > 0 |
| 666 | and self.get("is_return") |
| 667 | ) |
| 668 | and self.doctype in ["Sales Invoice", "Delivery Note", "Stock Entry"] |
Rohit Waghchaure | 6798b90 | 2023-05-09 16:07:14 +0530 | [diff] [blame] | 669 | ): |
| 670 | sl_dict[dimension.target_fieldname] = row.get(dimension.source_fieldname) |
| 671 | else: |
| 672 | fieldname_start_with = "to" |
| 673 | if self.doctype in ["Purchase Invoice", "Purchase Receipt"]: |
| 674 | fieldname_start_with = "from" |
| 675 | |
| 676 | fieldname = f"{fieldname_start_with}_{dimension.source_fieldname}" |
| 677 | sl_dict[dimension.target_fieldname] = row.get(fieldname) |
| 678 | |
| 679 | if not sl_dict.get(dimension.target_fieldname): |
| 680 | sl_dict[dimension.target_fieldname] = row.get(dimension.source_fieldname) |
| 681 | |
| 682 | elif row.get(dimension.source_fieldname): |
Rohit Waghchaure | 289e6cd | 2022-07-15 15:43:38 +0530 | [diff] [blame] | 683 | sl_dict[dimension.target_fieldname] = row.get(dimension.source_fieldname) |
Rohit Waghchaure | dbec5cf | 2022-06-22 12:24:08 +0530 | [diff] [blame] | 684 | |
Rohit Waghchaure | 0b39a01 | 2022-08-17 11:56:13 +0530 | [diff] [blame] | 685 | if not sl_dict.get(dimension.target_fieldname) and dimension.fetch_from_parent: |
| 686 | sl_dict[dimension.target_fieldname] = self.get(dimension.fetch_from_parent) |
| 687 | |
| 688 | # Get value based on doctype name |
| 689 | if not sl_dict.get(dimension.target_fieldname): |
Daizy Modi | 4efc947 | 2022-11-07 09:21:03 +0530 | [diff] [blame] | 690 | fieldname = next( |
| 691 | ( |
| 692 | field.fieldname |
| 693 | for field in frappe.get_meta(self.doctype).fields |
| 694 | if field.options == dimension.fetch_from_parent |
| 695 | ), |
| 696 | None, |
Rohit Waghchaure | 0b39a01 | 2022-08-17 11:56:13 +0530 | [diff] [blame] | 697 | ) |
| 698 | |
| 699 | if fieldname and self.get(fieldname): |
| 700 | sl_dict[dimension.target_fieldname] = self.get(fieldname) |
| 701 | |
Rohit Waghchaure | 75fcab0 | 2022-09-03 17:09:24 +0530 | [diff] [blame] | 702 | if sl_dict[dimension.target_fieldname] and self.docstatus == 1: |
| 703 | row.db_set(dimension.source_fieldname, sl_dict[dimension.target_fieldname]) |
Rohit Waghchaure | 2372999 | 2022-09-02 18:43:55 +0530 | [diff] [blame] | 704 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 705 | def make_sl_entries(self, sl_entries, allow_negative_stock=False, via_landed_cost_voucher=False): |
Rushabh Mehta | 1f84799 | 2013-12-12 19:12:19 +0530 | [diff] [blame] | 706 | from erpnext.stock.stock_ledger import make_sl_entries |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 707 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 708 | make_sl_entries(sl_entries, allow_negative_stock, via_landed_cost_voucher) |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 709 | |
Deepesh Garg | 2a9c5ba | 2020-04-30 10:38:58 +0530 | [diff] [blame] | 710 | def make_gl_entries_on_cancel(self): |
ruthra kumar | 46ea814 | 2023-07-28 08:29:19 +0530 | [diff] [blame] | 711 | cancel_exchange_gain_loss_journal(frappe._dict(doctype=self.doctype, name=self.name)) |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 712 | if frappe.db.sql( |
| 713 | """select name from `tabGL Entry` where voucher_type=%s |
| 714 | and voucher_no=%s""", |
| 715 | (self.doctype, self.name), |
| 716 | ): |
| 717 | self.make_gl_entries() |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 718 | |
Anand Doshi | a740f75 | 2014-06-25 13:31:02 +0530 | [diff] [blame] | 719 | def get_serialized_items(self): |
| 720 | serialized_items = [] |
Ankush Menat | a9c84f7 | 2021-06-11 16:00:48 +0530 | [diff] [blame] | 721 | item_codes = list(set(d.item_code for d in self.get("items"))) |
Anand Doshi | a740f75 | 2014-06-25 13:31:02 +0530 | [diff] [blame] | 722 | if item_codes: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 723 | serialized_items = frappe.db.sql_list( |
| 724 | """select name from `tabItem` |
| 725 | where has_serial_no=1 and name in ({})""".format( |
| 726 | ", ".join(["%s"] * len(item_codes)) |
| 727 | ), |
| 728 | tuple(item_codes), |
| 729 | ) |
Anand Doshi | a740f75 | 2014-06-25 13:31:02 +0530 | [diff] [blame] | 730 | |
| 731 | return serialized_items |
Rushabh Mehta | b16b9cd | 2015-08-03 16:13:33 +0530 | [diff] [blame] | 732 | |
Saurabh | 2e29206 | 2015-11-18 17:03:33 +0530 | [diff] [blame] | 733 | def validate_warehouse(self): |
Rohan Bansal | 7f8b95e | 2021-04-14 14:12:03 +0530 | [diff] [blame] | 734 | from erpnext.stock.utils import validate_disabled_warehouse, validate_warehouse_company |
Saurabh | 2e29206 | 2015-11-18 17:03:33 +0530 | [diff] [blame] | 735 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 736 | warehouses = list(set(d.warehouse for d in self.get("items") if getattr(d, "warehouse", None))) |
Saurabh | 2e29206 | 2015-11-18 17:03:33 +0530 | [diff] [blame] | 737 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 738 | target_warehouses = list( |
| 739 | set([d.target_warehouse for d in self.get("items") if getattr(d, "target_warehouse", None)]) |
| 740 | ) |
Deepesh Garg | 15ff6a5 | 2020-02-18 12:28:41 +0530 | [diff] [blame] | 741 | |
| 742 | warehouses.extend(target_warehouses) |
| 743 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 744 | from_warehouse = list( |
| 745 | set([d.from_warehouse for d in self.get("items") if getattr(d, "from_warehouse", None)]) |
| 746 | ) |
Deepesh Garg | 15ff6a5 | 2020-02-18 12:28:41 +0530 | [diff] [blame] | 747 | |
| 748 | warehouses.extend(from_warehouse) |
| 749 | |
Saurabh | 2e29206 | 2015-11-18 17:03:33 +0530 | [diff] [blame] | 750 | for w in warehouses: |
Jannat Patel | 30c8873 | 2021-02-11 11:46:48 +0530 | [diff] [blame] | 751 | validate_disabled_warehouse(w) |
Saurabh | 2e29206 | 2015-11-18 17:03:33 +0530 | [diff] [blame] | 752 | validate_warehouse_company(w, self.company) |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 753 | |
Anand Doshi | 6b71ef5 | 2016-01-06 16:32:06 +0530 | [diff] [blame] | 754 | def update_billing_percentage(self, update_modified=True): |
marination | d6596a1 | 2020-11-02 15:07:48 +0530 | [diff] [blame] | 755 | target_ref_field = "amount" |
| 756 | if self.doctype == "Delivery Note": |
| 757 | target_ref_field = "amount - (returned_qty * rate)" |
| 758 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 759 | self._update_percent_field( |
| 760 | { |
| 761 | "target_dt": self.doctype + " Item", |
| 762 | "target_parent_dt": self.doctype, |
| 763 | "target_parent_field": "per_billed", |
| 764 | "target_ref_field": target_ref_field, |
| 765 | "target_field": "billed_amt", |
| 766 | "name": self.name, |
| 767 | }, |
| 768 | update_modified, |
| 769 | ) |
Anand Doshi | a740f75 | 2014-06-25 13:31:02 +0530 | [diff] [blame] | 770 | |
Nabin Hait | 8af429d | 2016-11-16 17:21:59 +0530 | [diff] [blame] | 771 | def validate_inspection(self): |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 772 | """Checks if quality inspection is set/ is valid for Items that require inspection.""" |
| 773 | inspection_fieldname_map = { |
| 774 | "Purchase Receipt": "inspection_required_before_purchase", |
| 775 | "Purchase Invoice": "inspection_required_before_purchase", |
s-aga-r | 3fdcd33 | 2023-08-23 12:15:35 +0530 | [diff] [blame] | 776 | "Subcontracting Receipt": "inspection_required_before_purchase", |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 777 | "Sales Invoice": "inspection_required_before_delivery", |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 778 | "Delivery Note": "inspection_required_before_delivery", |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 779 | } |
| 780 | inspection_required_fieldname = inspection_fieldname_map.get(self.doctype) |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 781 | |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 782 | # return if inspection is not required on document level |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 783 | if ( |
| 784 | (not inspection_required_fieldname and self.doctype != "Stock Entry") |
| 785 | or (self.doctype == "Stock Entry" and not self.inspection_required) |
| 786 | or (self.doctype in ["Sales Invoice", "Purchase Invoice"] and not self.update_stock) |
| 787 | ): |
| 788 | return |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 789 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 790 | for row in self.get("items"): |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 791 | qi_required = False |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 792 | if inspection_required_fieldname and frappe.db.get_value( |
| 793 | "Item", row.item_code, inspection_required_fieldname |
| 794 | ): |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 795 | qi_required = True |
| 796 | elif self.doctype == "Stock Entry" and row.t_warehouse: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 797 | qi_required = True # inward stock needs inspection |
Rushabh Mehta | ffd80a6 | 2017-01-16 17:23:20 +0530 | [diff] [blame] | 798 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 799 | if qi_required: # validate row only if inspection is required on item level |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 800 | self.validate_qi_presence(row) |
| 801 | if self.docstatus == 1: |
| 802 | self.validate_qi_submission(row) |
| 803 | self.validate_qi_rejection(row) |
| 804 | |
| 805 | def validate_qi_presence(self, row): |
| 806 | """Check if QI is present on row level. Warn on save and stop on submit if missing.""" |
| 807 | if not row.quality_inspection: |
marination | 654e9d8 | 2021-06-21 16:51:12 +0530 | [diff] [blame] | 808 | msg = f"Row #{row.idx}: Quality Inspection is required for Item {frappe.bold(row.item_code)}" |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 809 | if self.docstatus == 1: |
marination | 654e9d8 | 2021-06-21 16:51:12 +0530 | [diff] [blame] | 810 | frappe.throw(_(msg), title=_("Inspection Required"), exc=QualityInspectionRequiredError) |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 811 | else: |
marination | 654e9d8 | 2021-06-21 16:51:12 +0530 | [diff] [blame] | 812 | frappe.msgprint(_(msg), title=_("Inspection Required"), indicator="blue") |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 813 | |
| 814 | def validate_qi_submission(self, row): |
| 815 | """Check if QI is submitted on row level, during submission""" |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 816 | action = frappe.db.get_single_value( |
| 817 | "Stock Settings", "action_if_quality_inspection_is_not_submitted" |
| 818 | ) |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 819 | qa_docstatus = frappe.db.get_value("Quality Inspection", row.quality_inspection, "docstatus") |
| 820 | |
barredterra | eb9ee3f | 2023-12-05 11:22:55 +0100 | [diff] [blame] | 821 | if qa_docstatus != 1: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 822 | link = frappe.utils.get_link_to_form("Quality Inspection", row.quality_inspection) |
| 823 | msg = ( |
| 824 | f"Row #{row.idx}: Quality Inspection {link} is not submitted for the item: {row.item_code}" |
| 825 | ) |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 826 | if action == "Stop": |
marination | 654e9d8 | 2021-06-21 16:51:12 +0530 | [diff] [blame] | 827 | frappe.throw(_(msg), title=_("Inspection Submission"), exc=QualityInspectionNotSubmittedError) |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 828 | else: |
Marica | 9ba3fce | 2021-06-22 11:20:17 +0530 | [diff] [blame] | 829 | frappe.msgprint(_(msg), alert=True, indicator="orange") |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 830 | |
| 831 | def validate_qi_rejection(self, row): |
| 832 | """Check if QI is rejected on row level, during submission""" |
marination | f67f13c | 2021-07-10 18:24:24 +0530 | [diff] [blame] | 833 | action = frappe.db.get_single_value("Stock Settings", "action_if_quality_inspection_is_rejected") |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 834 | qa_status = frappe.db.get_value("Quality Inspection", row.quality_inspection, "status") |
| 835 | |
| 836 | if qa_status == "Rejected": |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 837 | link = frappe.utils.get_link_to_form("Quality Inspection", row.quality_inspection) |
marination | 654e9d8 | 2021-06-21 16:51:12 +0530 | [diff] [blame] | 838 | msg = f"Row #{row.idx}: Quality Inspection {link} was rejected for item {row.item_code}" |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 839 | if action == "Stop": |
marination | 654e9d8 | 2021-06-21 16:51:12 +0530 | [diff] [blame] | 840 | frappe.throw(_(msg), title=_("Inspection Rejected"), exc=QualityInspectionRejectedError) |
marination | 9ac9a4e | 2021-06-21 16:18:35 +0530 | [diff] [blame] | 841 | else: |
marination | 654e9d8 | 2021-06-21 16:51:12 +0530 | [diff] [blame] | 842 | frappe.msgprint(_(msg), alert=True, indicator="orange") |
Manas Solanki | cc90241 | 2016-11-10 19:15:11 +0530 | [diff] [blame] | 843 | |
Nabin Hait | b2d3c0f | 2018-06-14 15:54:34 +0530 | [diff] [blame] | 844 | def update_blanket_order(self): |
Nabin Hait | d1f40ad | 2018-06-14 17:09:55 +0530 | [diff] [blame] | 845 | blanket_orders = list(set([d.blanket_order for d in self.items if d.blanket_order])) |
Nabin Hait | b2d3c0f | 2018-06-14 15:54:34 +0530 | [diff] [blame] | 846 | for blanket_order in blanket_orders: |
| 847 | frappe.get_doc("Blanket Order", blanket_order).update_ordered_qty() |
Manas Solanki | e5e87f7 | 2018-05-28 20:07:08 +0530 | [diff] [blame] | 848 | |
marination | fd04e96 | 2020-04-03 15:46:48 +0530 | [diff] [blame] | 849 | def validate_customer_provided_item(self): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 850 | for d in self.get("items"): |
marination | fd04e96 | 2020-04-03 15:46:48 +0530 | [diff] [blame] | 851 | # Customer Provided parts will have zero valuation rate |
Rohit Waghchaure | 2d5ccc0 | 2023-05-01 21:17:18 +0530 | [diff] [blame] | 852 | if frappe.get_cached_value("Item", d.item_code, "is_customer_provided_item"): |
marination | fd04e96 | 2020-04-03 15:46:48 +0530 | [diff] [blame] | 853 | d.allow_zero_valuation_rate = 1 |
Anand Doshi | 2ce39cf | 2014-04-07 18:51:58 +0530 | [diff] [blame] | 854 | |
Anupam Kumar | 7e1dcf9 | 2021-02-11 20:19:30 +0530 | [diff] [blame] | 855 | def set_rate_of_stock_uom(self): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 856 | if self.doctype in [ |
| 857 | "Purchase Receipt", |
| 858 | "Purchase Invoice", |
| 859 | "Purchase Order", |
| 860 | "Sales Invoice", |
| 861 | "Sales Order", |
| 862 | "Delivery Note", |
| 863 | "Quotation", |
| 864 | ]: |
Anupam Kumar | 7e1dcf9 | 2021-02-11 20:19:30 +0530 | [diff] [blame] | 865 | for d in self.get("items"): |
Rohit Waghchaure | 1358443 | 2021-03-26 14:11:50 +0530 | [diff] [blame] | 866 | d.stock_uom_rate = d.rate / (d.conversion_factor or 1) |
Anupam Kumar | 7e1dcf9 | 2021-02-11 20:19:30 +0530 | [diff] [blame] | 867 | |
Deepesh Garg | b4be292 | 2021-01-28 13:09:56 +0530 | [diff] [blame] | 868 | def validate_internal_transfer(self): |
rohitwaghchaure | 5136fe1 | 2023-10-22 20:03:02 +0530 | [diff] [blame] | 869 | if self.doctype in ("Sales Invoice", "Delivery Note", "Purchase Invoice", "Purchase Receipt"): |
| 870 | if self.is_internal_transfer(): |
| 871 | self.validate_in_transit_warehouses() |
| 872 | self.validate_multi_currency() |
| 873 | self.validate_packed_items() |
rohitwaghchaure | 8fdc244 | 2024-01-27 21:37:58 +0530 | [diff] [blame] | 874 | |
| 875 | if self.get("is_internal_supplier"): |
| 876 | self.validate_internal_transfer_qty() |
rohitwaghchaure | 5136fe1 | 2023-10-22 20:03:02 +0530 | [diff] [blame] | 877 | else: |
| 878 | self.validate_internal_transfer_warehouse() |
| 879 | |
| 880 | def validate_internal_transfer_warehouse(self): |
| 881 | for row in self.items: |
| 882 | if row.get("target_warehouse"): |
| 883 | row.target_warehouse = None |
| 884 | |
| 885 | if row.get("from_warehouse"): |
| 886 | row.from_warehouse = None |
Deepesh Garg | b4be292 | 2021-01-28 13:09:56 +0530 | [diff] [blame] | 887 | |
| 888 | def validate_in_transit_warehouses(self): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 889 | if ( |
| 890 | self.doctype == "Sales Invoice" and self.get("update_stock") |
| 891 | ) or self.doctype == "Delivery Note": |
| 892 | for item in self.get("items"): |
Deepesh Garg | b4be292 | 2021-01-28 13:09:56 +0530 | [diff] [blame] | 893 | if not item.target_warehouse: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 894 | frappe.throw( |
| 895 | _("Row {0}: Target Warehouse is mandatory for internal transfers").format(item.idx) |
| 896 | ) |
Deepesh Garg | b4be292 | 2021-01-28 13:09:56 +0530 | [diff] [blame] | 897 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 898 | if ( |
| 899 | self.doctype == "Purchase Invoice" and self.get("update_stock") |
| 900 | ) or self.doctype == "Purchase Receipt": |
| 901 | for item in self.get("items"): |
Deepesh Garg | b4be292 | 2021-01-28 13:09:56 +0530 | [diff] [blame] | 902 | if not item.from_warehouse: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 903 | frappe.throw( |
| 904 | _("Row {0}: From Warehouse is mandatory for internal transfers").format(item.idx) |
| 905 | ) |
Deepesh Garg | b4be292 | 2021-01-28 13:09:56 +0530 | [diff] [blame] | 906 | |
| 907 | def validate_multi_currency(self): |
| 908 | if self.currency != self.company_currency: |
| 909 | frappe.throw(_("Internal transfers can only be done in company's default currency")) |
| 910 | |
| 911 | def validate_packed_items(self): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 912 | if self.doctype in ("Sales Invoice", "Delivery Note Item") and self.get("packed_items"): |
Deepesh Garg | b4be292 | 2021-01-28 13:09:56 +0530 | [diff] [blame] | 913 | frappe.throw(_("Packed Items cannot be transferred internally")) |
| 914 | |
rohitwaghchaure | 8fdc244 | 2024-01-27 21:37:58 +0530 | [diff] [blame] | 915 | def validate_internal_transfer_qty(self): |
| 916 | if self.doctype not in ["Purchase Invoice", "Purchase Receipt"]: |
| 917 | return |
| 918 | |
| 919 | item_wise_transfer_qty = self.get_item_wise_inter_transfer_qty() |
| 920 | if not item_wise_transfer_qty: |
| 921 | return |
| 922 | |
| 923 | item_wise_received_qty = self.get_item_wise_inter_received_qty() |
| 924 | precision = frappe.get_precision(self.doctype + " Item", "qty") |
| 925 | |
| 926 | over_receipt_allowance = frappe.db.get_single_value( |
| 927 | "Stock Settings", "over_delivery_receipt_allowance" |
| 928 | ) |
| 929 | |
| 930 | parent_doctype = { |
| 931 | "Purchase Receipt": "Delivery Note", |
| 932 | "Purchase Invoice": "Sales Invoice", |
| 933 | }.get(self.doctype) |
| 934 | |
| 935 | for key, transferred_qty in item_wise_transfer_qty.items(): |
| 936 | recevied_qty = flt(item_wise_received_qty.get(key), precision) |
| 937 | if over_receipt_allowance: |
| 938 | transferred_qty = transferred_qty + flt( |
| 939 | transferred_qty * over_receipt_allowance / 100, precision |
| 940 | ) |
| 941 | |
| 942 | if recevied_qty > flt(transferred_qty, precision): |
| 943 | frappe.throw( |
| 944 | _("For Item {0} cannot be received more than {1} qty against the {2} {3}").format( |
| 945 | bold(key[1]), |
| 946 | bold(flt(transferred_qty, precision)), |
| 947 | bold(parent_doctype), |
| 948 | get_link_to_form(parent_doctype, self.get("inter_company_reference")), |
| 949 | ) |
| 950 | ) |
| 951 | |
| 952 | def get_item_wise_inter_transfer_qty(self): |
| 953 | reference_field = "inter_company_reference" |
| 954 | if self.doctype == "Purchase Invoice": |
| 955 | reference_field = "inter_company_invoice_reference" |
| 956 | |
| 957 | parent_doctype = { |
| 958 | "Purchase Receipt": "Delivery Note", |
| 959 | "Purchase Invoice": "Sales Invoice", |
| 960 | }.get(self.doctype) |
| 961 | |
| 962 | child_doctype = parent_doctype + " Item" |
| 963 | |
| 964 | parent_tab = frappe.qb.DocType(parent_doctype) |
| 965 | child_tab = frappe.qb.DocType(child_doctype) |
| 966 | |
| 967 | query = ( |
| 968 | frappe.qb.from_(parent_doctype) |
| 969 | .inner_join(child_tab) |
| 970 | .on(child_tab.parent == parent_tab.name) |
| 971 | .select( |
| 972 | child_tab.name, |
| 973 | child_tab.item_code, |
| 974 | child_tab.qty, |
| 975 | ) |
| 976 | .where((parent_tab.name == self.get(reference_field)) & (parent_tab.docstatus == 1)) |
| 977 | ) |
| 978 | |
| 979 | data = query.run(as_dict=True) |
| 980 | item_wise_transfer_qty = defaultdict(float) |
| 981 | for row in data: |
| 982 | item_wise_transfer_qty[(row.name, row.item_code)] += flt(row.qty) |
| 983 | |
| 984 | return item_wise_transfer_qty |
| 985 | |
| 986 | def get_item_wise_inter_received_qty(self): |
| 987 | child_doctype = self.doctype + " Item" |
| 988 | |
| 989 | parent_tab = frappe.qb.DocType(self.doctype) |
| 990 | child_tab = frappe.qb.DocType(child_doctype) |
| 991 | |
| 992 | query = ( |
| 993 | frappe.qb.from_(self.doctype) |
| 994 | .inner_join(child_tab) |
| 995 | .on(child_tab.parent == parent_tab.name) |
| 996 | .select( |
| 997 | child_tab.item_code, |
| 998 | child_tab.qty, |
| 999 | ) |
| 1000 | .where(parent_tab.docstatus < 2) |
| 1001 | ) |
| 1002 | |
| 1003 | if self.doctype == "Purchase Invoice": |
| 1004 | query = query.select( |
| 1005 | child_tab.sales_invoice_item.as_("name"), |
| 1006 | ) |
| 1007 | |
| 1008 | query = query.where( |
| 1009 | parent_tab.inter_company_invoice_reference == self.inter_company_invoice_reference |
| 1010 | ) |
| 1011 | else: |
| 1012 | query = query.select( |
| 1013 | child_tab.delivery_note_item.as_("name"), |
| 1014 | ) |
| 1015 | |
| 1016 | query = query.where(parent_tab.inter_company_reference == self.inter_company_reference) |
| 1017 | |
| 1018 | data = query.run(as_dict=True) |
| 1019 | item_wise_transfer_qty = defaultdict(float) |
| 1020 | for row in data: |
| 1021 | item_wise_transfer_qty[(row.name, row.item_code)] += flt(row.qty) |
| 1022 | |
| 1023 | return item_wise_transfer_qty |
| 1024 | |
marination | fac4035 | 2020-12-07 21:35:49 +0530 | [diff] [blame] | 1025 | def validate_putaway_capacity(self): |
| 1026 | # if over receipt is attempted while 'apply putaway rule' is disabled |
| 1027 | # and if rule was applied on the transaction, validate it. |
marination | 957615b | 2021-01-18 23:47:24 +0530 | [diff] [blame] | 1028 | from erpnext.stock.doctype.putaway_rule.putaway_rule import get_available_putaway_capacity |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1029 | |
| 1030 | valid_doctype = self.doctype in ( |
| 1031 | "Purchase Receipt", |
| 1032 | "Stock Entry", |
| 1033 | "Purchase Invoice", |
| 1034 | "Stock Reconciliation", |
| 1035 | ) |
marination | fac4035 | 2020-12-07 21:35:49 +0530 | [diff] [blame] | 1036 | |
rohitwaghchaure | b966c06 | 2024-02-12 12:49:09 +0530 | [diff] [blame] | 1037 | if not frappe.get_all("Putaway Rule", limit=1): |
| 1038 | return |
| 1039 | |
marination | 957615b | 2021-01-18 23:47:24 +0530 | [diff] [blame] | 1040 | if self.doctype == "Purchase Invoice" and self.get("update_stock") == 0: |
| 1041 | valid_doctype = False |
| 1042 | |
| 1043 | if valid_doctype: |
marination | fac4035 | 2020-12-07 21:35:49 +0530 | [diff] [blame] | 1044 | rule_map = defaultdict(dict) |
| 1045 | for item in self.get("items"): |
marination | 957615b | 2021-01-18 23:47:24 +0530 | [diff] [blame] | 1046 | warehouse_field = "t_warehouse" if self.doctype == "Stock Entry" else "warehouse" |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1047 | rule = frappe.db.get_value( |
| 1048 | "Putaway Rule", |
| 1049 | {"item_code": item.get("item_code"), "warehouse": item.get(warehouse_field)}, |
| 1050 | ["name", "disable"], |
| 1051 | as_dict=True, |
| 1052 | ) |
marination | 957615b | 2021-01-18 23:47:24 +0530 | [diff] [blame] | 1053 | if rule: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1054 | if rule.get("disabled"): |
| 1055 | continue # dont validate for disabled rule |
marination | 957615b | 2021-01-18 23:47:24 +0530 | [diff] [blame] | 1056 | |
| 1057 | if self.doctype == "Stock Reconciliation": |
| 1058 | stock_qty = flt(item.qty) |
| 1059 | else: |
| 1060 | stock_qty = flt(item.transfer_qty) if self.doctype == "Stock Entry" else flt(item.stock_qty) |
| 1061 | |
| 1062 | rule_name = rule.get("name") |
| 1063 | if not rule_map[rule_name]: |
| 1064 | rule_map[rule_name]["warehouse"] = item.get(warehouse_field) |
| 1065 | rule_map[rule_name]["item"] = item.get("item_code") |
| 1066 | rule_map[rule_name]["qty_put"] = 0 |
| 1067 | rule_map[rule_name]["capacity"] = get_available_putaway_capacity(rule_name) |
| 1068 | rule_map[rule_name]["qty_put"] += flt(stock_qty) |
marination | fac4035 | 2020-12-07 21:35:49 +0530 | [diff] [blame] | 1069 | |
| 1070 | for rule, values in rule_map.items(): |
| 1071 | if flt(values["qty_put"]) > flt(values["capacity"]): |
marination | 957615b | 2021-01-18 23:47:24 +0530 | [diff] [blame] | 1072 | message = self.prepare_over_receipt_message(rule, values) |
marination | fac4035 | 2020-12-07 21:35:49 +0530 | [diff] [blame] | 1073 | frappe.throw(msg=message, title=_("Over Receipt")) |
marination | 957615b | 2021-01-18 23:47:24 +0530 | [diff] [blame] | 1074 | |
| 1075 | def prepare_over_receipt_message(self, rule, values): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1076 | message = _( |
| 1077 | "{0} qty of Item {1} is being received into Warehouse {2} with capacity {3}." |
| 1078 | ).format( |
| 1079 | frappe.bold(values["qty_put"]), |
| 1080 | frappe.bold(values["item"]), |
| 1081 | frappe.bold(values["warehouse"]), |
| 1082 | frappe.bold(values["capacity"]), |
| 1083 | ) |
marination | 957615b | 2021-01-18 23:47:24 +0530 | [diff] [blame] | 1084 | message += "<br><br>" |
| 1085 | rule_link = frappe.utils.get_link_to_form("Putaway Rule", rule) |
Ankush Menat | ad6a265 | 2021-04-17 16:50:02 +0530 | [diff] [blame] | 1086 | message += _("Please adjust the qty or edit {0} to proceed.").format(rule_link) |
marination | 957615b | 2021-01-18 23:47:24 +0530 | [diff] [blame] | 1087 | return message |
marination | fac4035 | 2020-12-07 21:35:49 +0530 | [diff] [blame] | 1088 | |
Rohit Waghchaure | 2d5ccc0 | 2023-05-01 21:17:18 +0530 | [diff] [blame] | 1089 | def repost_future_sle_and_gle(self, force=False): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1090 | args = frappe._dict( |
| 1091 | { |
| 1092 | "posting_date": self.posting_date, |
| 1093 | "posting_time": self.posting_time, |
| 1094 | "voucher_type": self.doctype, |
| 1095 | "voucher_no": self.name, |
| 1096 | "company": self.company, |
| 1097 | } |
| 1098 | ) |
Ankush Menat | 3638fbf | 2022-03-01 18:17:14 +0530 | [diff] [blame] | 1099 | |
Rohit Waghchaure | 6e661e7 | 2023-05-16 16:23:52 +0530 | [diff] [blame] | 1100 | if self.docstatus == 2: |
| 1101 | force = True |
| 1102 | |
Rohit Waghchaure | 2d5ccc0 | 2023-05-01 21:17:18 +0530 | [diff] [blame] | 1103 | if force or future_sle_exists(args) or repost_required_for_queue(self): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1104 | item_based_reposting = cint( |
| 1105 | frappe.db.get_single_value("Stock Reposting Settings", "item_based_reposting") |
| 1106 | ) |
Ankush Menat | 45dd46b | 2021-11-02 10:50:52 +0530 | [diff] [blame] | 1107 | if item_based_reposting: |
| 1108 | create_item_wise_repost_entries(voucher_type=self.doctype, voucher_no=self.name) |
| 1109 | else: |
| 1110 | create_repost_item_valuation_entry(args) |
| 1111 | |
Sagar Sharma | f467394 | 2022-08-21 21:26:06 +0530 | [diff] [blame] | 1112 | def add_gl_entry( |
| 1113 | self, |
| 1114 | gl_entries, |
| 1115 | account, |
| 1116 | cost_center, |
| 1117 | debit, |
| 1118 | credit, |
| 1119 | remarks, |
| 1120 | against_account, |
| 1121 | debit_in_account_currency=None, |
| 1122 | credit_in_account_currency=None, |
| 1123 | account_currency=None, |
| 1124 | project=None, |
| 1125 | voucher_detail_no=None, |
| 1126 | item=None, |
| 1127 | posting_date=None, |
| 1128 | ): |
| 1129 | |
| 1130 | gl_entry = { |
| 1131 | "account": account, |
| 1132 | "cost_center": cost_center, |
| 1133 | "debit": debit, |
| 1134 | "credit": credit, |
| 1135 | "against": against_account, |
| 1136 | "remarks": remarks, |
| 1137 | } |
| 1138 | |
| 1139 | if voucher_detail_no: |
| 1140 | gl_entry.update({"voucher_detail_no": voucher_detail_no}) |
| 1141 | |
| 1142 | if debit_in_account_currency: |
| 1143 | gl_entry.update({"debit_in_account_currency": debit_in_account_currency}) |
| 1144 | |
| 1145 | if credit_in_account_currency: |
| 1146 | gl_entry.update({"credit_in_account_currency": credit_in_account_currency}) |
| 1147 | |
| 1148 | if posting_date: |
| 1149 | gl_entry.update({"posting_date": posting_date}) |
| 1150 | |
| 1151 | gl_entries.append(self.get_gl_dict(gl_entry, item=item)) |
| 1152 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1153 | |
Deepesh Garg | 2e52a63 | 2023-06-04 19:20:28 +0530 | [diff] [blame] | 1154 | @frappe.whitelist() |
Deepesh Garg | 0e68da5 | 2023-06-22 15:43:32 +0530 | [diff] [blame] | 1155 | def show_accounting_ledger_preview(company, doctype, docname): |
Smit Vora | 77cc91d | 2023-10-19 22:35:55 +0530 | [diff] [blame] | 1156 | filters = frappe._dict(company=company, include_dimensions=1) |
Deepesh Garg | 0e68da5 | 2023-06-22 15:43:32 +0530 | [diff] [blame] | 1157 | doc = frappe.get_doc(doctype, docname) |
Smit Vora | 77cc91d | 2023-10-19 22:35:55 +0530 | [diff] [blame] | 1158 | doc.run_method("before_gl_preview") |
Deepesh Garg | 0e68da5 | 2023-06-22 15:43:32 +0530 | [diff] [blame] | 1159 | |
| 1160 | gl_columns, gl_data = get_accounting_ledger_preview(doc, filters) |
| 1161 | |
Deepesh Garg | d9e7bc5 | 2023-06-22 16:07:32 +0530 | [diff] [blame] | 1162 | frappe.db.rollback() |
Deepesh Garg | 0e68da5 | 2023-06-22 15:43:32 +0530 | [diff] [blame] | 1163 | |
| 1164 | return {"gl_columns": gl_columns, "gl_data": gl_data} |
| 1165 | |
| 1166 | |
| 1167 | @frappe.whitelist() |
| 1168 | def show_stock_ledger_preview(company, doctype, docname): |
Smit Vora | 77cc91d | 2023-10-19 22:35:55 +0530 | [diff] [blame] | 1169 | filters = frappe._dict(company=company) |
Deepesh Garg | 2e52a63 | 2023-06-04 19:20:28 +0530 | [diff] [blame] | 1170 | doc = frappe.get_doc(doctype, docname) |
Smit Vora | 77cc91d | 2023-10-19 22:35:55 +0530 | [diff] [blame] | 1171 | doc.run_method("before_sl_preview") |
Deepesh Garg | e30c3ea | 2023-06-12 11:46:51 +0530 | [diff] [blame] | 1172 | |
Deepesh Garg | 011ac13 | 2023-06-12 18:42:49 +0530 | [diff] [blame] | 1173 | sl_columns, sl_data = get_stock_ledger_preview(doc, filters) |
Deepesh Garg | e30c3ea | 2023-06-12 11:46:51 +0530 | [diff] [blame] | 1174 | |
Deepesh Garg | d9e7bc5 | 2023-06-22 16:07:32 +0530 | [diff] [blame] | 1175 | frappe.db.rollback() |
Deepesh Garg | e30c3ea | 2023-06-12 11:46:51 +0530 | [diff] [blame] | 1176 | |
Deepesh Garg | 2e52a63 | 2023-06-04 19:20:28 +0530 | [diff] [blame] | 1177 | return { |
Deepesh Garg | 011ac13 | 2023-06-12 18:42:49 +0530 | [diff] [blame] | 1178 | "sl_columns": sl_columns, |
| 1179 | "sl_data": sl_data, |
Deepesh Garg | 2e52a63 | 2023-06-04 19:20:28 +0530 | [diff] [blame] | 1180 | } |
| 1181 | |
| 1182 | |
Deepesh Garg | 011ac13 | 2023-06-12 18:42:49 +0530 | [diff] [blame] | 1183 | def get_accounting_ledger_preview(doc, filters): |
| 1184 | from erpnext.accounts.report.general_ledger.general_ledger import get_columns as get_gl_columns |
| 1185 | |
| 1186 | gl_columns, gl_data = [], [] |
| 1187 | fields = [ |
| 1188 | "posting_date", |
| 1189 | "account", |
| 1190 | "debit", |
| 1191 | "credit", |
| 1192 | "against", |
| 1193 | "party", |
| 1194 | "party_type", |
Deepesh Garg | 0e68da5 | 2023-06-22 15:43:32 +0530 | [diff] [blame] | 1195 | "cost_center", |
Deepesh Garg | 011ac13 | 2023-06-12 18:42:49 +0530 | [diff] [blame] | 1196 | "against_voucher_type", |
| 1197 | "against_voucher", |
| 1198 | ] |
| 1199 | |
| 1200 | doc.docstatus = 1 |
Deepesh Garg | 0e68da5 | 2023-06-22 15:43:32 +0530 | [diff] [blame] | 1201 | |
| 1202 | if doc.get("update_stock") or doc.doctype in ("Purchase Receipt", "Delivery Note"): |
| 1203 | doc.update_stock_ledger() |
| 1204 | |
Deepesh Garg | 011ac13 | 2023-06-12 18:42:49 +0530 | [diff] [blame] | 1205 | doc.make_gl_entries() |
| 1206 | columns = get_gl_columns(filters) |
| 1207 | gl_entries = get_gl_entries_for_preview(doc.doctype, doc.name, fields) |
| 1208 | |
| 1209 | gl_columns = get_columns(columns, fields) |
| 1210 | gl_data = get_data(fields, gl_entries) |
| 1211 | |
| 1212 | return gl_columns, gl_data |
| 1213 | |
| 1214 | |
| 1215 | def get_stock_ledger_preview(doc, filters): |
| 1216 | from erpnext.stock.report.stock_ledger.stock_ledger import get_columns as get_sl_columns |
| 1217 | |
| 1218 | sl_columns, sl_data = [], [] |
| 1219 | fields = [ |
| 1220 | "item_code", |
| 1221 | "stock_uom", |
| 1222 | "actual_qty", |
| 1223 | "qty_after_transaction", |
| 1224 | "warehouse", |
| 1225 | "incoming_rate", |
| 1226 | "valuation_rate", |
| 1227 | "stock_value", |
| 1228 | "stock_value_difference", |
| 1229 | ] |
| 1230 | columns_fields = [ |
| 1231 | "item_code", |
| 1232 | "stock_uom", |
| 1233 | "in_qty", |
| 1234 | "out_qty", |
| 1235 | "qty_after_transaction", |
| 1236 | "warehouse", |
| 1237 | "incoming_rate", |
Deepesh Garg | 0e68da5 | 2023-06-22 15:43:32 +0530 | [diff] [blame] | 1238 | "in_out_rate", |
Deepesh Garg | 011ac13 | 2023-06-12 18:42:49 +0530 | [diff] [blame] | 1239 | "stock_value", |
| 1240 | "stock_value_difference", |
| 1241 | ] |
| 1242 | |
Deepesh Garg | 0e68da5 | 2023-06-22 15:43:32 +0530 | [diff] [blame] | 1243 | if doc.get("update_stock") or doc.doctype in ("Purchase Receipt", "Delivery Note"): |
Deepesh Garg | 011ac13 | 2023-06-12 18:42:49 +0530 | [diff] [blame] | 1244 | doc.docstatus = 1 |
| 1245 | doc.update_stock_ledger() |
| 1246 | columns = get_sl_columns(filters) |
| 1247 | sl_entries = get_sl_entries_for_preview(doc.doctype, doc.name, fields) |
| 1248 | |
| 1249 | sl_columns = get_columns(columns, columns_fields) |
| 1250 | sl_data = get_data(columns_fields, sl_entries) |
| 1251 | |
| 1252 | return sl_columns, sl_data |
| 1253 | |
| 1254 | |
| 1255 | def get_sl_entries_for_preview(doctype, docname, fields): |
| 1256 | sl_entries = frappe.get_all( |
| 1257 | "Stock Ledger Entry", filters={"voucher_type": doctype, "voucher_no": docname}, fields=fields |
| 1258 | ) |
| 1259 | |
| 1260 | for entry in sl_entries: |
| 1261 | if entry.actual_qty > 0: |
| 1262 | entry["in_qty"] = entry.actual_qty |
| 1263 | entry["out_qty"] = 0 |
| 1264 | else: |
| 1265 | entry["out_qty"] = abs(entry.actual_qty) |
| 1266 | entry["in_qty"] = 0 |
| 1267 | |
Deepesh Garg | 0e68da5 | 2023-06-22 15:43:32 +0530 | [diff] [blame] | 1268 | entry["in_out_rate"] = entry["valuation_rate"] |
| 1269 | |
Deepesh Garg | 011ac13 | 2023-06-12 18:42:49 +0530 | [diff] [blame] | 1270 | return sl_entries |
| 1271 | |
| 1272 | |
| 1273 | def get_gl_entries_for_preview(doctype, docname, fields): |
Deepesh Garg | e30c3ea | 2023-06-12 11:46:51 +0530 | [diff] [blame] | 1274 | return frappe.get_all( |
Deepesh Garg | 011ac13 | 2023-06-12 18:42:49 +0530 | [diff] [blame] | 1275 | "GL Entry", filters={"voucher_type": doctype, "voucher_no": docname}, fields=fields |
Deepesh Garg | e30c3ea | 2023-06-12 11:46:51 +0530 | [diff] [blame] | 1276 | ) |
| 1277 | |
| 1278 | |
Deepesh Garg | 011ac13 | 2023-06-12 18:42:49 +0530 | [diff] [blame] | 1279 | def get_columns(raw_columns, fields): |
Deepesh Garg | e30c3ea | 2023-06-12 11:46:51 +0530 | [diff] [blame] | 1280 | return [ |
Deepesh Garg | 011ac13 | 2023-06-12 18:42:49 +0530 | [diff] [blame] | 1281 | {"name": d.get("label"), "editable": False, "width": 110} |
Deepesh Garg | e30c3ea | 2023-06-12 11:46:51 +0530 | [diff] [blame] | 1282 | for d in raw_columns |
Deepesh Garg | 011ac13 | 2023-06-12 18:42:49 +0530 | [diff] [blame] | 1283 | if not d.get("hidden") and d.get("fieldname") in fields |
Deepesh Garg | e30c3ea | 2023-06-12 11:46:51 +0530 | [diff] [blame] | 1284 | ] |
| 1285 | |
| 1286 | |
| 1287 | def get_data(raw_columns, raw_data): |
| 1288 | datatable_data = [] |
| 1289 | for row in raw_data: |
| 1290 | data_row = [] |
| 1291 | for column in raw_columns: |
Deepesh Garg | 011ac13 | 2023-06-12 18:42:49 +0530 | [diff] [blame] | 1292 | data_row.append(row.get(column) or "") |
Deepesh Garg | e30c3ea | 2023-06-12 11:46:51 +0530 | [diff] [blame] | 1293 | |
| 1294 | datatable_data.append(data_row) |
| 1295 | |
| 1296 | return datatable_data |
| 1297 | |
| 1298 | |
Ankush Menat | 3638fbf | 2022-03-01 18:17:14 +0530 | [diff] [blame] | 1299 | def repost_required_for_queue(doc: StockController) -> bool: |
| 1300 | """check if stock document contains repeated item-warehouse with queue based valuation. |
| 1301 | |
| 1302 | if queue exists for repeated items then SLEs need to reprocessed in background again. |
| 1303 | """ |
| 1304 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1305 | consuming_sles = frappe.db.get_all( |
| 1306 | "Stock Ledger Entry", |
Ankush Menat | 3638fbf | 2022-03-01 18:17:14 +0530 | [diff] [blame] | 1307 | filters={ |
| 1308 | "voucher_type": doc.doctype, |
| 1309 | "voucher_no": doc.name, |
| 1310 | "actual_qty": ("<", 0), |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1311 | "is_cancelled": 0, |
Ankush Menat | 3638fbf | 2022-03-01 18:17:14 +0530 | [diff] [blame] | 1312 | }, |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1313 | fields=["item_code", "warehouse", "stock_queue"], |
Ankush Menat | 3638fbf | 2022-03-01 18:17:14 +0530 | [diff] [blame] | 1314 | ) |
| 1315 | item_warehouses = [(sle.item_code, sle.warehouse) for sle in consuming_sles] |
| 1316 | |
| 1317 | unique_item_warehouses = set(item_warehouses) |
| 1318 | |
| 1319 | if len(unique_item_warehouses) == len(item_warehouses): |
| 1320 | return False |
| 1321 | |
| 1322 | for sle in consuming_sles: |
| 1323 | if sle.stock_queue != "[]": # using FIFO/LIFO valuation |
| 1324 | return True |
| 1325 | return False |
| 1326 | |
Nabin Hait | 19f8fa5 | 2021-02-22 22:27:22 +0530 | [diff] [blame] | 1327 | |
Rohan Bansal | 7f8b95e | 2021-04-14 14:12:03 +0530 | [diff] [blame] | 1328 | @frappe.whitelist() |
| 1329 | def make_quality_inspections(doctype, docname, items): |
Rohan Bansal | a06ec03 | 2021-06-02 14:55:31 +0530 | [diff] [blame] | 1330 | if isinstance(items, str): |
| 1331 | items = json.loads(items) |
Rohan Bansal | 7f8b95e | 2021-04-14 14:12:03 +0530 | [diff] [blame] | 1332 | |
Rohan Bansal | a06ec03 | 2021-06-02 14:55:31 +0530 | [diff] [blame] | 1333 | inspections = [] |
Rohan Bansal | 7f8b95e | 2021-04-14 14:12:03 +0530 | [diff] [blame] | 1334 | for item in items: |
Rohan Bansal | 1cdf5a0 | 2021-05-26 14:42:15 +0530 | [diff] [blame] | 1335 | if flt(item.get("sample_size")) > flt(item.get("qty")): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1336 | frappe.throw( |
| 1337 | _( |
| 1338 | "{item_name}'s Sample Size ({sample_size}) cannot be greater than the Accepted Quantity ({accepted_quantity})" |
| 1339 | ).format( |
| 1340 | item_name=item.get("item_name"), |
| 1341 | sample_size=item.get("sample_size"), |
| 1342 | accepted_quantity=item.get("qty"), |
| 1343 | ) |
| 1344 | ) |
Rohan Bansal | 7f8b95e | 2021-04-14 14:12:03 +0530 | [diff] [blame] | 1345 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1346 | quality_inspection = frappe.get_doc( |
| 1347 | { |
| 1348 | "doctype": "Quality Inspection", |
| 1349 | "inspection_type": "Incoming", |
| 1350 | "inspected_by": frappe.session.user, |
| 1351 | "reference_type": doctype, |
| 1352 | "reference_name": docname, |
| 1353 | "item_code": item.get("item_code"), |
| 1354 | "description": item.get("description"), |
| 1355 | "sample_size": flt(item.get("sample_size")), |
| 1356 | "item_serial_no": item.get("serial_no").split("\n")[0] if item.get("serial_no") else None, |
| 1357 | "batch_no": item.get("batch_no"), |
| 1358 | } |
| 1359 | ).insert() |
Rohan Bansal | 7f8b95e | 2021-04-14 14:12:03 +0530 | [diff] [blame] | 1360 | quality_inspection.save() |
Rohan Bansal | 1cdf5a0 | 2021-05-26 14:42:15 +0530 | [diff] [blame] | 1361 | inspections.append(quality_inspection.name) |
Rohan Bansal | 7f8b95e | 2021-04-14 14:12:03 +0530 | [diff] [blame] | 1362 | |
Rohan Bansal | 1cdf5a0 | 2021-05-26 14:42:15 +0530 | [diff] [blame] | 1363 | return inspections |
Rohan Bansal | 7f8b95e | 2021-04-14 14:12:03 +0530 | [diff] [blame] | 1364 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1365 | |
Nabin Hait | b99c77b | 2020-12-25 18:12:35 +0530 | [diff] [blame] | 1366 | def is_reposting_pending(): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1367 | return frappe.db.exists( |
| 1368 | "Repost Item Valuation", {"docstatus": 1, "status": ["in", ["Queued", "In Progress"]]} |
| 1369 | ) |
| 1370 | |
Nabin Hait | b99c77b | 2020-12-25 18:12:35 +0530 | [diff] [blame] | 1371 | |
Rohit Waghchaure | 8520edc | 2021-06-15 10:21:44 +0530 | [diff] [blame] | 1372 | def future_sle_exists(args, sl_entries=None): |
| 1373 | key = (args.voucher_type, args.voucher_no) |
Rohit Waghchaure | d9dd64b | 2023-04-14 13:00:12 +0530 | [diff] [blame] | 1374 | if not hasattr(frappe.local, "future_sle"): |
| 1375 | frappe.local.future_sle = {} |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 1376 | |
Rohit Waghchaure | 8520edc | 2021-06-15 10:21:44 +0530 | [diff] [blame] | 1377 | if validate_future_sle_not_exists(args, key, sl_entries): |
| 1378 | return False |
| 1379 | elif get_cached_data(args, key): |
| 1380 | return True |
| 1381 | |
| 1382 | if not sl_entries: |
| 1383 | sl_entries = get_sle_entries_against_voucher(args) |
| 1384 | if not sl_entries: |
| 1385 | return |
| 1386 | |
| 1387 | or_conditions = get_conditions_to_validate_future_sle(sl_entries) |
| 1388 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1389 | data = frappe.db.sql( |
| 1390 | """ |
Rohit Waghchaure | 8520edc | 2021-06-15 10:21:44 +0530 | [diff] [blame] | 1391 | select item_code, warehouse, count(name) as total_row |
Deepesh Garg | 6f107da | 2021-10-12 20:15:55 +0530 | [diff] [blame] | 1392 | from `tabStock Ledger Entry` force index (item_warehouse) |
Rohit Waghchaure | 8520edc | 2021-06-15 10:21:44 +0530 | [diff] [blame] | 1393 | where |
| 1394 | ({}) |
| 1395 | and timestamp(posting_date, posting_time) |
| 1396 | >= timestamp(%(posting_date)s, %(posting_time)s) |
| 1397 | and voucher_no != %(voucher_no)s |
| 1398 | and is_cancelled = 0 |
| 1399 | GROUP BY |
| 1400 | item_code, warehouse |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1401 | """.format( |
| 1402 | " or ".join(or_conditions) |
| 1403 | ), |
| 1404 | args, |
| 1405 | as_dict=1, |
| 1406 | ) |
Rohit Waghchaure | 8520edc | 2021-06-15 10:21:44 +0530 | [diff] [blame] | 1407 | |
| 1408 | for d in data: |
| 1409 | frappe.local.future_sle[key][(d.item_code, d.warehouse)] = d.total_row |
| 1410 | |
| 1411 | return len(data) |
| 1412 | |
Rohit Waghchaure | 8520edc | 2021-06-15 10:21:44 +0530 | [diff] [blame] | 1413 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1414 | def validate_future_sle_not_exists(args, key, sl_entries=None): |
| 1415 | item_key = "" |
| 1416 | if args.get("item_code"): |
| 1417 | item_key = (args.get("item_code"), args.get("warehouse")) |
| 1418 | |
| 1419 | if not sl_entries and hasattr(frappe.local, "future_sle"): |
Rohit Waghchaure | d9dd64b | 2023-04-14 13:00:12 +0530 | [diff] [blame] | 1420 | if key not in frappe.local.future_sle: |
| 1421 | return False |
| 1422 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1423 | if not frappe.local.future_sle.get(key) or ( |
| 1424 | item_key and item_key not in frappe.local.future_sle.get(key) |
| 1425 | ): |
Rohit Waghchaure | 8520edc | 2021-06-15 10:21:44 +0530 | [diff] [blame] | 1426 | return True |
| 1427 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1428 | |
Rohit Waghchaure | 8520edc | 2021-06-15 10:21:44 +0530 | [diff] [blame] | 1429 | def get_cached_data(args, key): |
Rohit Waghchaure | 8520edc | 2021-06-15 10:21:44 +0530 | [diff] [blame] | 1430 | if key not in frappe.local.future_sle: |
Rohit Waghchaure | 2d5ccc0 | 2023-05-01 21:17:18 +0530 | [diff] [blame] | 1431 | frappe.local.future_sle[key] = frappe._dict({}) |
Rohit Waghchaure | 8520edc | 2021-06-15 10:21:44 +0530 | [diff] [blame] | 1432 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1433 | if args.get("item_code"): |
| 1434 | item_key = (args.get("item_code"), args.get("warehouse")) |
Rohit Waghchaure | 8520edc | 2021-06-15 10:21:44 +0530 | [diff] [blame] | 1435 | count = frappe.local.future_sle[key].get(item_key) |
| 1436 | |
| 1437 | return True if (count or count == 0) else False |
| 1438 | else: |
| 1439 | return frappe.local.future_sle[key] |
| 1440 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1441 | |
Rohit Waghchaure | 8520edc | 2021-06-15 10:21:44 +0530 | [diff] [blame] | 1442 | def get_sle_entries_against_voucher(args): |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1443 | return frappe.get_all( |
| 1444 | "Stock Ledger Entry", |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 1445 | filters={"voucher_type": args.voucher_type, "voucher_no": args.voucher_no}, |
| 1446 | fields=["item_code", "warehouse"], |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1447 | order_by="creation asc", |
| 1448 | ) |
| 1449 | |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 1450 | |
Rohit Waghchaure | 8520edc | 2021-06-15 10:21:44 +0530 | [diff] [blame] | 1451 | def get_conditions_to_validate_future_sle(sl_entries): |
Sagar Vora | 868c0bf | 2021-03-27 16:10:20 +0530 | [diff] [blame] | 1452 | warehouse_items_map = {} |
| 1453 | for entry in sl_entries: |
| 1454 | if entry.warehouse not in warehouse_items_map: |
| 1455 | warehouse_items_map[entry.warehouse] = set() |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 1456 | |
Sagar Vora | 868c0bf | 2021-03-27 16:10:20 +0530 | [diff] [blame] | 1457 | warehouse_items_map[entry.warehouse].add(entry.item_code) |
| 1458 | |
| 1459 | or_conditions = [] |
| 1460 | for warehouse, items in warehouse_items_map.items(): |
| 1461 | or_conditions.append( |
Noah Jacob | b5a1491 | 2021-06-15 12:44:04 +0530 | [diff] [blame] | 1462 | f"""warehouse = {frappe.db.escape(warehouse)} |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1463 | and item_code in ({', '.join(frappe.db.escape(item) for item in items)})""" |
| 1464 | ) |
Sagar Vora | 868c0bf | 2021-03-27 16:10:20 +0530 | [diff] [blame] | 1465 | |
Rohit Waghchaure | 8520edc | 2021-06-15 10:21:44 +0530 | [diff] [blame] | 1466 | return or_conditions |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 1467 | |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1468 | |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 1469 | def create_repost_item_valuation_entry(args): |
| 1470 | args = frappe._dict(args) |
| 1471 | repost_entry = frappe.new_doc("Repost Item Valuation") |
| 1472 | repost_entry.based_on = args.based_on |
| 1473 | if not args.based_on: |
Ankush Menat | 494bd9e | 2022-03-28 18:52:46 +0530 | [diff] [blame] | 1474 | repost_entry.based_on = "Transaction" if args.voucher_no else "Item and Warehouse" |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 1475 | repost_entry.voucher_type = args.voucher_type |
| 1476 | repost_entry.voucher_no = args.voucher_no |
| 1477 | repost_entry.item_code = args.item_code |
| 1478 | repost_entry.warehouse = args.warehouse |
| 1479 | repost_entry.posting_date = args.posting_date |
| 1480 | repost_entry.posting_time = args.posting_time |
| 1481 | repost_entry.company = args.company |
| 1482 | repost_entry.allow_zero_rate = args.allow_zero_rate |
| 1483 | repost_entry.flags.ignore_links = True |
Ankush Menat | aa024fc | 2021-11-18 12:51:26 +0530 | [diff] [blame] | 1484 | repost_entry.flags.ignore_permissions = True |
Nabin Hait | a77b8c9 | 2020-12-21 14:45:50 +0530 | [diff] [blame] | 1485 | repost_entry.save() |
Sagar Vora | 868c0bf | 2021-03-27 16:10:20 +0530 | [diff] [blame] | 1486 | repost_entry.submit() |
Ankush Menat | 6dc9b82 | 2021-10-28 15:53:18 +0530 | [diff] [blame] | 1487 | |
| 1488 | |
| 1489 | def create_item_wise_repost_entries(voucher_type, voucher_no, allow_zero_rate=False): |
| 1490 | """Using a voucher create repost item valuation records for all item-warehouse pairs.""" |
| 1491 | |
Ankush Menat | d220e08 | 2021-10-28 17:47:00 +0530 | [diff] [blame] | 1492 | stock_ledger_entries = get_items_to_be_repost(voucher_type, voucher_no) |
| 1493 | |
Ankush Menat | 6dc9b82 | 2021-10-28 15:53:18 +0530 | [diff] [blame] | 1494 | distinct_item_warehouses = set() |
Ankush Menat | 6dc9b82 | 2021-10-28 15:53:18 +0530 | [diff] [blame] | 1495 | repost_entries = [] |
| 1496 | |
| 1497 | for sle in stock_ledger_entries: |
| 1498 | item_wh = (sle.item_code, sle.warehouse) |
| 1499 | if item_wh in distinct_item_warehouses: |
| 1500 | continue |
| 1501 | distinct_item_warehouses.add(item_wh) |
| 1502 | |
| 1503 | repost_entry = frappe.new_doc("Repost Item Valuation") |
| 1504 | repost_entry.based_on = "Item and Warehouse" |
Ankush Menat | 6dc9b82 | 2021-10-28 15:53:18 +0530 | [diff] [blame] | 1505 | |
| 1506 | repost_entry.item_code = sle.item_code |
| 1507 | repost_entry.warehouse = sle.warehouse |
| 1508 | repost_entry.posting_date = sle.posting_date |
| 1509 | repost_entry.posting_time = sle.posting_time |
| 1510 | repost_entry.allow_zero_rate = allow_zero_rate |
| 1511 | repost_entry.flags.ignore_links = True |
Ankush Menat | 0a2964d | 2021-11-24 15:55:31 +0530 | [diff] [blame] | 1512 | repost_entry.flags.ignore_permissions = True |
Ankush Menat | 6dc9b82 | 2021-10-28 15:53:18 +0530 | [diff] [blame] | 1513 | repost_entry.submit() |
| 1514 | repost_entries.append(repost_entry) |
| 1515 | |
| 1516 | return repost_entries |