blacklight configurations need to indicate that exhibit, browse, and …

…search params are permitted
projectblacklight · May 11, 2022 · 47a06f3 · 47a06f3
1 parent 522ba26
commit 47a06f3
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/app/controllers/concerns/spotlight/catalog.rb b/app/controllers/concerns/spotlight/catalog.rb
@@ -10,6 +10,11 @@ module Catalog
 
     included do
       before_action :add_facet_visibility_field
+
+      blacklight_config.search_state_fields.tap do |allowed_fields|
+        # if the blacklight config may be filtering params, add the required fields for exihibits, browse and search
+        allowed_fields&.concat(%i[browse_category_id exhibit_id id] - Array(allowed_fields))
+      end
     end
 
     # Adds a facet to display document visibility for the current exhibit

diff --git a/app/models/spotlight/blacklight_configuration.rb b/app/models/spotlight/blacklight_configuration.rb
@@ -224,6 +224,11 @@ def blacklight_config
           config.navbar.partials[:search_history].if = false if config.navbar.partials.key? :search_history
         end
 
+        config.search_state_fields.tap do |allowed_fields|
+          # if the blacklight config may be filtering params, add the required fields for exihibits, browse and search
+          allowed_fields&.concat(%i[browse_category_id exhibit_id id] - Array(allowed_fields))
+        end
+
         config
       end
     end