From e64f4f632d8bd830f1d4e7828b4d8139e539aa33 Mon Sep 17 00:00:00 2001 From: Pravesh Sharma Date: Mon, 4 Dec 2023 17:19:05 +0530 Subject: [PATCH] 1. Fixed an issue where all rows and filter rows buttons of object explorer toolbar were disabled for views and other supported nodes. #7011 2. Fixed an issue where View/Edit data -> Filter dialog was not editable for catalog objects. --- web/pgadmin/static/js/UtilityView.jsx | 3 +-- .../static/js/helpers/ObjectExplorerToolbar.jsx | 12 ++++++------ 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/web/pgadmin/static/js/UtilityView.jsx b/web/pgadmin/static/js/UtilityView.jsx index 6ea221ffc..e2e0a3d5e 100644 --- a/web/pgadmin/static/js/UtilityView.jsx +++ b/web/pgadmin/static/js/UtilityView.jsx @@ -69,7 +69,6 @@ function UtilityViewContent({panelId, schema, treeNodeInfo, actionType, formType const pgAdmin = usePgAdmin(); const serverInfo = treeNodeInfo && ('server' in treeNodeInfo) && pgAdmin.Browser.serverInfo && pgAdmin.Browser.serverInfo[treeNodeInfo.server._id]; - const inCatalog = treeNodeInfo && ('catalog' in treeNodeInfo); const api = getApiInstance(); const url = ()=>{ return urlBase; @@ -144,7 +143,7 @@ function UtilityViewContent({panelId, schema, treeNodeInfo, actionType, formType type: serverInfo.server_type, version: serverInfo.version, }: undefined, - inCatalog: inCatalog, + inCatalog: false, }; let initData = ()=>new Promise((resolve, reject)=>{ diff --git a/web/pgadmin/static/js/helpers/ObjectExplorerToolbar.jsx b/web/pgadmin/static/js/helpers/ObjectExplorerToolbar.jsx index b6b0feabf..91c0ddcfe 100644 --- a/web/pgadmin/static/js/helpers/ObjectExplorerToolbar.jsx +++ b/web/pgadmin/static/js/helpers/ObjectExplorerToolbar.jsx @@ -23,8 +23,8 @@ ToolbarButton.propTypes = { export default function ObjectExplorerToolbar() { const [menus, setMenus] = useState({ 'query_tool': undefined, - 'view_all_rows_context_table': undefined, - 'view_filtered_rows_context_table': undefined, + 'view_all_rows_context': undefined, + 'view_filtered_rows_context': undefined, 'search_objects': undefined, 'psql': undefined, }); @@ -42,8 +42,8 @@ export default function ObjectExplorerToolbar() { setMenus({ 'query_tool': toolsMenus?.find((m)=>(m.name=='query_tool')), - 'view_all_rows_context_table': viewMenus?.find((m)=>(m.name=='view_all_rows_context_table')), - 'view_filtered_rows_context_table': viewMenus?.find((m)=>(m.name=='view_filtered_rows_context_table')), + 'view_all_rows_context': viewMenus?.find((m)=>(m.name=='view_all_rows_context_' + m.node)), + 'view_filtered_rows_context': viewMenus?.find((m)=>(m.name=='view_filtered_rows_context_' + m.node)), 'search_objects': toolsMenus?.find((m)=>(m.name=='search_objects')), 'psql': toolsMenus?.find((m)=>(m.name=='psql')) }); @@ -61,8 +61,8 @@ export default function ObjectExplorerToolbar() { } menuItem={menus['query_tool']} /> - } menuItem={menus['view_all_rows_context_table']} /> - } menuItem={menus['view_filtered_rows_context_table']} /> + } menuItem={menus['view_all_rows_context']} /> + } menuItem={menus['view_filtered_rows_context']} /> } menuItem={menus['search_objects']} /> {!_.isUndefined(menus['psql']) && } menuItem={menus['psql']} />}