diff --git a/src/main/java/org/olat/course/nodes/members/MembersPeekViewController.java b/src/main/java/org/olat/course/nodes/members/MembersPeekViewController.java index 7d5b21e45fddacaf99b52eb7a92eda2977c83bbe..e948a2fb90e1c4c118cb54c9ec3f7a78d159b5c3 100644 --- a/src/main/java/org/olat/course/nodes/members/MembersPeekViewController.java +++ b/src/main/java/org/olat/course/nodes/members/MembersPeekViewController.java @@ -140,7 +140,7 @@ public class MembersPeekViewController extends BasicController { MembersCourseNodeConfiguration nodeConfig = (MembersCourseNodeConfiguration)CourseNodeFactory.getInstance().getCourseNodeConfiguration("cmembers"); boolean deduplicateList = nodeConfig.isDeduplicateList(); - Predicate<Identity> deduplicatCatch = deduplicateList ? new Deduplicate() : new All(); + Predicate<Identity> deduplicatCatch = new Deduplicate(); if(withOwners) { List<Identity> filteredOwners = owners.stream() .filter(deduplicatCatch) @@ -149,6 +149,9 @@ public class MembersPeekViewController extends BasicController { } if(withCoaches) { + if(!deduplicateList) { + deduplicatCatch = new Deduplicate(); + } List<Identity> filteredCoaches = coaches.stream() .filter(deduplicatCatch) .collect(Collectors.toList()); @@ -156,6 +159,9 @@ public class MembersPeekViewController extends BasicController { } if(withParticipants) { + if(!deduplicateList) { + deduplicatCatch = new Deduplicate(); + } List<Identity> filteredParticipants = participants.stream() .filter(deduplicatCatch) .collect(Collectors.toList()); @@ -195,11 +201,4 @@ public class MembersPeekViewController extends BasicController { return true; } } - - private static class All implements Predicate<Identity> { - @Override - public boolean test(Identity t) { - return true; - } - } }