Fix order and patch notes

IQSS · Mar 21, 2024 · 4093e18 · 4093e18
1 parent 77e4eac
commit 4093e18
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/doc/release-notes/10338-expose-and-sort-publish-status-facet.md b/doc/release-notes/10338-expose-and-sort-publish-status-facet.md
@@ -1 +1 @@
-In version 6.1, the publication status facet location was unintentionally moved to the bottom, and it also prevented it from being visible to guest users. In version 6.2, we have restored its visibility to all users and moved it back to the top of the list.
+In version 6.1, the publication status facet location was unintentionally moved to the bottom. In this version, we have restored the original order.
diff --git a/src/main/java/edu/harvard/iq/dataverse/search/SearchServiceBean.java b/src/main/java/edu/harvard/iq/dataverse/search/SearchServiceBean.java
@@ -217,13 +217,7 @@ public SolrQueryResponse search(
 
         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
@@ -232,6 +226,13 @@ public SolrQueryResponse search(
             solrQuery.addFacetField(SearchFields.DATAVERSE_CATEGORY);
             solrQuery.addFacetField(SearchFields.METADATA_SOURCE);
             solrQuery.addFacetField(SearchFields.PUBLICATION_YEAR);
+            /*
+            * 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);
             solrQuery.addFacetField(SearchFields.DATASET_LICENSE);
             /**
              * @todo when a new method on datasetFieldService is available
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		In version 6.1, the publication status facet location was unintentionally moved to the bottom, and it also prevented it from being visible to guest users. In version 6.2, we have restored its visibility to all users and moved it back to the top of the list.
		In version 6.1, the publication status facet location was unintentionally moved to the bottom. In this version, we have restored the original order.