blacklight-spotlight 3.2.0 → 3.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 56b89965ba03cecc57a91706bab8ab9d93033988608116063650459bebd4182c
-  data.tar.gz: 9600146cf5c35a2d763c144d936ffd7e5bfc27a0f519e356dfba500696cbb8f1
+  metadata.gz: 8dafde1a36e1a3f3e608fd0af0c1fe94a8a19b37fcb026647cd01680bf3d6b02
+  data.tar.gz: b06879821f191b68bf252bca5f433ca74b11d030b0871890bbf1afdca990c1f7
 SHA512:
-  metadata.gz: 40df9ad1c5506195ee5601834532b10e0551e362ad546e6594e88e1f4f00bd59b23b58b788b2c0541de8f907d1cfee87ef5d37131a75a208fa0e903817061a2b
-  data.tar.gz: 8cb8bc71aa03bd906b2bdea4c51b0783d5c7be866ce0dd6ac25ac34f54c65ae106f271bd84762061c10adb795082c5cc0f358d51fcff24b947e1060b7704707d
+  metadata.gz: 6ea08929fec9c1f9381163ab0f8de5c4aa6c979ee5931be1f31095bc8f1b2197971b50a839d6d0ce15ad4a35dd444f9065921e2fdc9a36a5a530fd7992da9bd7
+  data.tar.gz: bb2d68c6f2d9ea92b5eba3f86e2dd3e5ba4368e31cf265200a32bf501370c2c4b72956605b7242a56d970f8d8fbfc3d7b27fd22e6aa0df62af6900dafae4aa88

data/README.md

@@ -9,8 +9,8 @@ Read more about what Spotlight is, our motivations for creating it, and how to i
 
 ## Requirements
 
