blacklight 7.26.1 → 7.28.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5943737d0e65f97d49929bd5e1698de5f42a3cea352150f56e2417834ef307ea
-  data.tar.gz: e2ddfa33c4e1cf788a22084e9465acad0a530ea9d714e820c91805f09a8a2b80
+  metadata.gz: d7e57b2c0a5ca770fa8fbc0560072609b5db93d10c952d33af0db3d69ae2dfc6
+  data.tar.gz: 231146a7df38ba031b116075b75bf9abe7f91e760485524fb8678a624d9be893
 SHA512:
-  metadata.gz: 2d8f7cfb29344d217c4f8f1929fa9ac422567b59ab47d893ccea78f425be5524f75e1456f19fc3bfc25b7211f17b152f7de0a02733bfe4c9bb7f9fd0b6044219
-  data.tar.gz: 6219cde27cd67bfb826532d2b08b6d50a720104a12525251dc79b59a35bb9be0ba5f776f5d6b2e09a3fcbecf410445ba827d4ba85b9106c00a52fe7424aea8f0
+  metadata.gz: 95bc567df4947b3a11b37749c0c9a123da3c5a20552e78195a77775638bc5075d1db2c6d11d8158864346a1346a29a296a3ce8a873b497a9946eb011cd53843f
+  data.tar.gz: 28efc83fd658e1e43a3f85c8bea5ec389ee913b540b413ce967925b98b48a5d4d9422a2df1fcbf14d905ba0e31cba3fe9267a3cee38e6371adaf83b963183f41

data/.github/workflows/ruby.yml

@@ -76,11 +76,11 @@ jobs:
     - name: Install dependencies
       run: bundle install
       env:
-        RAILS_VERSION: 6.0.3.7
+        RAILS_VERSION: 6.0.5.1
     - name: Run tests
       run: bundle exec rake ci
       env:
-        RAILS_VERSION: 6.0.3.7
+        RAILS_VERSION: 6.0.5.1
         ENGINE_CART_RAILS_OPTIONS: '--skip-git --skip-listen --skip-spring --skip-keeps --skip-action-cable --skip-coffee --skip-test'
   test_rails5_2:
     runs-on: ubuntu-latest
@@ -96,11 +96,11 @@ jobs:
     - name: Install dependencies
       run: bundle install
       env:
-        RAILS_VERSION: 5.2.4.6
+        RAILS_VERSION: 5.2.8.1
     - name: Run tests
       run: bundle exec rake ci
       env:
-        RAILS_VERSION: 5.2.4.6
+        RAILS_VERSION: 5.2.8.1
         ENGINE_CART_RAILS_OPTIONS: '--skip-git --skip-listen --skip-spring --skip-keeps --skip-action-cable --skip-coffee --skip-test'
 
   test_rails6_1:
@@ -117,11 +117,11 @@ jobs:
     - name: Install dependencies
       run: bundle install
       env:
-        RAILS_VERSION: 6.1.5
+        RAILS_VERSION: 6.1.6.1
     - name: Run tests
       run: bundle exec rake ci
       env:
-        RAILS_VERSION: 6.1.5
+        RAILS_VERSION: 6.1.6.1
         ENGINE_CART_RAILS_OPTIONS: '--skip-git --skip-keeps --skip-action-cable --skip-test'
   api_test:
     runs-on: ubuntu-latest

data/VERSION

@@ -1 +1 @@
-7.26.1
+7.28.0

data/app/components/blacklight/advanced_search_form_component.rb

@@ -43,7 +43,7 @@ module Blacklight
         search_field_control do
           fields_for('clause[]', i, include_id: false) do |f|
             content_tag(:div, class: 'form-group advanced-search-field row') do
-              f.label(:query, field.display_label('search'), class: "col-md-3 col-form-label text-md-right") +
+              f.label(:query, field.display_label('search'), class: "col-sm-3 col-form-label text-md-right") +
                 content_tag(:div, class: 'col-sm-9') do
                   f.hidden_field(:field, value: field.key) +
                     f.text_field(:query, value: query_for_search_clause(field.key), class: 'form-control')

data/app/components/blacklight/facet_component.rb

@@ -11,11 +11,19 @@ module Blacklight
     # @param [Blacklight::Solr::Response::Facets::FacetField] display_facet
     # @param [Blacklight::Configuration] blacklight_config
     # @param [Boolean] layout
