nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-04-09 09:55:08 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: remove unnecessary param, and use value from self

Browse Source 
(cherry picked from commit 98dfd64f63)
This commit is contained in:
Smit Vora
2026-04-07 13:17:22 +05:30
committed by Mergify
parent 6cebea314d
commit 7bef9542d4
1 changed files with 8 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
erpnext/stock/doctype/stock_entry/stock_entry.py
View File

@@ -345,9 +345,7 @@ class StockEntry(StockController, SubcontractingInwardController):
scale_factor = flt(self.fg_completed_qty) / source_fg_qty if source_fg_qty else 0
bundle_data = get_voucher_wise_serial_batch_from_bundle(voucher_no=[self.source_stock_entry])
source_rows_by_name = {
r.name: r for r in self.get_items_from_manufacture_stock_entry(self.source_stock_entry)
}
source_rows_by_name = {r.name: r for r in self.get_items_from_manufacture_stock_entry()}
for row in self.items:
if not row.ste_detail:
@@ -2378,7 +2376,6 @@ class StockEntry(StockController, SubcontractingInwardController):
self._append_disassembly_row_from_source(
disassemble_qty=disassemble_qty,
scale_factor=scale_factor,
source_stock_entry=self.source_stock_entry,
)
def _add_items_for_disassembly_from_work_order(self):
@@ -2399,8 +2396,8 @@ class StockEntry(StockController, SubcontractingInwardController):
scale_factor=scale_factor,
)
def _append_disassembly_row_from_source(self, disassemble_qty, scale_factor, source_stock_entry=None):
for source_row in self.get_items_from_manufacture_stock_entry(self.source_stock_entry):
def _append_disassembly_row_from_source(self, disassemble_qty, scale_factor):
for source_row in self.get_items_from_manufacture_stock_entry():
if source_row.is_finished_item:
qty = disassemble_qty
s_warehouse = self.from_warehouse or source_row.t_warehouse
@@ -2436,10 +2433,10 @@ class StockEntry(StockController, SubcontractingInwardController):
"use_serial_batch_fields": 1 if (source_row.batch_no or source_row.serial_no) else 0,
}
if source_stock_entry:
if self.source_stock_entry:
item.update(
{
"against_stock_entry": source_stock_entry,
"against_stock_entry": self.source_stock_entry,
"ste_detail": source_row.name,
}
)
@@ -2486,7 +2483,7 @@ class StockEntry(StockController, SubcontractingInwardController):
# Finished goods
self.load_items_from_bom()
def get_items_from_manufacture_stock_entry(self, stock_entry=None):
def get_items_from_manufacture_stock_entry(self):
SE = frappe.qb.DocType("Stock Entry")
SED = frappe.qb.DocType("Stock Entry Detail")
query = frappe.qb.from_(SED).join(SE).on(SED.parent == SE.name).where(SE.docstatus == 1)
@@ -2511,10 +2508,10 @@ class StockEntry(StockController, SubcontractingInwardController):
SED.bom_no,
]
if stock_entry:
if self.source_stock_entry:
return (
query.select(SED.name, SED.qty, SED.transfer_qty, *common_fields)
.where(SE.name == stock_entry)
.where(SE.name == self.source_stock_entry)
.orderby(SED.idx)
.run(as_dict=True)
)