public inbox for [email protected]  

help / color / mirror / Atom feed

From: Murtuza Zabuawala <[email protected]>
To: Dave Page <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: PATCH: To fix the issue in Debugger module (pgAdmin4)
Date: Mon, 26 Sep 2016 17:44:03 +0530
Message-ID: <CAKKotZRMetEEPzFTuqiw_8Vit2_STHSN22=K+5frJBu7=Jp6PQ@mail.gmail.com> (raw)
In-Reply-To: <CA+OCxoxkphr1YF6F53j3sTreTMvoBOKqV+wjF6CnxoQ-SA+rzA@mail.gmail.com>
References: <CAKKotZQz6mGEiky_Szo91tzm_RFvwJTYGriTmuxVYrhcWi5nJg@mail.gmail.com>
	<CA+OCxoxkphr1YF6F53j3sTreTMvoBOKqV+wjF6CnxoQ-SA+rzA@mail.gmail.com>
List-Unsubscribe:  <mailto:[email protected]?body=unsub%20pgadmin-hackers>

On Mon, Sep 26, 2016 at 5:08 PM, Dave Page <[email protected]> wrote:

> Hi
>
> On Mon, Sep 26, 2016 at 11:09 AM, Murtuza Zabuawala
> <[email protected]> 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
>

view thread (21+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: PATCH: To fix the issue in Debugger module (pgAdmin4)
  In-Reply-To: <CAKKotZRMetEEPzFTuqiw_8Vit2_STHSN22=K+5frJBu7=Jp6PQ@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox