blacklight 6.14.0 → 7.0.0

data/lib/blacklight/configuration.rb

@@ -4,31 +4,28 @@ module Blacklight
   # Blacklight::Configuration holds the configuration for a Blacklight::Controller, including
   # fields to display, facets to show, sort options, and search fields.
   class Configuration < OpenStructWithHashAccess
-    # Using required_dependency to work around Rails autoloading
-    # problems when developing blacklight. Without this, any change
-    # to this class breaks other classes in this namespace
-
-    require_dependency 'blacklight/configuration/context'
-    require_dependency 'blacklight/configuration/view_config'
-    require_dependency 'blacklight/configuration/tool_config'
-    # XXX this isn't very pretty, but it works.
-    require_dependency 'blacklight/configuration/fields'
-    require_dependency 'blacklight/configuration/field'
-    require_dependency 'blacklight/configuration/null_field'
-    require_dependency 'blacklight/configuration/search_field'
-    require_dependency 'blacklight/configuration/facet_field'
-    require_dependency 'blacklight/configuration/sort_field'
+    extend ActiveSupport::Autoload
+    eager_autoload do
+      autoload :Context
+      autoload :ViewConfig
+      autoload :ToolConfig
+      autoload :Fields
+      autoload :Field
+      autoload :NullField
+      autoload :SearchField
+      autoload :FacetField
+      autoload :SortField
+    end
+
     include Fields
 
-    # Set up Blacklight::Configuration.default_values to contain
-    # the basic, required Blacklight fields
+    # Set up Blacklight::Configuration.default_values to contain the basic, required Blacklight fields
     class << self
+      # rubocop:disable Metrics/MethodLength
       def default_values
         @default_values ||= begin
           {
-          ##
           # === Search request configuration
-          ##
           # HTTP method to use when making requests to solr; valid
           # values are :get and :post.
           http_method: :get,
@@ -38,24 +35,14 @@ module Blacklight
           default_solr_params: {},
           ##
           # === Single document request configuration
-          ##
-          # The solr rqeuest handler to use when requesting only a single document
-          document_solr_request_handler: 'document',
-          # THe path to send single document requests to solr
-          document_solr_path: nil,
-          document_unique_id_param: :id,
+          # The solr request handler to use when requesting only a single document
+          document_solr_request_handler: nil,
+          # The path to send single document requests to solr
+          document_solr_path: 'get',
+          document_unique_id_param: :ids,
           # Default values of parameters to send when requesting a single document
-          default_document_solr_params: {
-            ## Blacklight provides these settings in the /document request handler
-            ## by default, we just ask for all fields.
-            #fl: '*',
-            ## this is a fancy way to say "find the document by id using
-            ## the value in the id query parameter"
-            #q: "{!term f=#{unique_key} v=$id}",
-            ## disable features we don't need
-            #facet: false,
-            #rows: 1
-          },
+          default_document_solr_params: {},
+          fetch_many_document_params: {},
           document_pagination_params: {},
           ##
           # == Response models
@@ -67,16 +54,13 @@ module Blacklight
           response_model: nil,
           # the model to use for each response document
           document_model: nil,
-          # document presenter class used by helpers and views
-          document_presenter_class: nil, # deprecated
           # Class for paginating long lists of facet fields
           facet_paginator_class: nil,
           # repository connection configuration
           connection_config: nil,
           ##
           # == Blacklight view configuration
-          ##
-          navbar: OpenStructWithHashAccess.new(partials: { }),
+          navbar: OpenStructWithHashAccess.new(partials: {}),
           # General configuration for all views
           index: ViewConfig::Index.new(
             # document presenter class used by helpers and views
@@ -98,10 +82,10 @@ module Blacklight
           show: ViewConfig::Show.new(
             # document presenter class used by helpers and views
             document_presenter_class: nil,
-            # default route parameters for 'show' requests
-            # set this to a hash with additional arguments to merge into
-            # the route, or set `controller: :current` to route to the
-            # current controller.
+            display_type_field: 'format',
+            # Default route parameters for 'show' requests.
+            # Set this to a hash with additional arguments to merge into the route,
+            # or set `controller: :current` to route to the current controller.
             route: nil,
             # partials to render for each document(see #render_document_partials)
             partials: [:show_header, :show],
@@ -109,17 +93,17 @@ module Blacklight
           ),
           # Configurations for specific types of index views
           view: NestedOpenStructWithHashAccess.new(ViewConfig,
-            'list',
-            atom: {
-              if: false, # by default, atom should not show up as an alternative view
-              partials: [:document]
-            },
-            rss: {
-              if: false, # by default, rss should not show up as an alternative view
-              partials: [:document]
-          }),
+                                                   list: {},
+                                                   atom: {
+                                                     if: false, # by default, atom should not show up as an alternative view
+                                                     partials: [:document]
+                                                   },
+                                                   rss: {
+                                                     if: false, # by default, rss should not show up as an alternative view
+                                                     partials: [:document]
+                                                 }),
           #
-          # These fields are created and managed below by `defined_field_access`
+          # These fields are created and managed below by `define_field_access`
           # facet_fields
           # index_fields
           # show_fields
@@ -127,7 +111,6 @@ module Blacklight
           # search_fields
           ##
           # === Blacklight behavior configuration
-          ##
           # Maxiumum number of spelling suggestions to offer
           spell_max: 5,
           # Maximum number of results to show per page
@@ -141,9 +124,12 @@ module Blacklight
           default_more_limit: 20,
           # proc for determining whether the session is a crawler/bot
           # ex.: crawler_detector: lambda { |req| req.env['HTTP_USER_AGENT'] =~ /bot/ }
-          crawler_detector: nil
+          crawler_detector: nil,
+          autocomplete_suggester: 'mySuggester',
+          raw_endpoint: OpenStructWithHashAccess.new(enabled: false)
           }
         end
