Skip to content
Snippets Groups Projects
Commit 90530620 authored by srosse's avatar srosse
Browse files

OO-2093: prevent loop in loop

parent 243c917e
No related branches found
No related tags found
No related merge requests found
......@@ -30,8 +30,10 @@ import java.util.Arrays;
import java.util.Collection;
import java.util.Date;
import java.util.HashMap;
import java.util.HashSet;
import java.util.List;
import java.util.Map;
import java.util.Set;
import org.olat.admin.user.bulkChange.UserBulkChangeManager;
import org.olat.admin.user.bulkChange.UserBulkChangeStep00;
......@@ -352,15 +354,12 @@ public class UsermanagerUserSearchController extends BasicController implements
* @param tobeAddedIdentities
*/
public void addIdentitiesToSearchResult(UserRequest ureq, List<Identity> tobeAddedIdentities) {
Set<Identity> identitiesSet = new HashSet<>();
if(identitiesList != null) {
identitiesSet.addAll(identitiesList);
}
for (Identity toBeAdded : tobeAddedIdentities) {
boolean found = false;
for (Identity original : identitiesList) {
if (original.getKey().equals(toBeAdded.getKey())) {
found = true;
break;
}
}
if (!found) {
if (!identitiesSet.contains(toBeAdded)) {
identitiesList.add(toBeAdded);
}
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment