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
This commit is contained in:
@@ -83,7 +83,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()
|
||||
@@ -113,9 +115,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",
|
||||
@@ -124,6 +123,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