Re: Required inputs in issue #6138 - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Required inputs in issue #6138
Date
Msg-id CA+OCxoymw-hKw8nYbt-nTi=6h7tB_dRPyaQL-ib2s2uPzcoxTQ@mail.gmail.com
Whole thread Raw
In response to Required inputs in issue #6138  (Nikhil Mohite <nikhil.mohite@enterprisedb.com>)
Responses Re: Required inputs in issue #6138
List pgadmin-hackers


On Wed, 14 Jun 2023 at 13:43, Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:
Hi Hacker,

I am currently working on issue #6138, which was fixed earlier but the fix was not correct so working on it again.
Steps to reproduce the issue:
1. Open View/Edit Data for table1
2. Open the PSQL tool delete the same table1 and recreates the same name table table1 and try to re-execute the query from the already opened View/Edit data panel.
3. pgAdmin throws an error in the Messages tab List index out of range (pgAdmin query to get table columns from Postgres based on table oid which was deleted and the return result is empty which is not handled in pgAdmin).

Approach to fix this issue:
1. If the user re-executes the query from already opened View/Edit data we should fetch the table details by using schema and table name (not using table oid) and show the newly created table records in the result.
(This will only fix an issue with View/Edit Data however if the user switches to properties or some other panel it user will still get an error notification with content "could not find the specified table.")

I think this is fine. The View/Edit data window shows the query with table/schema name after all, so the user should expect it to work. 

--

pgadmin-hackers by date:

Previous
From: Sahil Harpal
Date:
Subject: Re: Pgadmin4 System Stats Extension Design
Next
From: Dave Page
Date:
Subject: Re: Pgadmin4 System Stats Extension Design