+        # rubocop:enable Metrics/MethodLength
       end
     end
 
@@ -177,6 +163,11 @@ module Blacklight
       super || ::SolrDocument
     end
 
+    # A class that builds documents
+    def document_factory
+      super || Blacklight::DocumentFactory
+    end
+
     # only here to support alias_method
     def document_model=(*args)
       super
@@ -186,17 +177,6 @@ module Blacklight
       super || Blacklight::Solr::Response
     end
 
-    # @deprecated
-    def document_presenter_class
-      super || Blacklight::DocumentPresenter
-    end
-
-    # @deprecated
-    def document_presenter_class=(klass)
-      super
-    end
-    deprecation_deprecate :document_presenter_class= => "replaced by show.presenter_class and index.presenter_class"
-
     def response_model=(*args)
       super
     end
@@ -205,6 +185,10 @@ module Blacklight
       super || Blacklight::Solr::Repository
     end
 
+    def repository
+      repository_class.new(self)
+    end
+
     def connection_config
       super || Blacklight.connection_config
     end
@@ -234,16 +218,14 @@ module Blacklight
     # Returns default search field, used for simpler display in history, etc.
     # if not set, defaults to first defined search field
     def default_search_field
-      field = super
-      field ||= search_fields.values.find { |f| f.default == true }
+      field = super || search_fields.values.find { |f| f.default == true }
       field || search_fields.values.first
     end
 
     # Returns default sort field, used for simpler display in history, etc.
     # if not set, defaults to first defined sort field
     def default_sort_field
-      field = super
-      field ||= sort_fields.values.find { |f| f.default == true }
+      field = super || sort_fields.values.find { |f| f.default == true }
       field || sort_fields.values.first
     end
 
@@ -271,11 +253,10 @@ module Blacklight
       if fields.empty?
         self.add_facet_fields_to_solr_request = true
       else
-        facet_fields.slice(*fields).each { |_k, v| v.include_in_request = true }
+        facet_fields.slice(*fields).each_value { |v| v.include_in_request = true }
       end
     end
 
-    ##
     # Add any configured facet fields to the default solr parameters hash
     # @overload add_field_configuration_to_solr_request!
     #    add all index, show, and facet fields to the solr request
@@ -285,9 +266,9 @@ module Blacklight
       if fields.empty?
         self.add_field_configuration_to_solr_request = true
       else
