Skip to content
Snippets Groups Projects
Commit f437d569 authored by Christoph Thelen's avatar Christoph Thelen
Browse files

Merge branch 'master' into issue-6674

parents bc25f066 b13dc14f
Branches
Tags
No related merge requests found
language: java
......@@ -37,6 +37,7 @@ public class UserSessionAspect {
* @param socketId
* @param session
*/
/* FIXME This is not working because of scoping problems
@AfterReturning(
pointcut="execution(public * de.thm.arsnova.services.SessionService.joinSession(..)) && args(keyword, socketId)",
returning="session"
......@@ -45,4 +46,5 @@ public class UserSessionAspect {
userSessionService.setSession(session);
userSessionService.setSocketId(socketId);
}
*/
}
......@@ -64,7 +64,6 @@ public class SessionService implements ISessionService {
}
@Override
@Authenticated
public final Session joinSession(final String keyword, final UUID socketId) {
/* Socket.IO solution */
......
......@@ -142,8 +142,12 @@ public class UserService implements IUserService {
}
String session = user2sessionLegacy.get(user);
if (session == null) {
return false;
session = user2session.get(user);
if (session == null) {
return false;
}
}
return keyword.equals(session);
}
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment