subject:"\[pgadmin\-hackers\] \[pgAdmin4\]\[PATCH\] To fix the issue of EXPLAIN option with IE"

Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAIN option with IE

2017-04-25 Thread Akshay Joshi

Thanks patch applied. On Mon, Apr 24, 2017 at 4:40 PM, Dave Page wrote: > Akshay, can you review/commit this please? > > On Fri, Apr 21, 2017 at 10:25 AM, Murtuza Zabuawala enterprisedb.com> wrote: > >> Hi, >> >> Please find the patch to fix the issue

Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAIN option with IE

2017-04-24 Thread Dave Page

Akshay, can you review/commit this please? On Fri, Apr 21, 2017 at 10:25 AM, Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > Hi, > > Please find the patch to fix the issue while using ANALYZE option in IE. > > *Issue:* > We were using .startsWith() function to check the sql if

[pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAIN option with IE

2017-04-21 Thread Murtuza Zabuawala

Hi, Please find the patch to fix the issue while using ANALYZE option in IE. *Issue:* We were using .startsWith() function to check the sql if it is already starts with explain clause or not, which was added in commit: c758920890309e50f47a9427039cf735ddf04d37 But IE do not support .startsWith()