From da3499e321ddcab2d9693d242403fa934784576e Mon Sep 17 00:00:00 2001 From: Natasa Kapravelou Date: Mon, 13 Dec 2010 18:08:16 +0200 Subject: [PATCH] Put the user's full name from the User object to the hashmap that keeps the correlation usernames and fullnames in order to avoid redundant calls to server in the filelist. This commit is related to Issue 67. --- src/gr/ebs/gss/client/FileList.java | 6 ------ src/gr/ebs/gss/client/UserDetailsPanel.java | 1 + 2 files changed, 1 insertion(+), 6 deletions(-) diff --git a/src/gr/ebs/gss/client/FileList.java b/src/gr/ebs/gss/client/FileList.java index cc4e23f..2cd38ce 100644 --- a/src/gr/ebs/gss/client/FileList.java +++ b/src/gr/ebs/gss/client/FileList.java @@ -17,7 +17,6 @@ * along with GSS. If not, see . */ package gr.ebs.gss.client; -import gr.ebs.gss.client.commands.GetUserCommand; import gr.ebs.gss.client.dnd.DnDSimpleFocusPanel; import gr.ebs.gss.client.dnd.DnDTreeItem; import gr.ebs.gss.client.rest.GetCommand; @@ -675,11 +674,6 @@ public class FileList extends Composite implements ClickHandler { else table.setHTML(i, 1, file.getName()); - if(GSS.get().findUserFullName(file.getOwner()) == null){ - GetUserCommand guc = new GetUserCommand(file.getOwner()); - guc.execute(); - GSS.get().putUserToMap(file.getOwner(), file.getOwner()); - } table.setText(i, 2, GSS.get().getUserFullName(file.getOwner())); table.setText(i, 3, file.getPath()); table.setText(i, 4, String.valueOf(file.getVersion())); diff --git a/src/gr/ebs/gss/client/UserDetailsPanel.java b/src/gr/ebs/gss/client/UserDetailsPanel.java index ed8d608..a04f4d2 100644 --- a/src/gr/ebs/gss/client/UserDetailsPanel.java +++ b/src/gr/ebs/gss/client/UserDetailsPanel.java @@ -70,6 +70,7 @@ public class UserDetailsPanel extends Composite { if (user == null) return !DONE; userInfoLabel.setHTML("" + user.getName() + " \u0387 " + user.getUsername() + ""); + GSS.get().putUserToMap(user.getUsername(), user.getName()); return DONE; } -- 1.7.10.4