Merge pull request #1272 from datacite/graph-search-order

Chang default DOI search order
datacite · Nov 14, 2024 · efe893b · efe893b
2 parents b47ed62 + 3ec1400
commit efe893b
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/app/controllers/datacite_dois_controller.rb b/app/controllers/datacite_dois_controller.rb
@@ -50,7 +50,7 @@ def index
       when "relevance"
         { "_score": { "order": "desc" } }
       else
-        { updated: { order: "desc" } }
+        { "_score": { "order": "desc" } }
       end
 
     page = page_from_params(params)

diff --git a/app/models/doi.rb b/app/models/doi.rb
@@ -2518,7 +2518,7 @@ def update_publisher_from_hash
 
     def update_publisher_from_string
       self.publisher_obj = { name: publisher_before_type_cast }
-     end
+    end
 
     def reset_publishers
       self.publisher_obj = nil

diff --git a/app/models/doi/graphql_query.rb b/app/models/doi/graphql_query.rb
@@ -4,7 +4,7 @@ module Doi::GraphqlQuery
   class Builder
     include Modelable
 
-    DEFAULT_CURSOR = [0, ""]
+    DEFAULT_CURSOR = ["Infinity", 0]
     DEFAULT_PAGE_SIZE = 0
     DEFAULT_FACET_COUNT = 10
 
@@ -29,7 +29,7 @@ def size
     end
 
     def sort
-      [{ created: "asc", uid: "asc" }]
+      [{ _score: "desc", uid: "asc" }]
     end
 
     def query_fields
@@ -47,16 +47,16 @@ def query_fields
 
     def cursor
       tmp_cursor = @options.dig(:page, :cursor)
-      if tmp_cursor.nil?
+      if tmp_cursor.blank?
         return DEFAULT_CURSOR
       end
 
       if tmp_cursor.is_a?(Array)
-        timestamp, uid = tmp_cursor
+        tmp_score, uid = tmp_cursor
       elsif tmp_cursor.is_a?(String)
-        timestamp, uid = tmp_cursor.split(",")
+        tmp_score, uid = tmp_cursor.split(",")
       end
-      [timestamp.to_i, uid.to_s]
+      [tmp_score.to_f, uid.to_s]
     end
 
     def search_after