diff --git a/site-manage/site-group-manager/src/main/java/org/sakaiproject/groupmanager/controller/GroupController.java b/site-manage/site-group-manager/src/main/java/org/sakaiproject/groupmanager/controller/GroupController.java index 8be535eb6e8f..7e306c1f2111 100644 --- a/site-manage/site-group-manager/src/main/java/org/sakaiproject/groupmanager/controller/GroupController.java +++ b/site-manage/site-group-manager/src/main/java/org/sakaiproject/groupmanager/controller/GroupController.java @@ -185,9 +185,9 @@ public String showGroup(Model model, } } - // For every member of the site or the filtered group, add it to the selector except if they were provided by a role. + // For every active member of the site or the filtered group, add it to the selector except if they were provided by a role. for (Member member : filterGroup == null ? site.getMembers() : filterGroup.getMembers()) { - if (!roleProviderList.contains(member.getRole().getId()) && sectionProvidedUsers.stream().noneMatch(m -> m.getUserId().equals(member.getUserId()))) { + if (!roleProviderList.contains(member.getRole().getId()) && sectionProvidedUsers.stream().noneMatch(m -> m.getUserId().equals(member.getUserId())) && member.isActive()) { Optional memberUserOptional = sakaiService.getUser(member.getUserId()); if (memberUserOptional.isPresent()) { siteMemberSet.add(memberUserOptional.get());