From 346d59ac3266c1382b7b61c5fe1137dd54ab32fe Mon Sep 17 00:00:00 2001 From: Ashesh Vashi Date: Fri, 24 Jul 2026 19:14:23 +0530 Subject: [PATCH] fix: report the correct error message on trigger node.sql failure MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The failure path for the second query (node.sql, resolving the trigger function's node data) reported errormsg=rset — the result dict from the first, already-successful get_function_oid.sql query — instead of res, the actual error from the failed query. Any failure here (permissions, catalog access) surfaced a stale, misleading payload instead of the real cause. --- .../servers/databases/schemas/tables/triggers/__init__.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/pgadmin/browser/server_groups/servers/databases/schemas/tables/triggers/__init__.py b/web/pgadmin/browser/server_groups/servers/databases/schemas/tables/triggers/__init__.py index 5e74f17f72d..9d557788aa3 100644 --- a/web/pgadmin/browser/server_groups/servers/databases/schemas/tables/triggers/__init__.py +++ b/web/pgadmin/browser/server_groups/servers/databases/schemas/tables/triggers/__init__.py @@ -327,7 +327,7 @@ def get_children_nodes(self, manager, **kwargs): ) status, res = self.conn.execute_2darray(sql) if not status: - return internal_server_error(errormsg=rset) + return internal_server_error(errormsg=res) if len(res['rows']) == 0: return gone(gettext(