blacklight 7.20.0 → 7.21.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 514bb2df051f28fda2f7b1a2f4b309bd20cf79843dbaa74b519232543f36bc70
-  data.tar.gz: 0bbb3de22863a1e3b9e0aa55d0b9e501550f955fcc2630685e78572b7baf3d05
+  metadata.gz: 689d00beb305c757d023bd04cbbe2a25c103d33bd510dbf66919157623091fea
+  data.tar.gz: f331369f02497dffde92e2b6869fcd5db1daf7d9ceb1b1ca15faa1b42ca93e0e
 SHA512:
-  metadata.gz: c8a023e934c9d2b1365f71104047b53d60d51073f53fecf80d8e159a66edf9dbb5148db8be095b522d04e9d5939cf623fe824fa9607bfc6ed123c3e5cb08dfb7
-  data.tar.gz: e04a8a63ca946e9d31696ecc76c8da4fca3495687174dba352f613f64388a7802df446d2c902507d99f767b163acea71d2dd021f90f60f2aebe97ae8b115e0d4
+  metadata.gz: 87dbb644ca803013438f197b3ad040b6f1c13c58945deae560fe8f2dfc27a5ed790ac5539c68d4121bad61da1aea7c375095cbcc504616906060bd7fc29a180d
+  data.tar.gz: fba8f46b736beb2373e36effd81b335b3e70ef7cdba5c8888a3169fe73e15ddd877857b45f267cd6ccba47e2248b36c3f242f277c27be6eabae12370425534b3

data/.babelrc

@@ -0,0 +1,11 @@
+{
+  "presets": [
+    [
+      "@babel/preset-env",
+      {
+        "useBuiltIns": "entry",
+        "corejs": "2.0"
+      }
+    ]
+  ]
+}

data/.github/workflows/ruby.yml

@@ -9,9 +9,9 @@ name: CI
 
 on:
   push:
-    branches: [ master ]
+    branches: [ main ]
   pull_request:
-    branches: [ master ]
+    branches: [ main ]
 
 jobs:
   lint:

data/Gemfile

@@ -27,7 +27,7 @@ else
   if ENV['RAILS_VERSION']
     if ENV['RAILS_VERSION'] == 'edge'
       gem 'rails', github: 'rails/rails'
-      ENV['ENGINE_CART_RAILS_OPTIONS'] = '--edge --skip-turbolinks'
+      ENV['ENGINE_CART_RAILS_OPTIONS'] = '--edge'
     else
       gem 'rails', ENV['RAILS_VERSION']
     end

data/README.md

@@ -39,7 +39,7 @@ rails generate blacklight:install
 
 ## Contributing Code
 
