Pass the correct data attribute value to the work search

Fixes #1176 Fixes samvera/hyku#1224
samvera · Jun 9, 2017 · 8a0a956 · 8a0a956
1 parent 752022f
commit 8a0a956
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/app/assets/javascripts/hyrax/autocomplete.es6 b/app/assets/javascripts/hyrax/autocomplete.es6
@@ -15,7 +15,7 @@ export default class Autocomplete {
         new Work(
           element,
           url,
-          element.data('id')
+          element.data('exclude-work')
         )
         break
       case 'based_near':

diff --git a/app/authorities/qa/authorities/find_works.rb b/app/authorities/qa/authorities/find_works.rb
@@ -1,6 +1,8 @@
 module Qa::Authorities
   class FindWorks < Qa::Authorities::Base
     def search(_q, controller)
+      # The FindWorksSearchBuilder expects a current_user
+      return [] unless controller.current_user
       repo = CatalogController.new.repository
       builder = Hyrax::FindWorksSearchBuilder.new(controller)
       response = repo.search(builder)

diff --git a/app/search_builders/hyrax/find_works_search_builder.rb b/app/search_builders/hyrax/find_works_search_builder.rb
@@ -8,7 +8,8 @@ class Hyrax::FindWorksSearchBuilder < Hyrax::SearchBuilder
   # Excludes the id that is part of the params
   def initialize(context)
     super(context)
-    @id = context.params[:id]
+    # Without an id this class will produce an invalid query.
+    @id = context.params[:id] || raise("missing required parameter: id")
     @q = context.params[:q]
   end