From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: PATCH: Preferences/Options dialog |
Date: | 2016-03-03 13:39:11 |
Message-ID: | CA+OCxowfabUn5Ri6s-Nbuo3811saMYJ6=50Zqqxy6KzNWoeRsQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi
On Mon, Feb 29, 2016 at 6:22 PM, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)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:
- The patch has bit-rotted, and needs to be rebased.
- 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.
I don't see anything else that seems horrendously wrong at the moment, but
will look more closely once the rebase has been done.
Thanks.
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2016-03-03 14:04:55 | pgAdmin 4 commit: In the current Backgrid implementation, a cell can be |
Previous Message | Surinder Kumar | 2016-03-03 12:49:40 | [pgAdmin4] [Patch]: Grant Wizard |