From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][Patch]: Fix RM #3191 : Debug option is not working |
Date: | 2018-07-12 09:25:33 |
Message-ID: | CA+OCxoweXsLHij_k=FHsd0mcdpTNfU0rdMQGTq0cmd5qDuJKjw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi
On Thu, Jul 12, 2018 at 6:13 AM, Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
> Hi Dave,
>
> On Wed, Jul 11, 2018 at 4:15 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>>
>>
>> On Wed, Jul 11, 2018 at 8:03 AM, Khushboo Vashi <
>> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>>
>>> Please find the attached updated patch with PEP8 fixes.
>>>
>>> On Wed, Jul 11, 2018 at 11:18 AM, Khushboo Vashi <
>>> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi,
>>>>
>>>> Please find the attached patch to fix #3191 : Debug option is not
>>>> working.
>>>>
>>>> Issues fixed:
>>>>
>>>> 1. EPAS packages' function/procedure does not honour INOUT arguments,
>>>> it converts INOUT to OUT.
>>>>
>>>> 2. Packages' functions and procedures are not getting listed in their
>>>> respected nodes in some scenarios like procedure having INOUT argument and
>>>> function with void return type
>>>>
>>>> 3. The Reverse engineering SQL is not correct for Packages'
>>>> functions/procedures
>>>>
>>>> 4. In case of INOUT argument, debugger asks for mendatory input which
>>>> should not.
>>>>
>>>
>> Hi
>>
>> Not sure if this is a result of your patch or not, but I'm seeing a
>> failure if I debug a procedure, then click run to debug it again in the
>> same debugger panel. The parameters dialogue opens, but nothing happens
>> when it's closed. Can you check please?
>>
>> This is working fine for me with/without my patch on PG 11.
>
Odd. I've attached a video of the issue so you can see exactly what I'm
doing. The server is PG 11b2, running on macOS.
> This is on PG 11:
>>
>> -- PROCEDURE: public.dummy_proc(integer)
>>
>> -- DROP PROCEDURE public.dummy_proc(integer);
>>
>> CREATE OR REPLACE PROCEDURE public.dummy_proc(
>> id integer)
>> LANGUAGE 'plpgsql'
>>
>> AS $BODY$BEGIN
>> raise notice 'id is %', id;
>> END;$BODY$;
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
> Thanks,
> Khushboo
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment | Content-Type | Size |
---|---|---|
debugger.mov | video/quicktime | 4.2 MB |
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2018-07-12 10:17:17 | Re: Regarding RM #3316 Pgadmin4 No Tray Crash |
Previous Message | Dave Page | 2018-07-12 09:06:43 | Re: v4 3.1 Korean message file |