+    # rubocop:disable Metrics/CyclomaticComplexity
     def initialize(display_facet_or_field_config: nil, display_facet: nil, field_config: nil, response: nil, blacklight_config: nil, **component_args)
-      if display_facet_or_field_config.is_a?(Blacklight::Configuration::Field) || field_config
+      if display_facet_or_field_config.is_a? Blacklight::FacetFieldPresenter
+        @facet_field_presenter = display_facet_or_field_config
+        @field_config = @facet_field_presenter.facet_field
+        @display_facet = @facet_field_presenter.display_facet
+      elsif display_facet_or_field_config.is_a?(Blacklight::Configuration::Field) || field_config
+        @facet_field_presenter = nil # we need the view context to generate this
         @field_config = display_facet_or_field_config || field_config
-        @display_facet = display_facet || (response && response.aggregations[@field_config.field])
+        @display_facet = display_facet ||
+                         response&.aggregations&.fetch(@field_config.field) { Blacklight::Solr::Response::Facets::NullFacetField.new(@field_config.field, response: response) }
       elsif (display_facet || display_facet_or_field_config).respond_to?(:name)
+        @facet_field_presenter = nil # we need the view context to generate this
         @display_facet = display_facet || display_facet_or_field_config
         @field_config = field_config || blacklight_config&.facet_configuration_for_field(@display_facet.name)
       else
@@ -24,20 +32,27 @@ module Blacklight
 
       @component_args = component_args
     end
+    # rubocop:enable Metrics/CyclomaticComplexity
 
     def render?
       helpers.should_render_field?(@field_config, @display_facet)
     end
 
     def call
+      return render_partial if @field_config.partial
+
       component = @field_config.component == true ? Blacklight::FacetFieldListComponent : @field_config.component
 
       render(
         component.new(
-          facet_field: helpers.facet_field_presenter(@field_config, @display_facet),
+          facet_field: @facet_field_presenter || helpers.facet_field_presenter(@field_config, @display_facet),
           **@component_args
         )
       )
     end
+
+    def render_partial
+      helpers.render(@field_config.partial, locals: { field_name: @field_config.field, facet_field: @field_config, display_facet: @display_facet }.merge(@component_args))
+    end
   end
 end

data/app/components/blacklight/search_context_component.html.erb

@@ -1,10 +1,10 @@
 <div class='pagination-search-widgets'>
 
   <div class="page-links">
-    <%= link_to_previous_document @search_context[:prev] %> |
+    <%= link_to_previous_document %> |
 
     <%= item_page_entry_info %> |
 
-    <%= link_to_next_document @search_context[:next] %>
+    <%= link_to_next_document %>
   </div>
 </div>

data/app/components/blacklight/search_context_component.rb

@@ -19,15 +19,15 @@ module Blacklight
       end
     end
 
-    def link_to_previous_document(*args)
+    def link_to_previous_document(document = nil, *args, **kwargs)
       Deprecation.silence(Blacklight::UrlHelperBehavior) do
-        helpers.link_to_previous_document(*args)
+        helpers.link_to_previous_document(document || @search_context[:prev], *args, **kwargs)
       end
     end
 
-    def link_to_next_document(*args)
+    def link_to_next_document(document = nil, *args, **kwargs)
       Deprecation.silence(Blacklight::UrlHelperBehavior) do
-        helpers.link_to_next_document(*args)
+        helpers.link_to_next_document(document || @search_context[:next], *args, **kwargs)
       end
     end
   end

data/app/controllers/concerns/blacklight/catalog.rb

@@ -60,9 +60,7 @@ module Blacklight::Catalog
   end
 
   def advanced_search
-    empty_service = search_service_class.new(config: blacklight_config, user_params: {}, **search_service_context)
-
-    (@response, _deprecated_document_list) = empty_service.search_results
+    (@response, _deprecated_document_list) = blacklight_advanced_search_form_search_service.search_results
   end
 
   # get a single document from the index
@@ -255,6 +253,7 @@ module Blacklight::Catalog
   # By default, any search action from a Blacklight::Catalog controller
   # should use the current controller when constructing the route.
   def search_action_url options = {}
+    options = options.to_h if options.is_a? Blacklight::SearchState
     url_for(options.reverse_merge(action: 'index'))
   end
 
@@ -334,4 +333,14 @@ module Blacklight::Catalog
     flash[:notice] = flash_notice
     redirect_to search_action_url
   end
+
+  def blacklight_advanced_search_form_search_service
+    form_search_state = search_state_class.new(blacklight_advanced_search_form_params, blacklight_config, self)
+
+    search_service_class.new(config: blacklight_config, search_state: form_search_state, user_params: form_search_state.to_h, **search_service_context)
+  end
+
+  def blacklight_advanced_search_form_params
+    {}
+  end
 end

data/app/helpers/blacklight/render_constraints_helper_behavior.rb

@@ -35,7 +35,7 @@ module Blacklight::RenderConstraintsHelperBehavior
                              end
 
     Deprecation.silence(Blacklight::RenderConstraintsHelperBehavior) do
-      render_constraints_query(params_or_search_state) + render_constraints_filters(params_or_search_state)
+      render_constraints_query(params_or_search_state) + render_constraints_clauses(params_or_search_state) + render_constraints_filters(params_or_search_state)
     end
   end
 
@@ -60,6 +60,24 @@ module Blacklight::RenderConstraintsHelperBehavior
     end
   end
 
+  ##
+  # Render the query constraints
+  #
+  # @deprecated
+  # @param [Blacklight::SearchState,ActionController::Parameters] params_or_search_state query parameters
+  # @return [String]
+  def render_constraints_clauses(params_or_search_state = search_state)
+    search_state = convert_to_search_state(params_or_search_state)
+
+    clause_presenters = search_state.clause_params.map do |key, clause|
+      field_config = blacklight_config.search_fields[clause[:field]]
+      Blacklight::ClausePresenter.new(key, clause, field_config, self, search_state)
+    end
+
+    render(Blacklight::ConstraintComponent.with_collection(clause_presenters))
+  end
+  deprecation_deprecate :render_constraints_clauses
+
   ##
   # Provide a url for removing a particular constraint. This can be overriden
   # in the case that you want parameters other than the defaults to be removed

data/app/helpers/blacklight/url_helper_behavior.rb

@@ -52,8 +52,8 @@ module Blacklight::UrlHelperBehavior
   ##
   # Link to the previous document in the current search context
   # @deprecated
-  def link_to_previous_document(previous_document)
-    link_opts = session_tracking_params(previous_document, search_session['counter'].to_i - 1).merge(class: "previous", rel: 'prev')
+  def link_to_previous_document(previous_document, classes: 'previous', **addl_link_opts)
+    link_opts = session_tracking_params(previous_document, search_session['counter'].to_i - 1).merge(class: classes, rel: 'prev').merge(addl_link_opts)
     link_to_unless previous_document.nil?, raw(t('views.pagination.previous')), url_for_document(previous_document), link_opts do
       tag.span raw(t('views.pagination.previous')), class: 'previous'
     end
@@ -63,8 +63,8 @@ module Blacklight::UrlHelperBehavior
   ##
   # Link to the next document in the current search context
   # @deprecated
-  def link_to_next_document(next_document)
-    link_opts = session_tracking_params(next_document, search_session['counter'].to_i + 1).merge(class: "next", rel: 'next')
+  def link_to_next_document(next_document, classes: 'next', **addl_link_opts)
+    link_opts = session_tracking_params(next_document, search_session['counter'].to_i + 1).merge(class: classes, rel: 'next').merge(addl_link_opts)
     link_to_unless next_document.nil?, raw(t('views.pagination.next')), url_for_document(next_document), link_opts do
       tag.span raw(t('views.pagination.next')), class: 'next'
     end

data/app/models/search.rb

@@ -3,7 +3,8 @@
 class Search < ApplicationRecord
   belongs_to :user, optional: true
 
-  serialize :query_params
+  # use a backwards-compatible serializer until the Rails API stabilizes and we can evaluate for major-revision compatibility
+  serialize :query_params, Blacklight::SearchParamsYamlCoder
 
   # A Search instance is considered a saved search if it has a user_id.
   def saved?

data/app/presenters/blacklight/clause_presenter.rb

@@ -25,7 +25,7 @@ module Blacklight
     end
 
     def remove_href(path = search_state)
-      view_context.search_action_path(path.reset_search(clause: path.clause_params.except(key)))
+      view_context.search_action_path(path.reset_search(clause: path.clause_params.except(key)).to_h)
     end
 
     private

