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

OO-4304: fix filter short cuts only tools for not configured instances

parent 6ac9c11b
No related branches found
No related tags found
No related merge requests found
...@@ -20,9 +20,9 @@ ...@@ -20,9 +20,9 @@
package org.olat.user; package org.olat.user;
import java.util.Collections; import java.util.Collections;
import java.util.Iterator;
import java.util.List; import java.util.List;
import java.util.Set; import java.util.Set;
import java.util.stream.Collectors;
import org.olat.admin.user.tools.UserToolExtension; import org.olat.admin.user.tools.UserToolExtension;
import org.olat.admin.user.tools.UserToolsModule; import org.olat.admin.user.tools.UserToolsModule;
...@@ -71,15 +71,10 @@ public class ToolsPrefsController extends FormBasicController { ...@@ -71,15 +71,10 @@ public class ToolsPrefsController extends FormBasicController {
if(enabled) { if(enabled) {
Set<String> aToolSet = userToolsModule.getAvailableUserToolSet(); Set<String> aToolSet = userToolsModule.getAvailableUserToolSet();
userTools = userToolsModule.getAllUserToolExtensions(ureq); List<UserToolExtension> userToolList = userToolsModule.getAllUserToolExtensions(ureq);
if(!aToolSet.isEmpty()) { userTools = userToolList.stream()
for(Iterator<UserToolExtension> it=userTools.iterator(); it.hasNext(); ) { .filter(tool -> (!tool.isShortCutOnly() && (aToolSet.isEmpty() || aToolSet.contains(tool.getUniqueExtensionID()))))
UserToolExtension userToolExt = it.next(); .collect(Collectors.toList());
if(!aToolSet.contains(userToolExt.getUniqueExtensionID()) || userToolExt.isShortCutOnly()) {
it.remove();
}
}
}
} else { } else {
userTools = Collections.emptyList(); userTools = Collections.emptyList();
} }
......
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