From: Charles du Jeu <charles.dujeu@gmail.com>
Date: Sun, 20 Sep 2009 14:45:09 +0000 (+0000)
Subject: Do not send empty string role
X-Git-Tag: argeo-commons-2.1.30~1766
X-Git-Url: https://git.argeo.org/?p=lgpl%2Fargeo-commons.git;a=commitdiff_plain;h=e12d56e671b665431a6af07627871eee88698cc5

Do not send empty string role

git-svn-id: https://svn.argeo.org/commons/trunk@2991 4cfe0d0a-d680-48aa-b62c-e0a02a3f76cc
---

diff --git a/security/runtime/org.argeo.security.ria/src/argeo-ria-lib/security/class/org/argeo/security/ria/UserEditorApplet.js b/security/runtime/org.argeo.security.ria/src/argeo-ria-lib/security/class/org/argeo/security/ria/UserEditorApplet.js
index 38ca9c85c..e85b52410 100644
--- a/security/runtime/org.argeo.security.ria/src/argeo-ria-lib/security/class/org/argeo/security/ria/UserEditorApplet.js
+++ b/security/runtime/org.argeo.security.ria/src/argeo-ria-lib/security/class/org/argeo/security/ria/UserEditorApplet.js
@@ -253,7 +253,14 @@ qx.Class.define("org.argeo.security.ria.UserEditorApplet",
   	saveUser : function(){
   		var user = this.getCurrentUser();
   		user.setName(this.usernameField.getValue());
-  		user.setRoles((this.rolesField.getValue()||"").split(","));
+  		var roles = this.rolesField.getValue();
+  		if(roles && roles != ""){
+	  		user.setRoles(roles.split(","));
+  		}else{
+  			user.setRoles([]);
+  		}
+  		user.setPassword(this.passPane.getData());
+  		
   		// GO TO AND RETURN FROM SERVER
   		if(user.isCreate()){
   			var userExists = false;