On Sun, Mar 6, 2016 at 3:00 PM, Ashesh Vashi <ashesh.va...@enterprisedb.com> wrote:
> Hi Dave, > > > On Fri, Mar 4, 2016 at 4:08 PM, Dave Page <dp...@pgadmin.org> wrote: > >> And here's the attachment. >> >> On Fri, Mar 4, 2016 at 10:27 AM, Dave Page <dp...@pgadmin.org> wrote: >> > >> > >> > On Thu, Mar 3, 2016 at 3:40 PM, Ashesh Vashi < >> ashesh.va...@enterprisedb.com> >> > wrote: >> >> >> >> On Thu, Mar 3, 2016 at 7:09 PM, Dave Page <dp...@pgadmin.org> wrote: >> >>> >> >>> Hi >> >>> >> >>> On Mon, Feb 29, 2016 at 6:22 PM, Ashesh Vashi >> >>> <ashesh.va...@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). >> > Hmm. > Nope - It is not (not at least on catalog/schema node). > I have try to reproduce at my end without any success. > > Did you reviewed the schema/catalog node patch before this? > If yes - you may need to remove the "*.pyc" from the source directory. > Hi Dave, I could find out an error in database module (due to an typo). Also, resolved few old references to pgadmin.settings.settings_model. Please find the latest patch with the changes. -- Thanks & Regards, Ashesh Vashi EnterpriseDB INDIA: Enterprise PostgreSQL Company <http://www.enterprisedb.com/> *http://www.linkedin.com/in/asheshvashi* <http://www.linkedin.com/in/asheshvashi> > > -- > > Thanks & Regards, > > Ashesh Vashi > EnterpriseDB INDIA: Enterprise PostgreSQL Company > <http://www.enterprisedb.com/> > > > *http://www.linkedin.com/in/asheshvashi* > <http://www.linkedin.com/in/asheshvashi> > > >> > >> > -- >> > Dave Page >> > Blog: http://pgsnake.blogspot.com >> > Twitter: @pgsnake >> > >> > EnterpriseDB UK: http://www.enterprisedb.com >> > The Enterprise PostgreSQL Company >> >> >> >> -- >> Dave Page >> Blog: http://pgsnake.blogspot.com >> Twitter: @pgsnake >> >> EnterpriseDB UK: http://www.enterprisedb.com >> The Enterprise PostgreSQL Company >> > >
preferences_v4.patch
Description: Binary data
-- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers