mods_display 1.0.0.alpha3 → 1.0.0

data/lib/mods_display/fields/nested_related_item.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ModsDisplay
   ##
   # This class will hopefully take over for related item support more broadly.
@@ -21,7 +23,12 @@ module ModsDisplay
     def to_html(view_context = ApplicationController.renderer)
       helpers = view_context.respond_to?(:simple_format) ? view_context : ApplicationController.new.view_context
 
-      component = ModsDisplay::ListFieldComponent.with_collection(fields, value_transformer: ->(value) { helpers.link_urls_and_email(value.to_s) }, list_html_attributes: { class: 'mods_display_nested_related_items' }, list_item_html_attributes: { class: 'mods_display_nested_related_item open' })
+      component = ModsDisplay::ListFieldComponent.with_collection(
+        fields,
+        value_transformer: ->(value) { helpers.link_urls_and_email(value.to_s) },
+        list_html_attributes: { class: 'mods_display_nested_related_items' },
+        list_item_html_attributes: { class: 'mods_display_nested_related_item open' }
+      )
 
       view_context.render component, layout: false
     end
@@ -29,7 +36,15 @@ module ModsDisplay
     private
 
     def related_item_mods_object(value)
-      mods = ::Stanford::Mods::Record.new.tap { |r| r.from_str("<mods>#{value.children.to_xml}</mods>", false) }
+      mods = ::Stanford::Mods::Record.new.tap do |r|
+        # dup'ing the value adds the appropriate namespaces, but...
+        munged_node = value.dup.tap do |x|
+          # ... the mods gem also expects the root of the document to have the root tag <mods>
+          x.name = 'mods'
+        end
+
+        r.from_nk_node(munged_node)
+      end
       related_item = ModsDisplay::HTML.new(mods)
 
       ModsDisplay::Values.new(
@@ -42,6 +57,7 @@ module ModsDisplay
       body = related_item.body
 
       return if body == '<dl></dl>'
+
       body
     end
 

data/lib/mods_display/fields/note.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ModsDisplay
   class Note < Field
     def fields
@@ -25,20 +27,20 @@ module ModsDisplay
       if element.attributes['type'].respond_to?(:value)
         return note_labels[element.attributes['type'].value] || "#{element.attributes['type'].value.capitalize}:"
       end
+
       I18n.t('mods_display.note')
     end
 
     def note_labels
       { 'statement of responsibility' => I18n.t('mods_display.statement_of_responsibility'),
         'date/sequential designation' => I18n.t('mods_display.date_sequential_designation'),
-        'publications'                => I18n.t('mods_display.publications'),
-        'references'                  => I18n.t('mods_display.references'),
-        'bibliography'                => I18n.t('mods_display.bibliography'),
-        'preferred citation'          => I18n.t('mods_display.preferred_citation'),
-        'biographical/historical'     => I18n.t('mods_display.biographical_historical'),
+        'publications' => I18n.t('mods_display.publications'),
+        'references' => I18n.t('mods_display.references'),
+        'bibliography' => I18n.t('mods_display.bibliography'),
+        'preferred citation' => I18n.t('mods_display.preferred_citation'),
+        'biographical/historical' => I18n.t('mods_display.biographical_historical'),
         'creation/production credits' => I18n.t('mods_display.creation_production_credits'),
-        'citation/reference'          => I18n.t('mods_display.citation_reference')
-       }
+        'citation/reference' => I18n.t('mods_display.citation_reference') }
     end
   end
 end

data/lib/mods_display/fields/related_item.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ModsDisplay
   class RelatedItem < Field
     include ModsDisplay::RelatedItemConcerns
@@ -6,10 +8,10 @@ module ModsDisplay
       return_fields = @values.map do |value|
         next if related_item_is_a_collection?(value)
         next if render_nested_related_item?(value)
-        case
-        when related_item_is_a_location?(value)
+
+        if related_item_is_a_location?(value)
           process_location value
