mods_display 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8ec2c58c34a3f05225a077459d6cc226cd587a0a
-  data.tar.gz: c0e3f4e6e7533f5b25c0eb735f0bd0f5cfd9ea47
+  metadata.gz: b6fbeb6e358da70acf9959feba56cea0b735f47b
+  data.tar.gz: b4236dd3026d7d7cb3c5a93217e64491c6fd6528
 SHA512:
-  metadata.gz: 567d5eed25681658bf0cd94141d83b846d99b9d6698e20462f52a83a52f833c4e757fdb2da218c1c0decec854bc9b2651d044397685847e8547127afca49d8bb
-  data.tar.gz: f3e05076e40607b5ca914401feb0acce6aacf80ba43361bfbdb05aa535d4372fae6832910c0f805be6b422141d310c17e3e57dba4062afba21c980c3a9a8447e
+  metadata.gz: b2e555e1dd3269a8ad2fea3aa1426e042b417992cba343369dc4c8ba3a27c66c18d2ab637df006e9a9f69550a31d44237ed7535994093da65bcf3f916460dbb2
+  data.tar.gz: 91caac5689b774665d1e027b20cb4dbdcded4f62950799c46c77a062781968588e36a8313049695cb9b6baaf54494057d65cbf1289de2191d02ec156093cc0b7

data/lib/mods_display/fields/subject.rb

@@ -1,8 +1,10 @@
 class ModsDisplay::Subject < ModsDisplay::Field
   
   def fields
-    return_values = []
+    return_fields = []
     @values.each do |value|
+      return_values = []
+      label = displayLabel(value) || "Subject"
       return_text = []
       selected_subjects(value).each do |child|
         if self.respond_to?(:"process_#{child.name}")
@@ -18,9 +20,11 @@ class ModsDisplay::Subject < ModsDisplay::Field
       unless return_text.empty?
         return_values << return_text.flatten
       end
+      unless return_values.empty?
+        return_fields << ModsDisplay::Values.new(:label => label, :values => return_values)
+      end
     end
-    return [] if return_values.empty?
-    [ModsDisplay::Values.new(:label => "Subject", :values => return_values)]
+    collapse_subjects return_fields
   end
   
   # Would really like to clean this up, but it works and is tested for now.
@@ -99,5 +103,35 @@ class ModsDisplay::Subject < ModsDisplay::Field
   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 and (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
+  end
 end

data/lib/mods_display/version.rb

@@ -1,3 +1,3 @@
 module ModsDisplay
-  VERSION = "0.2.0"
+  VERSION = "0.2.1"
 end

data/mods_display.gemspec

@@ -10,7 +10,7 @@ Gem::Specification.new do |gem|
   gem.email         = ["jessie.keck@gmail.com"]
   gem.description   = %q{MODS Display is a gem to centralize the display logic of MODS medadata.}
   gem.summary       = %q{The MODS Display gem allows implementers to configure a customized display of MODS metadata.  This display implements the specifications defined at Stanford for how to display MODS.}
-  gem.homepage      = ""
+  gem.homepage      = "https://github.com/sul-dlss/mods_display"
 
   gem.files         = `git ls-files`.split($/)
   gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }

data/spec/fields/subject_spec.rb

@@ -25,8 +25,8 @@ describe ModsDisplay::Subject do
     @geo_subject = Stanford::Mods::Record.new.from_str(hierarchical_geo_subjects, false).subject
     @name_subject = Stanford::Mods::Record.new.from_str(name_subjects, false).subject
     @complex_subject = Stanford::Mods::Record.new.from_str(complex_subjects, false).subject
+    @display_label = Stanford::Mods::Record.new.from_str(display_label_subjects, false).subject
   end
-
   describe "fields" do
     it "should split individual child elments of subject into separate parts" do
       fields = mods_display_subject(@subject).fields
@@ -43,6 +43,16 @@ describe ModsDisplay::Subject do
       fields.length.should == 1
       fields.first.values.should == [["United States", "California", "Stanford"]]
     end
+    it "should handle display labels properly" do
+      fields = mods_display_subject(@display_label).fields
+      fields.length.should == 3
+      fields.first.label.should  == "Subject"
+      fields.first.values.should == [["A Subject", "Another Subject"], ["B Subject", "Another B Subject"]]
+      fields[1].label.should     == "Subject Heading"
+      fields[1].values.should    == [["Jazz", "Japan", "History and criticism"]]
+      fields.last.label.should   == "Subject"
+      fields.last.values.should  == [["Bay Area", "Stanford"]]
+    end
     it "should handle blank subjects properly" do
       mods_display_subject(@blank_subject).fields.should == []
     end
@@ -88,6 +98,12 @@ describe ModsDisplay::Subject do
       html.scan(/<br\/>/).length.should == 1
       html.scan(/ &gt; /).length.should == 3
     end
+    it "should handle complex display labels" do
+      html = mods_display_subject(@display_label).to_html
+      html.scan(/<dt title='Subject'>Subject:<\/dt>/).length.should eq 2
+      html.scan(/<dt title='Subject Heading'>Subject Heading:<\/dt>/).length.should eq 1
+      html.scan(/<dd>/).length.should == 3
+    end
   end
 
 end

data/spec/fixtures/subjects_fixtures.rb

@@ -76,5 +76,27 @@ module SubjectsFixtures
       </mods>
     XML
   end
-
+  def display_label_subjects
+    <<-XML
+      <mods>
+        <subject>
+          <topic>A Subject</topic>
+          <geographical>Another Subject</geographical>
+        </subject>
+        <subject>
+          <topic>B Subject</topic>
+          <geographical>Another B Subject</geographical>
+        </subject>
+        <subject displayLabel="Subject Heading">
+          <topic>Jazz</topic>
+          <geographical>Japan</geographical>
+          <topic>History and criticism</topic>
+        </subject>
+        <subject>
+          <topic>Bay Area</topic>
+          <geographical>Stanford</geographical>
+        </subject>
+      </mods>
+    XML
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mods_display
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Jessie Keck
@@ -137,7 +137,7 @@ files:
 - spec/integration/html_spec.rb
 - spec/integration/installation_spec.rb
 - spec/spec_helper.rb
-homepage: ''
+homepage: https://github.com/sul-dlss/mods_display
 licenses: []
 metadata: {}
 post_install_message: