public inbox for [email protected]
help / color / mirror / Atom feedFrom: Khushboo Vashi <[email protected]>
To: Dave Page <[email protected]>
Cc: Nikhil Mohite <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: Required inputs in issue #6138
Date: Thu, 15 Jun 2023 15:30:59 +0530
Message-ID: <CAFOhELfKUZdr+VuS-9uMMiJhc8+PSbNbfhH6TnW_TR3r0xqVJA@mail.gmail.com> (raw)
In-Reply-To: <CA+OCxoymw-hKw8nYbt-nTi=6h7tB_dRPyaQL-ib2s2uPzcoxTQ@mail.gmail.com>
References: <CAOBg0AMGqtR9QVkaJuu2x7fccCpFWOzAz56DHAgGdx1-=us7dg@mail.gmail.com>
<CA+OCxoymw-hKw8nYbt-nTi=6h7tB_dRPyaQL-ib2s2uPzcoxTQ@mail.gmail.com>
On Thu, Jun 15, 2023 at 3:11 PM Dave Page <[email protected]> wrote:
>
>
> On Wed, 14 Jun 2023 at 13:43, Nikhil Mohite <
> [email protected]> wrote:
>
>> Hi Hacker,
>>
>> I am currently working on issue #6138
>> <https://github.com/pgadmin-org/pgadmin4/issues/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.
>
> The object for which the user has opened the View data panel is completely
changed. So as per me, it should raise the error object doesn't exist.
The table name is the same, but it is a different object, and when we
initialise the panel, we consider that object.
--
> Dave Page
> Blog: https://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EDB: https://www.enterprisedb.com
>
>
view thread (3+ messages)
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected]
Subject: Re: Required inputs in issue #6138
In-Reply-To: <CAFOhELfKUZdr+VuS-9uMMiJhc8+PSbNbfhH6TnW_TR3r0xqVJA@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox