Thread: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAIN option with IE
[pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAIN option with IE
From
Murtuza Zabuawala
Date:
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.
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,
Attachment
Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAINoption with IE
From
Dave Page
Date:
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.
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 ManagerPlease review.RM#2347--Regards,
--
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
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAINoption with IE
From
Akshay Joshi
Date:
Thanks patch applied.
On Mon, Apr 24, 2017 at 4:40 PM, Dave Page <dpage@pgadmin.org> wrote:
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.
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 ManagerPlease review.RM#2347--Regards,
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
Akshay Joshi
Principal Software Engineer

Phone: +91 20-3058-9517
Mobile: +91 976-788-8246
Mobile: +91 976-788-8246