data/app/services/blacklight/search_params_yaml_coder.rb

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+module Blacklight
+  # This is a custom YAML coder for (de)serializing blacklight search parameters that
+  # supports deserializing HashWithIndifferentAccess parameters (as was historically done by Blacklight).
+  class SearchParamsYamlCoder
+    # Serializes an attribute value to a string that will be stored in the database.
+    def self.dump(obj)
+      # Convert HWIA to an ordinary hash so we have some hope of using the regular YAML encoder in the future
+      obj = obj.to_hash if obj.is_a?(ActiveSupport::HashWithIndifferentAccess)
+
+      YAML.dump(obj)
+    end
+
+    # Deserializes a string from the database to an attribute value.
+    def self.load(yaml)
+      return yaml unless yaml.is_a?(String) && yaml.start_with?("---")
+
+      params = yaml_load(yaml)
+
+      params.with_indifferent_access
+    end
+
+    # rubocop:disable Security/YAMLLoad
+    if YAML.respond_to?(:unsafe_load)
+      def self.yaml_load(payload)
+        if ActiveRecord.try(:use_yaml_unsafe_load) || ActiveRecord::Base.try(:use_yaml_unsafe_load)
+          YAML.unsafe_load(payload)
+        else
+          permitted_classes = (ActiveRecord.try(:yaml_column_permitted_classes) || ActiveRecord::Base.try(:yaml_column_permitted_classes) || []) +
+                              Blacklight::Engine.config.blacklight.search_params_permitted_classes
+          YAML.safe_load(payload, permitted_classes: permitted_classes, aliases: true)
+        end
+      end
+    else
+      def self.yaml_load(payload)
+        if ActiveRecord.try(:use_yaml_unsafe_load) || ActiveRecord::Base.try(:use_yaml_unsafe_load)
+          YAML.load(payload)
+        else
+          permitted_classes = (ActiveRecord.try(:yaml_column_permitted_classes) || ActiveRecord::Base.try(:yaml_column_permitted_classes) || []) +
+                              Blacklight::Engine.config.blacklight.search_params_permitted_classes
+          YAML.safe_load(payload, permitted_classes: permitted_classes, aliases: true)
+        end
+      end
+    end
+    # rubocop:enable Security/YAMLLoad
+  end
+end

data/app/views/bookmarks/_tools.html.erb

@@ -1,5 +1,5 @@
 <% if render_show_doc_actions_method_from_blacklight? %>
-  <%= render(Blacklight::Document::ActionsComponent.new(document: nil, tag: 'div', classes: "#{controller_name}Tools", wrapping_tag: 'span', wrapping_classes: 'btn btn-outline-primary', link_classes: '', actions: document_actions(document_list, options: { document: nil }), options: { document_list: @response.documents }, url_opts: Blacklight::Parameters.sanitize(params.to_unsafe_h))) %>
+  <%= render(Blacklight::Document::ActionsComponent.new(document: nil, tag: 'div', classes: "#{controller_name}Tools", link_classes: 'btn btn-outline-primary', actions: document_actions(document_list, options: { document: nil }), options: { document_list: @response.documents }, url_opts: Blacklight::Parameters.sanitize(params.to_unsafe_h))) %>
 <% else %>
   <% Deprecation.warn(self, '#render_show_doc_actions is deprecated; use ActionComponents instead') %>
   <ul class="<%= controller_name %>Tools nav nav-pills">

data/app/views/catalog/_advanced_search_help.html.erb

@@ -1,6 +1,6 @@
 <div class='card card-default'>
   <div class="card-body">
-    <h4 class="card-title">Search tips</h4>
+    <h2 class="h4 card-title">Search tips</h4>
     <ul class="advanced-help">
      <li>Select "match all" to require all fields.
      </li>

data/lib/blacklight/engine.rb

@@ -59,6 +59,8 @@ module Blacklight
       outer_window: 2
     }
 
+    bl_global_config.search_params_permitted_classes = [ActiveSupport::HashWithIndifferentAccess, Symbol]
+
     # Anything that goes into Blacklight::Engine.config is stored as a class
     # variable on Railtie::Configuration.  we're going to encapsulate all the
     # Blacklight specific stuff in this single struct:

data/lib/blacklight/solr/response/facets.rb

@@ -99,6 +99,12 @@ module Blacklight::Solr::Response::Facets
     end
   end
 
+  class NullFacetField < FacetField
+    def initialize name, items = [], response: nil, **kwargs
+      super(name, items, response: response, **kwargs)
+    end
+  end
+
   ##
   # Get all the Solr facet data (fields, queries, pivots) as a hash keyed by
   # both the Solr field name and/or by the blacklight field name

