From 7267e151cf9715b53c2b7a8a9b692e5c0aa186f1 Mon Sep 17 00:00:00 2001 From: Toni Prieto Date: Wed, 1 Feb 2023 17:25:42 +0100 Subject: [PATCH] Remove statusFilter added to Solr browse indexes queries to solve an issue with the local administrators navigation --- .../java/org/dspace/browse/SolrBrowseDAO.java | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/dspace-api/src/main/java/org/dspace/browse/SolrBrowseDAO.java b/dspace-api/src/main/java/org/dspace/browse/SolrBrowseDAO.java index 860137194d..e02367f6eb 100644 --- a/dspace-api/src/main/java/org/dspace/browse/SolrBrowseDAO.java +++ b/dspace-api/src/main/java/org/dspace/browse/SolrBrowseDAO.java @@ -8,7 +8,6 @@ package org.dspace.browse; import java.io.Serializable; -import java.sql.SQLException; import java.util.ArrayList; import java.util.Collections; import java.util.Comparator; @@ -179,7 +178,6 @@ public class SolrBrowseDAO implements BrowseDAO { DiscoverQuery query = new DiscoverQuery(); addLocationScopeFilter(query); addDefaultFilterQueries(query); - addStatusFilter(query); if (distinct) { DiscoverFacetField dff; if (StringUtils.isNotBlank(startsWith)) { @@ -230,18 +228,6 @@ public class SolrBrowseDAO implements BrowseDAO { return sResponse; } - private void addStatusFilter(DiscoverQuery query) { - try { - if (!authorizeService.isAdmin(context) - && (authorizeService.isCommunityAdmin(context) - || authorizeService.isCollectionAdmin(context))) { - query.addFilterQueries(searcher.createLocationQueryForAdministrableItems(context)); - } - } catch (SQLException ex) { - log.error("Error looking up authorization rights of current user", ex); - } - } - private void addLocationScopeFilter(DiscoverQuery query) { if (container != null) { if (containerIDField.startsWith("collection")) { @@ -346,7 +332,6 @@ public class SolrBrowseDAO implements BrowseDAO { DiscoverQuery query = new DiscoverQuery(); addLocationScopeFilter(query); addDefaultFilterQueries(query); - addStatusFilter(query); query.setMaxResults(0); query.addFilterQueries("search.resourcetype:" + IndexableItem.TYPE);