On Wed, Apr 11, 2012 at 6:22 AM, Akshay Joshi
> On Fri, Mar 30, 2012 at 1:09 AM, Guillaume Lelarge <guillaume(at)lelarge(dot)info>
>> On Wed, 2012-03-28 at 10:28 +0100, Dave Page wrote:
>> > That looks OK to me - thanks. Guillaume, what do you think (it's
>> > fixing a bug in your code :-) ). Basically the problem is in the code
>> > you added to prevent the same properties dialogue being opened twice,
>> > or dropping of objects that are open in dialogues. The issue was that
>> > the code always assumed the selected item in the treeview was the item
>> > on which it was acting, which is not always the case (for example, you
>> > might double-click an item in the listview).
>> It seems good to me. I don't reproduce the bug, so I can't say if this
>> specific patch fixed it. But the patch seems good.
> Dave/Guillaume, If patch looks good to both of you then can anyone please
> commit it.
Done - thanks!
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
In response to
pgadmin-hackers by date
|Next:||From: Sachin Srivastava||Date: 2012-04-11 11:43:47|
|Subject: Re: PATCH: Error Message is displayed when the Package is Clicked in
pgAdmin (having functions with default parameters)|
|Previous:||From: Dave Page||Date: 2012-04-11 10:48:53|
|Subject: pgAdmin III commit: Ensure we track the dialogue in the correctobject |