Index: lams_common/src/java/org/lamsfoundation/lams/usermanagement/web/UsersAddAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_common/src/java/org/lamsfoundation/lams/usermanagement/web/Attic/UsersAddAction.java,v
diff -u -r1.1 -r1.2
--- lams_common/src/java/org/lamsfoundation/lams/usermanagement/web/UsersAddAction.java	28 Jan 2005 04:20:36 -0000	1.1
+++ lams_common/src/java/org/lamsfoundation/lams/usermanagement/web/UsersAddAction.java	28 Jan 2005 05:34:34 -0000	1.2
@@ -171,14 +171,13 @@
 			if ( users != null )
 			{
 				//remove users who already exists in current organisation
-				Iterator iter = users.iterator();
-				while(iter.hasNext()){
-					User user = (User)iter.next();
+				for(int i=0; i<users.size(); i++ ){
+					User user = (User)users.get(i);
 					Iterator iter2 = existingUsers.iterator();
 					while(iter2.hasNext()){
 						User existingUser = (User)iter2.next();
 						if(user.getUserId().equals(existingUser.getUserId())){
-							users.remove(user);
+							users.remove(i);
 						}
 					}
 				}
Index: lams_common/src/java/org/lamsfoundation/lams/usermanagement/web/UsersAddActionForm.java
===================================================================
RCS file: /usr/local/cvsroot/lams_common/src/java/org/lamsfoundation/lams/usermanagement/web/Attic/UsersAddActionForm.java,v
diff -u -r1.1 -r1.2
--- lams_common/src/java/org/lamsfoundation/lams/usermanagement/web/UsersAddActionForm.java	28 Jan 2005 04:20:36 -0000	1.1
+++ lams_common/src/java/org/lamsfoundation/lams/usermanagement/web/UsersAddActionForm.java	28 Jan 2005 05:34:41 -0000	1.2
@@ -144,6 +144,18 @@
 	}
 
 	/**
+	 * @return Returns the allRoleNames.
+	 */
+	public String[] getAllRoleNames() {
+		return allRoleNames;
+	}
+	/**
+	 * @param allRoleNames The allRoleNames to set.
+	 */
+	public void setAllRoleNames(String[] allRoleNames) {
+		this.allRoleNames = allRoleNames;
+	}
+	/**
 	 * @return Returns the potentialRoleNames.
 	 */
 	public String[] getPotentialRoleNames() {
Index: lams_common/web/admin/usersaddContent.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_common/web/admin/Attic/usersaddContent.jsp,v
diff -u -r1.2 -r1.3
--- lams_common/web/admin/usersaddContent.jsp	28 Jan 2005 04:21:08 -0000	1.2
+++ lams_common/web/admin/usersaddContent.jsp	28 Jan 2005 05:34:55 -0000	1.3
@@ -45,7 +45,7 @@
 		<tr> 
 			<td valign="top">
 				<H2>Add Existing Users To <bean:write name="UsersAddActionForm" property="name"/></H2>
-				<html:hidden name="UsersAddActionForm" property="organisationId"/>
+				<html:hidden name="UsersAddActionForm" property="orgId"/>
 				<html:errors/>							
 				<p class="body"><strong>Select an organisation from which to pick new users: </strong></p>
 				<select name="selectedOrgId" onchange="showName(this)">
@@ -71,7 +71,7 @@
 					<p class="body"><strong>Select Users and Assign Roles </strong></p>
 					<table>
 						<c:set var="me" value="${remoteUser}"/>
-						<logic:iterate id="user" indexId="index" name="UsersAddActionForm" property="potentialUsers" type="User" >
+						<logic:iterate id="user" name="UsersAddActionForm" property="potentialUsers" type="User" >
 							<logic:present name="user">
 								<TR>
 									<TD class="body" valign="top">
@@ -82,9 +82,9 @@
 									<TD class="body" >							  
 										<c:if test="${not ( user.login eq me)}">
 											<c:forEach var="item" items="${UsersAddActionForm.allRoleNames}">
-												<!--<input type="checkbox" name="potentialRoleNames" value="<c:out value="${user.login}">_<c:out value="${item}"/>">-->
+												<input type="checkbox" name="potentialRoleNames" value="<c:out value="${user.login}"/>_<c:out value="${item}"/>">						
 												<c:out value="${item}"/>
-											</c:forEach>	
+			  								</c:forEach>	
 										</c:if>
 									</TD>
 								</TR>