Show display names in group members tree

[pithos-web-client] / src / gr / grnet / pithos / web / client / AddUserDialog.java

diff --git a/src/gr/grnet/pithos/web/client/AddUserDialog.java b/src/gr/grnet/pithos/web/client/AddUserDialog.java
index cf3ed6f..8d3515d 100644 (file)
--- a/src/gr/grnet/pithos/web/client/AddUserDialog.java
+++ b/src/gr/grnet/pithos/web/client/AddUserDialog.java
@@ -169,8 +169,9 @@ public class AddUserDialog extends DialogBox {
         hide();\r
     }\r
 \r
-    private void doAddUser(final String userID) {\r
-        group.addMember(userID);\r
+    private void doAddUserByName(final String userDisplayName) {\r
+        final String userID = app.getIDForUserDisplayName(userDisplayName);\r
+        group.addMemberID(userID);\r
         String path = "?update=";\r
         PostRequest updateGroup = new PostRequest(app.getApiPath(), app.getUserID(), path) {\r
 \r
@@ -198,7 +199,7 @@ public class AddUserDialog extends DialogBox {
         };\r
         updateGroup.setHeader(Const.X_AUTH_TOKEN, app.getUserToken());\r
         String groupMembers = "";\r
-        for(String u : group.getMembers()) {\r
+        for(String u : group.getMemberIDs()) {\r
             groupMembers += (URL.encodePathSegment(u) + ",");\r
         }\r
         updateGroup.setHeader(Const.X_ACCOUNT_GROUP_ + URL.encodePathSegment(group.getName()), groupMembers);\r
@@ -218,15 +219,13 @@ public class AddUserDialog extends DialogBox {
         }\r
 \r
         if(app.hasIDForUserDisplayName(userDisplayName)) {\r
-            final String userID = app.getIDForUserDisplayName(userDisplayName);\r
-            doAddUser(userID);\r
+            doAddUserByName(userDisplayName);\r
         }\r
         else {\r
             new UpdateUserCatalogs(app, null, Arrays.asList(userDisplayName)) {\r
                 @Override\r
                 public void onSuccess(UserCatalogs requestedUserCatalogs, UserCatalogs updatedUserCatalogs) {\r
-                    final String userID = app.getIDForUserDisplayName(userDisplayName);\r
-                    doAddUser(userID);\r
+                    doAddUserByName(userDisplayName);\r
                 }\r
             }.scheduleDeferred();\r
         }\r