SAK-23825 Correctly get the exsiting role when multiple sections used.

The existing role and the new role where both set to the new value, this caused a lower role in a second section to incorrecly take precendence.

Patch from Juan José Meroño Sánchez.

git-svn-id: https://source.sakaiproject.org/svn/providers/trunk@310806 66ffb92e-73f9-0310-93c1-f5514f145a0a

providers/cm-authz-provider/src/java/org/sakaiproject/coursemanagement/impl/provider/CourseManagementGroupProvider.java

@@ -104,8 +104,8 @@ public Map<String, String> getUserRolesForGroup(String id) {
 				for(Iterator<Entry<String, String>> rrRoleIter = rrUserRoleMap.entrySet().iterator(); rrRoleIter.hasNext();) {
 					Entry<String, String> entry = rrRoleIter.next();
 					String userEid = entry.getKey();
-					String existingRole = entry.getValue();
-					String rrRole = rrUserRoleMap.get(userEid);
+					String existingRole = userRoleMap.get(userEid);
+					String rrRole = entry.getValue();
 
 					// The Role Resolver has found no role for this user
 					if(rrRole == null) {