DMC/scipro
7
0
Fork 0
Code Issues30 Pull Requests3 Actions Projects1 Releases Wiki Activity

Small fix to user switching, roles should be correctly reset now.

Browse Source
This commit is contained in:
Robin Eklund 2011-08-04 16:27:57 +02:00
parent 2c8df42a18
commit e9273a97eb
1 changed files with 1 additions and 1 deletions

2
src/main/java/se/su/dsv/scipro/SciProSession.java

@ -157,6 +157,7 @@ public class SciProSession extends WebSession {
user = userDao.save(user); user = userDao.save(user);
} }
//Assign roles //Assign roles
iRoles.clear();
if(roleDao.isStudent(user)){ if(roleDao.isStudent(user)){
iRoles.add(new Student()); iRoles.add(new Student());
} }
@ -208,7 +209,6 @@ public class SciProSession extends WebSession {
return false; return false;
logger.info("Currently logged in user: '"+user.getEmailAddress()+"' attempting switch to '"+suUser+"@"+suRealm+"'"); logger.info("Currently logged in user: '"+user.getEmailAddress()+"' attempting switch to '"+suUser+"@"+suRealm+"'");
if(suUser != null && roleDao.isSysadmin(user)){ if(suUser != null && roleDao.isSysadmin(user)){
iRoles.clear();
return signInAuthenticatedUser(suUser, suRealm); return signInAuthenticatedUser(suUser, suRealm);
}else{ }else{
logger.error("User does not have the privilege to switch user"); logger.error("User does not have the privilege to switch user");