blacklight 6.14.0 → 7.0.0

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

@@ -24,8 +24,8 @@ module Blacklight::Document::Extensions
   # Needs to be called in initializer of class including this module, to
   # apply all registered extensions on a per-document basis
   def apply_extensions
-    self.class.registered_extensions.each do | registration|
-      self.extend( registration[:module_obj] ) if registration[:condition_proc].nil? || registration[:condition_proc].call( self )
+    self.class.registered_extensions.each do |registration|
+      extend(registration[:module_obj]) if registration[:condition_proc].nil? || registration[:condition_proc].call(self)
     end
   end
 
@@ -43,15 +43,15 @@ module Blacklight::Document::Extensions
     end
 
     # Register an extension module with the class. A block taking one
-    # parameter can be supplied; the block will be passed an instance of 
+    # parameter can be supplied; the block will be passed an instance of
     # a Document, and the extension will be applied only if the block
     # evaluates as true. If no condition is given, the extension will
     # be applied to every instance of the class.
     #
     # SolrDocument.use_extension( SomeExtensionModule ) { | document | should_apply_some_extension?(document) }
     # SolrDocument.use_extension( SomeExtensionModule) # will be applied to all docs
-    def use_extension( module_obj, &condition )
-      registered_extensions << {module_obj: module_obj, condition_proc: condition}    
+    def use_extension(module_obj, &condition)
+      registered_extensions << { module_obj: module_obj, condition_proc: condition }
     end
   end
 end

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

@@ -7,7 +7,7 @@ module Blacklight::Document
       # Returns array of hashes of registered extensions. Each hash
       # has a :module_obj key and a :condition_proc key. Usually this
       # method is only used internally in #apply_extensions, but if you
-   
+
       # Class-level method for accessing/setting semantic mappings
       # for solr stored fields. Can be set by local app, key is
       # a symbol for a semantic, value is a solr _stored_ field.
@@ -17,22 +17,22 @@ module Blacklight::Document
       #
       # Currently documented semantic tokens, not all may be
       # used by core BL, but some may be used by plugins present
-      # or future. 
-      # :title, :author, :year, :language => User-presentable strings. 
+      # or future.
+      # :title, :author, :year, :language => User-presentable strings.
       def field_semantics
         @field_semantics ||= {}
-      end  
+      end
     end
-    
+
     # Returns a hash keyed by semantic tokens, value is an array of
     # strings. (Array to handle multi-value fields). If no value(s)
-    # available, empty array is returned. 
+    # available, empty array is returned.
     #
     # Default implementation here uses field_semantics
-    # to just take values from Solr stored fields. 
+    # to just take values from Solr stored fields.
     # Extensions can over-ride this method to provide better/different lookup,
     # but extensions should call super and modify hash returned, to avoid
-    # unintentionally erasing values provided by other extensions. 
+    # unintentionally erasing values provided by other extensions.
     def to_semantic_values
       @semantic_value_hash ||= self.class.field_semantics.each_with_object(Hash.new([])) do |(key, field_names), hash|
         ##
@@ -45,6 +45,6 @@ module Blacklight::Document
       end
 
       @semantic_value_hash ||= {}
-    end    
+    end
   end
 end

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

@@ -3,7 +3,7 @@
 module Blacklight::Document::Sms
   # Return a text string that will be the body of the email
   def to_sms_text
-    semantics = self.to_semantic_values
+    semantics = to_semantic_values
     body = []
     body << I18n.t('blacklight.sms.text.title', value: semantics[:title].first) unless semantics[:title].blank?
     body << I18n.t('blacklight.sms.text.author', value: semantics[:author].first) unless semantics[:author].blank?

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

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
-##
+
+require 'globalid'
+
 ##
 # = Introduction
 # Blacklight::Document is the module with logic for a class representing
@@ -14,9 +16,6 @@
 # transformation formats.
 #
 module Blacklight::Document
-  extend Deprecation
-  self.deprecation_horizon = 'blacklight 7.0'
-
   autoload :ActiveModelShim, 'blacklight/document/active_model_shim'
   autoload :SchemaOrg, 'blacklight/document/schema_org'
   autoload :CacheKey, 'blacklight/document/cache_key'
@@ -41,36 +40,13 @@ module Blacklight::Document
 
   attr_reader :response, :_source
   alias_method :solr_response, :response
-  delegate :[], :key?, :keys, :to_h, to: :_source
-
-  def initialize(source_doc={}, response=nil)
-    @_source = if source_doc.respond_to?(:to_hash)
-                 ActiveSupport::HashWithIndifferentAccess.new(source_doc)
-               else
-                 Deprecation.warn(Blacklight::Document, "Blacklight::Document#initialize expects a hash-like object, received #{source_doc.class}.")
-                 ActiveSupport::HashWithIndifferentAccess.new(source_doc.to_h)
-               end
-    @response = response
-    apply_extensions
-  end
 
-  # the wrapper method to the @_source object.
-  # If a method is missing, it gets sent to @_source
-  # with all of the original params and block
-  def method_missing(m, *args, &b)
-    return super if m == :to_hash
+  delegate :[], :key?, :keys, :to_h, :as_json, to: :_source
 
-    if _source_responds_to?(m)
-      Deprecation.warn(Blacklight::Solr::Document, "Blacklight::Document##{m} is deprecated; use obj.to_h.#{m} instead.")
-      _source.send(m, *args, &b)
-    else
-      super
-    end
-  end
-
-  def respond_to_missing? m, *args
-    return super if %i(empty? to_hash).include?(m)
-    _source_responds_to?(m, *args) || super
+  def initialize(source_doc = {}, response = nil)
+    @_source = ActiveSupport::HashWithIndifferentAccess.new(source_doc).freeze
+    @response = response
+    apply_extensions
   end
 
   # Helper method to check if value/multi-values exist for a given key.
@@ -105,7 +81,7 @@ module Blacklight::Document
   def fetch key, *default
     if key? key
       self[key]
-    elsif default.empty? and !block_given?
+    elsif default.empty? && !block_given?
       raise KeyError, "key not found \"#{key}\""
     else
       (yield(self) if block_given?) || default.first
@@ -120,11 +96,11 @@ module Blacklight::Document
     'catalog/document'
   end
 
-  def has_highlight_field? k
+  def has_highlight_field? _k
     false
   end
 
-  def highlight_field k
+  def highlight_field _k
     nil
   end
 
@@ -142,11 +118,21 @@ module Blacklight::Document
     def unique_key
       @unique_key ||= 'id'
     end
-  end
-
-  private
 
-  def _source_responds_to? *args
-    _source && self != _source && _source.respond_to?(*args)
+    # Define an attribute reader on a document model
+    # @Example
+    #   class SolrDocument
+    #     include Blacklight::Solr::Document
+    #     attribute :title, Blacklight::Types::String, 'title_tesim'
+    #   end
+    #
+    #   doc = SolrDocument.new(title_tesim: ["One flew over the cuckoo's nest"])
+    #   doc.title
+    #   #=> "One flew over the cuckoo's nest"
+    def attribute(name, type, field)
+      define_method name do
+        type.coerce(self[field])
+      end
+    end
   end
 end

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

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-require 'rsolr'
 ##
 ##
 # = Introduction
@@ -16,20 +15,21 @@ require 'rsolr'
 #
 
 module Blacklight::Solr::Document
-  autoload :MoreLikeThis, 'blacklight/solr/document/more_like_this'
-
   extend ActiveSupport::Concern
   include Blacklight::Document
   include Blacklight::Document::ActiveModelShim
-  include Blacklight::Solr::Document::MoreLikeThis
+
+  def more_like_this
+    response.more_like(self).map { |doc| self.class.new(doc, response) }
+  end
 
   def has_highlight_field? k
-    return false if response['highlighting'].blank? or response['highlighting'][self.id].blank?
-    
-    response['highlighting'][self.id].key? k.to_s
+    return false if response['highlighting'].blank? || response['highlighting'][id].blank?
+
+    response['highlighting'][id].key? k.to_s
   end
 
   def highlight_field k
-    response['highlighting'][self.id][k.to_s].map(&:html_safe) if has_highlight_field? k
+    response['highlighting'][id][k.to_s].map(&:html_safe) if has_highlight_field? k
   end
 end

data/app/models/concerns/blacklight/suggest/response.rb

@@ -2,17 +2,19 @@
 module Blacklight
   module Suggest
     class Response
-      attr_reader :response, :request_params, :suggest_path
+      attr_reader :response, :request_params, :suggest_path, :suggester_name
 
       ##
       # Creates a suggest response
       # @param [RSolr::HashWithResponse] response
       # @param [Hash] request_params
       # @param [String] suggest_path
-      def initialize(response, request_params, suggest_path)
+      # @param [String] suggester_name
+      def initialize(response, request_params, suggest_path, suggester_name)
         @response = response
         @request_params = request_params
         @suggest_path = suggest_path
+        @suggester_name = suggester_name
       end
 
       ##
@@ -20,7 +22,7 @@ module Blacklight
       # present
       # @return [Array]
       def suggestions
-        response.try(:[], suggest_path).try(:[], 'mySuggester').try(:[], request_params[:q]).try(:[], 'suggestions') || []
+        (response.try(:[], suggest_path).try(:[], suggester_name).try(:[], request_params[:q]).try(:[], 'suggestions') || []).uniq
       end
     end
   end

data/app/models/concerns/blacklight/user.rb

@@ -1,13 +1,14 @@
 # frozen_string_literal: true
 module Blacklight::User
   # This gives us an is_blacklight_user method that can be included in
-  # the containing applications models. 
-  # SEE ALSO:  The /lib/blacklight/engine.rb class for how when this 
+  # the containing applications models.
+  # SEE ALSO:  The /lib/blacklight/engine.rb class for how when this
   # is injected into the hosting application through ActiveRecord::Base extend
   def self.included(base)
     return unless base.respond_to? :has_many
-    base.send :has_many, :bookmarks, :dependent => :destroy, :as => :user
-    base.send :has_many, :searches,  :dependent => :destroy, :as => :user
+
+    base.send :has_many, :bookmarks, dependent: :destroy, as: :user
+    base.send :has_many, :searches,  dependent: :destroy, as: :user
   end
 
   def bookmarks_for_documents documents = []
@@ -21,9 +22,9 @@ module Blacklight::User
   def document_is_bookmarked?(document)
     bookmarks_for_documents([document]).any?
   end
-    
+
   # returns a Bookmark object if there is one for document_id, else
-  # nil. 
+  # nil.
   def existing_bookmark_for(document)
     bookmarks_for_documents([document]).first
   end

data/app/models/record_mailer.rb

@@ -1,25 +1,24 @@
 # frozen_string_literal: true
-# Only works for documents with a #to_marc right now. 
+# Only works for documents with a #to_marc right now.
 class RecordMailer < ActionMailer::Base
   def email_record(documents, details, url_gen_params)
-        
     title = begin
               documents.first.to_semantic_values[:title]
             rescue
               I18n.t('blacklight.email.text.default_title')
             end
-    subject = I18n.t('blacklight.email.text.subject', :count => documents.length, :title => title )
+    subject = I18n.t('blacklight.email.text.subject', count: documents.length, title: title)
 
     @documents      = documents
     @message        = details[:message]
     @url_gen_params = url_gen_params
 
-    mail(:to => details[:to],  :subject => subject)
+    mail(to: details[:to],  subject: subject)
   end
-  
+
   def sms_record(documents, details, url_gen_params)
     @documents      = documents
     @url_gen_params = url_gen_params
-    mail(:to => details[:to], :subject => "")
+    mail(to: details[:to], subject: "")
   end
 end

data/app/models/search.rb

@@ -1,22 +1,20 @@
 # frozen_string_literal: true
-class Search < ActiveRecord::Base
-  belongs_to_arguments = { optional: true } if Rails.version >= '5.0.0'
 
-  belongs_to :user, belongs_to_arguments
+class Search < ApplicationRecord
+  belongs_to :user, optional: true
 
   serialize :query_params
 
-  attr_accessible :query_params if Blacklight::Utils.needs_attr_accessible?
-
   # A Search instance is considered a saved search if it has a user_id.
   def saved?
-    self.user_id?
+    user_id?
   end
 
   # delete old, unsaved searches
   def self.delete_old_searches(days_old)
     raise ArgumentError, 'days_old is expected to be a number' unless days_old.is_a?(Numeric)
     raise ArgumentError, 'days_old is expected to be greater than 0' if days_old <= 0
-    self.where(['created_at < ? AND user_id IS NULL', Time.zone.today - days_old]).destroy_all
+
+    where(['created_at < ? AND user_id IS NULL', Time.zone.today - days_old]).destroy_all
   end
 end

data/app/presenters/blacklight/document_presenter.rb

@@ -1,153 +1,38 @@
 # frozen_string_literal: true
+
 module Blacklight
-  # @deprecated
+  # An abstract class that the view presenters for SolrDocuments descend from
   class DocumentPresenter
-    extend Deprecation
-    self.deprecation_horizon = 'Blacklight version 7.0.0'
-
-    # @param [SolrDocument] document
-    # @param [ActionController::Base] controller scope for linking and generating urls
-    # @param [Blacklight::Configuration] configuration
-    def initialize(document, controller, configuration = controller.blacklight_config)
-      @document = document
-      @configuration = configuration
-      @controller = controller
-    end
-
-    ##
-    # Get the value of the document's "title" field, or a placeholder
-    # value (if empty)
-    #
-    # @return [String]
-    # @deprecated use ShowPresenter#heading instead
-    def document_heading
-      show_presenter.heading
-    end
-    deprecation_deprecate document_heading: "use ShowPresenter#heading instead"
-
-    ##
-    # Create <link rel="alternate"> links from a documents dynamically
-    # provided export formats. Returns empty string if no links available.
-    #
-    # @param [Hash] options
-    # @option options [Boolean] :unique ensures only one link is output for every
-    #     content type, e.g. as required by atom
-    # @option options [Array<String>] :exclude array of format shortnames to not include in the output
-    # @deprecated moved to ShowPresenter#link_rel_alternates
-    def link_rel_alternates(options = {})
-      show_presenter.link_rel_alternates(options)
-    end
-    deprecation_deprecate link_rel_alternates: "use ShowPresenter#link_rel_alternates instead"
-
-    ##
-    # Get the document's "title" to display in the <title> element.
-    # (by default, use the #document_heading)
-    #
-    # @see #document_heading
-    # @return [String]
-    # @deprecated use ShowPresenter#html_title instead
-    def document_show_html_title
-      show_presenter.html_title
-    end
-    deprecation_deprecate document_show_html_title: "use ShowPresenter#html_title instead"
-
-    ##
-    # Render the document index heading
-    #
-    # @overload render_document_index_label(field, opts)
-    #   @param [Symbol, Proc, String] field Render the given field or evaluate the proc or render the given string
-    #   @param [Hash] opts
-    # @deprecated use IndexPresenter#label instead
-    def render_document_index_label(*args)
-      index_presenter.label(*args)
-    end
-    deprecation_deprecate render_document_index_label: "use IndexPresenter#label instead"
-
-    ##
-    # Render the index field label for a document
-    #
-    # Allow an extention point where information in the document
-    # may drive the value of the field
-    # @overload render_index_field_value(field, opts)
-    #   @param [String] field
-    #   @param [Hash] opts
-    #   @option opts [String] :value
-    # @deprecated use IndexPresenter#field_value instead
-    def render_index_field_value *args
-      index_presenter.field_value(*args)
+    attr_reader :document, :configuration, :view_context
+
+    # @return [Hash<String,Configuration::Field>]  all the fields for this index view that should be rendered
+    def fields_to_render
+      fields.select do |_name, field_config|
+        # rubocop:disable Style/PreferredHashMethods
+        render_field?(field_config) && has_value?(field_config)
+        # rubocop:enable Style/PreferredHashMethods
+      end
     end
