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

fix: coerce cost_allocation_per to float in BOM cost allocation

Browse Source
This commit is contained in:
Shllokkk
2026-09-10 22:23:57 +05:30
parent 5d9cbd9755
commit 1918ed6a3c
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
erpnext/manufacturing/doctype/bom/bom.py
View File

@@ -471,7 +471,7 @@ class BOM(WebsiteGenerator):
if item.valuation_type in ("Valuation Rate", "Manual"):
item.cost_allocation_per = 0
own_cost += flt(item.cost)
total_secondary_items_per += item.cost_allocation_per
total_secondary_items_per += flt(item.cost_allocation_per)
if self.cost_allocation_per == 100 and total_secondary_items_per:
self.cost_allocation_per -= total_secondary_items_per
@@ -487,9 +487,9 @@ class BOM(WebsiteGenerator):
)
def validate_total_cost_allocation(self):
total_cost_allocation_per = self.cost_allocation_per
total_cost_allocation_per = flt(self.cost_allocation_per)
for item in self.secondary_items:
total_cost_allocation_per += item.cost_allocation_per
total_cost_allocation_per += flt(item.cost_allocation_per)
if total_cost_allocation_per != 100:
frappe.throw(_("Cost allocation between finished goods and secondary items should equal 100%"))
@@ -1100,7 +1100,7 @@ class BOM(WebsiteGenerator):
for d in self.get("secondary_items"):
if d.valuation_type not in ("Valuation Rate", "Manual"):
d.cost = flt(allocation_basis * (d.cost_allocation_per / 100), precision)
d.cost = flt(allocation_basis * (flt(d.cost_allocation_per) / 100), precision)
d.base_cost = flt(d.cost * self.conversion_rate, precision)
if save:
d.db_update()