Re: 1.16.1: many operations cause display tree to collapse - Mailing list pgadmin-support

From Dave Page
Subject Re: 1.16.1: many operations cause display tree to collapse
Date
Msg-id CA+OCxoyB0EbQsgj2N15czucqDZ3D=fztfCk2j+_W1T7KxbX7aQ@mail.gmail.com
Whole thread Raw
In response to Re: 1.16.1: many operations cause display tree to collapse  (Dave Page <dpage@pgadmin.org>)
List pgadmin-support
OK, so I finally managed to track down the root cause of this bug -
specifically, it's this change:
http://git.postgresql.org/gitweb/?p=pgadmin3.git;a=commit;h=e97d351dcbc42a38f0054a81f37727d511087ffa

We're looking into the most appropriate way to fix it properly.

On Mon, May 13, 2013 at 8:43 AM, Dave Page <dpage@pgadmin.org> wrote:
> On Sat, May 11, 2013 at 2:56 AM, Aren Cambre <aren@arencambre.com> wrote:
>> Thanks. I'll shut up now. :-)
>>
>> You guys really need a bug tracker.
>
> We have one: http://code.pgadmin.org/trac/query
>
> It's only used for triaged bugs that aren't being immediately worked on though.
>
>> On Thu, May 9, 2013 at 1:02 AM, Dave Page <dpage@pgadmin.org> wrote:
>>>
>>> Yes.
>>>
>>>
>>> On Thursday, May 9, 2013, Aren Cambre wrote:
>>>>
>>>> Is anyone paying attention to this?
>>>>
>>>> On Sun, Apr 14, 2013 at 9:13 AM, Michael Shapiro <mshapiro51@gmail.com>
>>>> wrote:
>>>>>
>>>>> I can confirm this behavior as well.
>>>>>
>>>>> On Sun, Apr 14, 2013 at 8:52 AM, Aren Cambre <aren@arencambre.com>
>>>>> wrote:
>>>>>>
>>>>>> Just FYI, this bug is still present in 1.16.1 (Apr 2, 2013, rev:
>>>>>> REL-1_16_1).
>>>>>>
>>>>>> As for whether it happens when adding columns, my experience on Windows
>>>>>> 7 is that the tree doesn't collapse when when I add columns or do other
>>>>>> modifications through the table's Properties dialog. Otherwise, it happens
>>>>>> whenever I complete a change that was initiated by right-clicking on any of
>>>>>> the nodes below the table (Columns, Constraints, etc.).
>>>>>>
>>>>>> Aren
>>>>>>
>>>>>>
>>>>>> On Mon, Feb 18, 2013 at 10:19 AM, Dave Page <dpage@pgadmin.org> wrote:
>>>>>>>
>>>>>>> On Mon, Feb 4, 2013 at 3:54 PM, Dan Halbert <halbert@halwitz.org>
>>>>>>> wrote:
>>>>>>> >
>>>>>>> > On Monday, February 4, 2013 10:37am, "Dave Page" <dpage@pgadmin.org>
>>>>>>> > said:
>>>>>>> >
>>>>>>> > > OK, I finally managed to recreate this. It doesn't do it for me
>>>>>>> > > when
>>>>>>> > > adding columns as described, but does if I rename one. I'll have a
>>>>>>> > > poke around and figure out what's going on.
>>>>>>> >
>>>>>>> > > >>I can't reproduce this on Mac (the logic is the same on all
>>>>>>> > > >> platforms)
>>>>>>> > > >>- but what do you have File -> Preferences -> Browser -> UI
>>>>>>> > > >>Miscellaneous -> Refresh on click set to?
>>>>>>> > > >
>>>>>>> > > > I have the same issue on a Mac. It also happens when modifying
>>>>>>> > > > constraint and index names.
>>>>>>> > > > Mac OS X 10.6.8
>>>>>>> > > > pgAdmin3 1.16.1
>>>>>>> > > > UI Miscellaneous -> Refresh on click -> any setting
>>>>>>> >
>>>>>>> >
>>>>>>> >
>>>>>>> > Sorry, I never saw this query from last month. For me, I have
>>>>>>> > Refresh set to None.
>>>>>>> > I think it's a bit weird you are not seeing this problem on adding
>>>>>>> > columns, but two others of us are. Is there any way it could have to do with
>>>>>>> > the server version? I am using 1.16.1 against PostgreSQL 9.1, running on
>>>>>>> > Ubuntu 12.04, updated to the latest standard repo version there.
>>>>>>> > Dan
>>>>>>>
>>>>>>>
>>>>>>> Apologies for not getting further with this - it hasn't been
>>>>>>> forgotten!
>>>>>>>
>>>>>>> Akshay/Ashesh; as I mentioned in our meeting earlier today, I believe
>>>>>>> I've isolated the cause of this to this patch:
>>>>>>>
>>>>>>>
>>>>>>> http://git.postgresql.org/gitweb/?p=pgadmin3.git;a=commit;h=cb5dba55dc491107e4152bd80ec496294e449934
>>>>>>>
>>>>>>> Specifically, I think the mainForm->Refresh(tblobj) call that's been
>>>>>>> added to dlgProperty::ShowObject() is messing up the attempt to
>>>>>>> re-open the tree to the original state during the existing refresh.
>>>>>>> Can one or both of you investigate further please?
>>>>>>>
>>>>>>> Thanks.
>>>>>>>
>>>>>>> --
>>>>>>> Dave Page
>>>>>>> Blog: http://pgsnake.blogspot.com
>>>>>>> Twitter: @pgsnake
>>>>>>>
>>>>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>>>>> The Enterprise PostgreSQL Company
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> Sent via pgadmin-support mailing list (pgadmin-support@postgresql.org)
>>>>>>> To make changes to your subscription:
>>>>>>> http://www.postgresql.org/mailpref/pgadmin-support
>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EnterpriseDB UK: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>
>
>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company



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

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



pgadmin-support by date:

Previous
From: Dave Page
Date:
Subject: Re: pgagent on Greenplum
Next
From: Dave Page
Date:
Subject: Remove dialogue "Apply" buttons?