mirror of
https://github.com/frappe/erpnext.git
synced 2026-04-27 02:28:30 +00:00
fix: recalculate operating costs if workstation type is changed (#54390)
* fix: recalculate operating costs if workstation type is changed
* fix: do not overwrite op costs on every save
(cherry picked from commit 28f3429a54)
Co-authored-by: Mihir Kandoi <kandoimihir@gmail.com>
This commit is contained in:
@@ -82,7 +82,9 @@ class Workstation(Document):
|
||||
)
|
||||
|
||||
def before_save(self):
|
||||
self.set_data_based_on_workstation_type()
|
||||
if self.has_value_changed("workstation_type"):
|
||||
self.set_data_based_on_workstation_type()
|
||||
|
||||
self.set_hour_rate()
|
||||
self.set_total_working_hours()
|
||||
self.disabled_workstation()
|
||||
@@ -112,9 +114,6 @@ class Workstation(Document):
|
||||
|
||||
@frappe.whitelist()
|
||||
def set_data_based_on_workstation_type(self):
|
||||
if self.workstation_costs:
|
||||
return
|
||||
|
||||
if self.workstation_type:
|
||||
data = frappe.get_all(
|
||||
"Workstation Cost",
|
||||
@@ -123,6 +122,9 @@ class Workstation(Document):
|
||||
order_by="idx",
|
||||
)
|
||||
|
||||
if data:
|
||||
self.workstation_costs = []
|
||||
|
||||
for row in data:
|
||||
self.append(
|
||||
"workstation_costs",
|
||||
|
||||
Reference in New Issue
Block a user