-        when related_item_is_a_reference?(value)
+        elsif related_item_is_a_reference?(value)
           process_reference value
         else
           process_related_item(value)
@@ -33,12 +35,13 @@ module ModsDisplay
     end
 
     def process_related_item(item)
-      return unless item.titleInfo.length > 0
+      return unless item.titleInfo.length.positive?
+
       title = item.titleInfo.text.strip
       return_text = title
       location = nil
-      location = item.location.url.text if item.location.length > 0 &&
-                                           item.location.url.length > 0
+      location = item.location.url.text if item.location.length.positive? &&
+                                           item.location.url.length.positive?
       return_text = "<a href='#{location}'>#{title}</a>" if location && !title.empty?
 
       ModsDisplay::Values.new(label: related_item_label(item), values: [return_text]) unless return_text.empty?
@@ -54,8 +57,8 @@ module ModsDisplay
     def related_item_is_a_location?(item)
       !related_item_is_a_collection?(item) &&
         !related_item_is_a_reference?(item) &&
-        item.location.length > 0 &&
-        item.titleInfo.length < 1
+        item.location.length.positive? &&
+        item.titleInfo.empty?
     end
 
     def related_item_is_a_reference?(item)
@@ -66,7 +69,7 @@ module ModsDisplay
 
     def related_item_label(item)
       if displayLabel(item)
-        return displayLabel(item)
+        displayLabel(item)
       else
         case
         when related_item_is_a_location?(item)

data/lib/mods_display/fields/resource_type.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ModsDisplay
   class ResourceType < Field
     def fields

data/lib/mods_display/fields/sub_title.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ModsDisplay
   class SubTitle < ModsDisplay::Title
     def initialize(values)

data/lib/mods_display/fields/subject.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ModsDisplay
   class Subject < Field
     def fields
@@ -7,23 +9,19 @@ module ModsDisplay
         label = displayLabel(value) || I18n.t('mods_display.subject')
         return_text = []
         selected_subjects(value).each do |child|
-          if self.respond_to?(:"process_#{child.name}")
+          if respond_to?(:"process_#{child.name}")
             method_send = send(:"process_#{child.name}", child)
             return_text << method_send unless method_send.to_s.empty?
+          elsif child.text.include?('--')
+            return_text << child.text.split('--').map(&:strip)
           else
-            if child.text.include?('--')
-              return_text << child.text.split('--').map(&:strip)
-            else
-              return_text << child.text unless child.text.empty?
-            end
+            return_text << child.text unless child.text.empty?
           end
         end
         return_values << return_text.flatten unless return_text.empty?
-        unless return_values.empty?
-          return_fields << ModsDisplay::Values.new(label: label, values: return_values)
-        end
+        return_fields << ModsDisplay::Values.new(label: label, values: return_values) unless return_values.empty?
       end
-      collapse_subjects return_fields
+      collapse_fields return_fields
     end
 
     # Would really like to clean this up, but it works and is tested for now.
@@ -44,7 +42,7 @@ module ModsDisplay
     def process_name(element)
       name = ModsDisplay::Name.new([element]).fields.first
 
-      name.values.first if name
+      name&.values&.first
     end
 
     private
@@ -56,54 +54,23 @@ module ModsDisplay
     def values_from_subjects(element)
       return_values = []
       selected_subjects(element).each do |child|
-        if child.text.include?('--')
-          return_values << child.text.split('--').map(&:strip)
-        else
-          return_values << child.text.strip
-        end
+        return_values << if child.text.include?('--')
+                           child.text.split('--').map(&:strip)
+                         else
+                           child.text.strip
+                         end
       end
       return_values
     end
 
     def selected_subjects(element = @value)
-      element.children.select do |child|
-        !omit_elements.include?(child.name.to_sym)
+      element.children.reject do |child|
+        omit_elements.include?(child.name.to_sym)
       end
     end
 
     def omit_elements
