From: | Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Victoria Henry <vhenry(at)pivotal(dot)io> |
Cc: | Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][Patch]: RM 3362 - Fix the functions for PG v11, and add support procedure for PG v11 |
Date: | 2018-06-13 04:02:48 |
Message-ID: | CAFOhELdy8a=pjCFc_LdJ7MMmiqBcj+X8nSVKMvpEygn-e-=aiw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Victoria,
The updated patch is attached.
On Tue, Jun 12, 2018 at 9:36 PM, Victoria Henry <vhenry(at)pivotal(dot)io> wrote:
> Hi Khushboo,
>
> The following change is allowing the creation of procedures in postgresql
> versions less then 11 and also GreenPlum
>
> --- a/web/pgadmin/browser/server_groups/servers/databases/schemas/functions/static/js/procedure.js
> +++ b/web/pgadmin/browser/server_groups/servers/databases/schemas/functions/static/js/procedure.js
> @@ -71,11 +71,7 @@ define('pgadmin.node.procedure', [
> if ('catalog' in node_hierarchy)
> return false;
>
> - // Procedures supported only in PPAS
> - return (
> - 'server' in node_hierarchy &&
> - node_hierarchy['server'].server_type == 'ppas'
> - );
> + return true;
>
> Fixed.
> Now that the Procedures are a thing in Postgresql maybe they should live
> in their own module.
>
The main functionalities of the functions and procedures are almost same
and we have inherited most of the things from function itself.
So, as per me they should live in one module.
> In the tests for trigger functions we are not consistent on the naming of
> the utils , in some places we call it funcs_utils in others
> trigger_funcs_utils.
>
Fixed.
> Thanks
>
> Victoria & Joao
>
> Thanks,
Khushboo
> On Tue, Jun 12, 2018 at 3:10 AM Khushboo Vashi <
> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>
>> Hi,
>>
>> Please find the attached updated patch.
>>
>> On Fri, Jun 8, 2018 at 2:21 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Hi
>>>
>>> On Thu, Jun 7, 2018 at 11:27 AM, Khushboo Vashi <
>>> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi,
>>>>
>>>> Please ignore my previous patch, find the attached updated one.
>>>>
>>>
>>> I found a couple of issues with this:
>>>
>>> - Clicking the + button on the Parameters tab does nothing in either
>>> Create or Edit modes
>>>
>>> Fixed
>>
>>> - The debugger fails to start (though, perhaps that's because the plugin
>>> doesn't have Ashesh's latest patches in it).
>>>
>>> FYI, I was trying to test the debugger with:
>>>
>>> -- 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$;
>>>
>>> Fixed. Tested with the latest code of the plugin.
>>
>>> Thanks!
>>>
>>> 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 |
---|---|---|
RM_3362_ver3.patch | text/x-patch | 104.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Ashesh Vashi | 2018-06-13 04:11:02 | [PGAGENT] Upgrade pgAgent from unpackaged version |
Previous Message | Victoria Henry | 2018-06-12 19:39:09 | Re: [pgadmin4][patch] Use pytest test runner for unit tests |