diff --git a/src/main/java/org/gcube/portlets/user/workspace/client/AppControllerExplorer.java b/src/main/java/org/gcube/portlets/user/workspace/client/AppControllerExplorer.java index e15dcfd..c9b3bf4 100644 --- a/src/main/java/org/gcube/portlets/user/workspace/client/AppControllerExplorer.java +++ b/src/main/java/org/gcube/portlets/user/workspace/client/AppControllerExplorer.java @@ -1142,9 +1142,9 @@ public class AppControllerExplorer implements EventHandler, TreeAppControllerInt FileModel fileModel = event.getFileTarget(); if(explorerPanel.getAsycTreePanel().deleteItem(fileModel.getIdentifier())){ explorerPanel.getAsycTreePanel().selectItem(fileModel.getParentFileModel().getIdentifier()); //Select parent of item deleted - notifySubscriber(event); - } + + notifySubscriber(event); } }; diff --git a/src/main/java/org/gcube/portlets/user/workspace/server/GWTWorkspaceBuilder.java b/src/main/java/org/gcube/portlets/user/workspace/server/GWTWorkspaceBuilder.java index f5ed173..917ba00 100644 --- a/src/main/java/org/gcube/portlets/user/workspace/server/GWTWorkspaceBuilder.java +++ b/src/main/java/org/gcube/portlets/user/workspace/server/GWTWorkspaceBuilder.java @@ -1137,7 +1137,7 @@ public class GWTWorkspaceBuilder { if(searchFolderItem.getFolderItemType()==null){ logger.trace("Search folder item type is null for "+searchFolderItem.getId() + " name: "+searchFolderItem.getName()); //FOR DEBUG - System.out.println("Search folder item type is null for "+searchFolderItem.getId() + " name: "+searchFolderItem.getName()); +// System.out.println("Search folder item type is null for "+searchFolderItem.getId() + " name: "+searchFolderItem.getName()); return fileModel; } @@ -1577,7 +1577,7 @@ public class GWTWorkspaceBuilder { if(listUsers==null) return listContactsModel; - logger.trace("List size returned from Portal is: "+ listUsers.size()); + logger.trace("List size returned from Portal VO is: "+ listUsers.size()); logger.trace("Building list contact model list user model"); for (UserModel userModel : listUsers) { diff --git a/src/main/java/org/gcube/portlets/user/workspace/server/GWTWorkspaceServiceImpl.java b/src/main/java/org/gcube/portlets/user/workspace/server/GWTWorkspaceServiceImpl.java index c2aac85..27c1350 100644 --- a/src/main/java/org/gcube/portlets/user/workspace/server/GWTWorkspaceServiceImpl.java +++ b/src/main/java/org/gcube/portlets/user/workspace/server/GWTWorkspaceServiceImpl.java @@ -62,8 +62,11 @@ import org.gcube.portlets.user.workspace.shared.SessionExpiredException; import org.gcube.portlets.user.workspace.shared.accounting.GxtAccountingEntryType; import org.gcube.portlets.user.workspace.shared.accounting.GxtAccountingField; import org.gcube.portlets.user.workspaceapplicationhandler.ApplicationReaderFromGenericResource; +import org.gcube.vomanagement.usermanagement.GroupManager; import org.gcube.vomanagement.usermanagement.UserManager; +import org.gcube.vomanagement.usermanagement.impl.liferay.LiferayGroupManager; import org.gcube.vomanagement.usermanagement.impl.liferay.LiferayUserManager; +import org.gcube.vomanagement.usermanagement.model.UserModel; import com.google.gwt.user.server.rpc.RemoteServiceServlet; @@ -1260,8 +1263,10 @@ public class GWTWorkspaceServiceImpl extends RemoteServiceServlet implements GWT } UserManager userManag = new LiferayUserManager(); - workspaceLogger.trace("user manager getting list users..."); - return builder.buildGXTListContactsModel(userManag.listUsers()); + GroupManager gm = new LiferayGroupManager(); + String groupId = gm.getRootVO().getGroupId(); + workspaceLogger.trace("user manager getting list users by group: "+groupId); + return builder.buildGXTListContactsModel(userManag.listUsersByGroup(groupId)); } catch (Exception e) { workspaceLogger.error("Error in server get all contacts ", e); @@ -2339,64 +2344,27 @@ public class GWTWorkspaceServiceImpl extends RemoteServiceServlet implements GWT String userUsername = WsUtil.getAslSession(this.getThreadLocalRequest().getSession()).getUsername(); boolean isTestUser = userUsername.compareTo(WsUtil.TEST_USER)==0; - workspaceLogger.trace("is valid user: "+isTestUser + " is test mode: "+isTestMode()); +// //TODO COMMENT THIS FOR RELEASE +// workspaceLogger.trace("is "+WsUtil.TEST_USER+" user: "+isTestUser + " is test mode: "+isTestMode()); +// +// if(isTestUser && !isTestMode()){ +// workspaceLogger.error("workspace session is expired! username is: "+WsUtil.TEST_USER); +// return true; //is TEST_USER, session is expired +// } - if(isTestUser && !isTestMode()){ + + //TODO UNCOMMENT THIS FOR RELEASE + workspaceLogger.trace("is "+WsUtil.TEST_USER+" user: "+isTestUser); + + if(isTestUser){ workspaceLogger.error("workspace session is expired! username is: "+WsUtil.TEST_USER); return true; //is TEST_USER, session is expired } + workspaceLogger.trace("workspace session is valid! current username is: "+userUsername); return false; } - -// @Override -// public List getFullPathByItemIdentifier(String itemIdentifier) throws Exception { -// List listParents = new ArrayList(); -// workspaceLogger.trace("get Full Path Parents By Item Identifier "+ itemIdentifier); -// try { -// Workspace workspace = getWorkspace(); -// -// WorkspaceItem wsItem = workspace.getItem(itemIdentifier); -// -// workspaceLogger.trace("workspace retrieve item name: "+wsItem.getName()); -// -// GWTWorkspaceBuilder builder = getGWTWorkspaceBuilder(); -// -// if(wsItem.getType().equals(WorkspaceItemType.FOLDER)){ -// listParents.add(builder.buildGXTFolderModelItem((WorkspaceFolder) wsItem, null)); -// } -// -// -// while(wsItem!=null && wsItem.getParent()!=null){ -// -// WorkspaceFolder wsFolder = wsItem.getParent(); -// -// listParents.add(builder.buildGXTFolderModelItem(wsFolder, null)); -// wsItem = wsFolder; -// } -// -// Collections.reverse(listParents); -// -// //SET PARENTS -// for(int i=0; i