Thanks. --
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> On Mon, Mar 7, 2016 at 5:18 PM, Dave Page <dp...@pgadmin.org> wrote: > Thanks - applied. > > On Sun, Mar 6, 2016 at 11:12 AM, Ashesh Vashi < > ashesh.va...@enterprisedb.com> wrote: > >> 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 >>>> >>> >>> >> > > > -- > Dave Page > Blog: http://pgsnake.blogspot.com > Twitter: @pgsnake > > EnterpriseDB UK: http://www.enterprisedb.com > The Enterprise PostgreSQL Company >