Re: Fixing issues between objects and properties dialogs

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
Cc: "pgadmin-hackers(at)postgresql(dot)org" <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Fixing issues between objects and properties dialogs
Date: 2011-07-24 14:09:06
Message-ID: CA+OCxowOwfjbV-2nhq5ZgFopSaFb7YMoDVFJ9vLqA5S_eS=oTQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Sun, Jul 24, 2011 at 2:23 PM, Guillaume Lelarge
<guillaume(at)lelarge(dot)info> wrote:
> On Sun, 2011-07-24 at 14:11 +0200, Guillaume Lelarge wrote:
>> On Sun, 2011-07-24 at 12:58 +0100, Dave Page wrote:
>> > The former fix seems more appropriate. Operations in frmMain shouldn't
>> > cause dialogues to close.
>> >
>>
>> Fine with me. That should be simpler to do.
>>
>> > Simple fix seems to be to store a pointer to the dialog in pgObject;
>> > if not null, disallow drop, or if trying to view properties, use the
>> > pointer to raise the existing dialogue. Clear the pointer when the
>> > dialog is closed.
>> >
>>
>> I thought of something similar, but this is OK with me.
>>
>> I expect it to be quite a big patch. Should I work on 1.14 or master?
>> this is surely a bug, so 1.14 should be OK. But I'm afraid this is too
>> much changes to code it on a beta release. Any strong opinion on this
>> matter?
>>
>
> And another question. Let's say I open a table properties' dialog, and I
> try to refresh the whole database. Should it refresh everything except
> this table, or nothing and complains about properties' dialogs being
> open, preventing the object to be refreshed?

Nothing. "There are properties dialogues open for one or more objects
that would be refreshed. Please close the properties dialogues and try
again."

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Guillaume Lelarge 2011-07-24 15:26:03 pgAdmin III commit: Fix altering comments on an existing function
Previous Message Dave Page 2011-07-24 14:07:28 Re: Fixing issues between objects and properties dialogs