fix: don't access db unnecessarily

frappe · Sep 19, 2024 · 40c3cea · 40c3cea
1 parent 1807ca8
commit 40c3cea
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/hrms/overrides/employee_payment_entry.py b/hrms/overrides/employee_payment_entry.py
@@ -65,7 +65,10 @@ def set_missing_ref_details(
 						continue
 
 					if field == "exchange_rate" or not d.get(field) or force:
-						d.db_set(field, value)
+						if self.get("_action") in ("submit", "cancel"):
+							d.db_set(field, value)
+						else:
+							d.set(field, value)
 
 
 @frappe.whitelist()