-        index_fields.slice(*fields).each { |_k, v| v.include_in_request = true }
-        show_fields.slice(*fields).each { |_k, v| v.include_in_request = true }
-        facet_fields.slice(*fields).each { |_k, v| v.include_in_request = true }
+        index_fields.slice(*fields).each_value { |v| v.include_in_request = true }
+        show_fields.slice(*fields).each_value { |v| v.include_in_request = true }
+        facet_fields.slice(*fields).each_value { |v| v.include_in_request = true }
       end
     end
 
@@ -304,17 +285,22 @@ module Blacklight
         end
       end
     end
-    alias_method :inheritable_copy, :deep_copy
 
-    ##
+    # builds a copy for the provided controller class
+    def build(klass)
+      deep_copy.tap do |conf|
+        conf.klass = klass
+      end
+    end
+    alias_method :inheritable_copy, :build
+
     # Get a view configuration for the given view type
     # including default values from the index configuration
+    # @param [Symbol,#to_sym] view_type
+    # @return [Blacklight::Configuration::ViewConfig]
     def view_config(view_type)
-      if view_type == :show
-        self.index.merge self.show
-      else
-        self.index.merge view.fetch(view_type, {})
-      end
+      view_type = view_type.to_sym unless view_type.is_a? Symbol
+      index.merge(view_type == :show ? show : view.fetch(view_type, {}))
     end
 
     # YARD will include inline disabling as docs, cannot do multiline inside @!macro.  AND this must be separate from doc block.
@@ -329,6 +315,7 @@ module Blacklight
     def add_show_tools_partial(name, opts = {})
       opts[:partial] ||= 'document_action'
       add_action(show.document_actions, name, opts)
+      klass && ActionBuilder.new(klass, name, opts).build
     end
     # rubocop:enable Metrics/LineLength
 
@@ -350,13 +337,39 @@ module Blacklight
       add_action(navbar.partials, name, opts)
     end
 
-    private
+    ##
+    # Add a section of config that only applies to documents with a matching display type
+    def for_display_type display_type, &_block
+      self.fields_for_type ||= {}
 
-      def add_action(config_hash, name, opts)
-        config = Blacklight::Configuration::ToolConfig.new opts
-        config.name = name
-        yield(config) if block_given?
-        config_hash[name] = config
+      (fields_for_type[display_type] ||= self.class.new).tap do |conf|
+        yield(conf) if block_given?
       end
+    end
+
+    ##
+    # Return a list of fields for the index display that should be used for the
+    # provided document.  This respects any configuration made using for_display_type
+    def index_fields_for(document)
+      display_type = document.first(index.display_type_field)
+      for_display_type(display_type).index_fields.merge(index_fields)
+    end
+
+    ##
+    # Return a list of fields for the show page that should be used for the
+    # provided document.  This respects any configuration made using for_display_type
+    def show_fields_for(document)
+      display_type = document.first(show.display_type_field)
+      for_display_type(display_type).show_fields.merge(show_fields)
+    end
+
+    private
+
+    def add_action(config_hash, name, opts)
+      config = Blacklight::Configuration::ToolConfig.new opts
+      config.name = name
+      yield(config) if block_given?
+      config_hash[name] = config
+    end
   end
 end

data/lib/blacklight/engine.rb

@@ -3,12 +3,9 @@ module Blacklight
   class Engine < Rails::Engine
     engine_name "blacklight"
 
-    require 'bootstrap-sass'
-    require 'twitter-typeahead-rails'
-
     # BlacklightHelper is needed by all helpers, so we inject it
-    # into action view base here. 
-    initializer 'blacklight.helpers' do |app|
+    # into action view base here.
+    initializer 'blacklight.helpers' do
       ActionView::Base.send :include, BlacklightHelper
     end
 
@@ -28,7 +25,10 @@ module Blacklight
     end
 
     initializer "blacklight.assets.precompile" do |app|
-      app.config.assets.precompile += %w(favicon.ico)
+      # When Rails has been generated in API mode, it does not have sprockets available
+      if defined? Sprockets
+        app.config.assets.precompile += %w(favicon.ico)
+      end
     end
 
     Blacklight::Engine.config.sms_mappings = {
@@ -43,7 +43,11 @@ module Blacklight
     }
 
     config.bookmarks_http_method = :post
-    
+
     config.email_regexp = defined?(Devise) ? Devise.email_regexp : /\A[^@\s]+@[^@\s]+\z/
+
+    config.action_dispatch.rescue_responses.merge!(
+      "Blacklight::Exceptions::RecordNotFound" => :not_found
+    )
   end
 end

data/lib/blacklight/exceptions.rb

@@ -4,13 +4,7 @@ module Blacklight
     class AccessDenied < StandardError
     end
 
-    # When a request for a single solr document by id
-    # is not successful, we can raise this exception. 
-    # Deprecated; this will be removed in Blacklight 6.0:
-    class InvalidSolrID < RuntimeError
-    end
-    # In Blacklight 6.0, this exception can subclass RuntimeError directly
-    class RecordNotFound < InvalidSolrID
+    class RecordNotFound < RuntimeError
     end
 
     class InvalidRequest < StandardError
@@ -20,5 +14,8 @@ module Blacklight
     end
 
     class ECONNREFUSED < ::Errno::ECONNREFUSED; end
+
+    class IconNotFound < StandardError
+    end
   end
 end

data/lib/blacklight/{utils.rb → nested_open_struct_with_hash_access.rb}

@@ -1,96 +1,24 @@
 # frozen_string_literal: true
 require 'ostruct'
 module Blacklight
-  module Utils
-    def self.needs_attr_accessible?
-      protected_attributes_enabled?
-    end
-
-    def self.protected_attributes_enabled?
-      defined?(ActiveModel::MassAssignmentSecurity)
-    end
-  end
-
   ##
-  # An OpenStruct that responds to common Hash methods
-  class OpenStructWithHashAccess < OpenStruct
-    delegate :keys, :each, :map, :has_key?, :key?, :include?, :empty?, :length, :delete, :delete_if, :keep_if, :clear, :reject!, :select!, :replace, :fetch, :to_json, :as_json, :any?, to: :to_h
-
-    ##
-    # Expose the internal hash
-    # @return [Hash]
-    def to_h
-      @table
-    end
-    
-    def select *args, &block
-      self.class.new to_h.select(*args, &block)
-    end
-
-    def sort_by *args, &block
-      self.class.new Hash[to_h.sort_by(*args, &block)]
-    end
-
-    def sort_by! *args, &block
-      replace Hash[to_h.sort_by(*args, &block)]
-      self
-    end
-
-    ##
-    # Merge the values of this OpenStruct with another OpenStruct or Hash
-    # @param [Hash,#to_h] other_hash
-    # @return [OpenStructWithHashAccess] a new instance of an OpenStructWithHashAccess
-    def merge other_hash
-      self.class.new to_h.merge((other_hash if other_hash.is_a? Hash) || other_hash.to_h)
-    end
-
-    ##
-    # Merge the values of another OpenStruct or Hash into this object
-    # @param [Hash,#to_h] other_hash
-    # @return [OpenStructWithHashAccess] a new instance of an OpenStructWithHashAccess
-    def merge! other_hash
-      @table.merge!((other_hash if other_hash.is_a? Hash) || other_hash.to_h)
-    end
-
-    def deep_dup
-      self.class.new @table.deep_dup
-    end
-  end
-
-  ##
-  # An OpenStruct refinement that converts any hash-keys into  
+  # An OpenStruct refinement that converts any hash-keys into
   # additional instances of NestedOpenStructWithHashAccess
   class NestedOpenStructWithHashAccess < OpenStructWithHashAccess
     attr_reader :nested_class
-    delegate :default_proc=, :to => :to_h
+    delegate :default_proc=, to: :to_h
 
-    def initialize klass, *args
+    def initialize(klass, hash = {})
       @nested_class = klass
-      hash = {}
-
-      hashes_and_keys = args.flatten
-      lazy_configs = hashes_and_keys.extract_options!
-
-      hashes_and_keys.each do |v|
+      value = hash.transform_values do |v|
         if v.is_a? Hash
-          key = v.first
-          value = v[key]
-
-          hash[key] = nested_class.new value
+          nested_class.new(v)
         else
-          hash[v] = nested_class.new
+          v
         end
       end
 
-      lazy_configs.each do |k,v|
-        hash[k] = if v.is_a? nested_class
-                    v
-                  else
-                    nested_class.new v
-                  end
-      end
-
-      super hash
+      super value
       set_default_proc!
     end
 
@@ -131,7 +59,7 @@ module Blacklight
     end
 
     def deep_dup
-      self.class.new self.nested_class, @table.deep_dup
+      self.class.new nested_class, @table.deep_dup
     end
 
     def select *args, &block
@@ -172,7 +100,7 @@ module Blacklight
 
     def set_default_proc!
       self.default_proc = lambda do |hash, key|
-        hash[key] = self.nested_class.new
+        hash[key] = nested_class.new
       end
     end
   end

data/lib/blacklight/open_struct_with_hash_access.rb

@@ -0,0 +1,49 @@
+# frozen_string_literal: true
+require 'ostruct'
+module Blacklight
+  ##
+  # An OpenStruct that responds to common Hash methods
+  class OpenStructWithHashAccess < OpenStruct
+    delegate :keys, :each, :map, :has_key?, :key?, :include?, :empty?, :length, :delete, :delete_if, :keep_if, :clear, :reject!, :select!, :replace, :fetch, :to_json, :as_json, :any?, to: :to_h
+
+    ##
+    # Expose the internal hash
+    # @return [Hash]
+    def to_h
+      @table
+    end
+
+    def select *args, &block
+      self.class.new to_h.select(*args, &block)
+    end
+
+    def sort_by *args, &block
+      self.class.new Hash[to_h.sort_by(*args, &block)]
+    end
+
+    def sort_by! *args, &block
+      replace Hash[to_h.sort_by(*args, &block)]
+      self
+    end
+
+    ##
+    # Merge the values of this OpenStruct with another OpenStruct or Hash
+    # @param [Hash,#to_h] other_hash
+    # @return [OpenStructWithHashAccess] a new instance of an OpenStructWithHashAccess
+    def merge other_hash
+      self.class.new to_h.merge((other_hash if other_hash.is_a? Hash) || other_hash.to_h)
+    end
+
+    ##
+    # Merge the values of another OpenStruct or Hash into this object
+    # @param [Hash,#to_h] other_hash
+    # @return [OpenStructWithHashAccess] a new instance of an OpenStructWithHashAccess
+    def merge! other_hash
+      @table.merge!((other_hash if other_hash.is_a? Hash) || other_hash.to_h)
+    end
+
+    def deep_dup
+      self.class.new @table.deep_dup
+    end
+  end
+end

data/lib/blacklight/parameters.rb

@@ -6,7 +6,7 @@ module Blacklight
     # from the provided parameters.
     # @param [Hash] params parameters
     def self.sanitize params
-      params.reject { |k,v| v.nil? }
+      params.reject { |_k, v| v.nil? }
             .except(:action, :controller, :id, :commit, :utf8)
     end
   end

data/lib/blacklight/routes/exportable.rb

@@ -6,9 +6,7 @@ module Blacklight
         @defaults = defaults
       end
 
-      def call(mapper, options = {})
-        options = @defaults.merge(options)
-
+      def call(mapper, _options = {})
         mapper.member do
           mapper.match 'email', via: [:get, :post]
           mapper.match 'sms', via: [:get, :post]

data/lib/blacklight/routes/searchable.rb

@@ -6,14 +6,14 @@ module Blacklight
         @defaults = defaults
       end
 
-      def call(mapper, options = {})
-        options = @defaults.merge(options)
-
+      def call(mapper, _options = {})
         mapper.match '/', action: 'index', as: 'search', via: [:get, :post]
 
         mapper.post ":id/track", action: 'track', as: 'track'
+        mapper.get ":id/raw", action: 'raw', as: 'raw', defaults: { format: 'json' }
 
         mapper.get "opensearch"
+        mapper.get 'suggest', as: 'suggest_index', defaults: { format: 'json' }
         mapper.get "facet/:id", action: 'facet', as: 'facet'
       end
     end

data/lib/blacklight/runtime_registry.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_support/per_thread_registry'
 
 module Blacklight