Index: lams_common/src/java/org/lamsfoundation/lams/usermanagement/service/UserManagementService.java
===================================================================
diff -u -r9ad43dec7eb0960145bc92da0ae6435a2c412a58 -r270a0d4c31f0b1d491d6d87a75fc0146217a4925
--- lams_common/src/java/org/lamsfoundation/lams/usermanagement/service/UserManagementService.java	(.../UserManagementService.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
+++ lams_common/src/java/org/lamsfoundation/lams/usermanagement/service/UserManagementService.java	(.../UserManagementService.java)	(revision 270a0d4c31f0b1d491d6d87a75fc0146217a4925)
@@ -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<UserOrganisationRole>();