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

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAINoption with IE
Дата
Msg-id CA+OCxoytpKWGnpSdaUXa1_pHSyXUiN29DnU1x49rOx5Q7YN+mA@mail.gmail.com
обсуждение исходный текст
Ответ на [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAIN option with IE  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
Ответы Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAINoption with IE
Список 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.


FeatureChromeFirefox (Gecko)EdgeInternet ExplorerOperaSafari
Basic support4117 (17)(Yes)No support289


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

В списке pgadmin-hackers по дате отправления: