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.56 -r1.57 --- lams_common/src/java/org/lamsfoundation/lams/usermanagement/service/UserManagementService.java 6 Sep 2006 01:35:44 -0000 1.56 +++ lams_common/src/java/org/lamsfoundation/lams/usermanagement/service/UserManagementService.java 12 Sep 2006 07:30:37 -0000 1.57 @@ -643,12 +643,13 @@ for (String roleId : rolesList) { for (UserOrganisationRole uor : uors) { if (uor.getRole().getRoleId().toString().equals(roleId)) { + // remove from the Copys the ones we are keeping rolesCopy.remove(roleId); uorsCopy.remove(uor); - log.debug("removing role: "+uor.getRole().getName()); } } } + log.debug("removing roles: "+uorsCopy); uors.removeAll(uorsCopy); } else { uors = new HashSet();