Index: lams_admin/src/java/org/lamsfoundation/lams/admin/web/UserAction.java
===================================================================
diff -u -rb09297590093f8cbfaf009d9117abc4f339c0865 -rff056fe666b6a9a6c93cb2c02fa520345c1b077f
--- lams_admin/src/java/org/lamsfoundation/lams/admin/web/UserAction.java	(.../UserAction.java)	(revision b09297590093f8cbfaf009d9117abc4f339c0865)
+++ lams_admin/src/java/org/lamsfoundation/lams/admin/web/UserAction.java	(.../UserAction.java)	(revision ff056fe666b6a9a6c93cb2c02fa520345c1b077f)
@@ -100,13 +100,15 @@
 			
 			// get system's roles
 			List allRoles = service.findAll(Role.class);
-			ArrayList<RoleDTO> rolelist = new ArrayList<RoleDTO>();
+			/*ArrayList<RoleDTO> rolelist = new ArrayList<RoleDTO>();
 			for(int i=0; i<allRoles.size(); i++){
 				Role r = (Role)allRoles.get(i);
 			    rolelist.add(new RoleDTO(r));
 			}
 			Collections.sort(rolelist);
-			request.setAttribute("rolelist",rolelist);
+			request.setAttribute("rolelist",rolelist);*/
+			Collections.sort(allRoles);
+			request.setAttribute("rolelist",allRoles);
 			
 			// get user's roles
 			Iterator iter = user.getUserOrganisations().iterator();
Index: lams_admin/src/java/org/lamsfoundation/lams/admin/web/UserManageAction.java
===================================================================
diff -u -rb09297590093f8cbfaf009d9117abc4f339c0865 -rff056fe666b6a9a6c93cb2c02fa520345c1b077f
--- lams_admin/src/java/org/lamsfoundation/lams/admin/web/UserManageAction.java	(.../UserManageAction.java)	(revision b09297590093f8cbfaf009d9117abc4f339c0865)
+++ lams_admin/src/java/org/lamsfoundation/lams/admin/web/UserManageAction.java	(.../UserManageAction.java)	(revision ff056fe666b6a9a6c93cb2c02fa520345c1b077f)
@@ -25,6 +25,7 @@
 package org.lamsfoundation.lams.admin.web;
 
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 
 import javax.servlet.http.HttpServletRequest;
@@ -123,6 +124,7 @@
 			List roles;
 			try{
 				roles = service.getRolesForUserByOrganisation(user, orgId);
+				Collections.sort(roles);
 			} catch(NullPointerException e){
 				roles = new ArrayList();
 				log.debug("no roles found for user: "+user);