diff --git a/web/pgadmin/browser/server_groups/servers/tablespaces/templates/tablespaces/js/tablespaces.js b/web/pgadmin/browser/server_groups/servers/tablespaces/templates/tablespaces/js/tablespaces.js index 5f406924a..25169585e 100644 --- a/web/pgadmin/browser/server_groups/servers/tablespaces/templates/tablespaces/js/tablespaces.js +++ b/web/pgadmin/browser/server_groups/servers/tablespaces/templates/tablespaces/js/tablespaces.js @@ -95,7 +95,8 @@ function($, _, S, pgAdmin, pgBrowser, alertify) { } },{ id: 'spcuser', label:'{{ _('Owner') }}', cell: 'string', - type: 'text', control: 'node-list-by-name', node: 'role' + type: 'text', control: 'node-list-by-name', node: 'role', + select2: {allowClear: false} },{ id: 'acl', label: '{{ _('Privileges') }}', type: 'text', group: '{{ _('Security') }}', mode: ['properties'], disabled: true @@ -128,14 +129,12 @@ function($, _, S, pgAdmin, pgBrowser, alertify) { errmsg = null, changedAttrs = this.sessAttrs, msg = undefined; - if (_.has(changedAttrs, 'name') && - (_.isUndefined(this.get('name')) - || String(this.get('name')).replace(/^\s+|\s+$/g, '') == '')) { + if (_.isUndefined(this.get('name')) + || String(this.get('name')).replace(/^\s+|\s+$/g, '') == '') { msg = '{{ _('Name cannot be empty.') }}'; this.errorModel.set('name', msg); - } else if (_.has(changedAttrs, 'spclocation') && - (_.isUndefined(this.get('spclocation')) - || String(this.get('spclocation')).replace(/^\s+|\s+$/g, '') == '')) { + } else if (_.isUndefined(this.get('spclocation')) + || String(this.get('spclocation')).replace(/^\s+|\s+$/g, '') == '') { msg = '{{ _('Location cannot be empty.') }}'; this.errorModel.set('spclocation', msg); } else { diff --git a/web/pgadmin/browser/server_groups/servers/templates/servers/servers.js b/web/pgadmin/browser/server_groups/servers/templates/servers/servers.js index 37f862c41..d1c1cd324 100644 --- a/web/pgadmin/browser/server_groups/servers/templates/servers/servers.js +++ b/web/pgadmin/browser/server_groups/servers/templates/servers/servers.js @@ -20,15 +20,26 @@ function($, _, S, pgAdmin, pgBrowser, alertify) { }], validate: function() { var err = {}, - errmsg = null, - data = this.toJSON(); + errmsg = null; + this.errorModel.clear(); - if (_.isUndefined(data.label) || - _.isNull(data.label) || - String(data.label).replace(/^\s+|\s+$/g, '') == '') { - return _("Please specify the value for all the security providers."); - } - return null; + if (_.isUndefined(this.get('provider')) || + _.isNull(this.get('provider')) || + String(this.get('provider')).replace(/^\s+|\s+$/g, '') == '') { + errmsg = '{{ _('Provider must be specified.') }}'; + this.errorModel.set('provider', errmsg); + return errmsg; + } + + if (_.isUndefined(this.get('label')) || + _.isNull(this.get('label')) || + String(this.get('label')).replace(/^\s+|\s+$/g, '') == '') { + errmsg = '{{ _('Label must be specified.') }}'; + this.errorModel.set('label', errmsg); + return errmsg; + } + + return null; } });