Postgres Pro
Downloads
Home   >   mailing lists

Thread: Numeric control optionally allow null values [pgadmin4]

Numeric control optionally allow null values [pgadmin4]

From
Harshal Dhumal
Date:
Hi,


Please find attached patch for numeric control with optionally allow null values

Null values: undefined/empty string/null
By default numeric control value can not be null. Use flag allowNull: true to allow null values.

Usage:

id: 'fillfactor', label: '{{ _('Fill factor') }}', deps: ['index'],
type: 'numeric', group: '{{ _('Definition') }}', allowNull: true,


-- 
Harshal Dhumal
Software Engineer 



Attachment

Re: Numeric control optionally allow null values [pgadmin4]

From
Harshal Dhumal
Date:
Hi,

Please find attached patch for numeric and integer control with optionally null value support.

Usage:

1] Integer
id: 'fillfactor', label: '{{ _('Fill factor') }}', deps: ['index'],
type: 'int', group: '{{ _('Definition') }}', allowNull: true,

2] Numeric

id: 'fillfactor', label: '{{ _('Fill factor') }}', deps: ['index'],
type: 'numeric', group: '{{ _('Definition') }}', allowNull: true,





-- 
Harshal Dhumal
Software Engineer 




On Fri, Mar 4, 2016 at 10:55 AM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi,


Please find attached patch for numeric control with optionally allow null values

Null values: undefined/empty string/null
By default numeric control value can not be null. Use flag allowNull: true to allow null values.

Usage:

id: 'fillfactor', label: '{{ _('Fill factor') }}', deps: ['index'],
type: 'numeric', group: '{{ _('Definition') }}', allowNull: true,


-- 
Harshal Dhumal
Software Engineer 




Attachment

Re: Numeric control optionally allow null values [pgadmin4]

From
Dave Page
Date:
Hi

On Fri, Mar 4, 2016 at 6:28 AM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi,

Please find attached patch for numeric and integer control with optionally null value support.

Usage:

1] Integer
id: 'fillfactor', label: '{{ _('Fill factor') }}', deps: ['index'],
type: 'int', group: '{{ _('Definition') }}', allowNull: true,

2] Numeric

id: 'fillfactor', label: '{{ _('Fill factor') }}', deps: ['index'],
type: 'numeric', group: '{{ _('Definition') }}', allowNull: true,

You've used regexps to validate numbers in this patch, however they assume that the only value number formats are (ignoring a leading -):

1234567 (integer) 
1234567.89 (numeric)

However, this is only the standard in some countries - for example, a French user might write a numeric as:

1 234 567,89 

Even in India that might be written as:

12,34,567.89

In Germany:

1.234.567,89

The numeric inputs should be validated based on the client's locale.

Thanks.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company