-      [:cartographics, :geographicCode, :text]
-    end
-
-    # Providing subject specific collapsing method so we can
-    # collapse the labels w/o flattening all the subject fields.
-    def collapse_subjects(display_fields)
-      return_values = []
-      current_label = nil
-      prev_label = nil
-      buffer = []
-      display_fields.each_with_index do |field, index|
-        current_label = field.label
-        current_values = field.values
-        if display_fields.length == 1
-          return_values << ModsDisplay::Values.new(label: current_label, values: current_values)
-        elsif index == (display_fields.length - 1)
-          # need to deal w/ when we have a last element but we have separate labels in the buffer.
-          if current_label != prev_label
-            return_values << ModsDisplay::Values.new(label: prev_label, values: [buffer.flatten(1)])
-            return_values << ModsDisplay::Values.new(label: current_label, values: current_values)
-          else
-            buffer.concat(current_values)
-            return_values << ModsDisplay::Values.new(label: current_label, values: buffer.flatten(0))
-          end
-        elsif prev_label && (current_label != prev_label)
-          return_values << ModsDisplay::Values.new(label: prev_label, values: buffer.flatten(0))
-          buffer = []
-        end
-        buffer.concat(current_values)
-        prev_label = current_label
-      end
-      return_values
+      %i[cartographics geographicCode text]
     end
   end
 end

data/lib/mods_display/fields/title.rb

@@ -1,14 +1,14 @@
+# frozen_string_literal: true
+
 module ModsDisplay
   class Title < Field
     def fields
       return_values = []
-      if @values
-        @values.each do |value|
-          return_values << ModsDisplay::Values.new(
-            label: displayLabel(value) || title_label(value),
-            values: [assemble_title(value)]
-          )
-        end
+      @values&.each do |value|
+        return_values << ModsDisplay::Values.new(
+          label: displayLabel(value) || title_label(value),
+          values: [assemble_title(value)]
+        )
       end
       collapse_fields(return_values)
     end
@@ -20,8 +20,6 @@ module ModsDisplay
     end
 
     def assemble_title(element)
-      return displayForm(element) if displayForm(element)
-
       title = ''
       previous_element = nil
 
@@ -29,20 +27,21 @@ module ModsDisplay
         str = value.text.strip
         next if str.empty?
 
-        delimiter = case
-        when title.empty?, title.end_with?(' ')
-          nil
-        when title.end_with?('.', ',', ':', ';')
-          ' '
-        when value.name == 'subTitle'
-          ' : '
-        when value.name == 'partName' && previous_element.name == 'partNumber'
-          ', '
-        when value.name == 'partNumber', value.name == 'partName'
-          '. '
-        else
-          ' '
-        end
+        delimiter = if title.empty? || title.end_with?(' ')
+                      nil
+                    elsif previous_element&.name == 'nonSort' && title.end_with?('-', '\'')
+                      nil
+                    elsif title.end_with?('.', ',', ':', ';')
+                      ' '
+                    elsif value.name == 'subTitle'
+                      ' : '
+                    elsif value.name == 'partName' && previous_element.name == 'partNumber'
+                      ', '
+                    elsif value.name == 'partNumber' || value.name == 'partName'
+                      '. '
+                    else
+                      ' '
+                    end
 
         title += delimiter if delimiter
         title += str
@@ -50,7 +49,18 @@ module ModsDisplay
         previous_element = value
       end
 
-      title
+      if element['type'] == 'uniform' && element['nameTitleGroup'].present?
+        [uniform_title_name_part(element), title].compact.join('. ')
+      else
+        title
+      end
+    end
+
+    def uniform_title_name_part(element)
+      names = element.xpath("//m:name[@nameTitleGroup=\"#{element['nameTitleGroup']}\"]", **Mods::Record::NS_HASH)
+      names = element.xpath("//name[@nameTitleGroup=\"#{element['nameTitleGroup']}\"]") if names.empty?
+
+      ModsDisplay::Name.new(names).fields.first&.values&.first&.to_s
     end
 
     def title_parts
@@ -62,14 +72,15 @@ module ModsDisplay
          title_labels.key?(element.attributes['type'].value)
         return title_labels[element.attributes['type'].value]
       end
