Re: [pgadmin-hackers] server and database node type check in is_connected check [pgadmin4]

2016-05-24 Thread Ashesh Vashi
Hi Harshal, I've committed a fix for the same. Please take a look at the commit-id: 748497916e71134d891e4b6f37f7b61749f9c878 for details. -- Thanks & Regards, Ashesh Vashi EnterpriseDB INDIA: Enterprise PostgreSQL Company *http://www.linkedin.com/in/asheshvashi*

Re: [pgadmin-hackers] server and database node type check in is_connected check [pgadmin4]

2016-05-24 Thread Ashesh Vashi
On Tue, May 24, 2016 at 12:26 PM, Harshal Dhumal < harshal.dhu...@enterprisedb.com> wrote: > Hi Ashesh, > > For change server password we have added check is_connected() in > server.js. Also same function is also available in database.js. So it was > causing change server password menu to enable

Re: [pgadmin-hackers] server and database node type check in is_connected check [pgadmin4]

2016-05-24 Thread Harshal Dhumal
Hi Ashesh, For change server password we have added check is_connected() in server.js. Also same function is also available in database.js. So it was causing change server password menu to enable on database node as well if database is connected. This issue will appear for any other menu if that

Re: [pgadmin-hackers] server and database node type check in is_connected check [pgadmin4]

2016-05-24 Thread Ashesh Vashi
On Tue, May 24, 2016 at 12:06 PM, Harshal Dhumal < harshal.dhu...@enterprisedb.com> wrote: > Hi, > > > PFA patch for server and database node: > > Issue fixed: Added node type check in is_connected check. > Can you please describe the situation, when these functions are being executed outside

[pgadmin-hackers] server and database node type check in is_connected check [pgadmin4]

2016-05-24 Thread Harshal Dhumal
Hi, PFA patch for server and database node: Issue fixed: Added node type check in is_connected check. -- *Harshal Dhumal* *Software Engineer * EenterpriseDB diff --git a/web/pgadmin/browser/server_groups/servers/databases/templates/databases/js/databases.js