Re: PATCH: Preferences/Options dialog - Mailing list pgadmin-hackers

From Dave Page
Subject Re: PATCH: Preferences/Options dialog
Date
Msg-id CA+OCxoz0koZd_tteTfPpz33Q+8P_MhoBRZRLJa0AODH+uHFuZA@mail.gmail.com
Whole thread Raw
In response to Re: PATCH: Preferences/Options dialog  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
Responses Re: PATCH: Preferences/Options dialog
List pgadmin-hackers


On Thu, Mar 3, 2016 at 3:40 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Thu, Mar 3, 2016 at 7:09 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Feb 29, 2016 at 6:22 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave,

Please find the patch adding support for the preferences/options dialog.
Initial patch was shared by Khushboo with me, but - I made some design changes.

With this patch:
* Each preferences can be saved/retrieved per module basis.
* An object is created which represent that module in the preferences by default.
* Module needs to override the register_preferences(...) method to register certain preference.
* You can access other modules preference using static methods of the preferences.
* A collection module (PGChildModule) will register show_node_<NODE_TYPE>, and also refers the 'show_system_objects' in it, which can be accessed using self.pref_show_system_objects, or can use class property 'show_system_obejcts', also there is one more property 'show_node', which uses the 'pref_show_node' object of that module.

I've done an initial review:
Thanks. 

- The patch has bit-rotted, and needs to be rebased.
Done. 

- web/pgadmin/preferences/__init__.py has an out of date copyright notice with no blank line after it, and no pydoc comment to introduce the file.
Done. 

I don't see anything else that seems horrendously wrong at the moment, but will look more closely once the rebase has been done.
Please find the updated patch.


Thanks. Is it dependent on any other patches? I'm getting the attached message when expanding the server group node (I have the schema/catalog patch applied). 

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

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

pgadmin-hackers by date:

Previous
From: Murtuza Zabuawala
Date:
Subject: PATCH: Enhancement to backform controls [pgAdmin4]
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Allow the connection driver to return notices/message