Re: PATCH: To fix the issue in Debugger module (pgAdmin4) - Mailing list pgadmin-hackers

From Murtuza Zabuawala
Subject Re: PATCH: To fix the issue in Debugger module (pgAdmin4)
Date
Msg-id CAKKotZQGdhiFrtNK0xz8OVLKOA2QCk_yfOZDFT+HEMfELzbk4A@mail.gmail.com
Whole thread Raw
In response to Re: PATCH: To fix the issue in Debugger module (pgAdmin4)  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
Responses Re: PATCH: To fix the issue in Debugger module (pgAdmin4)  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
Hi Dave,

PFA updated patch to fix `<br>` tag display.
Please clear cache & try again with this updated patch.

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


On Mon, Sep 26, 2016 at 5:44 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:


On Mon, Sep 26, 2016 at 5:08 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Sep 26, 2016 at 11:09 AM, Murtuza Zabuawala
<murtuza.zabuawala@enterprisedb.com> wrote:
> Hi,
>
> PFA patch to fix the issue where it was not disabling buttons after
> execution gets finished.
> RM#1227
>
> Issue:
> If user clicks on buttons after execution is complete then it was throwing
> error, expected behaviour was all button should gets disabled except execute
> button.

This is an improvement I think, but not complete:

- The info messages are now shown, but:
  - Not until execution ends, which limits their usefulness for
additional debugging. Not sure if that's easily changeable though. 
  - The line breaks in messages are displaying like this: "INFO:
Employee 1 not found <br>SELECT 1" 
It's showing properly on my side, Please clear browser cache & try again. 
- The first execution of the function worked OK, but following the
second execution, I again saw: "Debugger: Step into execution error"
as I stepped out of the RETURN statement on line 18.
This is transient issue, so need more debugging.
 

Thanks.

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

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


Attachment

pgadmin-hackers by date:

Previous
From: Murtuza Zabuawala
Date:
Subject: Re: PATCH: To fix the issue in Debugger module (pgAdmin4)
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: pgAgent misc fixes: