Fixing issues between objects and properties dialogs - Mailing list pgadmin-hackers

From Guillaume Lelarge
Subject Fixing issues between objects and properties dialogs
Date
Msg-id 1311507431.2130.31.camel@laptop
Whole thread Raw
Responses Re: Fixing issues between objects and properties dialogs  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
Hi,

There is at least one bug that I know since I started to use pgAdmin,
and I've seen no work on it.

When someone connects to a database, the treeview is built from
informations gathered in the database. Not only do we fill the treeview,
but we also create a lot of pgObject objects (pgTable, pgSequence, you
name it). These objects are used when someone clicks on a treeview node
to display informations in the properties tab of the browser, and in the
SQL pane. They are also used when the user opens an object properties'
dialog. The object is used all the time by the properties' dialog while
it's only usable till the object's node is refreshed in the treeview.

Try this:

* open an object properties' dialog
* refresh the object's node in the treeview
* change something in the properties' dialog
* click OK

Or try this

* open an object properties' dialog
* drop the object in the treeview
* change something in the properties' dialog
* click OK

Sure crash in both case, whatever the object you're working with.

Question is: how can we fix that?

I see two potential fixes:

 * prevent refresh/drop while a properties dialog is open

 * close a properties dialog if the object associated is about to be
   refreshed/dropped

Not sure which one is better. I prefer the latter, but it'll be more
difficult to code.

Also, it would be a good time to prevent a misbehaviour. Nothing
prevents a user to get two properties' dialog for the same object.

Comments?


--
Guillaume
  http://blog.guillaume.lelarge.info
  http://www.dalibo.com


pgadmin-hackers by date:

Previous
From: Guillaume Lelarge
Date:
Subject: Patch to add support for security labels on local objects
Next
From: Dave Page
Date:
Subject: Re: Fixing issues between objects and properties dialogs