fixed some unnecessary user creation

Author: FreakyBytes

src/main/java/de/unirostock/sems/cbarchive/web/Tools.java

@@ -116,14 +116,18 @@ public static UserManager getUser (CookieManager cookies) throws IOException
 
 		Cookie userInfo		= cookies.getCookie( Fields.COOKIE_USER );
 
-		UserManager user = new UserManager(pathCookie.getValue());
-		if( userInfo != null && !userInfo.getValue().isEmpty() )
-			user.setData( UserData.fromJson( userInfo.getValue() ) ); 
+		UserManager user = null;
+		if( WorkspaceManager.getInstance().hasWorkspace( pathCookie.getValue() ) ) {
+			// workspace exists
+			user = new UserManager(pathCookie.getValue());
+			// parse vCard info
+			if( userInfo != null && !userInfo.getValue().isEmpty() )
+				user.setData( UserData.fromJson( userInfo.getValue() ) ); 
+		}
 
 		storeUserCookies(cookies, user);
 
 		return user;
-
 	}
 
 	public static void storeUserCookies(CookieManager cookies, UserManager user) {

src/main/java/de/unirostock/sems/cbarchive/web/UserManager.java

@@ -71,6 +71,8 @@ public UserManager( String workspaceId ) throws IOException {
 			LOGGER.debug("Workspace with id ", workspaceId, " not found. Create new one with random id.");
 			workspace = workspaceManager.createWorkspace();
 		}
+		else
+			LOGGER.debug("Got it...");
 
 		// updates the last-seen time stamp
 		workspace.updateLastseen();

src/main/java/de/unirostock/sems/cbarchive/web/rest/ShareApi.java

@@ -86,7 +86,10 @@ public class ShareApi extends RestHelper {
 	@Produces( MediaType.TEXT_PLAIN )
 	public Response setUserPath( @CookieParam(Fields.COOKIE_PATH) String oldUserPath, @PathParam("user_path") String userPath, @CookieParam(Fields.COOKIE_WORKSPACE_HISTORY) String historyCookie, @Context HttpServletRequest requestContext) {
 
+		// clean up user path
+		userPath = userPath.trim();
 		LOGGER.debug("Got share link to workspace ", userPath);
+		
 		// user stuff
 		UserManager user = null;
 		try {

src/main/java/de/unirostock/sems/cbarchive/web/servlet/DownloadServlet.java

@@ -40,7 +40,6 @@
 import de.unirostock.sems.cbarchive.web.Fields;
 import de.unirostock.sems.cbarchive.web.Tools;
 import de.unirostock.sems.cbarchive.web.UserManager;
-import de.unirostock.sems.cbarchive.web.WorkspaceManager;
 import de.unirostock.sems.cbarchive.web.dataholder.Archive;
 import de.unirostock.sems.cbarchive.web.exception.CombineArchiveWebCriticalException;
 import de.unirostock.sems.cbarchive.web.exception.CombineArchiveWebException;