bug5230: Restricted all AbstractRole (and Subclass) constructors to

require the new transitive flag
This commit is contained in:
Dennis Aulenbacher
2021-03-01 17:44:58 +01:00
parent c240a55406
commit 73fda94935
14 changed files with 45 additions and 49 deletions
@@ -291,7 +291,7 @@ public class UserStoreImpl implements UserStore {
}
groupQualifierForMigratedRole = serverGroup;
}
result = new Role(roleDefinition, groupQualifierForMigratedRole, /* user qualification */ null);
result = new Role(roleDefinition, groupQualifierForMigratedRole, /* user qualification */ null, true);
break;
}
}
@@ -1501,7 +1501,7 @@ public class UserStoreImpl implements UserStore {
removeRoleFromUser(checkUser.getName(), removeOrAdjust);
if (removeOrAdjust.getQualifiedForUser() != null) {
addRoleForUser(checkUser.getName(), new Role(removeOrAdjust.getRoleDefinition(), null,
removeOrAdjust.getQualifiedForUser()));
removeOrAdjust.getQualifiedForUser(), removeOrAdjust.isTransitive()));
}
} catch (UserManagementException e) {
logger.log(Level.WARNING,
@@ -251,7 +251,8 @@ public class DomainObjectFactoryImpl implements DomainObjectFactory {
" for user "+user.getName()+". Removing role.");
} else {
user.addRole(new Role(roleWithUserQualifierProxy.getRoleDefinition(),
roleWithUserQualifierProxy.getQualifiedForTenant(), resolvedUserQualifier));
roleWithUserQualifierProxy.getQualifiedForTenant(), resolvedUserQualifier,
roleWithUserQualifierProxy.isTransitive()));
}
}
}