data/spec/components/blacklight/facet_component_spec.rb

@@ -0,0 +1,102 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe Blacklight::FacetComponent, type: :component do
+  subject(:rendered) do
+    render_inline_to_capybara_node(component)
+  end
+
+  let(:component) { described_class.new(**component_kwargs) }
+  let(:component_kwargs) { { field_config: facet_config, display_facet: display_facet } }
+  let(:items) { [{ label: "Book", value: 'Book', hits: 20 }] }
+
+  let(:display_facet) do
+    instance_double(Blacklight::Solr::Response::Facets::FacetField, name: 'field', items: items, limit: nil, sort: :index, offset: 0, prefix: nil)
+  end
+
+  let(:facet_config) { Blacklight::Configuration::FacetField.new(key: 'field', component: Blacklight::FacetFieldListComponent).normalize! }
+
+  before do
+    allow(controller).to receive(:view_context).and_return(controller.view_context)
+    allow(controller.view_context).to receive(:facet_limit_for).and_return(20)
+  end
+
+  it 'delegates to the configured component to render something' do
+    expect(rendered).to have_selector 'ul.facet-values'
+  end
+
+  context 'with a facet configured to use a partial' do
+    let(:facet_config) do
+      Blacklight::Configuration::FacetField.new(key: 'field', partial: 'catalog/facet_partial').normalize!
+    end
+
+    before do
+      controller.view_context.view_paths.unshift(RSpec::Rails::ViewExampleGroup::StubResolverCache.resolver_for('catalog/_facet_partial.html.erb' => 'facet partial'))
+    end
+
+    it 'renders the partial' do
+      expect(rendered).to have_content 'facet partial'
+    end
+  end
+
+  context 'with a field and response' do
+    let(:component_kwargs) do
+      { display_facet_or_field_config: facet_config, response: response }
+    end
+
+    let(:response) { instance_double(Blacklight::Solr::Response, aggregations: { 'field' => display_facet }) }
+
+    it 'extracts the facet data from the response to pass on to the rendering component' do
+      allow(facet_config.component).to receive(:new).and_call_original
+
+      rendered
+
+      expect(facet_config.component).to have_received(:new).with(facet_field: have_attributes(facet_field: facet_config, display_facet: display_facet))
+    end
+
+    context 'when the field is not in the response' do
+      let(:facet_config) { Blacklight::Configuration::FacetField.new(key: 'some_other_field', component: Blacklight::FacetFieldListComponent).normalize! }
+
+      it 'uses a null field to pass through the response information anyway' do
+        allow(facet_config.component).to receive(:new).and_call_original
+
+        rendered
+
+        expect(facet_config.component).to have_received(:new).with(facet_field: have_attributes(facet_field: facet_config, display_facet: have_attributes(items: [], response: response)))
+      end
+    end
+  end
+
+  context 'with a display facet and configuration' do
+    let(:component_kwargs) do
+      { display_facet_or_field_config: display_facet, blacklight_config: blacklight_config }
+    end
+
+    let(:blacklight_config) { Blacklight::Configuration.new.tap { |config| config.facet_fields['field'] = facet_config } }
+
+    it 'pulls the facet config from the blacklight config' do
+      allow(facet_config.component).to receive(:new).and_call_original
+
+      rendered
+
+      expect(facet_config.component).to have_received(:new).with(facet_field: have_attributes(facet_field: facet_config, display_facet: display_facet))
+    end
+  end
+
+  context 'with a presenter' do
+    let(:component_kwargs) do
+      { display_facet_or_field_config: presenter }
+    end
+
+    let(:presenter) { Blacklight::FacetFieldPresenter.new(facet_config, display_facet, controller.view_context) }
+
+    it 'renders the component with the provided presenter' do
+      allow(facet_config.component).to receive(:new).and_call_original
+
+      rendered
+
+      expect(facet_config.component).to have_received(:new).with(facet_field: presenter)
+    end
+  end
+end

data/spec/helpers/blacklight/render_constraints_helper_behavior_spec.rb

@@ -6,6 +6,7 @@ RSpec.describe Blacklight::RenderConstraintsHelperBehavior do
   let(:config) do
     Blacklight::Configuration.new do |config|
       config.add_facet_field 'type'
+      config.add_search_field 'title'
     end
   end
 
@@ -31,6 +32,21 @@ RSpec.describe Blacklight::RenderConstraintsHelperBehavior do
     end
   end
 
