Do not send empty string role

[lgpl/argeo-commons.git] / security / runtime / org.argeo.security.ria / src / argeo-ria-lib / security / class / org / argeo / security / ria / UserEditorApplet.js

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 38ca9c85c5dce21b7841d7a691c4de40e922469a..e85b5241093f7c40f1660940160fe230539d3f74 100644 (file)
--- 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;