-    deprecation_deprecate render_index_field_value: "use IndexPresenter#field_value instead"
 
-    ##
-    # Render the show field value for a document
-    #
-    #   Allow an extention point where information in the document
-    #   may drive the value of the field
-    #
-    # @overload render_index_field_value(field, options)
-    #   @param [String] field
-    #   @param [Hash] options
-    #   @option options [String] :value
-    #   @deprecated use ShowPresenter#field_value
-    def render_document_show_field_value *args
-      show_presenter.field_value(*args)
-    end
-    deprecation_deprecate render_document_show_field_value: "use ShowPresenter#field_value instead"
+    private
 
     ##
-    # Get the value for a document's field, and prepare to render it.
-    # - highlight_field
-    # - accessor
-    # - solr field
-    #
-    # Rendering:
-    #   - helper_method
-    #   - link_to_search
-    # @param [String] _field name
-    # @param [Blacklight::Configuration::Field] field_config solr field configuration
-    # @param [Hash] options additional options to pass to the rendering helpers
-    # @deprecated
-    def get_field_values _field, field_config, options = {}
-      field_values(field_config, options)
+    # Check to see if the given field should be rendered in this context
+    # @param [Blacklight::Configuration::Field] field_config
+    # @return [Boolean]
+    def render_field?(field_config)
+      view_context.should_render_field?(field_config, document)
     end
-    deprecation_deprecate get_field_values: 'Use field_values instead'
 
     ##
-    # Get the value for a document's field, and prepare to render it.
-    # - highlight_field
-    # - accessor
-    # - solr field
-    #
-    # Rendering:
-    #   - helper_method
-    #   - link_to_search
-    # @param [Blacklight::Configuration::Field] field_config solr field configuration
-    # @param [Hash] options additional options to pass to the rendering helpers
-    def field_values(field_config, options={})
-      FieldPresenter.new(@controller, @document, field_config, options).render
-    end
-    deprecation_deprecate field_values: 'Use ShowPresenter or IndexPresenter field_values instead'
-
-    # @deprecated
-    def render_field_value(values, field_config = Configuration::NullField.new)
-      field_values(field_config, value: Array(values))
+    # Check if a document has (or, might have, in the case of accessor methods) a value for
+    # the given solr field
+    # @param [Blacklight::Configuration::Field] field_config
+    # @return [Boolean]
+    def has_value? field_config
+      document.has?(field_config.field) ||
+        (document.has_highlight_field? field_config.field if field_config.highlight) ||
+        field_config.accessor
     end
-    deprecation_deprecate render_field_value: 'Use FieldPresenter instead'
-
-    # @deprecated
-    def render_values(values, field_config = Configuration::NullField.new)
-      field_values(field_config, value: Array(values))
-    end
-    deprecation_deprecate render_values: 'Use FieldPresenter instead'
-
-    private
-
-      def index_presenter
-        @controller.index_presenter(@document)
-      end
-
-      def show_presenter
-        @controller.show_presenter(@document)
-      end
   end
 end

data/app/presenters/blacklight/field_presenter.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Blacklight
-  # Renders a field and handles link_to_search or helper_method if supplied
+  # Renders a field and handles link_to_facet or helper_method if supplied
   class FieldPresenter
     def initialize(controller, document, field_config, options)
       @controller = controller
@@ -9,7 +11,7 @@ module Blacklight
     end
 
     attr_reader :controller, :document, :field_config, :options
-    
+
     def render
       if options[:value]
         # This prevents helper methods from drawing.
@@ -24,8 +26,8 @@ module Blacklight
 
     private
 
-      def retrieve_values
-        FieldRetriever.new(document, field_config).fetch
-      end
+    def retrieve_values
+      FieldRetriever.new(document, field_config).fetch
+    end
   end
 end