Index: lams_central/src/java/org/lamsfoundation/lams/workspace/service/WorkspaceManagementService.java =================================================================== RCS file: /usr/local/cvsroot/lams_central/src/java/org/lamsfoundation/lams/workspace/service/WorkspaceManagementService.java,v diff -u -r1.54 -r1.55 --- lams_central/src/java/org/lamsfoundation/lams/workspace/service/WorkspaceManagementService.java 28 Aug 2009 07:20:56 -0000 1.54 +++ lams_central/src/java/org/lamsfoundation/lams/workspace/service/WorkspaceManagementService.java 18 Sep 2009 06:16:29 -0000 1.55 @@ -774,9 +774,13 @@ if(isUserAuthorizedToModifyFolderContents(targetFolderID,userID)){ WorkspaceFolder currentFolder = (WorkspaceFolder)baseDAO.find(WorkspaceFolder.class,currentFolderID); if(currentFolder!=null){ - WorkspaceFolder targetFolder = (WorkspaceFolder)baseDAO.find(WorkspaceFolder.class,targetFolderID); - currentFolder.setParentWorkspaceFolder(targetFolder); - baseDAO.update(currentFolder); + if (currentFolderID != targetFolderID) { + WorkspaceFolder targetFolder = (WorkspaceFolder)baseDAO.find(WorkspaceFolder.class,targetFolderID); + currentFolder.setParentWorkspaceFolder(targetFolder); + baseDAO.update(currentFolder); + } else { + // if making a folder it's own parent, do nothing + } flashMessage = new FlashMessage(MSG_KEY_MOVE, targetFolderID); }else throw new WorkspaceFolderException();