-1. Ruby (2.5.0 or greater)
-2. Rails (5.1 or greater)
+1. Ruby (2.7 or greater)
+2. Rails (5.2 or greater)
 3. Java (7 or greater) *for Solr*
 4. ImageMagick (http://www.imagemagick.org/script/index.php) due to [carrierwave](https://github.com/carrierwaveuploader/carrierwave#adding-versions)
 

data/app/assets/stylesheets/spotlight/_breadcrumbs.scss

@@ -1,8 +1,6 @@
-.breadcrumb-item {
-  .truncated-value {
+.truncated-value {
     overflow: hidden;
     text-overflow: ellipsis;
     white-space: nowrap;
-    width: 30ex;
-  }
+    max-width: 30ex;
 }

data/app/builders/spotlight/bootstrap_breadcrumbs_builder.rb

@@ -25,7 +25,7 @@ module Spotlight
 
       html_class = 'active' if current
 
-      @context.content_tag(:li, class: "breadcrumb-item #{html_class}") do
+      @context.content_tag(:li, class: "breadcrumb-item truncated-value #{html_class}") do
         @context.link_to_unless(current, element_label(element), compute_path(element), element.options&.except(:current))
       end
     end
@@ -33,7 +33,7 @@ module Spotlight
     private
 
     def element_label(element)
-      @context.content_tag(:span, class: 'truncated-value') { compute_name(element) }
+      @context.content_tag(:span) { compute_name(element) }
     end
   end
 end

data/app/controllers/concerns/spotlight/controller.rb

@@ -67,38 +67,39 @@ module Spotlight
       end
     end
 
-    def search_action_url(*args)
+    def search_action_url(*args, **kwargs)
       if current_exhibit
-        exhibit_search_action_url(*args)
+        exhibit_search_action_url(*args, **kwargs)
       else
-        main_app.search_catalog_url(*args)
+        main_app.search_catalog_url(*args, **kwargs)
       end
     end
 
-    def search_facet_path(*args)
+    def search_facet_path(*args, **kwargs)
       if current_exhibit
-        exhibit_search_facet_path(*args)
+        exhibit_search_facet_path(*args, **kwargs)
       else
-        main_app.catalog_facet_url(*args)
+        main_app.catalog_facet_url(*args, **kwargs)
       end
     end
 
-    def exhibit_search_action_url(*args)
+    def exhibit_search_action_url(*args, **kwargs)
       options = args.extract_options!
+      options = options.merge(kwargs)
       only_path = options[:only_path]
       options.except! :exhibit_id, :only_path
 
       if only_path
-        spotlight.search_exhibit_catalog_path(current_exhibit, *args, options)
+        spotlight.search_exhibit_catalog_path(current_exhibit, *args, **options)
       else
-        spotlight.search_exhibit_catalog_url(current_exhibit, *args, options)
+        spotlight.search_exhibit_catalog_url(current_exhibit, *args, **options)
       end
     end
 
-    def exhibit_search_facet_path(*args)
+    def exhibit_search_facet_path(*args, **kwargs)
       options = args.extract_options!
-      options = Blacklight::Parameters.sanitize(params.to_unsafe_h.with_indifferent_access).merge(options).except(:exhibit_id, :only_path)
-      spotlight.facet_exhibit_catalog_url(current_exhibit, *args, options)
+      options = Blacklight::Parameters.sanitize(params.to_unsafe_h.with_indifferent_access).merge(options).merge(kwargs).except(:exhibit_id, :only_path)
+      spotlight.facet_exhibit_catalog_url(current_exhibit, *args, **options&.symbolize_keys)
     end
   end
 end

data/app/controllers/spotlight/attachments_controller.rb

@@ -17,7 +17,7 @@ module Spotlight
 
     private
 
-    # Only allow a trusted parameter "white list" through.
+    # Only allow trusted parameters through.
     def attachment_params
       params.require(:attachment).permit(:name, :file, :uid)
     end

data/app/controllers/spotlight/custom_search_fields_controller.rb

@@ -57,7 +57,7 @@ module Spotlight
       add_breadcrumb t(:'spotlight.configuration.sidebar.search_configuration'), edit_exhibit_search_configuration_path(@exhibit)
     end
 
-    # Only allow a trusted parameter "white list" through.
+    # Only allow trusted parameters through.
     def custom_search_field_params
       params.require(:custom_search_field).permit(:slug, :field, :label)
     end

data/app/controllers/spotlight/pages_controller.rb

@@ -35,7 +35,7 @@ module Spotlight
 
     # GET /pages/1
     def show
-      fresh_when([@page.exhibit, @page, request.query_parameters.to_query])
+      fresh_when([@page.exhibit, @page])
     end
 
     def preview
@@ -189,7 +189,7 @@ module Spotlight
       add_breadcrumb t(key, title: current_exhibit.title), spotlight.exhibit_root_path(current_exhibit)
     end
 
-    # Only allow a trusted parameter "white list" through.
+    # Only allow trusted parameters through.
     def page_params
       params.require(controller_name.singularize).permit(allowed_page_params)
     end

data/app/controllers/spotlight/roles_controller.rb

@@ -7,7 +7,7 @@ module Spotlight
   class RolesController < Spotlight::ApplicationController
     before_action :authenticate_user!
     load_and_authorize_resource :exhibit, class: Spotlight::Exhibit
-    load_and_authorize_resource through: :exhibit, except: [:update_all]
+    load_and_authorize_resource through: :exhibit, except: %i[index update_all]
 
     def index
       role = @exhibit.roles.build

data/app/controllers/spotlight/searches_controller.rb

@@ -117,9 +117,14 @@ module Spotlight
     end
 
     def query_params
-      params.to_unsafe_h.with_indifferent_access.except(:exhibit_id, :search, *blacklisted_search_session_params).reject { |_k, v| v.blank? }
+      params.to_unsafe_h.with_indifferent_access.except(:exhibit_id, :search, *excluded_search_session_params).reject { |_k, v| v.blank? }
     end
 
+    def excluded_search_session_params
+      blacklisted_search_session_params
+    end
+
+    # @deprecated
     def blacklisted_search_session_params
       %i[id commit counter total search_id page per_page authenticity_token utf8 action controller]
     end

data/app/helpers/spotlight/application_helper.rb

@@ -162,7 +162,7 @@ module Spotlight
       if current_exhibit_theme && current_exhibit.theme != 'default'
         stylesheet_link_tag "#{tag}_#{current_exhibit_theme}"
       else
-        Rails.logger.warn "Exhibit theme '#{current_exhibit_theme}' not in white-list of available themes: #{current_exhibit.themes}"
+        Rails.logger.warn "Exhibit theme '#{current_exhibit_theme}' not in the list of available themes: #{current_exhibit.themes}"
         stylesheet_link_tag(tag)
       end
     end

data/app/helpers/spotlight/main_app_helpers.rb

@@ -40,7 +40,7 @@ module Spotlight
       if current_exhibit_theme && current_exhibit&.theme != 'default'
         stylesheet_link_tag "#{tag}_#{current_exhibit_theme}"
       else
-        Rails.logger.warn "Exhibit theme '#{current_exhibit_theme}' not in white-list of available themes: #{current_exhibit&.themes}"
+        Rails.logger.warn "Exhibit theme '#{current_exhibit_theme}' not in the list of available themes: #{current_exhibit&.themes}"
         stylesheet_link_tag(tag)
       end
     end

data/app/helpers/spotlight/pages_helper.rb

@@ -16,6 +16,18 @@ module Spotlight
       end
     end
 
+    def sir_trevor_format(text, format: :markdown)
+      if format.to_s.to_sym == :html
+        sir_trevor_html(text)
+      else
+        sir_trevor_markdown(text)
+      end
+    end
+
+    def sir_trevor_html(text)
+      sanitize(text, tags: %w[b i a br p sup])
+    end
+
     ##
     # Override the default #sir_trevor_markdown so we can use
     # a more complete markdown rendered

data/app/helpers/spotlight/title_helper.rb

@@ -27,10 +27,8 @@ module Spotlight
       content_tag(:h1, html_content, class: 'page-header')
     end
 
-    # rubocop:disable Naming/AccessorMethodName
     def set_html_page_title(title = nil)
       @page_title = strip_tags(t(:'spotlight.html_title', title: title || t(:'.title', default: :'.header'), application_name: application_name)).html_safe
     end
-    # rubocop:enable Naming/AccessorMethodName
   end
 end

data/app/mailers/spotlight/confirmation_mailer.rb

@@ -6,8 +6,9 @@ module Spotlight
   class ConfirmationMailer < ActionMailer::Base
     include Devise::Mailers::Helpers
 
-    def confirmation_instructions(record, token, opts)
+    def confirmation_instructions(record, token, opts, exhibit: nil)
       @token = token
+      @exhibit = exhibit
       initialize_from_record(record)
       mail headers_for(:confirmation_instructions, opts)
     end

data/app/models/sir_trevor_rails/block.rb

@@ -0,0 +1,85 @@
+# frozen_string_literal: true
+
+require 'ostruct'
+
+module SirTrevorRails
+  # Forked from (former) upstream sir-trevor-rails 0.6.2 gem:
+  # https://github.com/madebymany/sir-trevor-rails/blob/931b9554f5268158b4da8817477cdc82e4e2e69c/lib/sir_trevor_rails/block.rb
+  # Copyright (c) 2013-2014 by ITV plc - http://www.itv.com
+  class Block < OpenStruct
+    DEFAULT_FORMAT = :markdown
+
+    def self.from_hash(hash, parent = nil)
+      hash = hash.deep_dup.with_indifferent_access
+      type_klass(hash).new(hash, parent)
+    end
+
+    def format
+      send(:[], :format).present? ? send(:[], :format).to_sym : DEFAULT_FORMAT
+    end
+
+    # Sets a list of custom block types to speed up lookup at runtime.
+    def self.custom_block_types
+      # You can define your custom block types directly here or in your engine config.
+      Spotlight::Engine.config.sir_trevor_widgets
+    end
+
+    def initialize(hash, parent)
+      @raw_data = hash
+      @parent  = parent
+      @type    = hash[:type].to_sym
+      super(hash[:data])
+    end
+
+    attr_reader :parent, :type
+
+    def to_partial_path
+      "sir_trevor/blocks/#{self.class.name.demodulize.underscore}"
+    end
+
+    def as_json(*_attrs)
+      {
+        type: @type.to_s,
+        data: marshal_dump
+      }
+    end
+
+    # Infers the block class.
+    # Safe lookup that tries to identify user created block class.
+    #
+    # @param [Symbol] type
+    def self.block_class(type)
+      type_name = type.to_s.camelize
+      block_name = "#{type_name}Block"
+      if custom_block_types.include?(type_name)
+        begin
+          block_name.constantize
+        rescue NameError
+          block_class!(block_name)
+        end
+      else
+        block_class!(block_name)
+      end
+    end
+
+    # Infers the block class.
+    # Failover from block_class.
+    # Safe lookup against the SirTevor::Blocks namespace
+    # If no block is found, create one with given name and inherit from Block class
+    #
+    # @param [Constant] block_name
+    def self.block_class!(block_name)
+      SirTrevorRails::Blocks.const_get(block_name)
+    rescue NameError
+      SirTrevorRails::Blocks.const_set(block_name, Class.new(Block))
+    end
+
+    def self.type_klass(hash)
+      if self == Block
+        block_class(hash[:type].to_sym)
+      else
+        self
+      end
+    end
+  end
+end

data/app/models/sir_trevor_rails/blocks/displayable.rb

@@ -14,7 +14,7 @@ module SirTrevorRails
       end
 
       def ordered_items
-        items.sort_by { |x| x[:weight] }.map { |x| x[:id] }
+        items.sort_by { |x| x[:weight].to_i }.map { |x| x[:id] }
       end
 
       private

data/app/models/spotlight/contact_email.rb

@@ -33,7 +33,7 @@ module Spotlight
     end
 
     def send_devise_notification(notification, *args)
-      notice = notification_mailer.send(notification, self, *args)
+      notice = notification_mailer.send(notification, self, *args, exhibit: exhibit)
       if notice.respond_to? :deliver_now
         notice.deliver_now
       else

data/app/models/spotlight/page_configurations.rb

@@ -57,7 +57,7 @@ module Spotlight
     end
 
     def available_view_configs
-      available_view_fields.map { |k, v| { key: k, label: v.display_label(k) } }
+      available_view_fields.map { |k, v| { key: k, label: v.display_label } }
     end
 
     def attachment_endpoint

data/app/models/spotlight/page_content/sir_trevor.rb

@@ -5,12 +5,21 @@ module Spotlight
     # Sir-Trevor created content
     class SirTrevor
       def self.parse(page, attribute)
-        content = page.read_attribute(attribute)
-        content ||= [].to_json
+        blocks = page.read_attribute(attribute)
+        blocks ||= [].to_json
 
-        return SirTrevorRails::BlockArray.new if content.blank?
+        return [] if blocks.blank?
 
-        SirTrevorRails::BlockArray.from_json(content, page)
+        blocks = JSON.parse(blocks, symbolize_names: true) if blocks.is_a?(String)
+
+        if blocks.is_a?(Hash)
+          blocks = blocks[:data] || blocks['data'] or
+            raise IndexError, 'No block data found'
+        end
+
+        blocks.map do |obj|
+          SirTrevorRails::Block.from_hash(obj, page)
+        end
       end
     end
   end

data/app/models/translation.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+unless defined?(Translation)
+  Translation = I18n::Backend::ActiveRecord::Translation
+  Translation.include Spotlight::CustomTranslationExtension
+
+  # Work-around for https://github.com/svenfuchs/i18n-active_record/pull/133
+  if Translation.respond_to?(:to_hash)
+    class << Translation
+      alias to_h to_hash
+      remove_method :to_hash
+    end
+
+    I18n::Backend::ActiveRecord.define_method(:init_translations) do
+      @translations = Translation.to_h
+    end
+  end
+end

data/app/services/spotlight/exhibit_import_export_service.rb

@@ -49,7 +49,7 @@ module Spotlight
       deserialize_featured_image(exhibit, :masthead, hash[:masthead]) if hash[:masthead]
       deserialize_featured_image(exhibit, :thumbnail, hash[:thumbnail]) if hash[:thumbnail]
 
-      exhibit.blacklight_configuration.update hash[:blacklight_configuration].deep_stringify_keys if hash[:blacklight_configuration]
+      exhibit.blacklight_configuration.update hash[:blacklight_configuration].with_indifferent_access if hash[:blacklight_configuration]
 
       hash[:main_navigations].each do |attr|
         ar = exhibit.main_navigations.find_or_initialize_by(nav_type: attr[:nav_type])

data/app/views/shared/_site_sidebar.html.erb

@@ -7,7 +7,7 @@
     <li class="nav-item"><%= link_to t(:'spotlight.sites.edit.page_title'), spotlight.edit_site_path, class: 'nav-link' %></li>
   <% end %>
 
-  <% if can? :manange, Spotlight::Exhibit %>
+  <% if can? :manage, Spotlight::Exhibit %>
     <li class="nav-item"><%= link_to t(:'spotlight.sites.edit_exhibits.page_title'), spotlight.edit_site_exhibits_path, class: 'nav-link' %></li>
     <li class="nav-item"><%= link_to t(:'spotlight.admin_users.index.page_title'), spotlight.admin_users_path, class: 'nav-link' %></li>
   <% end %>

data/app/views/spotlight/catalog/admin.html.erb

@@ -10,7 +10,7 @@
 <%- if @response.empty? %>
   <%= render "zero_results" %>
 <%- else %>
-  <%= render_document_index %>
+  <%= render_document_index(@response.documents) %>
 <%- end %>
 
 <%= render 'results_pagination' %>

data/app/views/spotlight/confirmation_mailer/confirmation_instructions.html.erb

@@ -1,5 +1,5 @@
 <p><%= t(:'.welcome', email: @email) %></p>
-
-<p><%= t(:'.instructions') %></p>
-
+<br /><br />
+<%= simple_format t(:'.instructions', exhibit_title: @exhibit&.title) %>
+<br /><br />
 <p><%= link_to t(:'.confirm'), spotlight.contact_email_confirmation_url(:confirmation_token => @token) %></p>

data/app/views/spotlight/exhibits/_import.html.erb

@@ -4,7 +4,7 @@
       <p class="instructions"><%= t :'.instructions' %></p>
       <div class="form-group">
         <%= file_field_tag :file, class: 'form-control' %>
-        <%= hidden_field_tag :tab, 'import' %>
+        <%= hidden_field_tag :tab, 'import', id: nil %>
       </div>
       <div class="form-actions">
         <div class="primary-actions">

data/app/views/spotlight/exhibits/_languages.html.erb

@@ -4,7 +4,7 @@
   <%= bootstrap_form_for [current_exhibit, Spotlight::Language.new], layout: :horizontal, html: { class: 'row' }, label_col: "col-sm-3", control_col: "col-sm-9" do |f| %>
     <div class='col-sm-6'>
       <%= f.select('locale', add_exhibit_language_dropdown_options, prompt: t('.selection_prompt')) %>
-      <%= hidden_field_tag :tab, 'language' %>
+      <%= hidden_field_tag :tab, 'language', id: nil %>
     </div>
     <div class='col-sm-6'>
       <%= f.submit nil, class: 'btn btn-primary' %>
@@ -48,7 +48,7 @@
       </div>
       <div class="form-actions">
         <div class="primary-actions">
-          <%= hidden_field_tag :tab, 'language' %>
+          <%= hidden_field_tag :tab, 'language', id: nil %>
           <%= f.submit nil, class: 'btn btn-primary' %>
         </div>
       </div>

data/app/views/spotlight/filters/_form.html.erb

@@ -4,7 +4,7 @@
 
     <%= f.text_field :field %>
     <%= f.text_field :value %>
-    <%= hidden_field_tag :tab, 'filter' %>
+    <%= hidden_field_tag :tab, 'filter', id: nil %>
 
     <div class="form-actions">
       <div class="primary-actions">

data/app/views/spotlight/resources/upload/_form.html.erb

@@ -11,7 +11,7 @@
   <% end %>
   <div class="form-actions">
     <div class="primary-actions">
-      <%= hidden_field_tag :tab, 'upload' %>
+      <%= hidden_field_tag :tab, 'upload', id: nil %>
       <%= cancel_link @resource, :back, class: 'btn-sizing' %>
       <%= f.submit t('.add_item_and_continue'), name: 'add-and-continue', class: 'btn btn-secondary' %>
       <%= f.submit t('.add_item'), class: 'btn btn-primary' %>

data/app/views/spotlight/search_configurations/_document_index_view_types.html.erb

@@ -1,7 +1,7 @@
 <%= f.form_group :document_index_view_types, label: {text: t(:'.label'), class: 'pt-0'} do %>
   <%= f.fields_for :document_index_view_types, @blacklight_configuration.document_index_view_types_selected_hash do |vt| %>
     <% @blacklight_configuration.default_blacklight_config.view.select { |_k, v| v.if != false }.each do |key, view| %>
-      <%= vt.check_box key, label: view.display_label(key) %>
+      <%= vt.check_box key, label: view.display_label %>
     <% end %>
   <% end %>
 <% end %>

data/app/views/spotlight/search_configurations/_facet_metadata.html.erb

@@ -1,8 +1,10 @@
 <%= content_tag :span, t(:'.document_count', count: metadata[:document_count]) %>
 <% if metadata[:document_count] > 0 %>
   •
-  <% if metadata[:value_count] > Spotlight::FieldMetadata::FACET_LIMIT %>
-    <%= content_tag :span, t(:'.too_many_values_count', count: Spotlight::FieldMetadata::FACET_LIMIT) %>
+  <% limit = field_config&.limit || Spotlight::FieldMetadata::FACET_LIMIT %>
+  <% limit = blacklight_config.default_facet_limit if limit == true %>
+  <% if metadata[:value_count] >= limit %>
+    <%= content_tag :span, t(:'.too_many_values_count', count: limit) %>
   <% else %>
     <%= content_tag :span, t(:'.value_count', count: metadata[:value_count]) %>
   <% end %>

data/app/views/spotlight/search_configurations/_facets.html.erb

@@ -16,7 +16,7 @@
               </div>
             <% when :additional_options  %>
               <div class="facet-metadata mr-3">
-                <%= render partial: 'facet_metadata', locals: { metadata: metadata } %>
+                <%= render partial: 'facet_metadata', locals: { blacklight_config: @blacklight_configuration.blacklight_config, field_config: config, metadata: metadata } %>
               </div>
             <% else %>
             <% end %>

data/app/views/spotlight/searches/_form.html.erb

@@ -40,7 +40,7 @@
         <% end %>
         <%= f.form_group label: { text: t(:".default_index_view_type"), class: 'pt-0' } do %>
           <% available_document_index_views.each do |key, view| %>
-            <%= f.radio_button :default_index_view_type, key, label: view.display_label(key) %>
+            <%= f.radio_button :default_index_view_type, key, label: view.display_label %>
           <% end %>
         <% end %>
         <% unless @search.query_params.blank? %>

data/app/views/spotlight/sir_trevor/blocks/_video_block.html.erb

@@ -0,0 +1,5 @@
+<% if video.source == 'vimeo' %>
+  <%= render partial: '/spotlight/sir_trevor/blocks/videos/vimeo', vimeo: video %>
+<% elsif video.source == 'youtube' %>
+  <%= render partial: '/spotlight/sir_trevor/blocks/videos/youtube', youtube: video %>
+<% end %>

data/app/views/spotlight/translations/_browse_categories.html.erb

@@ -2,7 +2,7 @@
   <%= bootstrap_form_for current_exhibit, url: spotlight.exhibit_translations_path(current_exhibit), layout: :horizontal do |f| %>
     <% # Add hidden fields for the language and tab so the redirect knows how to come back here %>
     <%= hidden_field_tag :language, @language %>
-    <%= hidden_field_tag :tab, 'browse' %>
+    <%= hidden_field_tag :tab, 'browse', id: nil %>
     <div class="row">
       <div class="col-4 text-right">
         <span class="browse-translations-header">

data/app/views/spotlight/translations/_groups.html.erb

@@ -2,7 +2,7 @@
   <%= bootstrap_form_for current_exhibit, url: spotlight.exhibit_translations_path(current_exhibit), layout: :horizontal do |f| %>
     <% # Add hidden fields for the language and tab so the redirect knows how to come back here %>
     <%= hidden_field_tag :language, @language %>
-    <%= hidden_field_tag :tab, 'groups' %>
+    <%= hidden_field_tag :tab, 'groups', id: nil %>
 
     <% current_exhibit.groups.each do |group| %>
       <% title_translation = Translation.find_or_initialize_by(exhibit: current_exhibit, key: "#{group.slug}.title", locale: @language) %>

data/app/views/spotlight/translations/_import.html.erb

@@ -8,7 +8,7 @@
 
       <div class="input-group col px-0">
         <%= file_field_tag :file, class: 'form-control' %>
-        <%= hidden_field_tag :tab, 'import' %>
+        <%= hidden_field_tag :tab, 'import', id: nil %>
         <div class="input-group-append">
           <%= f.submit t(:'.import_submit'), class: 'btn btn-primary' %>
         </div>

data/app/views/spotlight/translations/_metadata.html.erb

@@ -2,7 +2,7 @@
   <%= bootstrap_form_for current_exhibit, url: spotlight.exhibit_translations_path(current_exhibit), layout: :horizontal do |f| %>
     <% # Add a hidden field for the language so the redirect knows how to come back here %>
     <%= hidden_field_tag :language, @language %>
-    <%= hidden_field_tag :tab, 'metadata' %>
+    <%= hidden_field_tag :tab, 'metadata', id: nil %>
 
     <% non_custom_metadata_fields.each do |key, field_config| %>
       <% translation = Translation.find_or_initialize_by(exhibit: current_exhibit, key: "blacklight.search.fields.#{key}", locale: @language) %>

data/app/views/spotlight/translations/_search_fields.html.erb

@@ -2,7 +2,7 @@
   <%= bootstrap_form_for current_exhibit, url: spotlight.exhibit_translations_path(current_exhibit), layout: :horizontal do |f| %>
     <% # Add hidden fields for the language and tab so the redirect knows how to come back here %>
     <%= hidden_field_tag :language, @language %>
-    <%= hidden_field_tag :tab, 'search_fields' %>
+    <%= hidden_field_tag :tab, 'search_fields', id: nil %>
 
     <div class='translation-field-based-search-fields'>
       <h2 class='translation-subheading'>

data/config/i18n-tasks.yml

@@ -65,6 +65,7 @@ ignore_unused:
   - helpers.action.{edit,edit_long,new,view} # app/helpers/spotlight/crud_link_helpers.rb
   - helpers.action.spotlight/search.edit_long # app/views/spotlight/searches/_search.html.erb
   - spotlight.about_pages.page_options.published # app/views/spotlight/about_pages/_page_options.html.erb
+  - spotlight.job_trackers.show.messages.status.{completed,enqueued,failed,missing,pending} # app/views/spotlight/job_trackers/show.html.erb
   - helpers.submit.contact_form.create # app/views/spotlight/shared/_report_a_problem.html.erb
   - activerecord.help.spotlight/exhibit.tag_list # app/views/spotlight/exhibits/_form.html.erb
   - helpers.label.solr_document.exhibit_tag_list # app/views/spotlight/catalog/_edit_default.html.erb
@@ -77,7 +78,7 @@ ignore_unused:
   # TODO Look into these as its unclear
   - activerecord.models.spotlight.page
   #- # perhaps removed here? https://github.com/projectblacklight/spotlight/commit/d4fdf04565ab3d648f0cb2a1238d84f262509fcd
-  - devise.mailer.invitation_instructions.subject # Does this even work? https://github.com/projectblacklight/spotlight/blob/master/app/mailers/spotlight/invitation_mailer.rb#L16
+  - devise.mailer.*.* # Does this even work? https://github.com/projectblacklight/spotlight/blob/master/app/mailers/spotlight/invitation_mailer.rb#L16
   - helpers.submit.solr_document.{batch_error,batch_updated,create,created,destroyed,submit,update,updated} # Do we need this?
   - helpers.submit.user.{batch_error,batch_updated,create,created,destroyed,submit,update,updated} # Do we need this?
   - helpers.action.destroy # no idea here