fix asc oid sorting (#1064)

Co-authored-by: JP Engstrom <jpengstrom@MWPF2JK9NR.yu.yale.edu> Co-authored-by: JP Engstrom <jpengstrom@macbook-pro.lan>
yalelibrary · Oct 21, 2024 · 83002db · 83002db
1 parent 884898a
commit 83002db
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/controllers/catalog_controller.rb b/app/controllers/catalog_controller.rb
@@ -539,7 +539,7 @@ class CatalogController < ApplicationController
     config.add_sort_field 'creator_ssim desc, title_ssim asc, archivalSort_ssi asc', label: 'Creator (Z --> A)'
     config.add_sort_field 'title_ssim asc, oid_ssi desc, archivalSort_ssi asc', label: 'Title (A --> Z)'
     config.add_sort_field 'title_ssim desc, oid_ssi desc, archivalSort_ssi asc', label: 'Title (Z --> A)'
-    config.add_sort_field 'year_isim asc, id desc, archivalSort_ssi asc', label: 'Year (ascending)'
+    config.add_sort_field 'year_isim asc, id asc, archivalSort_ssi asc', label: 'Year (ascending)'
     config.add_sort_field 'year_isim desc, id desc, archivalSort_ssi asc', label: 'Year (descending)'
 
     # If there are more than this many search results, no spelling ("did you

diff --git a/spec/system/advanced_search_spec.rb b/spec/system/advanced_search_spec.rb
@@ -265,7 +265,7 @@
 
     it 'can sort by date asc' do
       within '#sort' do
-        find("option[value='year_isim asc, id desc, archivalSort_ssi asc']").click
+        find("option[value='year_isim asc, id asc, archivalSort_ssi asc']").click
       end
       click_on 'SEARCH'
       within '#documents' do