europeana-blacklight 0.3.2 → 0.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d892f236445610a975019d0aed2ff675ad0801f4
-  data.tar.gz: 8d65e30ff5da21e187c64999a25fed6f2a29d225
+  metadata.gz: 9c009d6e64d47908e4814d19edf92c58764e208e
+  data.tar.gz: dae88754b44e1fabf5ba8dd7731559c4aa3fef18
 SHA512:
-  metadata.gz: 8a72fc7996052249b3ada8dee2421b22d3969cd51f927db36441c8909f847fc52100c89a02903fdd48a4eb576bdbb7f9a4ce3d02202088e0a7158b7af1e2156f
-  data.tar.gz: 9b7142747a9b6687dfc1f11b6343490bf113119b6d949108d09a421317907a5864a7cb906270fc71b684ff94c89454d16d9be840825d1ad4f7ebe7f2f2f48c6e
+  metadata.gz: 8795c06e1000dc9af5538daca798639ccf323db798aa08a2f3708ffc81fb77f390ae929cbe7b74f70a9f1ebf6bd9f1a9593090cc2a7c7a7c65d6aabe17e01cbf
+  data.tar.gz: b93869c448d02abb45e6cc847bcf65c630988a62a5c261b9d9006ea9afc77bfe680d50de7d106787e5669e3dd39f181e52d809df0c2a0b8f8cc12c3fd210672b

data/europeana-blacklight.gemspec

@@ -21,7 +21,7 @@ Gem::Specification.new do |spec|
   spec.required_ruby_version = '>= 2.0.0'
 
   spec.add_dependency 'activesupport', '>= 4.0', '< 5.0'
-  spec.add_dependency 'blacklight', '>= 5.12.0', '< 5.16.0'
+  spec.add_dependency 'blacklight', '>= 5.12.0', '< 6.0'
   spec.add_dependency 'europeana-api', '~> 0.4.1'
   spec.add_dependency 'iso-639', '~> 0.2.5'
   spec.add_dependency 'kaminari', '~> 0.16'

data/lib/europeana/blacklight/document.rb

@@ -7,59 +7,20 @@ module Europeana
     ##
     # A Europeana document
     class Document
+      autoload :LangMaps, 'europeana/blacklight/document/lang_maps'
       autoload :MoreLikeThis, 'europeana/blacklight/document/more_like_this'
       autoload :Relations, 'europeana/blacklight/document/relations'
 
       include ActiveModel::Conversion
       include ::Blacklight::Document
       include ::Blacklight::Document::ActiveModelShim
+      include LangMaps
       include MoreLikeThis
       include Relations
 
       attr_writer :provider_id, :record_id
 
       class << self
-        # @todo Are three-letter language codes valid in EDM?
-        # @todo Empty key acceptance is a workaround for malformed API data
-        #   output; remove when fixed at source
-        def lang_map?(obj)
-          return false unless obj.is_a?(Hash)
-          obj.keys.map(&:to_s).all? { |key| known_lang_map_key?(key) }
-        end
-
-        def known_lang_map_key?(key)
-          key = key.dup.downcase
-          ['def', '', 'sh'].include?(key) || (!ISO_639.find(key.split('-').first).nil?)
-        end
-
-        def localize_lang_map(lang_map)
-          if lang_map.is_a?(Array)
-            return lang_map.map { |l| localize_lang_map(l) }
-          end
-
-          return lang_map unless lang_map?(lang_map)
-
-          lang_map_value(lang_map, I18n.locale.to_s) ||
-            lang_map_value(lang_map, I18n.default_locale.to_s) ||
-            lang_map[:def] ||
-            lang_map.values
-        end
-
-        def lang_map_value(lang_map, locale)
-          iso_locale = ISO_639.find(locale)
-          return nil unless lang_map.key?(iso_locale.alpha2) || lang_map.key?(iso_locale.alpha3)
-
-          alpha2 = lang_map[iso_locale.alpha2]
-          alpha3 = lang_map[iso_locale.alpha3]
-          if alpha2 && alpha3
-            [alpha2, alpha3].flatten
-          elsif alpha2
-            alpha2
-          else
-            alpha3
-          end
-        end
-
         def model_name
           @_model_name ||= begin
             ActiveModel::Name.new(self, nil, 'Document')
@@ -67,8 +28,6 @@ module Europeana
         end
       end
 
-      delegate :lang_map?, :localize_lang_map, to: :class
-
       def initialize(source_doc = {}, response = nil)
         fields, @relations = extract_relations(source_doc)
         super(fields, response)

data/lib/europeana/blacklight/document/lang_maps.rb

