Index: lams_common/src/java/org/lamsfoundation/lams/usermanagement/service/UserManagementService.java
===================================================================
RCS file: /usr/local/cvsroot/lams_common/src/java/org/lamsfoundation/lams/usermanagement/service/UserManagementService.java,v
diff -u -r1.35 -r1.36
--- lams_common/src/java/org/lamsfoundation/lams/usermanagement/service/UserManagementService.java	27 Jun 2006 07:46:14 -0000	1.35
+++ lams_common/src/java/org/lamsfoundation/lams/usermanagement/service/UserManagementService.java	28 Jun 2006 04:31:59 -0000	1.36
@@ -93,9 +93,10 @@
 		try{
 			if(object instanceof User){
 				User user = (User)object;
-				object = createWorkspaceForUser(user);
 				if(user.getUserId()==null){
 					user.setPassword(HashUtil.sha1(user.getPassword()));
+					baseDAO.insertOrUpdate(user);  // creating a workspace needs a userId
+					object = createWorkspaceForUser(user);
 				}
 			}
 			baseDAO.insertOrUpdate(object);
@@ -376,16 +377,14 @@
 	}
 
 	private User createWorkspaceForUser(User user) {
-		if(user.getUserId()==null){//new User
-			Workspace workspace = new Workspace(user.getFullName());
-			save(workspace);
-			WorkspaceFolder folder = new WorkspaceFolder(workspace.getName(),user.getUserId(),new Date(),new Date(),WorkspaceFolder.NORMAL);
-			save(folder);
-			if ( workspace.getFolders() == null )
-				workspace.setFolders(new HashSet());
-			workspace.getFolders().add(folder);
-			user.setWorkspace(workspace);
-		}
+		Workspace workspace = new Workspace(user.getFullName());
+		save(workspace);
+		WorkspaceFolder folder = new WorkspaceFolder(workspace.getName(),user.getUserId(),new Date(),new Date(),WorkspaceFolder.NORMAL);
+		save(folder);
+		if ( workspace.getFolders() == null )
+			workspace.setFolders(new HashSet());
+		workspace.getFolders().add(folder);
+		user.setWorkspace(workspace);
 		return user;
 	}