From a02b1a4813ef2780d7c827c9ca5f4879d115c336 Mon Sep 17 00:00:00 2001 From: Murtuza Zabuawala Date: Wed, 10 Feb 2016 13:01:14 +0530 Subject: [PATCH] Adding validation for the previleges control. --- .../server_groups/servers/static/js/privilege.js | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/web/pgadmin/browser/server_groups/servers/static/js/privilege.js b/web/pgadmin/browser/server_groups/servers/static/js/privilege.js index 01788e294..2b8ceb4a8 100644 --- a/web/pgadmin/browser/server_groups/servers/static/js/privilege.js +++ b/web/pgadmin/browser/server_groups/servers/static/js/privilege.js @@ -177,6 +177,22 @@ }, validate: function() { + var err = {}, + errmsg = null, + changedAttrs = this.sessAttrs, + msg = undefined; + // We will throw error if user have not entered + // either grantee or privileges + if (_.has(changedAttrs, 'grantor')) { + if (_.isUndefined(this.get('grantee')) || + this.get('privileges').length == 0) { + errmsg = 'Please specify grantee/privileges'; + this.errorModel.set('grantee', errmsg); + return errmsg; + } + } else { + this.errorModel.unset('grantee'); + } return null; } });