Change should still be inside the logic for the facets

IQSS · Mar 21, 2024 · 77e4eac · 77e4eac
1 parent d3ef749
commit 77e4eac
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/src/main/java/edu/harvard/iq/dataverse/search/SearchServiceBean.java b/src/main/java/edu/harvard/iq/dataverse/search/SearchServiceBean.java
@@ -214,16 +214,17 @@ public SolrQueryResponse search(
         }
 
         List<DataverseMetadataBlockFacet> metadataBlockFacets = new LinkedList<>();
-
-        /*
-         * We talked about this in slack on 2021-09-14, Users can see objects on draft/unpublished 
-         *  if the owner gives permissions to all users so it makes sense to expose this facet 
-         *  to all users. The request of this change started because the order of the facets were 
-         *  changed with the PR #9635 and this was unintended.
-         */
-         solrQuery.addFacetField(SearchFields.PUBLICATION_STATUS);
 
         if (addFacets) {
+
+            /*
+            * We talked about this in slack on 2021-09-14, Users can see objects on draft/unpublished 
+            *  if the owner gives permissions to all users so it makes sense to expose this facet 
+            *  to all users. The request of this change started because the order of the facets were 
+            *  changed with the PR #9635 and this was unintended.
+            */
+            solrQuery.addFacetField(SearchFields.PUBLICATION_STATUS);
+
             // -----------------------------------
             // Facets to Retrieve
             // -----------------------------------