-Code contributions are always welcome, instructions for contributing can be found at [CONTRIBUTING.md](https://github.com/projectblacklight/blacklight/blob/master/CONTRIBUTING.md).
+Code contributions are always welcome, instructions for contributing can be found at [CONTRIBUTING.md](https://github.com/projectblacklight/blacklight/blob/main/CONTRIBUTING.md).
 
 ## Configuring Apache Solr
 You'll also want some information about how Blacklight expects [Apache Solr](http://lucene.apache.org/solr ) to run, which you can find in [README_SOLR](https://github.com/projectblacklight/blacklight/wiki/README_SOLR)

data/VERSION

@@ -1 +1 @@
-7.20.0
+7.21.2

data/app/assets/javascripts/blacklight/blacklight.js

@@ -1,18 +1,22 @@
-Blacklight = function () {
+"use strict";
+
+var Blacklight = function () {
   var buffer = new Array();
   return {
-    onLoad: function (func) {
+    onLoad: function onLoad(func) {
       buffer.push(func);
     },
-    activate: function () {
+    activate: function activate() {
       for (var i = 0; i < buffer.length; i++) {
         buffer[i].call();
       }
     },
-    listeners: function () {
+    listeners: function listeners() {
       var listeners = [];
 
-      if (typeof Turbolinks !== 'undefined' && Turbolinks.supported) {
+      if (typeof Turbo !== 'undefined') {
+        listeners.push('turbo:load');
+      } else if (typeof Turbolinks !== 'undefined' && Turbolinks.supported) {
         // Turbolinks 5
         if (Turbolinks.BrowserAdapter) {
           listeners.push('turbolinks:load');
@@ -37,10 +41,14 @@ Blacklight.listeners().forEach(function (listener) {
   });
 });
 Blacklight.onLoad(function () {
-  const elem = document.querySelector('.no-js');
+  var elem = document.querySelector('.no-js'); // The "no-js" class may already have been removed because this function is
+  // run on every turbo:load event, in that case, it won't find an element.
+
+  if (!elem) return;
   elem.classList.remove('no-js');
   elem.classList.add('js');
 });
+window.Blacklight = Blacklight;
 /*global Bloodhound */
 
 Blacklight.onLoad(function () {
@@ -86,7 +94,7 @@ Blacklight.onLoad(function () {
     $(Blacklight.doBookmarkToggleBehavior.selector).blCheckboxSubmit({
       // cssClass is added to elements added, plus used for id base
       cssClass: 'toggle-bookmark',
-      success: function (checked, response) {
+      success: function success(checked, response) {
         if (response.bookmarks) {
           $('[data-role=bookmark-counter]').text(response.bookmarks.count);
         }
@@ -104,8 +112,8 @@ Blacklight.onLoad(function () {
   // Button clicks should change focus. As of 10/3/19, Firefox for Mac and
   // Safari both do not set focus to a button on button click.
   // See https://zellwk.com/blog/inconsistent-button-behavior/ for background information
-  document.querySelectorAll('button.collapse-toggle').forEach(button => {
-    button.addEventListener('click', () => {
+  document.querySelectorAll('button.collapse-toggle').forEach(function (button) {
+    button.addEventListener('click', function () {
       event.target.focus();
     });
   });
@@ -192,12 +200,12 @@ Blacklight.onLoad(function () {
           dataType: 'json',
           type: form.attr('method').toUpperCase(),
           data: form.serialize(),
-          error: function () {
+          error: function error() {
             label.removeAttr('disabled');
             checkbox.removeAttr('disabled');
             options.error.call();
           },
-          success: function (data, status, xhr) {
+          success: function success(data, status, xhr) {
             //if app isn't running at all, xhr annoyingly
             //reports success with status 0.
             if (xhr.status != 0) {
@@ -223,10 +231,10 @@ Blacklight.onLoad(function () {
   $.fn.blCheckboxSubmit.defaults = {
     //cssClass is added to elements added, plus used for id base
     cssClass: 'blCheckboxSubmit',
-    error: function () {
+    error: function error() {
       alert("Error");
     },
-    success: function () {} //callback
+    success: function success() {} //callback
 
   };
 })(jQuery);
@@ -242,12 +250,12 @@ Blacklight.doResizeFacetLabelsAndCounts = function () {
   }
 
   document.querySelectorAll('.facet-values, .pivot-facet').forEach(function (elem) {
-    const nodes = elem.querySelectorAll('.facet-count'); // TODO: when we drop ie11 support, this can become the spread operator:
+    var nodes = elem.querySelectorAll('.facet-count'); // TODO: when we drop ie11 support, this can become the spread operator:
 
-    const longest = Array.from(nodes).sort(longer)[0];
+    var longest = Array.from(nodes).sort(longer)[0];
 
     if (longest && longest.textContent) {
-      const width = longest.textContent.length + 1 + 'ch';
+      var width = longest.textContent.length + 1 + 'ch';
       elem.querySelector('.facet-count').style.width = width;
     }
   });
@@ -364,7 +372,7 @@ Blacklight.modal.onFailure = function (jqXHR, textStatus, errorThrown) {
 Blacklight.modal.receiveAjax = function (contents) {
   // does it have a data- selector for container?
   // important we don't execute script tags, we shouldn't.
-  // code modelled off of JQuery ajax.load. https://github.com/jquery/jquery/blob/master/src/ajax/load.js?source=c#L62
+  // code modelled off of JQuery ajax.load. https://github.com/jquery/jquery/blob/main/src/ajax/load.js?source=c#L62
   var container = $('<div>').append(jQuery.parseHTML(contents)).find(Blacklight.modal.containerSelector).first();
 
   if (container.length !== 0) {
@@ -442,14 +450,26 @@ Blacklight.doSearchContextBehavior = function () {
     return Blacklight.do_search_context_behavior();
   }
 
-  const elements = document.querySelectorAll('a[data-context-href]'); // Equivalent to Array.from(), but supports ie11
+  var elements = document.querySelectorAll('a[data-context-href]'); // Equivalent to Array.from(), but supports ie11
 
-  const nodes = Array.prototype.slice.call(elements);
+  var nodes = Array.prototype.slice.call(elements);
   nodes.forEach(function (element) {
     element.addEventListener('click', function (e) {
       Blacklight.handleSearchContextMethod.call(e.currentTarget, e);
     });
   });
+};
+
+Blacklight.csrfToken = function () {
+  var _document$querySelect;
+
+  return (_document$querySelect = document.querySelector('meta[name=csrf-token]')) === null || _document$querySelect === void 0 ? void 0 : _document$querySelect.content;
+};
+
+Blacklight.csrfParam = function () {
+  var _document$querySelect2;
+
+  return (_document$querySelect2 = document.querySelector('meta[name=csrf-param]')) === null || _document$querySelect2 === void 0 ? void 0 : _document$querySelect2.content;
 }; // this is the Rails.handleMethod with a couple adjustments, described inline:
 // first, we're attaching this directly to the event handler, so we can check for meta-keys
 
@@ -462,22 +482,21 @@ Blacklight.handleSearchContextMethod = function (event) {
 
   var link = this; // instead of using the normal href, we need to use the context href instead
 
-  let href = link.getAttribute('data-context-href');
-  let target = link.getAttribute('target');
-  let csrfToken = document.querySelector('meta[name=csrf-token]')?.content
-  let csrfParam = document.querySelector('meta[name=csrf-param]')?.content
-  let form = document.createElement('form');
+  var href = link.getAttribute('data-context-href');
+  var target = link.getAttribute('target');
+  var csrfToken = Blacklight.csrfToken();
+  var csrfParam = Blacklight.csrfParam();
+  var form = document.createElement('form');
   form.method = 'post';
   form.action = href;
-  let formContent = `<input name="_method" value="post" type="hidden" />
-    <input name="redirect" value="${link.getAttribute('href')}" type="hidden" />`; // check for meta keys.. if set, we should open in a new tab
+  var formContent = "<input name=\"_method\" value=\"post\" type=\"hidden\" />\n    <input name=\"redirect\" value=\"".concat(link.getAttribute('href'), "\" type=\"hidden\" />"); // check for meta keys.. if set, we should open in a new tab
 
   if (event.metaKey || event.ctrlKey) {
     target = '_blank';
   }
 
   if (csrfParam !== undefined && csrfToken !== undefined) {
-    formContent += `<input name="${csrfParam}" value="${csrfToken}" type="hidden" />`;
+    formContent += "<input name=\"".concat(csrfParam, "\" value=\"").concat(csrfToken, "\" type=\"hidden\" />");
   } // Must trigger submit by click on a button, else "submit" event handler won't work!
   // https://developer.mozilla.org/en-US/docs/Web/API/HTMLFormElement/submit
 

data/app/assets/stylesheets/blacklight/_constraints.scss

@@ -1,6 +1,6 @@
 .constraints-container {
   @extend .mb-2;
-  @extend .d-flex;
+  display: flex;
 }
 
 .applied-filter {

data/app/assets/stylesheets/blacklight/_controls.scss

@@ -1,5 +1,6 @@
 .search-widgets {
-  @extend .d-flex;
+  display: flex;
+
   > * {
     @extend .mx-1;
   }

data/app/components/blacklight/constraints_component.rb

@@ -63,14 +63,10 @@ module Blacklight
 
       Deprecation.silence(Blacklight::SearchState) do
         @search_state.filters.map do |facet|
-          missing_facet = @search_state.params.dig("f", "-#{facet.key}:").present?
           facet.values.map do |val|
-            next if val.blank? && !missing_facet
+            next if val.blank?
 
-            if missing_facet && val.blank?
-              missing = I18n.t("blacklight.search.facets.missing")
-              yield facet_item_presenter(facet.config, missing, facet.key)
-            elsif val.is_a?(Array)
+            if val.is_a?(Array)
               yield inclusive_facet_item_presenter(facet.config, val, facet.key) if val.any?(&:present?)
             else
               yield facet_item_presenter(facet.config, val, facet.key)

data/app/components/blacklight/system/modal_component.html.erb

@@ -6,8 +6,8 @@
       <h1 class="modal-title"><%= title %></h1>
     <% end) %>
 
-    <button type="button" class="blacklight-modal-close btn-close close" data-dismiss="modal" aria-label="<%= t('blacklight.modal.close') %>">
-      <span aria-hidden="true">&times;</span>
+    <button type="button" class="blacklight-modal-close btn-close close" data-bs-dismiss="modal" data-dismiss="modal" aria-label="<%= t('blacklight.modal.close') %>">
+      <span aria-hidden="true" class="visually-hidden">&times;</span>
     </button>
   </div>
 

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

@@ -260,7 +260,7 @@ module Blacklight::Catalog
 
   # Email Action (this will render the appropriate view on GET requests and process the form and send the email on POST requests)
   def email_action documents
-    mail = RecordMailer.email_record(documents, { to: params[:to], message: params[:message] }, url_options)
+    mail = RecordMailer.email_record(documents, { to: params[:to], message: params[:message], config: blacklight_config }, url_options)
     if mail.respond_to? :deliver_now
       mail.deliver_now
     else
@@ -271,7 +271,7 @@ module Blacklight::Catalog
   # SMS action (this will render the appropriate view on GET requests and process the form and send the email on POST requests)
   def sms_action documents
     to = "#{params[:to].gsub(/[^\d]/, '')}@#{params[:carrier]}"
-    mail = RecordMailer.sms_record(documents, { to: to }, url_options)
+    mail = RecordMailer.sms_record(documents, { to: to, config: blacklight_config }, url_options)
     if mail.respond_to? :deliver_now
       mail.deliver_now
     else

data/app/helpers/blacklight/render_partials_helper_behavior.rb

@@ -9,7 +9,10 @@ module Blacklight::RenderPartialsHelperBehavior
   # @param [Hash] locals to pass to the render call
   # @return [String]
   def render_document_index documents = nil, locals = {}
-    documents ||= @response.documents
+    unless documents
+      Deprecation.warn(self, "Calling render_document_index without documents is deprecated and will be removed in version 8")
+      documents = @response.documents
+    end
     render_document_index_with_view(document_index_view_type, documents, locals)
   end
 

data/app/javascript/blacklight/core.js

@@ -1,4 +1,4 @@
-Blacklight = function() {
+const Blacklight = function() {
   var buffer = new Array;
   return {
     onLoad: function(func) {
@@ -13,7 +13,9 @@ Blacklight = function() {
 
     listeners: function () {
       var listeners = [];
-      if (typeof Turbolinks !== 'undefined' && Turbolinks.supported) {
+      if (typeof Turbo !== 'undefined') {
+        listeners.push('turbo:load');
+      } else if (typeof Turbolinks !== 'undefined' && Turbolinks.supported) {
         // Turbolinks 5
         if (Turbolinks.BrowserAdapter) {
           listeners.push('turbolinks:load');
@@ -39,7 +41,14 @@ Blacklight.listeners().forEach(function(listener) {
 })
 
 Blacklight.onLoad(function () {
-  const elem = document.querySelector('.no-js')
+  const elem = document.querySelector('.no-js');
+
+  // The "no-js" class may already have been removed because this function is
+  // run on every turbo:load event, in that case, it won't find an element.
+  if (!elem) return;
+
   elem.classList.remove('no-js')
   elem.classList.add('js')
 })
+
+window.Blacklight = Blacklight

data/app/javascript/blacklight/modal.js

@@ -120,7 +120,7 @@ Blacklight.modal.onFailure = function(jqXHR, textStatus, errorThrown) {
 Blacklight.modal.receiveAjax = function (contents) {
     // does it have a data- selector for container?
     // important we don't execute script tags, we shouldn't.
-    // code modelled off of JQuery ajax.load. https://github.com/jquery/jquery/blob/master/src/ajax/load.js?source=c#L62
+    // code modelled off of JQuery ajax.load. https://github.com/jquery/jquery/blob/main/src/ajax/load.js?source=c#L62
     var container =  $('<div>').
       append( jQuery.parseHTML(contents) ).find( Blacklight.modal.containerSelector ).first();
     if (container.length !== 0) {

data/app/models/concerns/blacklight/document/email.rb

@@ -2,13 +2,25 @@
 # This module provides the body of an email export based on the document's semantic values
 module Blacklight::Document::Email
   # Return a text string that will be the body of the email
-  def to_email_text
-    semantics = to_semantic_values
+  def to_email_text(config = nil)
     body = []
-    body << I18n.t('blacklight.email.text.title', value: semantics[:title].join(" ")) if semantics[:title].present?
-    body << I18n.t('blacklight.email.text.author', value: semantics[:author].join(" ")) if semantics[:author].present?
-    body << I18n.t('blacklight.email.text.format', value: semantics[:format].join(" ")) if semantics[:format].present?
-    body << I18n.t('blacklight.email.text.language', value: semantics[:language].join(" ")) if semantics[:language].present?
+
+    if config
+      body = config.email_fields.map do |name, field|
+        values = [self[name]].flatten
+        "#{field.label} #{values.join(' ')}" if self[name].present?
+      end
+    end
+
+    # Use to_semantic_values for backwards compatibility
+    if body.empty?
+      semantics = to_semantic_values
+      body << I18n.t('blacklight.email.text.title', value: semantics[:title].join(" ")) if semantics[:title].present?
+      body << I18n.t('blacklight.email.text.author', value: semantics[:author].join(" ")) if semantics[:author].present?
+      body << I18n.t('blacklight.email.text.format', value: semantics[:format].join(" ")) if semantics[:format].present?
+      body << I18n.t('blacklight.email.text.language', value: semantics[:language].join(" ")) if semantics[:language].present?
+    end
+
     return body.join("\n") unless body.empty?
   end
 end

data/app/models/concerns/blacklight/document/sms.rb

@@ -2,11 +2,23 @@
 # This module provides the body of an email export based on the document's semantic values
 module Blacklight::Document::Sms
   # Return a text string that will be the body of the email
-  def to_sms_text
-    semantics = to_semantic_values
+  def to_sms_text(config = nil)
     body = []
-    body << I18n.t('blacklight.sms.text.title', value: semantics[:title].first) if semantics[:title].present?
-    body << I18n.t('blacklight.sms.text.author', value: semantics[:author].first) if semantics[:author].present?
+
+    if config
+      body = config.sms_fields.map do |name, field|
+        values = [self[name]].flatten
+        "#{field.label} #{values.first}" if self[name].present?
+      end
+    end
+
+    # Use to_semantic_values for backwards compatibility
+    if body.empty?
+      semantics = to_semantic_values
+      body << I18n.t('blacklight.sms.text.title', value: semantics[:title].first) if semantics[:title].present?
+      body << I18n.t('blacklight.sms.text.author', value: semantics[:author].first) if semantics[:author].present?
+    end
+
     return body.join unless body.empty?
   end
 end

data/app/models/record_mailer.rb

@@ -3,7 +3,12 @@
 class RecordMailer < ActionMailer::Base
   def email_record(documents, details, url_gen_params)
     title = begin
-              documents.first.to_semantic_values[:title]
+              title_field = details[:config].email.title_field
+              if title_field
+                [documents.first[title_field]].flatten.first
+              else
+                documents.first.to_semantic_values[:title]
+              end
             rescue
               I18n.t('blacklight.email.text.default_title')
             end
@@ -14,6 +19,7 @@ class RecordMailer < ActionMailer::Base
 
     @documents      = documents
     @message        = details[:message]
+    @config         = details[:config]
     @url_gen_params = url_gen_params
 
     mail(to: details[:to],  subject: subject)
@@ -21,7 +27,9 @@ class RecordMailer < ActionMailer::Base
 
   def sms_record(documents, details, url_gen_params)
     @documents      = documents
+    @config         = details[:config]
     @url_gen_params = url_gen_params
+
     mail(to: details[:to], subject: "")
   end
 end

data/app/presenters/blacklight/facet_item_presenter.rb

@@ -56,6 +56,8 @@ module Blacklight
         view_context.public_send(facet_config.helper_method, label_value)
       elsif facet_config.query && facet_config.query[label_value]
         facet_config.query[label_value][:label]
+      elsif value == Blacklight::SearchState::FilterField::MISSING
+        I18n.t("blacklight.search.facets.missing")
       elsif facet_config.date
         localization_options = facet_config.date == true ? {} : facet_config.date
         I18n.l(Time.zone.parse(label_value), **localization_options)

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

@@ -14,7 +14,7 @@
     <%= render 'sort_and_per_page' %>
     <%= render partial: 'tools', locals: { document_list: @response.documents } %>
     <h2 class='section-heading sr-only visually-hidden'><%= t('blacklight.bookmarks.list_title') %></h2>
-    <%= render_document_index %>
+    <%= render_document_index @response.documents %>
     <%= render 'results_pagination' %>
   <% end %>
 </div>

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

@@ -26,7 +26,7 @@
 <%- elsif render_grouped_response? %>
   <%= Deprecation.silence(Blacklight::RenderPartialsHelperBehavior) { render_grouped_document_index } %>
 <%- else %>
-  <%= render_document_index %>
+  <%= render_document_index @response.documents %>
 <%- end %>
 
 <%= render 'results_pagination' %>

data/app/views/record_mailer/sms_record.text.erb

@@ -1,4 +1,4 @@
 <% @documents.each do |document| %>
-<%= document.to_sms_text %>
+<%= document.to_sms_text(@config) %>
 <%= t('blacklight.sms.text.url', :url => polymorphic_url(document, @url_gen_params) ) %>
 <% end %>

data/blacklight.gemspec

@@ -32,7 +32,7 @@ Gem::Specification.new do |s|
   s.add_dependency "deprecation"
   s.add_dependency "i18n", '>= 1.7.0' # added named parameters
   s.add_dependency "ostruct", '>= 0.3.2'
-  s.add_dependency "view_component", '~> 2.42.0'
+  s.add_dependency "view_component", '~> 2.42'
 
   s.add_development_dependency "rsolr", ">= 1.0.6", "< 3"  # Library for interacting with rSolr.
   s.add_development_dependency "rspec-rails", "~> 4.0.0.beta2"

data/config/locales/blacklight.de.yml

@@ -232,7 +232,7 @@ de:
     entry_name:
       default:
         one: Eintrag
-        other: Enträge
+        other: Einträge
       grouped:
         default: 'gruppiertes Ergebnis'
 

data/lib/blacklight/configuration.rb

@@ -101,6 +101,9 @@ module Blacklight
             show: { top_level_config: :show },
             citation: { parent_config: :show }
           ),
+          # SMS and Email configurations.
+          sms: ViewConfig.new,
+          email: ViewConfig.new,
           # Configurations for specific types of index views
           view: NestedOpenStructWithHashAccess.new(ViewConfig,
                                                    list: {},
@@ -167,6 +170,12 @@ module Blacklight
     # solr fields to use for sorting results
     define_field_access :sort_field
 
+    # solr fields to use in text message
+    define_field_access :sms_field
+
+    # solr fields to use in email message
+    define_field_access :email_field
+
     def initialize(hash = {})
       super(self.class.default_values.deep_dup.merge(hash))
       yield(self) if block_given?

data/lib/blacklight/engine.rb

@@ -48,5 +48,7 @@ module Blacklight
     config.action_dispatch.rescue_responses["Blacklight::Exceptions::RecordNotFound"] = :not_found
 
     config.enable_search_bar_autofocus = false
+
+    config.facet_missing_param = '[* TO *]'
   end
 end

data/lib/blacklight/search_state/filter_field.rb

@@ -4,6 +4,8 @@ module Blacklight
   class SearchState
     # Modeling access to filter query parameters
     class FilterField
+      MISSING = { missing: true }.freeze
+
       # @param [Blacklight::Configuration::FacetField] config
       attr_reader :config
 
@@ -25,7 +27,9 @@ module Blacklight
       def add(item)
         new_state = search_state.reset_search
 
-        if item.respond_to?(:fq)
+        if item.try(:missing)
+          # if this is a 'missing' facet value, the :fq is only for backwards compatibility
+        elsif item.respond_to?(:fq)
           Array(item.fq).each do |f, v|
             new_state = new_state.filter(f).add(v)
           end
@@ -35,21 +39,28 @@ module Blacklight
           return new_state.filter(item.field).add(item)
         end
 
+        url_key = key
         params = new_state.params
         param = :f
         value = as_url_parameter(item)
+
+        if value == Blacklight::SearchState::FilterField::MISSING
+          url_key = "-#{key}"
+          value = Blacklight::Engine.config.facet_missing_param
+        end
+
         param = :f_inclusive if value.is_a?(Array)
 
         # value could be a string
         params[param] = (params[param] || {}).dup
 
         if value.is_a? Array
-          params[param][key] = value
+          params[param][url_key] = value
         elsif config.single
-          params[param][key] = [value]
+          params[param][url_key] = [value]
         else
-          params[param][key] = Array(params[param][key] || []).dup
-          params[param][key].push(value)
+          params[param][url_key] = Array(params[param][url_key] || []).dup
+          params[param][url_key].push(value)
         end
 
         new_state.reset(params)
@@ -63,19 +74,26 @@ module Blacklight
           return new_state.filter(item.field).remove(item)
         end
 
+        url_key = config.key
         params = new_state.params
 
         param = :f
         value = as_url_parameter(item)
+
+        if value == Blacklight::SearchState::FilterField::MISSING
+          url_key = "-#{key}"
+          value = Blacklight::Engine.config.facet_missing_param
+        end
+
         param = :f_inclusive if value.is_a?(Array)
 
         # need to dup the facet values too,
         # if the values aren't dup'd, then the values
         # from the session will get remove in the show view...
         params[param] = (params[param] || {}).dup
-        params[param][key] = (params[param][key] || []).dup
+        params[param][url_key] = (params[param][url_key] || []).dup
 
-        collection = params[param][key]
+        collection = params[param][url_key]
         # collection should be an array, because we link to ?f[key][]=value,
         # however, Facebook (and maybe some other PHP tools) tranform that parameters
         # into ?f[key][0]=value, which Rails interprets as a Hash.
@@ -84,18 +102,10 @@ module Blacklight
           collection = collection.values
         end
 
-        params[param][key] = collection - Array(value)
-        params[param].delete(key) if params[param][key].empty?
+        params[param][url_key] = collection - Array(value)
+        params[param].delete(url_key) if params[param][url_key].empty?
         params.delete(param) if params[param].empty?
 
-        # Handle missing field queries.
-        missing = I18n.t("blacklight.search.facets.missing")
-        if (item.respond_to?(:fq) && item.fq == "-#{key}:[* TO *]") ||
-           item == missing
-          params[param].delete("-#{key}:")
-          params[param].delete(key) if params[param][key] == [""]
-        end
-
         new_state.reset(params)
       end
 
@@ -104,8 +114,9 @@ module Blacklight
         params = search_state.params
         f = Array(params.dig(:f, key))
         f_inclusive = [params.dig(:f_inclusive, key)] if params.dig(:f_inclusive, key).present?
+        f_missing = [Blacklight::SearchState::FilterField::MISSING] if params.dig(:f, "-#{key}")&.any? { |v| v == Blacklight::Engine.config.facet_missing_param }
 
-        f + (f_inclusive || [])
+        f + (f_inclusive || []) + (f_missing || [])
       end
       delegate :any?, to: :values
 
@@ -121,6 +132,8 @@ module Blacklight
 
         if value.is_a?(Array)
           (params.dig(:f_inclusive, key) || []).to_set == value.to_set
+        elsif value == Blacklight::SearchState::FilterField::MISSING
+          (params.dig(:f, "-#{key}") || []).include?(Blacklight::Engine.config.facet_missing_param)
         else
           (params.dig(:f, key) || []).include?(value)
         end
@@ -130,7 +143,9 @@ module Blacklight
 
       # TODO: this code is duplicated in Blacklight::FacetsHelperBehavior
       def as_url_parameter(item)
-        if item.respond_to? :value
+        if item.respond_to?(:missing) && item.missing
+          Blacklight::SearchState::FilterField::MISSING
+        elsif item.respond_to? :value
           item.value
         else
           item

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

@@ -171,7 +171,8 @@ module Blacklight::Solr::Response::Facets
         # legacy solr facet.missing serialization
         if value.nil?
           i.label = I18n.t(:"blacklight.search.fields.facet.missing.#{facet_field_name}", default: [:"blacklight.search.facets.missing"])
-          i.fq = "-#{facet_field_name}:[* TO *]"
+          i.fq = "-#{facet_field_name}:[* TO *]" # this explicit fq is deprecated; the missing attribute below is a better thing to check for this case
+          i.value = Blacklight::SearchState::FilterField::MISSING
           i.missing = true
         end
 

data/lib/blacklight/solr/search_builder_behavior.rb

@@ -10,7 +10,6 @@ module Blacklight::Solr
         :add_facetting_to_solr, :add_solr_fields_to_query, :add_paging_to_solr,
         :add_sorting_to_solr, :add_group_config_to_solr,
         :add_facet_paging_to_solr, :add_adv_search_clauses,
-        :add_missing_field_query,
         :add_additional_filters
       ]
     end
@@ -82,19 +81,6 @@ module Blacklight::Solr
       end
     end
 
-    ##
-    # Build and append a missing field query.
-    ##
-    def add_missing_field_query(solr_parameters)
-      return unless solr_parameters["facet.missing"]
-
-      solr_parameters[:fq] = [] if solr_parameters[:fq].blank?
-
-      solr_parameters[:fq].append(*(blacklight_params["f"] || [])
-        .select { |f| f.match(/^-/) }
-        .map { |k, _v| "#{k}[* TO *]" })
-    end
-
     def add_additional_filters(solr_parameters, additional_filters = nil)
       q = additional_filters || @additional_filters
 
@@ -384,6 +370,8 @@ module Blacklight::Solr
       elsif value.is_a?(Range)
         prefix = "{!#{local_params.join(' ')}}" unless local_params.empty?
         "#{prefix}#{solr_field}:[#{value.first} TO #{value.last}]"
+      elsif value == Blacklight::SearchState::FilterField::MISSING
+        "-#{solr_field}:[* TO *]"
       else
         "{!term f=#{solr_field}#{(' ' + local_params.join(' ')) unless local_params.empty?}}#{convert_to_term_value(value)}"
       end

data/package.json

@@ -1,7 +1,7 @@
 {
   "name": "blacklight-frontend",
-  "version": "7.20.0",
-  "description": "[![Build Status](https://travis-ci.com/projectblacklight/blacklight.png?branch=master)](https://travis-ci.com/projectblacklight/blacklight) [![Gem Version](https://badge.fury.io/rb/blacklight.png)](http://badge.fury.io/rb/blacklight) [![Coverage Status](https://coveralls.io/repos/github/projectblacklight/blacklight/badge.svg?branch=master)](https://coveralls.io/github/projectblacklight/blacklight?branch=master)",
+  "version": "7.21.2",
+  "description": "[![Build Status](https://travis-ci.com/projectblacklight/blacklight.png?branch=main)](https://travis-ci.com/projectblacklight/blacklight) [![Gem Version](https://badge.fury.io/rb/blacklight.png)](http://badge.fury.io/rb/blacklight) [![Coverage Status](https://coveralls.io/repos/github/projectblacklight/blacklight/badge.svg?branch=main)](https://coveralls.io/github/projectblacklight/blacklight?branch=main)",
   "main": "app/assets/javascripts/blacklight",
   "scripts": {
     "js-compile-bundle": "shx cat app/javascript/blacklight/core.js app/javascript/blacklight/autocomplete.js app/javascript/blacklight/bookmark_toggle.js app/javascript/blacklight/button_focus.js app/javascript/blacklight/checkbox_submit.js app/javascript/blacklight/facet_load.js app/javascript/blacklight/modal.js app/javascript/blacklight/search_context.js | shx sed \"s/^(import|export).*//\" | babel --filename app/javascript/blacklight/blacklight.js > app/assets/javascripts/blacklight/blacklight.js"
@@ -23,8 +23,10 @@
   "devDependencies": {
     "@babel/cli": "^7.2.3",
     "@babel/core": "^7.2.3",
+    "@babel/preset-env": "^7.16.0",
     "shx": "^0.3.2"
   },
+  "browserslist": "> 0.25%, not dead",
   "dependencies": {
     "bloodhound-js": "^1.2.3",
     "bootstrap": ">=4.3.1 <6.0.0",

data/spec/controllers/catalog_controller_spec.rb

@@ -501,6 +501,12 @@ RSpec.describe CatalogController, api: true do
     end
 
     describe "email", api: false do
+      let(:config) { Blacklight::Configuration.new }
+
+      before do
+        allow(controller).to receive(:blacklight_config).and_return(config)
+      end
+
       it "gives error if no TO parameter" do
         post :email, params: { id: doc_id }
         expect(request.flash[:error]).to eq "You must enter a recipient in order to send this message"
@@ -518,7 +524,7 @@ RSpec.describe CatalogController, api: true do
 
       it "redirects back to the record upon success" do
         allow(RecordMailer).to receive(:email_record)
-          .with(anything, { to: 'test_email@projectblacklight.org', message: 'xyz' }, hash_including(host: 'test.host'))
+          .with(anything, { to: 'test_email@projectblacklight.org', message: 'xyz', config: config }, hash_including(host: 'test.host'))
           .and_return double(deliver: nil)
         post :email, params: { id: doc_id, to: 'test_email@projectblacklight.org', message: 'xyz' }
         expect(request.flash[:error]).to be_nil
@@ -533,6 +539,12 @@ RSpec.describe CatalogController, api: true do
     end
 
     describe "sms", api: false do
+      let(:config) { Blacklight::Configuration.new }
+
+      before do
+        allow(controller).to receive(:blacklight_config).and_return(config)
+      end
+
       it "gives error if no phone number is given" do
         post :sms, params: { id: doc_id, carrier: 'att' }
         expect(request.flash[:error]).to eq "You must enter a recipient's phone number in order to send this message"
@@ -562,7 +574,7 @@ RSpec.describe CatalogController, api: true do
       it "sends to the appropriate carrier email address" do
         expect(RecordMailer)
           .to receive(:sms_record)
-          .with(anything, { to: '5555555555@txt.att.net' }, hash_including(host: 'test.host'))
+          .with(anything, { to: '5555555555@txt.att.net', config: config }, hash_including(host: 'test.host'))
           .and_return double(deliver: nil)
         post :sms, params: { id: doc_id, to: '5555555555', carrier: 'txt.att.net' }
       end

data/spec/features/axe_spec.rb

@@ -1,34 +1,33 @@
 # frozen_string_literal: true
 
 RSpec.describe 'Accessibility testing', api: false, js: true do
-  xit 'validates the home page' do
+  it 'validates the home page' do
     visit root_path
     expect(page).to be_accessible
   end
 
-  xit 'validates the catalog page' do
+  it 'validates the catalog page' do
     visit root_path
     fill_in "q", with: 'history'
     click_button 'search'
 
-    # aria-allowed-role doesn't like nav[role="region"]
-    expect(page).to be_accessible(skipping: ['aria-allowed-role'])
+    expect(page).to be_accessible
 
     within '.card.blacklight-language_ssim' do
       click_button 'Language'
       click_link "Tibetan"
     end
 
-    expect(page).to be_accessible(skipping: ['aria-allowed-role'])
+    expect(page).to be_accessible
   end
 
-  xit 'validates the single results page' do
+  it 'validates the single results page' do
     visit solr_document_path('2007020969')
     expect(page).to be_accessible
   end
 
   def be_accessible(skipping: [])
     # typeahead does funny things with the search bar
-    be_axe_clean.excluding('.tt-hint').skipping(skipping + [('color-contrast' if Bootstrap::VERSION < '5')])
+    be_axe_clean.excluding('.tt-hint').skipping(skipping + [('color-contrast' if Bootstrap::VERSION < '5')].compact)
   end
 end

data/spec/features/facet_missing_spec.rb

@@ -32,8 +32,12 @@ RSpec.describe "Facet missing" do
   end
 
   context "unselecting the facet missing facet" do
-    it "unselects the missig field facet" do
-      visit root_path + "?f[-subject_geo_ssim:[* TO *]][]=&f[subject_geo_ssim][]="
+    it "unselects the missing field facet" do
+      visit root_path
+
+      within "#facet-subject_geo_ssim" do
+        click_link "[Missing]"
+      end
 
       within "#facet-subject_geo_ssim" do
         click_link "remove"
@@ -45,8 +49,12 @@ RSpec.describe "Facet missing" do
   end
 
   context "unselecting the facet missing constraint" do
-    it "unselects the missig field facet" do
-      visit root_path + "?f[-subject_geo_ssim:[* TO *]][]=&f[subject_geo_ssim][]="
+    it "unselects the missing field facet" do
+      visit root_path
+
+      within "#facet-subject_geo_ssim" do
+        click_link "[Missing]"
+      end
 
       within ".filter-subject_geo_ssim" do
         click_link "Remove constraint Region: [Missing]"

data/spec/lib/blacklight/search_state/filter_field_spec.rb

@@ -162,28 +162,14 @@ RSpec.describe Blacklight::SearchState::FilterField do
 
     context "With facet.missing field" do
       let(:params) do
-        { f: { some_field: [""], "-some_field:": [""] } }
+        { f: { "-some_field": ["[* TO *]"] } }
       end
 
       it "removes facet.missing facet params" do
         filter = search_state.filter("some_field")
-        new_state = filter.remove(OpenStruct.new(fq: "-some_field:[* TO *]"))
+        new_state = filter.remove(OpenStruct.new(key: 'some_field', missing: true))
 
-        expect(new_state.params).to eq("f" => {})
-      end
-    end
-
-    context "With facet.missing field value" do
-      let(:params) do
-        { f: { some_field: [""], "-some_field:": [""] } }
-      end
-
-      it "removes facet.missing facet params" do
-        missing = I18n.t("blacklight.search.facets.missing")
-        filter = search_state.filter("some_field")
-        new_state = filter.remove(missing)
-
-        expect(new_state.params).to eq("f" => {})
+        expect(new_state.params).to eq({})
       end
     end
   end

data/spec/models/blacklight/configuration_spec.rb

@@ -543,6 +543,98 @@ RSpec.describe "Blacklight::Configuration", api: true do
     end
   end
 
+  describe "add_sms_field" do
+    it "takes hash form" do
+      config.add_sms_field("title_tsim", label: "Title")
+
+      expect(config.sms_fields["title_tsim"]).not_to be_nil
+      expect(config.sms_fields["title_tsim"].label).to eq "Title"
+    end
+
+    it "takes ShowField argument" do
+      config.add_sms_field("title_tsim", Blacklight::Configuration::SmsField.new(field: "title_display", label: "Title"))
+
+      expect(config.sms_fields["title_tsim"]).not_to be_nil
+      expect(config.sms_fields["title_tsim"].label).to eq "Title"
+    end
+
+    it "takes block form" do
+      config.add_sms_field("title_tsim") do |f|
+        f.label = "Title"
+      end
+
+      expect(config.sms_fields["title_tsim"]).not_to be_nil
+      expect(config.sms_fields["title_tsim"].label).to eq "Title"
+    end
+
+    it "creates default label humanized from field" do
+      config.add_sms_field("my_custom_field")
+
+      expect(config.sms_fields["my_custom_field"].label).to eq "My Custom Field"
+    end
+
+    it "raises on nil solr field name" do
+      expect { config.add_sms_field(nil) }.to raise_error ArgumentError
+    end
+
+    it "takes wild-carded field names and dereference them to solr fields" do
+      allow(config).to receive(:reflected_fields).and_return(
+        "some_field_display" => {},
+        "another_field_display" => {},
+        "a_facet_field" => {}
+      )
+      config.add_sms_field "*_display"
+
+      expect(config.sms_fields.keys).to eq %w[some_field_display another_field_display]
+    end
+  end
+
+  describe "add_email_field" do
+    it "takes hash form" do
+      config.add_email_field("title_tsim", label: "Title")
+
+      expect(config.email_fields["title_tsim"]).not_to be_nil
+      expect(config.email_fields["title_tsim"].label).to eq "Title"
+    end
+
+    it "takes ShowField argument" do
+      config.add_email_field("title_tsim", Blacklight::Configuration::EmailField.new(field: "title_display", label: "Title"))
+
+      expect(config.email_fields["title_tsim"]).not_to be_nil
+      expect(config.email_fields["title_tsim"].label).to eq "Title"
+    end
+
+    it "takes block form" do
+      config.add_email_field("title_tsim") do |f|
+        f.label = "Title"
+      end
+
+      expect(config.email_fields["title_tsim"]).not_to be_nil
+      expect(config.email_fields["title_tsim"].label).to eq "Title"
+    end
+
+    it "creates default label humanized from field" do
+      config.add_email_field("my_custom_field")
+
+      expect(config.email_fields["my_custom_field"].label).to eq "My Custom Field"
+    end
+
+    it "raises on nil solr field name" do
+      expect { config.add_email_field(nil) }.to raise_error ArgumentError
+    end
+
+    it "takes wild-carded field names and dereference them to solr fields" do
+      allow(config).to receive(:reflected_fields).and_return(
+        "some_field_display" => {},
+        "another_field_display" => {},
+        "a_facet_field" => {}
+      )
+      config.add_email_field "*_display"
+
+      expect(config.email_fields.keys).to eq %w[some_field_display another_field_display]
+    end
+  end
+
   describe "#default_search_field" do
     it "uses the field with a :default key" do
       config.add_search_field('search_field_1')

data/spec/models/blacklight/document/email_spec.rb

@@ -1,6 +1,10 @@
 # frozen_string_literal: true
 
 RSpec.describe "Blacklight::Document::Email" do
+  let(:config) do
+    Blacklight::Configuration.new
+  end
+
   before(:all) do
     SolrDocument.use_extension(Blacklight::Document::Email)
   end
@@ -22,4 +26,32 @@ RSpec.describe "Blacklight::Document::Email" do
     doc = SolrDocument.new(id: "1234")
     expect(doc.to_email_text).to be_nil
   end
+
+  context "we pass in configuration with email fields set" do
+    it "uses the email fields for to_email_text" do
+      config.add_email_field("foo", label: "Foo:")
+      config.add_email_field("bar", label: "Bar:")
+      doc = SolrDocument.new(id: "1234", foo: ["Fuzz Fuzz", "Fizz Fizz"], bar: ["Buzz Buzz", "Bizz Bizz"])
+
+      expect(doc.to_email_text(config)).to eq("Foo: Fuzz Fuzz Fizz Fizz\nBar: Buzz Buzz Bizz Bizz")
+    end
+  end
+
+  context "we pass in configuration with email fields no set" do
+    it "falls back on default semantics setup" do
+      doc = SolrDocument.new(id: "1234", title_tsim: ["My Title", "My Alt. Title"])
+      email_body = doc.to_email_text(config)
+      expect(email_body).to match(/Title: My Title/)
+    end
+  end
+
+  context "document field is single valued" do
+    it "handles the single value field correctly" do
+      config.add_email_field("foo", label: "Foo:")
+      config.add_email_field("bar", label: "Bar:")
+      doc = SolrDocument.new(id: "1234", foo: "Fuzz Fuzz", bar: ["Buzz Buzz", "Bizz Bizz"])
+
+      expect(doc.to_email_text(config)).to eq("Foo: Fuzz Fuzz\nBar: Buzz Buzz Bizz Bizz")
+    end
+  end
 end

data/spec/models/blacklight/document/sms_spec.rb

@@ -1,6 +1,10 @@
 # frozen_string_literal: true
 
 RSpec.describe "Blacklight::Document::Email" do
+  let(:config) do
+    Blacklight::Configuration.new
+  end
+
   before(:all) do
     SolrDocument.use_extension(Blacklight::Document::Sms)
   end
@@ -22,4 +26,33 @@ RSpec.describe "Blacklight::Document::Email" do
     doc = SolrDocument.new(id: "1234")
     expect(doc.to_sms_text).to be_nil
   end
+
+  context "we pass in configuration with sms fields set" do
+    it "uses the sms fields for to_sms_text" do
+      config.add_sms_field("foo", label: "Foo:")
+      config.add_sms_field("bar", label: " by")
+      doc = SolrDocument.new(id: "1234", foo: ["Fuzz Fuzz", "Fizz Fizz"], bar: ["Buzz Buzz", "Bizz Bizz"])
+
+      expect(doc.to_sms_text(config)).to eq("Foo: Fuzz Fuzz by Buzz Buzz")
+    end
+  end
+
+  context "we pass in configuration with sms fields no set" do
+    it "falls back on default semantics setup" do
+      doc = SolrDocument.new(id: "1234", title_tsim: ["My Title", "My Alt. Title"])
+      sms_text = doc.to_sms_text(config)
+      expect(sms_text).to match(/My Title/)
+      expect(sms_text).not_to match(/My Alt\. Title/)
+    end
+  end
+
+  context "document field is single valued" do
+    it "handles the single value field correctly" do
+      config.add_sms_field("foo", label: "Foo:")
+      config.add_sms_field("bar", label: " by")
+      doc = SolrDocument.new(id: "1234", foo: "Fuzz Fuzz", bar: ["Buzz Buzz", "Bizz Bizz"])
+
+      expect(doc.to_sms_text(config)).to eq("Foo: Fuzz Fuzz by Buzz Buzz")
+    end
+  end
 end

data/spec/models/blacklight/solr/response/facets_spec.rb

@@ -155,7 +155,7 @@ RSpec.describe Blacklight::Solr::Response::Facets, api: true do
     end
 
     it "marks the facet.missing field with a human-readable label and fq" do
-      missing = subject.aggregations["some_field"].items.find { |i| i.value.nil? }
+      missing = subject.aggregations["some_field"].items.find(&:missing)
 
       expect(missing.label).to eq "[Missing]"
       expect(missing.fq).to eq "-some_field:[* TO *]"

data/spec/models/blacklight/solr/search_builder_spec.rb

@@ -27,10 +27,6 @@ RSpec.describe Blacklight::Solr::SearchBuilderBehavior, api: true do
     it "uses the class-level default_processor_chain" do
       expect(subject.processor_chain).to eq search_builder_class.default_processor_chain
     end
-
-    it "appends the :add_missing_field_query processor" do
-      expect(subject.processor_chain).to include(:add_missing_field_query)
-    end
   end
 
   context 'with merged parameters from the defaults + the search field' do
@@ -826,13 +822,4 @@ RSpec.describe Blacklight::Solr::SearchBuilderBehavior, api: true do
       expect(subject.to_hash).to include q: '{!lucene}id:(1 OR 2 OR 3)'
     end
   end
-
-  describe "#add_missing_field_query" do
-    it "precesses facet.missing query" do
-      subject.with("f" => { "-hello:" => [""] })
-      solr_params = { "facet.missing" => true, fq: [] }
-
-      expect(subject.add_missing_field_query(solr_params)).to eq(["-hello:[* TO *]"])
-    end
-  end
 end

data/spec/models/record_mailer_spec.rb

@@ -1,17 +1,24 @@
 # frozen_string_literal: true
 
 RSpec.describe RecordMailer do
+  let(:config) do
+    Blacklight::Configuration.new
+  end
+
+  let(:document) do
+    SolrDocument.new(id: "123456", format: ["book"], title_tsim: "The horn", language_ssim: "English", author_tsim: "Janetzky, Kurt")
+  end
+
   before do
     allow(described_class).to receive(:default).and_return(from: 'no-reply@projectblacklight.org')
     SolrDocument.use_extension(Blacklight::Document::Email)
     SolrDocument.use_extension(Blacklight::Document::Sms)
-    document = SolrDocument.new(id: "123456", format: ["book"], title_tsim: "The horn", language_ssim: "English", author_tsim: "Janetzky, Kurt")
     @documents = [document]
   end
 
   describe "email" do
     before do
-      details = { to: 'test@test.com', message: "This is my message" }
+      details = { to: 'test@test.com', message: "This is my message", config: config }
       @email = described_class.email_record(@documents, details, host: 'projectblacklight.org', protocol: 'https')
     end
 
@@ -42,6 +49,27 @@ RSpec.describe RecordMailer do
       @https_email = described_class.email_record(@documents, details, host: 'projectblacklight.org', protocol: 'https')
       expect(@https_email.body).to match %r{https://projectblacklight.org/}
     end
+
+    context "email title_field is configured and multi valued" do
+      let(:document) { SolrDocument.new(id: "123456", foo: ["Fizz Fizz", "Fuzz Fuzz"], format: ["book"], title_tsim: "The horn", language_ssim: "English", author_tsim: "Janetzky, Kurt") }
+
+      it "uses configured email title_field" do
+        config.email.title_field = "foo"
+        expect(@email.subject).to match("Fizz Fizz")
+        expect(@email.subject).not_to match("Fuzz Fuzz")
+        expect(@email.subject).not_to match("The horn")
+      end
+    end
+
+    context "email title_field is configured and single valued" do
+      let(:document) { SolrDocument.new(id: "123456", foo: "Fizz Fizz", format: ["book"], title_tsim: "The horn", language_ssim: "English", author_tsim: "Janetzky, Kurt") }
+
+      it "uses configured email title_field" do
+        config.email.title_field = "foo"
+        expect(@email.subject).to match("Fizz Fizz")
+        expect(@email.subject).not_to match("The horn")
+      end
+    end
   end
 
   describe "SMS" do

data/spec/spec_helper.rb

@@ -10,9 +10,6 @@ SimpleCov.start do
   add_filter "/spec/"
 end
 
-require 'rsolr'
-require 'blacklight'
-
 require 'engine_cart'
 EngineCart.load_application!
 
@@ -25,6 +22,9 @@ require 'selenium-webdriver'
 require 'equivalent-xml'
 require 'axe-rspec'
 
+require 'rsolr'
+require 'blacklight'
+
 Capybara.javascript_driver = :headless_chrome
 
 Capybara.register_driver :headless_chrome do |app|

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: blacklight
 version: !ruby/object:Gem::Version
-  version: 7.20.0
+  version: 7.21.2
 platform: ruby
 authors:
 - Jonathan Rochkind
@@ -17,7 +17,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-11-01 00:00:00.000000000 Z
+date: 2021-11-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -129,14 +129,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.42.0
+        version: '2.42'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.42.0
+        version: '2.42'
 - !ruby/object:Gem::Dependency
   name: rsolr
   requirement: !ruby/object:Gem::Requirement
@@ -378,6 +378,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".babelrc"
 - ".docker/app/Dockerfile"
 - ".docker/app/entrypoint.sh"
 - ".env"
@@ -933,7 +934,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.1.4
 signing_key: 
 specification_version: 4
 summary: Blacklight provides a discovery interface for any Solr (http://lucene.apache.org/solr)