about summary refs log tree commit diff
path: root/app
diff options
context:
space:
mode:
authorMatt Jankowski <mjankowski@thoughtbot.com>2017-05-30 21:11:54 -0400
committerGitHub <noreply@github.com>2017-05-30 21:11:54 -0400
commit7f554306523f0ba531df768f21f6609d186406ea (patch)
treefcbe9892d9d6919321f6a93e15d791cc331af614 /app
parent82356233621300b51b3e2a2c093e9c4107e12e81 (diff)
Refactor api/v1/search controller (#3468)
Diffstat (limited to 'app')
-rw-r--r--app/controllers/api/v1/search_controller.rb19
1 files changed, 18 insertions, 1 deletions
diff --git a/app/controllers/api/v1/search_controller.rb b/app/controllers/api/v1/search_controller.rb
index 6b1292458..1ee2589a0 100644
--- a/app/controllers/api/v1/search_controller.rb
+++ b/app/controllers/api/v1/search_controller.rb
@@ -1,9 +1,26 @@
 # frozen_string_literal: true
 
 class Api::V1::SearchController < ApiController
+  RESULTS_LIMIT = 5
+
   respond_to :json
 
   def index
-    @search = OpenStruct.new(SearchService.new.call(params[:q], 5, params[:resolve] == 'true', current_account))
+    @search = OpenStruct.new(search_results)
+  end
+
+  private
+
+  def search_results
+    SearchService.new.call(
+      params[:q],
+      RESULTS_LIMIT,
+      resolving_search?,
+      current_account
+    )
+  end
+
+  def resolving_search?
+    params[:resolve] == 'true'
   end
 end