Re: [pgadmin4] Edb package support. - Mailing list pgadmin-hackers
From | Dave Page |
---|---|
Subject | Re: [pgadmin4] Edb package support. |
Date | |
Msg-id | CA+OCxowEFy-DKgCt3EF3V5tLH+xC9UYA4QbyNDzhEtVp9Yo7rg@mail.gmail.com Whole thread Raw |
In response to | Re: [pgadmin4] Edb package support. (Harshal Dhumal <harshal.dhumal@enterprisedb.com>) |
List | pgadmin-hackers |
Thanks - applied with some minor tweaks to groupings. On Mon, Aug 22, 2016 at 10:40 AM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote: > Hi, > > PFA updated patch for packages > > -- > Harshal Dhumal > Software Engineer > > EnterpriseDB India: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > On Thu, Aug 18, 2016 at 12:34 PM, Khushboo Vashi > <khushboo.vashi@enterprisedb.com> wrote: >> >> Hi Harshal, >> >> Please see below review comments. >> >> 1. Please make one line space between every SQL block as per pgAdmin 4 >> standard. > > Fixed >> >> 2. The RE SQL of package is not correct. The package name is displayed >> twice as shown below >> >> CREATE OR REPLACE PACKAGE enterprisedb.empinfo >> >> IS >> >> edb.empinfo is >> >> emp_name character varying(10); >> >> procedure get_name(p_empno numeric); >> >> function display_counter() return integer; >> >> END empinfo; >> >> >> Same applies for CREATE OR REPLACE PACKAGE BODY also. > > > Fixed >> >> 3. The RE SQL of functions, procedures and variable are not correct. >> Currently it is showing the entire package body. > > > Entire package body is stored as free text in single field in database. > There is no way that we can detect sql for particular function/procedure and > therefor I'm showing entire package body. (refer pgadmin3). > And for variables it's showing correct sql (refer pgadmin3) > >> 4. The Create scripts are incorrect for the package and its sub-nodes. If >> I execute that script, it gives me an error. > > Fixed for package. (fixed in point no. 2) > Create script is not applicable for functions, procedures, variables. I have > disable menu for package child nodes. > >> >> 5. Not able to update the package header as well as body as in the edit >> window the script values are incorrect. So, at the end it will create the >> wrong SQL. > > > Fixed (fixed in point no. 2) >> >> 6. Please check and change the comments for all the JS files if they are >> copied from another node. Ex: package.js is showing sequence node comments. > > Fixed. >> >> 7. Please maintain 4 tab indentation for SQL templates > > > Fixed >> >> >> NOTE: I have not repeated Dave's comments, so please incorporate those >> comments in this list. >> >> >> Thanks, >> >> Khushboo >> >> >> >> >> >> >> >> >> >> On Thu, Aug 18, 2016 at 10:50 AM, Khushboo Vashi >> <khushboo.vashi@enterprisedb.com> wrote: >>> >>> >>> >>> On Wed, Aug 17, 2016 at 4:35 PM, Dave Page <dpage@pgadmin.org> wrote: >>>> >>>> Hi, >>>> >>>> I did some quick testing, and found the following issues: >>>> >>>> - Package ACL is not shown in properties > > Fxied > >>>> >>>> >>>> - Package header and body are not shown in properties > > > Fixed >>>> >>>> >>>> - System package? is not shown in properties > > Fixed > >>>> >>>> >>>> - The header for the RE SQL on a package procedure should read: >>>> >>>> -- Package Procedure >>>> >>>> (not -- Package Function) > > > Fixed > >>>> >>>> >>>> Can you also have someone (Khushboo?) do a code review, if that hasn't >>>> happened already? >>>> >>> >>> Sure, will do this. >>> >>>> >>>> Thanks. >>>> >>>> On Tue, Aug 16, 2016 at 1:31 PM, Harshal Dhumal >>>> <harshal.dhumal@enterprisedb.com> wrote: >>>> > Hi, >>>> > >>>> > PFA initial patch for edb packages. >>>> > >>>> > Other changes: >>>> > 1] Added 'canEdit' flag for node to enable/disabled node edit. (by >>>> > default >>>> > it's enable) >>>> > 2] Privileges: Do not show 'ALL' in query if object has only one >>>> > applicable >>>> > privilege instead show that privilege. >>>> > >>>> > >>>> > -- >>>> > Harshal Dhumal >>>> > Software Engineer >>>> > >>>> > EnterpriseDB India: http://www.enterprisedb.com >>>> > The Enterprise PostgreSQL Company >>>> > >>>> > >>>> > -- >>>> > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) >>>> > To make changes to your subscription: >>>> > http://www.postgresql.org/mailpref/pgadmin-hackers >>>> > >>>> >>>> >>>> >>>> -- >>>> Dave Page >>>> Blog: http://pgsnake.blogspot.com >>>> Twitter: @pgsnake >>>> >>>> EnterpriseDB UK: http://www.enterprisedb.com >>>> The Enterprise PostgreSQL Company >>>> >>>> >>>> -- >>>> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) >>>> To make changes to your subscription: >>>> http://www.postgresql.org/mailpref/pgadmin-hackers >>> >>> >> > > > > -- > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company
pgadmin-hackers by date: