Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in browser tree

2017-04-12 Thread Dave Page
Thanks, committed. On Wed, Apr 12, 2017 at 1:26 PM, Murtuza Zabuawala wrote: > Hi Dave, > > Apologies for previous patch, please find updated patch. > > -- > Regards, > Murtuza Zabuawala > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > On Wed, Apr 12, 2017 at 5:

Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in browser tree

2017-04-12 Thread Murtuza Zabuawala
Hi Dave, Apologies for previous patch, please find updated patch. -- Regards, Murtuza Zabuawala EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company On Wed, Apr 12, 2017 at 5:52 PM, Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > Please disregard previous

Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in browser tree

2017-04-12 Thread Murtuza Zabuawala
Please disregard previous patch. -- Regards, Murtuza Zabuawala EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company On Wed, Apr 12, 2017 at 5:50 PM, Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > Hi Dave, > > PFA rebased patch. > > Thanks, > Murtuza > > O

Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in browser tree

2017-04-12 Thread Murtuza Zabuawala
Hi Dave, PFA rebased patch. Thanks, Murtuza On Wed, Apr 12, 2017 at 5:45 PM, Dave Page wrote: > Can you rebase this please? > > On Tue, Apr 11, 2017 at 2:04 PM, Murtuza Zabuawala > wrote: > > Hi, > > > > PFA minor patch to fix the issue where we were not updating inode > > information on refr

Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in browser tree

2017-04-12 Thread Dave Page
Can you rebase this please? On Tue, Apr 11, 2017 at 2:04 PM, Murtuza Zabuawala wrote: > Hi, > > PFA minor patch to fix the issue where we were not updating inode > information on refresh. > RM#2336 > > -- > Regards, > Murtuza Zabuawala > EnterpriseDB: http://www.enterprisedb.com > The Enterprise

Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in browser tree

2017-04-12 Thread Dave Page
Thanks, patch applied. On Mon, Apr 10, 2017 at 2:16 PM, Murtuza Zabuawala wrote: > Sure, https://redmine.postgresql.org/issues/2331 > > -- > Regards, > Murtuza Zabuawala > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > On Mon, Apr 10, 2017 at 6:43 PM, Dave Page

[pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in browser tree

2017-04-11 Thread Murtuza Zabuawala
Hi, PFA minor patch to fix the issue where we were not updating inode information on refresh. RM#2336 -- Regards, Murtuza Zabuawala EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company diff --git a/web/pgadmin/browser/templates/browser/js/browser.js b/web/pgadmin/browser/t

Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in browser tree

2017-04-10 Thread Murtuza Zabuawala
Sure, https://redmine.postgresql.org/issues/2331 -- Regards, Murtuza Zabuawala EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company On Mon, Apr 10, 2017 at 6:43 PM, Dave Page wrote: > Can you create an RM case for this please? > > Thanks. > > On Mon, Apr 10, 2017 at 2:11

Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in browser tree

2017-04-10 Thread Dave Page
Can you create an RM case for this please? Thanks. On Mon, Apr 10, 2017 at 2:11 PM, Murtuza Zabuawala wrote: > Hi, > > PFA patch to fix the issue in browser tree where it was adding new nodes at > wrong place due to wrong conditions in the logic. > > Thanks to Ashesh for helping me. > > > -- > R

[pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in browser tree

2017-04-10 Thread Murtuza Zabuawala
Hi, PFA patch to fix the issue in browser tree where it was adding new nodes at wrong place due to wrong conditions in the logic. Thanks to Ashesh for helping me. -- Regards, Murtuza Zabuawala EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company diff --git a/web/pgadmin/b