Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAINoption with IE - Mailing list pgadmin-hackers

Akshay, can you review/commit this please?

On Fri, Apr 21, 2017 at 10:25 AM, Murtuza Zabuawala <murtuza.zabuawala@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 it is already starts with explain clause or not, which was added in commit: c758920890309e50f47a9427039cf735ddf04d37

But IE do not support .startsWith() function.

• Desktop 
• Mobile

Feature	Chrome	Firefox (Gecko)	Edge	Internet Explorer	Opera	Safari
Basic support	41	17 (17)	(Yes)	No support	28	9

But IE do not support .startsWith() function.

I have also changed the code for File Manager module as well for the same issue.

So affected modules are,

- Query tool

- File Manager

Please review.

RM#2347

--

Regards,

Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

--

Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company