+  describe '#render_constraints_clauses' do
+    subject { helper.render_constraints_clauses(params) }
+
+    let(:my_engine) { double("Engine") }
+    let(:params) { ActionController::Parameters.new(clause: { "0": { field: 'title', query: 'nature' } }, f: { type: 'journal' }) }
+
+    it 'renders the clause constraint' do
+      expect(subject).to have_selector '.constraint-value', text: /Title\s+nature/
+    end
+
+    it "has a link relative to the current url" do
+      expect(subject).to have_link 'Remove constraint Title: nature', href: '/catalog?f%5Btype%5D%5B%5D=journal'
+    end
+  end
+
   describe '#render_filter_element' do
     subject { helper.render_filter_element('type', ['journal'], path) }
 

data/spec/models/search_spec.rb

@@ -1,32 +1,51 @@
 # frozen_string_literal: true
 
 RSpec.describe Search do
+  let(:search) { described_class.new(user: user) }
   let(:user) { User.create! email: 'xyz@example.com', password: 'xyz12345' }
+  let(:hash_params) { { q: "query", f: { facet: "filter" } } }
+  let(:query_params) { hash_params }
 
   describe "query_params" do
-    before do
-      @search = described_class.new(user: user)
-      @query_params = { q: "query", f: "facet" }
+    shared_examples "persisting query_params" do
+      it "can save and retrieve the hash" do
+        search.query_params = query_params
+        search.save!
+        expect(described_class.find(search.id).query_params).to eq query_params.with_indifferent_access
+      end
     end
 
-    it "can save and retrieve the hash" do
-      @search.query_params = @query_params
-      @search.save!
-      expect(described_class.find(@search.id).query_params).to eq @query_params
+    context "are an indifferent hash" do
+      include_context "persisting query_params" do
+        let(:query_params) { hash_params.with_indifferent_access }
+      end
+    end
+
+    context "are a string-keyed hash" do
+      include_context "persisting query_params" do
+        let(:query_params) { hash_params.with_indifferent_access.to_hash }
+      end
+    end
+
+    context "include symbol keys" do
+      include_context "persisting query_params" do
+        let(:query_params) { hash_params }
+      end
     end
   end
 
   describe "saved?" do
     it "is true when user_id is not NULL and greater than 0" do
-      @search = described_class.new(user: user)
-      @search.save!
-
-      expect(@search).to be_saved
+      search.save!
+      expect(search).to be_saved
     end
 
-    it "is false when user_id is NULL or less than 1" do
-      @search = described_class.create
-      expect(@search).not_to be_saved
+    context "when user_id is NULL or less than 1" do
+      let(:search) { described_class.create }
+
+      it "is false" do
+        expect(search).not_to be_saved
+      end
     end
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: blacklight
 version: !ruby/object:Gem::Version
-  version: 7.26.1
+  version: 7.28.0
 platform: ruby
 authors:
 - Jonathan Rochkind
@@ -17,7 +17,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-06-27 00:00:00.000000000 Z
+date: 2022-07-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -577,6 +577,7 @@ files:
 - app/presenters/blacklight/thumbnail_presenter.rb
 - app/services/blacklight/document_factory.rb
 - app/services/blacklight/field_retriever.rb
+- app/services/blacklight/search_params_yaml_coder.rb
 - app/services/blacklight/search_service.rb
 - app/values/blacklight/types.rb
 - app/views/blacklight/nav/_bookmark.html.erb
@@ -780,6 +781,7 @@ files:
 - spec/components/blacklight/document/group_component_spec.rb
 - spec/components/blacklight/document_component_spec.rb
 - spec/components/blacklight/document_metadata_component_spec.rb
+- spec/components/blacklight/facet_component_spec.rb
 - spec/components/blacklight/facet_field_checkboxes_component_spec.rb
 - spec/components/blacklight/facet_field_list_component_spec.rb
 - spec/components/blacklight/facet_item_component_spec.rb
@@ -960,6 +962,7 @@ test_files:
 - spec/components/blacklight/document/group_component_spec.rb
 - spec/components/blacklight/document_component_spec.rb
 - spec/components/blacklight/document_metadata_component_spec.rb
+- spec/components/blacklight/facet_component_spec.rb
 - spec/components/blacklight/facet_field_checkboxes_component_spec.rb
 - spec/components/blacklight/facet_field_list_component_spec.rb
 - spec/components/blacklight/facet_item_component_spec.rb