@@ -0,0 +1,65 @@
+module Europeana
+  module Blacklight
+    class Document
+      ##
+      # Methods for working with "LangMap" data types in API JSON responses
+      # @see http://labs.europeana.eu/api/getting-started#datatypes
+      module LangMaps
+        extend ActiveSupport::Concern
+
+        # @see https://www.loc.gov/standards/iso639-2/php/code_changes.php
+        DEPRECATED_ISO_LANG_CODES = %w(in iw jaw ji jw mo mol scc scr sh)
+
+        # Special keys API may return in a LangMap, not ISO codes
+        # @todo Empty key acceptance is a workaround for malformed API data
+        #   output; remove when fixed at source
+        NON_ISO_LANG_CODES = ['def', '']
+
+        class_methods do
+          # @todo Are three-letter language codes valid in EDM?
+          def lang_map?(obj)
+            return false unless obj.is_a?(Hash)
+            obj.keys.map(&:to_s).all? { |key| known_lang_map_key?(key) }
+          end
+
+          def known_lang_map_key?(key)
+            key = key.dup.downcase
+            DEPRECATED_ISO_LANG_CODES.include?(key) ||
+              NON_ISO_LANG_CODES.include?(key) ||
+              (!ISO_639.find(key.split('-').first).nil?)
+          end
+
+          def localize_lang_map(lang_map)
+            if lang_map.is_a?(Array)
+              return lang_map.map { |l| localize_lang_map(l) }
+            end
+
+            return lang_map unless lang_map?(lang_map)
+
+            lang_map_value(lang_map, I18n.locale.to_s) ||
+              lang_map_value(lang_map, I18n.default_locale.to_s) ||
+              lang_map[:def] ||
+              lang_map.values
+          end
+
+          def lang_map_value(lang_map, locale)
+            iso_locale = ISO_639.find(locale)
+            return nil unless lang_map.key?(iso_locale.alpha2) || lang_map.key?(iso_locale.alpha3)
+
+            alpha2 = lang_map[iso_locale.alpha2]
+            alpha3 = lang_map[iso_locale.alpha3]
+            if alpha2 && alpha3
+              [alpha2, alpha3].flatten
+            elsif alpha2
+              alpha2
+            else
+              alpha3
+            end
+          end
+        end
+
+        delegate :lang_map?, :localize_lang_map, to: :class
+      end
+    end
+  end
+end

data/lib/europeana/blacklight/document/more_like_this.rb

@@ -26,7 +26,7 @@ module Europeana
 
         def more_like_this_field_terms(*fields)
           fields.flatten.map do |field|
-            fetch(field, []).compact
+            fetch(field, []).compact[0..9]
           end.flatten
         end
 
@@ -42,7 +42,11 @@ module Europeana
 
         def more_like_this_param_query(param, terms, boost)
           return nil unless terms.present?
-          or_terms = terms.map do |v|
+
+          string_terms = terms.select { |t| t.is_a?(String) }
+          return nil unless string_terms.present?
+
+          or_terms = string_terms.map do |v|
             '"' + Europeana::API::Search.escape(v) + '"'
           end.join(' OR ')
           "#{param}: (#{or_terms})^#{boost}"

data/lib/europeana/blacklight/search_builder.rb

@@ -99,15 +99,16 @@ module Europeana
         salient_facets.each_pair do |facet_field, value_list|
           Array(value_list).reject(&:blank?).each do |value|
             api_parameters[:qf] ||= []
-            if Europeana::API::Search::Fields::MEDIA.include?(facet_field)
-              api_parameters[:qf] << "#{facet_field}:#{value}"
-            else
-              api_parameters[:qf] << "#{facet_field}:\"#{value}\""
-            end
+            api_parameters[:qf] << "#{facet_field}:" + quote_facet_value(facet_field, value)
           end
         end
       end
 
+      def quote_facet_value(facet_field, value)
+        return value if Europeana::API::Search::Fields::MEDIA.include?(facet_field)
+        '"' + value.gsub('"', '\"') + '"'
+      end
+
       ##
       # Filter results by a query facet
       def add_query_facet_to_api(_api_parameters)
@@ -173,8 +174,17 @@ module Europeana
       ##
       # Europeana API start param counts from 1
       def start(start = nil)
-        super_start = super
-        super_start == self ? super_start : super_start + 1
+        if start
+          params_will_change!
+          @start = start.to_i
+          self
+        else
+          @start ||= (page - 1) * (rows || 10) + 1
+
+          val = @start || 1
+          val = 1 if @start < 1
+          val
+        end
       end
 
       protected

data/lib/europeana/blacklight/search_helper.rb

@@ -3,14 +3,16 @@ module Europeana
     ##
     # Local overrides for {Blacklight::SearchHelper}
     module SearchHelper
+      # index arg counts from 0; API start param counts from 1
       def previous_and_next_document_params(index, window = 1)
+        start = index + 1
         api_params = {}
 
-        if index > 1
-          api_params[:start] = index - window # get one before
+        if start > 1
+          api_params[:start] = start - window # get one before
           api_params[:rows] = 2 * window + 1 # and one after
         else
-          api_params[:start] = 1 # there is no previous doc
+          api_params[:start] = start # there is no previous doc
           api_params[:rows] = 2 * window # but there should be one after
         end
 

data/lib/europeana/blacklight/version.rb

@@ -1,5 +1,5 @@
 module Europeana
   module Blacklight
-    VERSION = '0.3.2'
+    VERSION = '0.3.3'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: europeana-blacklight
 version: !ruby/object:Gem::Version
-  version: 0.3.2
+  version: 0.3.3
 platform: ruby
 authors:
 - Richard Doe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-14 00:00:00.000000000 Z
+date: 2016-01-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -39,7 +39,7 @@ dependencies:
         version: 5.12.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 5.16.0
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -49,7 +49,7 @@ dependencies:
         version: 5.12.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 5.16.0
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: europeana-api
   requirement: !ruby/object:Gem::Requirement
@@ -187,6 +187,7 @@ files:
 - lib/europeana/blacklight.rb
 - lib/europeana/blacklight/api_repository.rb
 - lib/europeana/blacklight/document.rb
+- lib/europeana/blacklight/document/lang_maps.rb
 - lib/europeana/blacklight/document/more_like_this.rb
 - lib/europeana/blacklight/document/relations.rb
 - lib/europeana/blacklight/document_presenter.rb