fix: check for session user rather than owner

frappe · Nov 23, 2022 · b482e38 · b482e38
1 parent be15419
commit b482e38
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/erpnext/stock/doctype/repost_item_valuation/repost_item_valuation.py b/erpnext/stock/doctype/repost_item_valuation/repost_item_valuation.py
@@ -36,11 +36,12 @@ def validate_accounts_freeze(self):
 		)
 		if not acc_settings.acc_frozen_upto:
 			return
-		if acc_settings.frozen_accounts_modifier and self.owner in get_users_with_role(
-			acc_settings.frozen_accounts_modifier
-		):
-			return
 		if getdate(self.posting_date) <= getdate(acc_settings.acc_frozen_upto):
+			if acc_settings.frozen_accounts_modifier and frappe.session.user in get_users_with_role(
+				acc_settings.frozen_accounts_modifier
+			):
+				frappe.msgprint(_("Caution: This might alter frozen accounts."))
+				return
 			frappe.throw(
 				_("You cannot repost item valuation before {}").format(acc_settings.acc_frozen_upto)
 			)