From: | Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: pgAdmin4 PATCH: Domain Module |
Date: | 2016-03-17 17:39:17 |
Message-ID: | CAFOhELc0PU6yuMGRX5xDj=njVnqVj7Jj632VY4AS9j0NRn60Bg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Dave,
I have a query regarding your below feedback :
* - Default values should be auto-quoted when necessary (ie. strings, on a
text-based domain).*
To resolve this, I have checked the typcategory field from pg_type for the
base_type selected for the Domain.
If the typcategory is String type (i.e. S), then only I have used
qtLiteral function to quote the default value.
Is this right approach or not?
Thanks,
Khushboo
On Wed, Mar 16, 2016 at 9:40 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> Hi
>
> On Wed, Mar 16, 2016 at 2:03 PM, Khushboo Vashi
> <khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
> > Hi,
> >
> > Please find the updated Domain Module Patch.
> >
> > To test this patch, please apply Backgrid Textarea Cell Patch before
> this.
>
> Thanks. I believe with the following fixes, we'll be done :-)
>
> - Default values should be auto-quoted when necessary (ie. strings, on
> a text-based domain).
>
> - "System Domain?" should be in the General section, between owner and
> comment.
>
> - The switches should use the same colouring/styling as other objects, e.g.
>
> options: {
> 'onText': 'Yes', 'offText': 'No',
> 'onColor': 'success', 'offColor': 'primary',
> 'size': 'small'
> }
>
> - Please remove the Schema property from the main properties tab (not
> the properties dialogue).
>
> - No icon is show for Checks on the Dependents tab for a domain.
>
> - The add button on the Security Labels tab is spelt "Add". Why is
> that? Other instances of this grid use "ADD" which is the default in
> backform.pgadmin.js.
>
> - Dependencies on domain check constraints are listed as being on a
> "Type" not a "Domain".
>
> - If adding a domain constraint using the grid on the Domain dialogue,
> I cannot specify "NOT VALID". We need a checkbox for that in a narrow
> columns at the end. Unchecking it for an existing constraint should be
> the equivalent of doing "ALTER DOMAIN ... VALIDATE CONSTRAINT"
>
> - If I switch the "Don't Validate" switch on a constraint, there are
> leading blank lines in the generated SQL. The same occurs when adding
> a comment to a constraint.
>
> - I think we need to reverse the meaning of "Don't Validate" and
> rename to match the "Valid?" field that's on the properties list.
> Otherwise it's not clear they're the same thing.
>
> - s/Not Null/Not Null?/
>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
From | Date | Subject | |
---|---|---|---|
Next Message | Ashesh Vashi | 2016-03-17 17:46:15 | Server side session management using the SQLite (per session) database |
Previous Message | Ashesh Vashi | 2016-03-17 17:32:59 | On restarting/reloading the server establish the connections with database server smoothly |