using find_users

app/models/user.rb

@@ -43,7 +43,7 @@ class User < ActiveRecord::Base
   after_destroy :destroy_drupal_user
 
   def self.search(query)
-    User.where('MATCH(username, bio) AGAINST(?)', query)
+    User.where('MATCH(username, bio) AGAINST (?)', query)
   end
 
   def new_contributor

app/services/search_service.rb

@@ -50,19 +50,19 @@ def textSearch_all(search_criteria)
   # If no sort_by value present, then it returns a list of profiles ordered by id DESC
   # a recent activity may be a node creation or a node revision
   def textSearch_profiles(search_criteria)
-    user_scope = User.where('username LIKE ? AND rusers.status = 1', '%' + search_criteria.query + '%')
+    user_scope = SrchScope.find_users(search_criteria.query, limit = 10)
+                          .reorder('')
 
     user_scope =
       if search_criteria.sort_by == "recent"
         user_scope.joins(:revisions)
                   .order("node_revisions.timestamp #{search_criteria.order_direction}")
-                  .distinct # do we need unique/distinct on user id ?
-
+                  .distinct
       else
-        user_scope.order(id: :desc) # order by what when order_by is not present? id? username?
+        user_scope.order(id: :desc)
       end
 
-    users = user_scope.limit(10) # verify limit?
+    users = user_scope.limit(10)
 
     sresult = DocList.new
     users.each do |match|

app/services/srch_scope.rb

@@ -3,12 +3,12 @@ class SrchScope
   def self.find_users(input, limit)
     if ActiveRecord::Base.connection.adapter_name == 'Mysql2'
       User.search(input)
-          .order('id DESC') # do we need this line?
-          .where('rusers.status = ?', 1)
+          .order('id DESC')
+          .where(status: 1)
           .limit(limit)
     else
-      User.order('id DESC')
-          .where('username LIKE ? AND rusers.status = 1', '%' + input + '%')
+      User.where('username LIKE ? AND rusers.status = 1', '%' + input + '%')
+          .order('id DESC')
           .limit(limit)
     end
   end

test/fixtures/nodes.yml

@@ -273,7 +273,7 @@ post_test1:
 
 post_test2:
   nid: 24
-  uid: 16
+  uid: 17
   title: "Post Test 2"
   path: "/notes/user2/<%= DateTime.new(2018,8,8).strftime("%m-%d-%Y") %>/post-test2"
   created: <%= DateTime.new(2018,8,8).to_i %>
@@ -285,7 +285,7 @@ post_test2:
 
 post_test3:
   nid: 25
-  uid: 17
+  uid: 16
   title: "Post Test 3"
   path: "/notes/user3/<%= DateTime.new(2018,8,15).strftime("%m-%d-%Y") %>/post-test3"
   created: <%= DateTime.new(2018,8,15).to_i %>

test/fixtures/revisions.yml

@@ -287,14 +287,14 @@ post_test1:
 
 post_test2:
   nid: 24
-  uid: 16
+  uid: 17
   title: Post test review 2
   body: Post test review 2
   timestamp: <%= DateTime.new(2018,8,8).to_i %>
 
 post_test3:
   nid: 25
-  uid: 17
+  uid: 16
   title: Post test review 2
   body: Post test review 2
   timestamp: <%= DateTime.new(2018,8,15).to_i %>

test/functional/search_api_test.rb

@@ -117,8 +117,8 @@ def app
 
      json = JSON.parse(last_response.body)
 
-     assert_equal "steff3",     json['items'][0]['docTitle']
-     assert_equal "steff2",     json['items'][1]['docTitle']
+     assert_equal "steff2",     json['items'][0]['docTitle']
+     assert_equal "steff3",     json['items'][1]['docTitle']
      assert_equal "steff1",     json['items'][2]['docTitle']
 
      assert matcher =~ json
@@ -142,8 +142,8 @@ def app
      json = JSON.parse(last_response.body)
 
      assert_equal "/profile/steff1",     json['items'][0]['docUrl']
-     assert_equal "/profile/steff2",     json['items'][1]['docUrl']
-     assert_equal "/profile/steff3",     json['items'][2]['docUrl']
+     assert_equal "/profile/steff3",     json['items'][1]['docUrl']
+     assert_equal "/profile/steff2",     json['items'][2]['docUrl']
 
      assert matcher =~ json
   end