From d117e2c08bdb8881e8f0e49b5d47def8a89cf588 Mon Sep 17 00:00:00 2001 From: Ankush Menat Date: Tue, 5 Sep 2023 16:38:09 +0530 Subject: [PATCH] fix!: Last overriden method should be considered --- frappe/api/v2.py | 4 ++-- frappe/handler.py | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/frappe/api/v2.py b/frappe/api/v2.py index 4594d3243c..130072d50e 100644 --- a/frappe/api/v2.py +++ b/frappe/api/v2.py @@ -30,8 +30,8 @@ def handle_rpc_call(method: str, doctype: str | None = None): # Login works implicitly right now. return - for hook in frappe.get_hooks("override_whitelisted_methods", {}).get(method, []): - # override using the first hook + for hook in reversed(frappe.get_hooks("override_whitelisted_methods", {}).get(method, [])): + # override using the last hook method = hook break diff --git a/frappe/handler.py b/frappe/handler.py index fe438d4cae..db99a47a43 100644 --- a/frappe/handler.py +++ b/frappe/handler.py @@ -61,7 +61,7 @@ def handle(): def execute_cmd(cmd, from_async=False): """execute a request as python module""" for hook in reversed(frappe.get_hooks("override_whitelisted_methods", {}).get(cmd, [])): - # override using the first hook + # override using the last hook cmd = hook break