Re: PATCH: Preferences/Options dialog

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: PATCH: Preferences/Options dialog
Дата
Msg-id CA+OCxowfabUn5Ri6s-Nbuo3811saMYJ6=50Zqqxy6KzNWoeRsQ@mail.gmail.com
обсуждение исходный текст
Ответ на PATCH: Preferences/Options dialog  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
Ответы Re: PATCH: Preferences/Options dialog  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
Список pgadmin-hackers
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:

- 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

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Surinder Kumar
Дата:
Сообщение: [pgAdmin4] [Patch]: Grant Wizard
Следующее
От: Dave Page
Дата:
Сообщение: pgAdmin 4 commit: In the current Backgrid implementation, a cell can be