+
       I18n.t('mods_display.title')
     end
 
     def title_labels
       { 'abbreviated' => I18n.t('mods_display.abbreviated_title'),
-        'translated'  => I18n.t('mods_display.translated_title'),
+        'translated' => I18n.t('mods_display.translated_title'),
         'alternative' => I18n.t('mods_display.alternative_title'),
-        'uniform'     => I18n.t('mods_display.uniform_title') }
+        'uniform' => I18n.t('mods_display.uniform_title') }
     end
   end
 end

data/lib/mods_display/fields/values.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ModsDisplay
   class Values
     attr_accessor :label, :values

data/lib/mods_display/html.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ModsDisplay
   class HTML
     MODS_DISPLAY_FIELD_MAPPING = {
@@ -33,9 +35,8 @@ module ModsDisplay
     end
 
     def title
-      unless mods_field(:title).fields.empty?
-        return mods_field(:title).fields.first.values
-      end
+      return mods_field(:title).fields.first.values unless mods_field(:title).fields.empty?
+
       ''
     end
 

data/lib/mods_display/related_item_concerns.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ModsDisplay
   module RelatedItemConcerns
     private
@@ -16,12 +18,12 @@ module ModsDisplay
 
     def related_item_is_a_constituent?(item)
       item.attributes['type'].respond_to?(:value) &&
-      item.attributes['type'].value == 'constituent'
+        item.attributes['type'].value == 'constituent'
     end
 
     def related_item_is_host?(item)
       item.attributes['type'].respond_to?(:value) &&
-      item.attributes['type'].value == 'host'
+        item.attributes['type'].value == 'host'
     end
   end
 end

data/lib/mods_display/relator_codes.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ModsDisplay
   module RelatorCodes
     def relator_codes

data/lib/mods_display/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ModsDisplay
-  VERSION = '1.0.0.alpha3'
+  VERSION = '1.0.0'
 end

data/lib/mods_display.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'mods_display/version'
 require 'mods_display/html'
 require 'mods_display/country_codes'
@@ -34,14 +36,14 @@ require 'stanford-mods'
 
 require 'i18n'
 require 'i18n/backend/fallbacks'
-I18n::Backend::Simple.send(:include, I18n::Backend::Fallbacks)
-I18n.load_path += Dir["#{File.expand_path('../..', __FILE__)}/config/locales/*.yml"]
+I18n::Backend::Simple.include I18n::Backend::Fallbacks
+I18n.load_path += Dir["#{File.expand_path('..', __dir__)}/config/locales/*.yml"]
 I18n.backend.load_translations
 
 begin
   require 'rails'
 rescue LoadError
-  #do nothing
+  # do nothing
 end
 
 require 'mods_display/engine' if defined?(Rails)

data/mods_display.gemspec

@@ -21,7 +21,7 @@ Gem::Specification.new do |gem|
   gem.executables = gem.files.grep(%r{\Aexe/}) { |f| File.basename(f) }
   gem.require_paths = ["lib"]
 
-  gem.add_dependency 'stanford-mods', '~> 2.1'
+  gem.add_dependency 'stanford-mods', '>= 3.0.0.alpha1', '< 4'
   gem.add_dependency 'i18n'
   gem.add_dependency 'view_component'
 

metadata

@@ -1,29 +1,35 @@
 --- !ruby/object:Gem::Specification
 name: mods_display
 version: !ruby/object:Gem::Version
-  version: 1.0.0.alpha3
+  version: 1.0.0
 platform: ruby
 authors:
 - Jessie Keck
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-01-18 00:00:00.000000000 Z
+date: 2022-03-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: stanford-mods
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2.1'
+        version: 3.0.0.alpha1
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.0.0.alpha1
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '2.1'
+        version: '4'
 - !ruby/object:Gem::Dependency
   name: i18n
   requirement: !ruby/object:Gem::Requirement
@@ -235,9 +241,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubygems_version: 3.2.32
 signing_key: