Re: [pgAdmin4][Patch]: RM 3927 Unable to debug the procedure inside package

Поиск
Список
Период
Сортировка
От Khushboo Vashi
Тема Re: [pgAdmin4][Patch]: RM 3927 Unable to debug the procedure inside package
Дата
Msg-id CAFOhELfkaH0Edrv7VNmQXNjypo3pyL7NtyV9e-64Q8oGoif5Bg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin4][Patch]: RM 3927 Unable to debug the procedure inside package  (Dave Page <dpage@pgadmin.org>)
Ответы Re: [pgAdmin4][Patch]: RM 3927 Unable to debug the procedure inside package  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Список pgadmin-hackers
Дерево обсуждения
Hi Akshay,

The issue has been fixed with this patch but I feel we need code refactoring of the existing code especially the loop in which we find the required string.
What's your thought?

Also, please remove the datagrid/__init__.py file from the patch which I assume you have sent by mistake.

Thanks,
Khushboo
 

On Tue, Jan 29, 2019 at 4:49 PM Dave Page <dpage@pgadmin.org> wrote:
Khushboo, can you review this please? If you're happy with it, Akshay
can commit.

Thanks.

On Mon, Jan 28, 2019 at 12:33 PM Akshay Joshi
<akshay.joshi@enterprisedb.com> wrote:
>
> Hi Hackers,
>
> Attached is the patch to fix RM 3927 "Unable to debug the procedure inside package". Problem with the existing code is we assume that "PLDBGBREAK" will be always found in the first element of the notification list.
>
> In this patch we loop through the list and found the string. Apart from that patch contains small fix to cancel the transaction of Query Tool when panel is closed.
>
> Please review it.
>
> --
> Akshay Joshi
> Sr. Software Architect
>
>
> Phone: +91 20-3058-9517
> Mobile: +91 976-788-8246



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

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

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

Предыдущее
От: Akshay Joshi
Дата:
Сообщение: Re: [pgAdmin4][RM3936] Feature tests failing intermittently for SQLeditor related test cases
Следующее
От: Ashesh Vashi
Дата:
Сообщение: pgAdmin 4 commit: Show the help string separate block in the calendarc