blacklight 9.0.0 → 9.1.0

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

@@ -5,7 +5,7 @@
              method: :post do %>
 <div class="modal-body">
   <%= render '/shared/flash_msg' %>
-    <div class="row">
+    <div class="row mb-3">
       <label class="col-form-label col-sm-2" for="to">
         <%= t('blacklight.sms.form.to') %>
       </label>
@@ -13,17 +13,17 @@
         <%= telephone_field_tag :to, params[:to], class: 'form-control', required: true %>
       </div>
     </div>
-    <div class="row">
+    <div class="row mb-3">
       <label class="col-form-label col-sm-2" for="carrier">
         <%= t('blacklight.sms.form.carrier') %>
       </label>
       <div class="col-sm-10">
-          <%= select_tag(:carrier, options_for_select(sms_mappings.to_a.sort.unshift([t('blacklight.sms.form.carrier_prompt'),'']), params[:carrier]), class: 'form-control') %><br/>
+          <%= select_tag(:carrier, options_for_select(sms_mappings.to_a.sort.unshift([t('blacklight.sms.form.carrier_prompt'),'']), params[:carrier]), class: 'form-control') %><br>
       </div>
 
     </div>
     <% @documents&.each do |doc| %>
-       <%=hidden_field_tag "id[]", doc.id %>
+       <%= hidden_field_tag "id[]", doc.id %>
     <% end %>
 </div>
 <div class="modal-footer">

data/app/views/catalog/index.rss.builder

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
 xml.instruct! :xml, version: "1.0"
-xml.rss(version: "2.0") {
-  xml.channel {
+xml.rss(version: "2.0") do
+  xml.channel do
     xml.title(t('blacklight.search.page_title.title', constraints: render_search_to_page_title(params), application_name: application_name))
     xml.link(search_action_url(search_state.to_h.merge(only_path: false)))
     xml.description(t('blacklight.search.page_title.title', constraints: render_search_to_page_title(params), application_name: application_name))
@@ -10,5 +10,5 @@ xml.rss(version: "2.0") {
     @response.documents.each_with_index do |document, document_counter|
       xml << render_xml_partials(document, blacklight_config.view_config(:rss).partials, document_counter: document_counter)
     end
-  }
-}
+  end
+end

data/app/views/catalog/opensearch.json.jbuilder

@@ -0,0 +1,3 @@
+# frozen_string_literal: true
+
+json.array! [search_state.query_param] + @presenter.documents.map { |document| document_presenter(document).heading }.uniq

data/app/views/catalog/opensearch.xml.builder

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 xml.instruct! :xml, version: '1.0'
-xml.OpenSearchDescription(xmlns: 'http://a9.com/-/spec/opensearch/1.1/') {
+xml.OpenSearchDescription(xmlns: 'http://a9.com/-/spec/opensearch/1.1/') do
   xml.ShortName application_name
   xml.Description "#{application_name} Search"
   xml.Image asset_url('favicon.ico'), height: 16, width: 16, type: 'image/x-icon'
@@ -9,4 +9,4 @@ xml.OpenSearchDescription(xmlns: 'http://a9.com/-/spec/opensearch/1.1/') {
   xml.Url type: 'text/html', method: 'get', template: "#{url_for controller: 'catalog', only_path: false}?q={searchTerms}&page={startPage?}"
   xml.Url type: 'application/rss+xml', method: 'get', template: "#{url_for controller: 'catalog', only_path: false}.rss?q={searchTerms}&page={startPage?}"
   xml.Url type: 'application/x-suggestions+json', method: 'get', template: "#{url_for controller: 'catalog', action: 'opensearch', format: 'json', only_path: false}?q={searchTerms}"
-}
+end

data/app/views/layouts/blacklight/base.html.erb

@@ -6,9 +6,21 @@
     <meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
 
     <title><%= render_page_title %></title>
+
+    <% if dark_mode_support? %>
     <script>
-      document.querySelector('html').classList.remove('no-js');
+      (() => {
+        const theme = localStorage.getItem('theme') || 'auto';
+        if (theme === 'auto') {
+          document.documentElement.setAttribute('data-bs-theme', window.matchMedia('(prefers-color-scheme: dark)').matches ? 'dark' : 'light');
+        } else {
+          document.documentElement.setAttribute('data-bs-theme', theme);
+        }
+        document.querySelector('html').classList.remove('no-js');
+      })();
     </script>
+    <% end %>
+    
     <%= opensearch_description_tag application_name, opensearch_catalog_url(format: 'xml') %>
     <%= favicon_link_tag %>
     <%= stylesheet_link_tag "application", media: "all", "data-turbo-track": "reload"  %>

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

@@ -4,19 +4,32 @@
   <% end %>
 
   <% if has_user_authentication_provider? %>
-    <% if current_user %>
+    <% user = current_user if respond_to?(:current_user) %>
+    <% if user %>
+    <% if blacklight_logout_path %>
     <li class="nav-item">
-      <%= link_to t('blacklight.header_links.logout'), destroy_user_session_path, class: 'nav-link' %>
+      <%= link_to t('blacklight.header_links.logout'), blacklight_logout_path, { class: 'nav-link' }.merge(blacklight_logout_link_options) %>
     </li>
-    <% unless current_user.to_s.blank? -%>
+    <% end %>
+    <% if blacklight_account_path && user.to_s.present? -%>
     <li class="nav-item">
-      <%= link_to current_user, edit_user_registration_path, class: 'nav-link' %>
+      <%= link_to user, blacklight_account_path, class: 'nav-link' %>
     </li>
     <% end %>
     <% else %>
+    <% if blacklight_login_path %>
     <li class="nav-item">
-      <%= link_to t('blacklight.header_links.login'), new_user_session_path, class: 'nav-link' %>
+      <%= link_to t('blacklight.header_links.login'), blacklight_login_path, class: 'nav-link' %>
     </li>
     <% end %>
+    <% end %>
+  <% end %>
+
+  <% if dark_mode_support? %>
+    <li class="nav-item py-2 py-md-1 col-12 col-md-auto">
+      <div class="vr d-none d-md-flex h-100 mx-md-2 text-white"></div>
+      <hr class="d-md-none my-2 text-white-50">
+    </li>
+    <%= render Blacklight::ThemeSwitcherComponent.new %>
   <% end %>
 </ul>

data/blacklight.gemspec

@@ -23,15 +23,15 @@ Gem::Specification.new do |s|
   s.executables   = s.files.grep(%r{^exe/}) { |f| File.basename(f) }
   s.require_paths = ["lib"]
 
-  s.required_ruby_version = '>= 3.1'
+  s.required_ruby_version = '>= 3.3'
 
-  s.add_dependency "rails", '>= 7.0', '< 9'
+  s.add_dependency "rails", '>= 7.2', '< 9'
   s.add_dependency "globalid"
   s.add_dependency "jbuilder", '~> 2.7'
   s.add_dependency "kaminari", ">= 0.15" # the pagination (page 1,2,3, etc..) of our search results
   s.add_dependency "i18n", '>= 1.7.0' # added named parameters
   s.add_dependency "ostruct", '>= 0.3.2'
-  s.add_dependency "view_component", '>= 3.0', '< 5.0'
+  s.add_dependency "view_component", '~> 4.0'
   s.add_dependency "zeitwerk"
 
   s.add_development_dependency "rsolr", ">= 1.0.6", "< 3"  # Library for interacting with rSolr.

data/config/locales/blacklight.ar.yml

@@ -212,6 +212,11 @@ ar:
       success: تم الإرسال كرسالة نصية قصيرة
       text:
         url: "الرابط: %{url}"
+    theme:
+      auto: تلقائي
+      dark: داكن
+      light: فاتح
+      toggle: تبديل المظهر
     tools:
       citation: استشهاد مرجعي
       clear: مسح

data/config/locales/blacklight.ca.yml

@@ -76,6 +76,11 @@ ca:
       forget: "oblida-ho"
       save: "desa-ho"
 
+    theme:
+      auto: Automàtic
+      dark: Fosc
+      light: Clar
+      toggle: Canvia el tema
     tools:
       title: "Eines"
       citation: "Citació"

data/config/locales/blacklight.de.yml

@@ -91,8 +91,7 @@ de:
         group:
           close: Facetten ausblenden
           open: Facetten zeigen
-        missing:
-          - fehlt
+        missing: "[fehlt]"
         more_html: mehr <span class="visually-hidden">%{field_name}</span> »
         pivot:
           hide: Schließen
@@ -199,6 +198,11 @@ de:
       success: SMS verschickt
       text:
         url: "Link: %{url}"
+    theme:
+      auto: Automatisch
+      dark: Dunkel
+      light: Hell
+      toggle: Design umschalten
     tools:
       citation: Zitieren
       clear: Löschen

data/config/locales/blacklight.en.yml

@@ -217,6 +217,11 @@ en:
       success: SMS Sent
       text:
         url: "Link: %{url}"
+    theme:
+      auto: Auto
+      dark: Dark
+      light: Light
+      toggle: Toggle theme
     tools:
       citation: Cite
       clear: Clear

data/config/locales/blacklight.es.yml

@@ -198,6 +198,11 @@ es:
       success: SMS enviado
       text:
         url: "Link: %{url}"
+    theme:
+      auto: Automático
+      dark: Oscuro
+      light: Claro
+      toggle: Cambiar tema
     tools:
       citation: Cité
       clear: Borrar

data/config/locales/blacklight.fr.yml

@@ -198,6 +198,11 @@ fr:
       success: SMS envoyé
       text:
         url: "Lien : %{url}"
+    theme:
+      auto: Automatique
+      dark: Sombre
+      light: Clair
+      toggle: Changer le thème
     tools:
       citation: Citer
       clear: Effacer

data/config/locales/blacklight.hu.yml

@@ -196,6 +196,11 @@ hu:
       success: SMS elküldve
       text:
         url: "Link: %{url}"
+    theme:
+      auto: Automatikus
+      dark: Sötét
+      light: Világos
+      toggle: Téma váltása
     tools:
       citation: Hivatkozás
       clear: Eltávolítás

data/config/locales/blacklight.it.yml

@@ -91,8 +91,7 @@ it:
         group:
           close: Nascondi sfaccettature
           open: Mostra sfaccettature
-        missing:
-          - Mancante
+        missing: "[Mancante]"
         more_html: altri <span class="visually-hidden">%{field_name}</span> »
         pivot:
           hide: Chiudi
@@ -199,6 +198,11 @@ it:
       success: SMS inviato
       text:
         url: "Link: %{url}"
+    theme:
+      auto: Automatico
+      dark: Scuro
+      light: Chiaro
+      toggle: Cambia tema
     tools:
       citation: Cita
       clear: Cancella

data/config/locales/blacklight.nl.yml

@@ -196,6 +196,11 @@ nl:
       success: SMS Verzonden
       text:
         url: "Link: %{url}"
+    theme:
+      auto: Automatisch
+      dark: Donker
+      light: Licht
+      toggle: Thema wisselen
     tools:
       citation: Citeer
       clear: Leeg maken

data/config/locales/blacklight.pt-BR.yml

@@ -89,8 +89,7 @@ pt-BR:
         group:
           close: Ocultar facetas
           open: Mostrar facetas
-        missing:
-          - Ausência
+        missing: "[Ausência]"
         more_html: mais <span class="visually-hidden">%{field_name}</span> »
         pivot:
           hide: Fechar
@@ -197,6 +196,11 @@ pt-BR:
       success: SMS Enviado
       text:
         url: "Link: %{url}"
+    theme:
+      auto: Automático
+      dark: Escuro
+      light: Claro
+      toggle: Alternar tema
     tools:
       citation: Citar
       clear: Limpar

data/config/locales/blacklight.sq.yml

@@ -196,6 +196,11 @@ sq:
       success: SMS-i u dëgua
       text:
         url: "Link: %{url}"
+    theme:
+      auto: Automatik
+      dark: E errët
+      light: E çelët
+      toggle: Ndrysho temën
     tools:
       citation: Citati
       clear: Fshij

data/config/locales/blacklight.zh.yml

@@ -196,6 +196,11 @@ zh:
       success: 已发送短信
       text:
         url: "链接: %{url}"
+    theme:
+      auto: 自动
+      dark: 深色
+      light: 浅色
+      toggle: 切换主题
     tools:
       citation: 引用
       clear: 清除

data/lib/blacklight/component.rb

@@ -15,7 +15,7 @@ module Blacklight
       # provided by Blacklight.
       def sidecar_files(*args, **kwargs)
         upstream_sidecar_files(*args, **kwargs).map do |path|
-          components_path = ViewComponent::VERSION::MAJOR == 3 ? view_component_path : config.generate.path
+          components_path = config.generate.path
 
           app_path = Rails.root.join(path.slice(path.index(components_path)..-1).to_s).to_s
 

data/lib/blacklight/configuration/action_config_map_entry.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+module Blacklight
+  class Configuration::ActionConfigMapEntry < OpenStructWithHashAccess
+    # @!attribute parent_action_key
+    # Pull in the configuration for this action from another action's config
+    # @return [Symbol]
+
+    def parent_config = parent_action_key
+
+    def parent_config=(value)
+      self.parent_action_key = value
+    end
+
+    #
+    # @!attribute blacklight_config_property
+    # Pull in the configuration for this action from a top-level config
+    # @return [Symbol]
+
+    def top_level_config = blacklight_config_property
+
+    def top_level_config=(value)
+      self.blacklight_config_property = value
+    end
+
+    #
+    # @!attribute default
+    # Pull in additional default configuration for this action from a top-level config
+    # @return [Array<Symbol>]
+  end
+end

data/lib/blacklight/configuration/display_field.rb

@@ -35,6 +35,7 @@ module Blacklight
     #   @return [Blacklight::FieldPresenter]
     # @!attribute component
     #   @return [Blacklight::MetadataFieldComponent]
+    # @!attribute layout_component
 
     ##
     # Default rendering pipeline:

data/lib/blacklight/configuration/field.rb

@@ -17,20 +17,20 @@ module Blacklight
 
     def normalize! _blacklight_config = nil
       self.field ||= key
-      self.key ||= self.field
+      self.key ||= field
 
       self.label ||= default_label
 
       self.if = true if self.if.nil?
       self.unless = false if self.unless.nil?
 
-      self.field &&= self.field.to_s
+      self.field &&= field.to_s
 
       self
     end
 
     def validate!
-      raise ArgumentError, "Must supply a field name" if self.field.nil?
+      raise ArgumentError, "Must supply a field name" if field.nil?
     end
 
     def display_label(context = nil, **)
@@ -44,10 +44,10 @@ module Blacklight
     end
 
     def default_label
-      if self.key.respond_to?(:titleize)
-        self.key.titleize
+      if key.respond_to?(:titleize)
+        key.titleize
       else
-        self.key.to_s.titleize
+        key.to_s.titleize
       end
     end
 

data/lib/blacklight/configuration/fields.rb

@@ -146,7 +146,7 @@ module Blacklight
 
         @table[:reflected_fields] ||= Rails.cache.fetch("blacklight_configuration/admin/reflected_fields", expires_in: 1.hour) do
           repository.reflect_fields
-        rescue => e
+        rescue StandardError => e
           Blacklight.logger&.warn "Error retrieving field metadata: #{e}"
           false
         end

data/lib/blacklight/configuration/sort_field.rb

@@ -10,13 +10,13 @@ module Blacklight
       self.field ||= label&.parameterize
       self.field ||= sort
 
-      self.sort ||= self.field
+      self.sort ||= field
 
       self
     end
 
     def validate!
-      raise ArgumentError.new, "Must supply a sort string" if self.sort.nil?
+      raise ArgumentError.new, "Must supply a sort string" if sort.nil?
     end
   end
 end

data/lib/blacklight/configuration.rb

@@ -47,6 +47,12 @@ module Blacklight
       property :header_component, default: Blacklight::HeaderComponent
       property :full_width_layout, default: false
 
+      # @!attribute dark_mode_support
+      # @since v9.1.0
+      # @return [Boolean] Whether to enable the light/dark mode them switcher.
+      # @note Requires Bootstrap v5.3+; see: https://getbootstrap.com/docs/5.3/customize/color-modes/#dark-mode
+      property :dark_mode_support, default: true
+
       bootstrap_version = ENV['BOOTSTRAP_VERSION'].presence || "5"
       property :bootstrap_version, default: /(\d)(?:\.\d){0,2}/.match(bootstrap_version)[1].to_i
 
@@ -208,14 +214,14 @@ module Blacklight
 
       # @!attribute action_mapping
       # @since v7.16.0
-      # @return [Hash{Symbol => Blacklight::Configuration::ViewConfig}]
+      # @return [Hash{Symbol => Blacklight::Configuration::ActionConfigMapEntry}]
       property :action_mapping, default: NestedOpenStructWithHashAccess.new(
-        ViewConfig,
-        default: { top_level_config: :index },
-        show: { top_level_config: :show },
-        citation: { parent_config: :show },
-        email_record: { top_level_config: :email },
-        sms_record: { top_level_config: :sms }
+        ActionConfigMapEntry,
+        default: { blacklight_config_property: :index, default: [:index] },
+        show: { blacklight_config_property: :show },
+        citation: { parent_action_key: :show },
+        email_record: { blacklight_config_property: :email },
+        sms_record: { blacklight_config_property: :sms }
       )
 
       # @!attribute sms
@@ -548,7 +554,7 @@ module Blacklight
         conf.klass = klass
       end
     end
-    alias_method :inheritable_copy, :build
+    alias inheritable_copy build
 
     # Get a view configuration for the given view type + action. The effective
     # view configuration is inherited from:
@@ -570,12 +576,7 @@ module Blacklight
         view_type = nil
       end
 
-      @view_config[[view_type, action_name]] ||= if view_type.nil?
-                                                   action_config(action_name)
-                                                 else
-                                                   base_config = action_config(action_name)
-                                                   base_config.merge(view.fetch(view_type, {}))
-                                                 end
+      @view_config[[view_type, action_name]] ||= action_config(action_name, (view.fetch(view_type, nil) if view_type))
     end
 
     # YARD will include inline disabling as docs, cannot do multiline inside @!macro.  AND this must be separate from doc block.
@@ -688,8 +689,8 @@ module Blacklight
       end
     end
 
-    def action_config(action, default: :index)
-      action_config = action_mapping[action]
+    def action_config(action_name, view_type_specific_config, default: :index)
+      action_config = action_mapping[action_name]
       action_config ||= action_mapping[:default]
 
       if action_config.parent_config && action_config.parent_config != :default
@@ -700,9 +701,19 @@ module Blacklight
       end
       action_config = action_config.reverse_merge(action_mapping[:default]) if action_config != action_mapping[:default]
 
-      action_config = action_config.reverse_merge(self[action_config.top_level_config]) if action_config.top_level_config
-      action_config = action_config.reverse_merge(show) if default == :show && action_config.top_level_config != :show
-      action_config.reverse_merge(index)
+      view_config = if action_config.blacklight_config_property
+                      self[action_config.blacklight_config_property]
+                    else
+                      self[default]
+                    end
+
+      view_config = Array(action_config.default - [action_config.blacklight_config_property || default]).inject(view_config) do |config, top_level_config|
+        config.reverse_merge(self[top_level_config])
+      end
+
+      view_config = view_config.merge(view_type_specific_config) if view_type_specific_config
+
+      view_config
     end
   end
 end

data/lib/blacklight/parameters.rb

@@ -79,7 +79,7 @@ module Blacklight
     # Facebook's crawler turns array query parameters into a hash with numeric keys. Once we know
     # the expected parameter structure, we can unmangle those parameters to match our expected values.
     def deep_unmangle_params!(params, permitted_params)
-      permitted_params.select { |p| p.is_a?(Hash) }.each do |permission|
+      permitted_params.grep(Hash).each do |permission|
         permission.each do |key, permitted_value|
           next unless params[key].is_a?(ActionController::Parameters)