Skip to content

Commit

Permalink
fix: get Valuation Rate instead of BOM rate
Browse files Browse the repository at this point in the history
  • Loading branch information
s-aga-r committed Aug 28, 2023
1 parent 592c7b5 commit 9d330a1
Showing 1 changed file with 14 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
import erpnext
from erpnext.accounts.utils import get_account_currency
from erpnext.controllers.subcontracting_controller import SubcontractingController
from erpnext.stock.stock_ledger import get_valuation_rate


class SubcontractingReceipt(SubcontractingController):
Expand Down Expand Up @@ -159,6 +160,17 @@ def get_scrap_items(self, recalculate_rate=False):
bom = frappe.get_doc("BOM", item.bom)
for scrap_item in bom.scrap_items:
qty = flt(item.qty) * (flt(scrap_item.stock_qty) / flt(bom.quantity))
rate = (
get_valuation_rate(
scrap_item.item_code,
self.set_warehouse,
self.doctype,
self.name,
currency=erpnext.get_company_currency(self.company),
company=self.company,
)
or scrap_item.rate
)
self.append(
"items",
{
Expand All @@ -169,12 +181,12 @@ def get_scrap_items(self, recalculate_rate=False):
"qty": qty,
"stock_uom": scrap_item.stock_uom,
"recalculate_rate": 0,
"rate": scrap_item.rate,
"rate": rate,
"rm_cost_per_qty": 0,
"service_cost_per_qty": 0,
"additional_cost_per_qty": 0,
"scrap_cost_per_qty": 0,
"amount": qty * scrap_item.rate,
"amount": qty * rate,
"warehouse": self.set_warehouse,
"rejected_warehouse": self.rejected_warehouse,
},
Expand Down

0 comments on commit 9d330a1

Please sign in to comment.