mods_display 0.2.4 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fb219cbcafb42b00a015bb20fea661cf8a611ea0
-  data.tar.gz: c1890ddae8b1b70ecbf78dd038177302d1f07984
+  metadata.gz: 7405311ccd79996582dd2eacb5a4aed58f01bc10
+  data.tar.gz: 8239610c70f063e06bdf0b27dd1a11d21824b050
 SHA512:
-  metadata.gz: 5da361849ae26f9589712d5baad5be6e2efb58d86075425abe2e305fc6954ac7f61a7b0b2d8ce34bf2661503680cc287af66ccd39da44651a9fda9033cea3306
-  data.tar.gz: 2ca0bb8679a66f76fd67b80c1a1c8bb39a6ad35280dd1fcca382cb8f38800521d42669b5694c7aae7da4ca9d75dc90d5087d2241db72ef5523b5218844584b29
+  metadata.gz: 0db38505cefc90acce173c4092574fe50bd7781326f5aff9b490a2491c3a6795641a3bb5055a72b0da6a202e48c62b8a798b9ef10d07a9286e1e0c491a6a2da6
+  data.tar.gz: 66abdd1b57edd01c590c4cc3e821d14db40cc377b24d737be32116f67f9363556157b1f625e5eaf0c4514be975489f2825acc5775bd2b66dc323528b6157325d

data/README.md

@@ -168,14 +168,22 @@ When getting JUST the main (first) title out of the metadata, it will be useful
 You can also access the array of ModsDisplay::Values objects for a given class directly by calling the name of the class. The class names are not always intuitive for public consumption so you may want to check the code the particular method to call.
 
     render_mods_display(@model).abstract
-    => [#<ModsDisplay::Values @label="Abstract", @values=["Hey. I'm an abstract."]>]
+    => [#<ModsDisplay::Values @label="Abstract:", @values=["Hey. I'm an abstract."]>]
 
 Given that this semantics that we're concerned with here are more about titles and data construction rather than XML it may be required that you find something by the label. A common example of this is the imprint class.  The imprint class can retun other publication data that is not the imprint statement.  You'll want to select (using your favorite enumerable method) the element in the array that is an imprint.
 
     imprint = render_mods_display(@model).imprint.find do |data|
-      data.label == "Imprint"
+      data.label == "Imprint:"
     end.values
 
+## Release/Upgrade Notes
+
+#### v0.3.0
+
+Labels now have internationalization support.  We have added colons to the english labels due to certain languages' punctuation rules requiring different spacing between the label and colon.
+
+Given that fact, you will want to update any pre 0.3.0 code that searches for elements by label in a way that would fail with the presence of a colon.
+
 ## Contributing
 
 1. Fork it

data/config/locales/en.yml

@@ -0,0 +1,67 @@
+en:
+  mods_display:
+    abbreviated_title: "Abbreviated title:"
+    abstract: "Abstract:"
+    access_condition: "Access condition:"
+    alternative_title: "Alternative title:"
+    author_creator: "Author/Creator:"
+    bibliography: "Bibliography:"
+    biographical_historical: "Biographical/Historical:"
+    citation_reference: "Citation/Reference:"
+    collection: "Collection:"
+    contact: "Contact:"
+    contributor: "Contributor:"
+    copyright: "Copyright:"
+    copyright_date: "Copyright date:"
+    creation_production_credits: "Creation/Production credits:"
+    date_captured: "Date captured:"
+    date_created: "Date created:"
+    date_modified: "Date modified:"
+    date_sequential_designation: "Date/Sequential designation:"
+    date_valid: "Date valid:"
+    digital_origin: "Digital origin:"
+    doi: "DOI:"
+    format: "Format:"
+    frequency: "Frequency:"
+    genre: "Genre:"
+    handle: "Handle:"
+    identifier: "Identifier:"
+    imprint: "Imprint:"
+    isbn: "ISBN:"
+    ismn: "ISMN:"
+    isrc: "ISRC:"
+    issn: "ISSN:"
+    issuance: "Issuance:"
+    issue_number: "Issue number:"
+    language: "Language:"
+    lccn: "LCCN:"
+    location: "Location:"
+    map_data: "Map data:"
+    matrix_number: "Matric number:"
+    music_plate: "Music plate:"
+    music_publisher: "Music publisher:"
+    note: "Note:"
+    oclc: "OCLC:"
+    physical_description: "Physical description:"
+    place: "Place:"
+    preferred_citation: "Preferred citation:"
+    publications: "Publications:"
+    publisher: "Publisher:"
+    referenced_by: "Referenced by:"
+    references: "References:"
+    related_item: "Related item:"
+    repository: "Repository:"
+    restriction_on_access: "Restriction on access:"
+    sici: "SICI:"
+    statement_of_responsibility: "Statement of responsibility:"
+    stock_number: "Stock number:"
+    subject: "Subject:"
+    table_of_contents: "Table of contents:"
+    target_audience: "Target audience:"
+    title: "Title:"
+    translated_title: "Translated title:"
+    type_of_resource: "Type of resource:"
+    uniform_title: "Uniform title:"
+    upc: "UPC:"
+    use_and_reproduction: "Use and reproduction:"
+    videorecording_identifier: "Videorecording identifier:"

data/lib/mods_display.rb

@@ -36,4 +36,11 @@ require "mods_display/fields/resource_type"
 require "mods_display/fields/subject"
 require "mods_display/fields/sub_title"
 require "mods_display/fields/title"
-require "mods_display/fields/values"
+require "mods_display/fields/values"
+
+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.default_locale = :en
+I18n.backend.load_translations

data/lib/mods_display/fields/abstract.rb

@@ -2,7 +2,7 @@ class ModsDisplay::Abstract < ModsDisplay::Field
 
   private
   def displayLabel(element)
-    super(element) || "Abstract"
+    super(element) || I18n.t('mods_display.abstract')
   end
 
 end

data/lib/mods_display/fields/access_condition.rb

@@ -52,7 +52,7 @@ class ModsDisplay::AccessCondition < ModsDisplay::Field
     if access_labels.has_key?(type)
       return access_labels[type]
     end
-    "Access condition"
+    I18n.t('mods_display.access_condition')
   end
 
   def normalize_type(element)
@@ -64,10 +64,10 @@ class ModsDisplay::AccessCondition < ModsDisplay::Field
   end
 
   def access_labels
-    {"useandreproduction"  => "Use and reproduction",
-     "restrictiononaccess" => "Restriction on access",
-     "copyright"           => "Copyright",
-     "license"             => "License"
+    {"useandreproduction"  => I18n.t('mods_display.use_and_reproduction'),
+     "restrictiononaccess" => I18n.t('mods_display.restriction_on_access'),
+     "copyright"           => I18n.t('mods_display.copyright'),
+     "license"             => I18n.t('mods_display.license')
      }
   end
 end

data/lib/mods_display/fields/audience.rb

@@ -2,7 +2,7 @@ class ModsDisplay::Audience < ModsDisplay::Field
 
   private
   def displayLabel(element)
-    super(element) || "Target audience"
+    super(element) || I18n.t('mods_display.target_audience')
   end
 
 end

data/lib/mods_display/fields/cartographics.rb

@@ -10,7 +10,7 @@ class ModsDisplay::Cartographics < ModsDisplay::Field
           projection = field.projection.empty? ? nil : field.projection.text
           coordinates = field.coordinates.empty? ? nil : field.coordinates.text
           post_scale = [projection, coordinates].compact.length > 0 ? [projection, coordinates].compact.join(" ") : nil
-          return_fields << ModsDisplay::Values.new({:label => (displayLabel(field) || label || "Map data"),
+          return_fields << ModsDisplay::Values.new({:label => (displayLabel(field) || label || I18n.t('mods_display.map_data')),
                                                     :values => [[scale, post_scale].compact.join(" ; ")]})
         end
       end

data/lib/mods_display/fields/collection.rb

@@ -1,7 +1,7 @@
 class ModsDisplay::Collection < ModsDisplay::Field
 
   def label
-    super || "Collection"
+    super || I18n.t('mods_display.collection')
   end
 
   def fields

data/lib/mods_display/fields/contact.rb

@@ -1,7 +1,7 @@
 class ModsDisplay::Contact < ModsDisplay::Field
   def fields
     return_fields = contact_fields.map do |value|
-      ModsDisplay::Values.new(:label => displayLabel(value) || "Contact", :values => [value.text])
+      ModsDisplay::Values.new(:label => displayLabel(value) || I18n.t('mods_display.contact'), :values => [value.text])
     end
     collapse_fields(return_fields)
   end

data/lib/mods_display/fields/contents.rb

@@ -2,7 +2,7 @@ class ModsDisplay::Contents < ModsDisplay::Field
 
   private
   def displayLabel(element)
-    super(element) || "Table of contents"
+    super(element) || I18n.t('mods_display.table_of_contents')
   end
 
 end

data/lib/mods_display/fields/description.rb

@@ -8,7 +8,7 @@ class ModsDisplay::Description < ModsDisplay::Field
   end
 
   def description_label(element)
-    label || displayLabel(element) || labels[element.name.to_sym] || "Physical description"
+    label || displayLabel(element) || labels[element.name.to_sym] || I18n.t('mods_display.physical_description')
   end
 
   private
@@ -20,8 +20,8 @@ class ModsDisplay::Description < ModsDisplay::Field
   end
 
   def labels
-    {:digitalOrigin => "Digital origin",
-     :note          => "Note"
+    {:digitalOrigin => I18n.t('mods_display.digital_origin'),
+     :note          => I18n.t('mods_display.note')
      }
   end
 

data/lib/mods_display/fields/field.rb

@@ -23,7 +23,7 @@ class ModsDisplay::Field
     output = ""
     fields.each do |field|
       if field.values.any?{|f| f && !f.empty? }
-        output << "<dt#{label_class} title='#{field.label}'>#{field.label}:</dt>"
+        output << "<dt#{label_class} #{sanitized_field_title(field.label)}>#{field.label}</dt>"
         output << "<dd#{value_class}>"
           output << field.values.map do |val|
             @config.link ? link_to_value(val.to_s) : link_urls_and_email(val.to_s)
@@ -57,10 +57,14 @@ class ModsDisplay::Field
   def displayLabel(element)
     if (element.respond_to?(:attributes) and
         element.attributes["displayLabel"].respond_to?(:value))
-      element.attributes["displayLabel"].value
+      "#{element.attributes["displayLabel"].value}:"
     end
   end
 
+  def sanitized_field_title(label)
+    "title='#{label.gsub(/:$/,'').strip}'"
+  end
+
   def replace_tokens(object, value)
     object = object.dup
     if object.is_a?(Hash)

data/lib/mods_display/fields/format.rb

@@ -11,11 +11,11 @@ class ModsDisplay::Format < ModsDisplay::Field
     unless @values.physical_description.nil?
       @values.physical_description.each do |description|
         unless description.form.nil? or description.form.empty?
-          return_fields << ModsDisplay::Values.new(:label  => displayLabel(description) || "Format",
+          return_fields << ModsDisplay::Values.new(:label  => displayLabel(description) || format_label,
                                                    :values => [description.form.map{|f| f.text.strip }.uniq.join(", ")])
         end
         unless description.extent.nil? or description.extent.empty?
-          return_fields << ModsDisplay::Values.new(:label  => displayLabel(description) || "Format",
+          return_fields << ModsDisplay::Values.new(:label  => displayLabel(description) || format_label,
                                                    :values => [description.extent.map{|e| e.text }.join(", ")])
         end
       end
@@ -36,4 +36,7 @@ class ModsDisplay::Format < ModsDisplay::Field
     format.strip.downcase.gsub(/\/|\\|\s+/, "_")
   end
 
+  def format_label
+    I18n.t('mods_display.format')
+  end
 end

data/lib/mods_display/fields/genre.rb

@@ -10,6 +10,6 @@ class ModsDisplay::Genre < ModsDisplay::Field
   private
 
   def displayLabel(element)
-    super(element) || "Genre"
+    super(element) || I18n.t('mods_display.genre')
   end
 end

data/lib/mods_display/fields/identifier.rb

@@ -11,30 +11,30 @@ class ModsDisplay::Identifier < ModsDisplay::Field
 
   def identifier_label(element)
     if element.attributes["type"].respond_to?(:value)
-      return identifier_labels[element.attributes["type"].value] || element.attributes["type"].value
+      return identifier_labels[element.attributes["type"].value] || "#{element.attributes["type"].value}:"
     end
-    "Identifier"
+    I18n.t('mods_display.identifier')
   end
 
   def identifier_labels
-    {"local"                     => "Identifier",
-     "isbn"                      => "ISBN",
-     "issn"                      => "ISSN",
-     "issn-l"                    => "ISSN",
-     "doi"                       => "DOI",
-     "hdl"                       => "Handle",
-     "isrc"                      => "ISRC",
-     "ismn"                      => "ISMN",
-     "issue number"              => "Issue number",
-     "lccn"                      => "LCCN",
-     "oclc"                      => "OCLC",
-     "matrix number"             => "Matrix number",
-     "music publisher"           => "Music publisher",
-     "music plate"               => "Music plate",
-     "sici"                      => "SICI",
-     "upc"                       => "UPC",
-     "videorecording identifier" => "Videorecording identifier",
-     "stock number"              => "Stock number"}
+    {"local"                     => I18n.t('mods_display.identifier'),
+     "isbn"                      => I18n.t('mods_display.isbn'),
+     "issn"                      => I18n.t('mods_display.issn'),
+     "issn-l"                    => I18n.t('mods_display.issn'),
+     "doi"                       => I18n.t('mods_display.doi'),
+     "hdl"                       => I18n.t('mods_display.handle'),
+     "isrc"                      => I18n.t('mods_display.isrc'),
+     "ismn"                      => I18n.t('mods_display.ismn'),
+     "issue number"              => I18n.t('mods_display.issue_number'),
+     "lccn"                      => I18n.t('mods_display.lccn'),
+     "oclc"                      => I18n.t('mods_display.oclc'),
+     "matrix number"             => I18n.t('mods_display.matrix_number'),
+     "music publisher"           => I18n.t('mods_display.music_publisher'),
+     "music plate"               => I18n.t('mods_display.music_plate'),
+     "sici"                      => I18n.t('mods_display.sici'),
+     "upc"                       => I18n.t('mods_display.upc'),
+     "videorecording identifier" => I18n.t('mods_display.videorecording_identifier'),
+     "stock number"              => I18n.t('mods_display.stock_number')}
   end
 
 end

data/lib/mods_display/fields/imprint.rb

@@ -33,7 +33,7 @@ class ModsDisplay::Imprint < ModsDisplay::Field
         editionPlace = [edition, placePub].compact.join(" - ")
         editionPlace = nil if editionPlace.strip.empty?
         unless [editionPlace, parts].compact.join(", ").strip.empty?
-          return_fields << ModsDisplay::Values.new(:label => displayLabel(value) || "Imprint", :values => [[editionPlace, parts].compact.join(", ")])
+          return_fields << ModsDisplay::Values.new(:label => displayLabel(value) || I18n.t('mods_display.imprint'), :values => [[editionPlace, parts].compact.join(", ")])
         end
         if dates(value).length > 0
           return_fields.concat(dates(value))
@@ -219,7 +219,7 @@ class ModsDisplay::Imprint < ModsDisplay::Field
     display_form = element.children.find do |child|
       child.name == "displayForm"
     end
-    ModsDisplay::Values.new(:label => displayLabel(element) || "Imprint", :values => [display_form.text]) if display_form
+    ModsDisplay::Values.new(:label => displayLabel(element) || I18n.t('mods_display.imprint'), :values => [display_form.text]) if display_form
   end
 
   private
@@ -233,13 +233,13 @@ class ModsDisplay::Imprint < ModsDisplay::Field
   end
 
   def pub_info_labels
-    {:dateCreated   => "Date created",
-     :dateCaptured  => "Date captured",
-     :dateValid     => "Date valid",
-     :dateModified  => "Date modified",
-     :copyrightDate => "Copyright date",
-     :issuance      => "Issuance",
-     :frequency     => "Frequency"
+    {:dateCreated   => I18n.t('mods_display.date_created'),
+     :dateCaptured  => I18n.t('mods_display.date_captured'),
+     :dateValid     => I18n.t('mods_display.date_valid'),
+     :dateModified  => I18n.t('mods_display.date_modified'),
+     :copyrightDate => I18n.t('mods_display.copyright_date'),
+     :issuance      => I18n.t('mods_display.issuance'),
+     :frequency     => I18n.t('mods_display.frequency')
     }
   end
 

data/lib/mods_display/fields/language.rb

@@ -4,7 +4,7 @@ class ModsDisplay::Language < ModsDisplay::Field
       if value.respond_to?(:languageTerm)
         value.languageTerm.map do |term|
           if term.attributes["type"].respond_to?(:value) and term.attributes["type"].value == "code"
-            ModsDisplay::Values.new(:label => displayLabel(value) || displayLabel(term) || "Language", :values => [displayForm(value) || language_codes[term.text]].flatten)
+            ModsDisplay::Values.new(:label => displayLabel(value) || displayLabel(term) || I18n.t('mods_display.language'), :values => [displayForm(value) || language_codes[term.text]].flatten)
           end
         end.flatten.compact
       end

data/lib/mods_display/fields/location.rb

@@ -6,13 +6,13 @@ class ModsDisplay::Location < ModsDisplay::Field
       location.children.each do |child|
         if location_field_keys.include? child.name.to_sym
           if child.name.to_sym == :url
-            loc_label = displayLabel(location) || "Location"
-            value = "<a href='#{child.text}'>#{displayLabel(child) || child.text}</a>"
+            loc_label = displayLabel(location) || I18n.t('mods_display.location')
+            value = "<a href='#{child.text}'>#{(displayLabel(child) || child.text).gsub(/:$/,'')}</a>"
           else
-            loc_label = location_label(child) || displayLabel(location) || "Location"
+            loc_label = location_label(child) || displayLabel(location) || I18n.t('mods_display.location')
             value = child.text
           end
-          return_fields << ModsDisplay::Values.new(:label  => loc_label || displayLabel(location) || "Location",
+          return_fields << ModsDisplay::Values.new(:label  => loc_label || displayLabel(location) || I18n.t('mods_display.location'),
                                                    :values => [value])
         end
       end
@@ -35,7 +35,7 @@ class ModsDisplay::Location < ModsDisplay::Field
   end
 
   def location_labels
-    {"repository" => "Repository"}
+    {"repository" => I18n.t('mods_display.repository')}
   end
 
 end

data/lib/mods_display/fields/name.rb

@@ -18,7 +18,7 @@ class ModsDisplay::Name < ModsDisplay::Field
     return nil if fields.empty? or @config.ignore?
     output = ""
     fields.each do |field|
-      output << "<dt#{label_class} title='#{field.label}'>#{field.label}:</dt>"
+      output << "<dt#{label_class} #{sanitized_field_title(field.label)}>#{field.label}</dt>"
       output << "<dd#{value_class}>"
         output << field.values.map do |val|
           if @config.link
@@ -38,9 +38,9 @@ class ModsDisplay::Name < ModsDisplay::Field
 
   def name_label(element)
     if !has_role?(element) || is_primary?(element) || has_author_or_creator_roles?(element)
-      "Author/Creator"
+      I18n.t('mods_display.author_creator')
     else
-      "Contributor"
+      I18n.t('mods_display.contributor')
     end
   end
 

data/lib/mods_display/fields/note.rb

@@ -20,22 +20,21 @@ class ModsDisplay::Note < ModsDisplay::Field
 
   def note_label(element)
     if element.attributes["type"].respond_to?(:value)
-      return note_labels[element.attributes["type"].value] || element.attributes["type"].value.capitalize
+      return note_labels[element.attributes["type"].value] || "#{element.attributes["type"].value.capitalize}:"
     end
-    "Note"
+    I18n.t('mods_display.note')
   end
 
   def note_labels
-    {"statement of responsibility" => "Statement of responsibility",
-     "date/sequential designation" => "Date/Sequential designation",
-     "publications"                => "Publications",
-     "references"                  => "References",
-     "bibliography"                => "Bibliography",
-     "preferred citation"          => "Preferred citation",
-     "date/sequential designation" =>	"Date/Sequential designation",
-     "biographical/historical"     => "Biographical/Historical",
-     "creation/production credits" => "Creation/Production credits",
-     "citation/reference"          => "Citation/Reference"
+    {"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'),
+     "creation/production credits" => I18n.t('mods_display.creation_production_credits'),
+     "citation/reference"          => I18n.t('mods_display.citation_reference')
      }
   end
 

data/lib/mods_display/fields/related_item.rb

@@ -74,11 +74,11 @@ class ModsDisplay::RelatedItem < ModsDisplay::Field
     else
       case
       when related_item_is_a_location?(item)
-        return "Location"
+        return I18n.t('mods_display.location')
       when related_item_is_a_reference?(item)
-        return "Referenced by"
+        return I18n.t('mods_display.referenced_by')
       end
-      "Related item"
+      I18n.t('mods_display.related_item')
     end
   end
 end

data/lib/mods_display/fields/resource_type.rb

@@ -9,6 +9,6 @@ class ModsDisplay::ResourceType < ModsDisplay::Field
 
   private
   def displayLabel(element)
-    super(element) || "Type of resource"
+    super(element) || I18n.t('mods_display.type_of_resource')
   end
 end

data/lib/mods_display/fields/subject.rb

@@ -4,7 +4,7 @@ class ModsDisplay::Subject < ModsDisplay::Field
     return_fields = []
     @values.each do |value|
       return_values = []
-      label = displayLabel(value) || "Subject"
+      label = displayLabel(value) || I18n.t('mods_display.subject')
       return_text = []
       selected_subjects(value).each do |child|
         if self.respond_to?(:"process_#{child.name}")
@@ -32,7 +32,7 @@ class ModsDisplay::Subject < ModsDisplay::Field
     return nil if fields.empty? or @config.ignore?
     output = ""
     fields.each do |field|
-      output << "<dt#{label_class} title='#{field.label}'>#{field.label}:</dt>"
+      output << "<dt#{label_class} #{sanitized_field_title(field.label)}>#{field.label}</dt>"
       output << "<dd#{value_class}>"
         subs = []
         field.values.each do |subjects|

data/lib/mods_display/fields/title.rb

@@ -46,14 +46,14 @@ class ModsDisplay::Title < ModsDisplay::Field
         title_labels.has_key?(element.attributes["type"].value))
       return title_labels[element.attributes["type"].value]
     end
-    "Title"
+    I18n.t('mods_display.title')
   end
 
   def title_labels
-    {"abbreviated" => "Abbreviated title",
-     "translated"  => "Translated title",
-     "alternative" => "Alternative title",
-     "uniform"     => "Uniform title"}
+    {"abbreviated" => I18n.t('mods_display.abbreviated_title'),
+     "translated"  => I18n.t('mods_display.translated_title'),
+     "alternative" => I18n.t('mods_display.alternative_title'),
+     "uniform"     => I18n.t('mods_display.uniform_title')}
   end
 
 end

data/lib/mods_display/version.rb

@@ -1,3 +1,3 @@
 module ModsDisplay
-  VERSION = "0.2.4"
+  VERSION = "0.3.0"
 end

data/mods_display.gemspec

@@ -18,6 +18,7 @@ Gem::Specification.new do |gem|
   gem.require_paths = ["lib"]
   
   gem.add_dependency 'stanford-mods'
+  gem.add_dependency 'i18n'
   
   gem.add_development_dependency 'rake'
   gem.add_development_dependency 'rspec'

data/spec/fields/abstract_spec.rb

@@ -14,7 +14,7 @@ describe ModsDisplay::Abstract do
     it "should get a default 'Abstract' label" do
       fields = mods_display_abstract(@link).fields
       fields.length.should == 1
-      fields.first.label.should == "Abstract"
+      fields.first.label.should == "Abstract:"
     end
   end
 

data/spec/fields/access_condition_spec.rb

@@ -23,7 +23,7 @@ describe ModsDisplay::AccessCondition do
     it "should normalize types and assign proper labels" do
       fields = mods_display_access_condition(@restrict_condition).fields
       fields.length.should == 1
-      fields.first.label.should == "Restriction on access"
+      fields.first.label.should == "Restriction on access:"
       fields.first.values.each_with_index do |value, index|
         value.should match /^Restrict Access Note#{index+1}/
       end

data/spec/fields/audience_spec.rb

@@ -11,10 +11,10 @@ describe ModsDisplay::Contents do
   end
   describe "label" do
     it "should have a default label" do
-      mods_display_audience(@audience).label.should == "Target audience"
+      mods_display_audience(@audience).label.should == "Target audience:"
     end
     it "should use the displayLabel attribute when one is available" do
-      mods_display_audience(@display_label).label.should == "Special Label"
+      mods_display_audience(@display_label).label.should == "Special Label:"
     end
   end  
   

data/spec/fields/collection_spec.rb

@@ -12,10 +12,10 @@ describe ModsDisplay::Collection do
   end
   describe "label" do
     it "should default to Collection" do
-      mods_display_collection(@collection).label.should == "Collection"
+      mods_display_collection(@collection).label.should == "Collection:"
     end
     it "should get the displayLabel if available" do
-      mods_display_collection(@display_label).label.should == "Special Collection"
+      mods_display_collection(@display_label).label.should == "Special Collection:"
     end
   end
   describe "fields" do

data/spec/fields/contents_spec.rb

@@ -11,10 +11,10 @@ describe ModsDisplay::Contents do
   end
   describe "label" do
     it "should have a default label" do
-      mods_display_contents(@contents).label.should == "Table of contents"
+      mods_display_contents(@contents).label.should == "Table of contents:"
     end
     it "should use the displayLabel attribute when one is available" do
-      mods_display_contents(@display_label).label.should == "Special Label"
+      mods_display_contents(@display_label).label.should == "Special Label:"
     end
   end  
   

data/spec/fields/description_spec.rb

@@ -13,16 +13,16 @@ describe ModsDisplay::Description do
   end
   describe "labels" do
     it "should use the displayLabel if one is provided" do
-      mods_display_description(@display_label).fields.first.label.should == "SpecialLabel"
+      mods_display_description(@display_label).fields.first.label.should == "SpecialLabel:"
     end
     it "should get the default label for a child element" do
-      mods_display_description(@form).fields.first.label.should == "Note"
+      mods_display_description(@form).fields.first.label.should == "Note:"
     end
     it "should get multiple lables for mixed content" do
-      mods_display_description(@mixed).fields.map{|v| v.label }.should == ["Note", "Digital origin"]
+      mods_display_description(@mixed).fields.map{|v| v.label }.should == ["Note:", "Digital origin:"]
     end
     it "should get the display label from child elements" do
-      mods_display_description(@child_display_label).fields.map{|f| f.label }.should == ["Note Label"]
+      mods_display_description(@child_display_label).fields.map{|f| f.label }.should == ["Note Label:"]
     end
   end
   

data/spec/fields/genre_spec.rb

@@ -14,12 +14,12 @@ describe ModsDisplay::Abstract do
     it "should return a default 'Genre' label" do
       fields = mods_display_genre(@genre).fields
       fields.length.should == 1
-      fields.first.label.should == "Genre"
+      fields.first.label.should == "Genre:"
     end
     it "should use a display label when one is available" do
       fields = mods_display_genre(@display_label).fields
       fields.length.should == 1
-      fields.first.label.should == "Special label"
+      fields.first.label.should == "Special label:"
     end
   end
   describe "fields" do

data/spec/fields/identifier_spec.rb

@@ -14,16 +14,16 @@ describe ModsDisplay::Note do
   end
   describe "label" do
     it "should have a default label" do
-      mods_display_id(@id).fields.first.label.should == "Identifier"
+      mods_display_id(@id).fields.first.label.should == "Identifier:"
     end
     it "should use the displayLabel attribute when one is available" do
-      mods_display_id(@display_label).fields.first.label.should == "Special Label"
+      mods_display_id(@display_label).fields.first.label.should == "Special Label:"
     end
     it "should use get a label from a list of translations" do
-      mods_display_id(@issue_label).fields.first.label.should == "Issue number"
+      mods_display_id(@issue_label).fields.first.label.should == "Issue number:"
     end
     it "should use use the raw type attribute if one is present" do
-      mods_display_id(@type_label).fields.first.label.should == "Some other Type"
+      mods_display_id(@type_label).fields.first.label.should == "Some other Type:"
     end
   end
   
@@ -32,15 +32,15 @@ describe ModsDisplay::Note do
       fields = mods_display_id(@complex_label).fields
       fields.length.should == 3
       
-      fields.first.label.should == "Identifier"
+      fields.first.label.should == "Identifier:"
       fields.first.values.length.should == 2
       fields.first.values.should == ["12345", "54321"]
       
-      fields[1].label.should == "Issue number"
+      fields[1].label.should == "Issue number:"
       fields[1].values.length.should == 1
       fields[1].values.should == ["12345"]
       
-      fields.last.label.should == "Identifier"
+      fields.last.label.should == "Identifier:"
       fields.last.values.length.should == 1
       fields.last.values.should == ["98765"]
     end

data/spec/fields/imprint_spec.rb

@@ -39,18 +39,18 @@ describe ModsDisplay::Imprint do
 
   describe "labels" do
     it "should get the Imprint label by default" do
-      mods_display_imprint(@imprint).fields.first.label.should == "Imprint"
+      mods_display_imprint(@imprint).fields.first.label.should == "Imprint:"
     end
     it "should get the label from non-imprint origin info fields" do
       fields = mods_display_imprint(@edition_and_date).fields
-      fields.first.label.should == "Date valid"
-      fields.last.label.should == "Issuance"
+      fields.first.label.should == "Date valid:"
+      fields.last.label.should == "Issuance:"
     end
     it "should get multiple labels when we have mixed content" do
-      mods_display_imprint(@mixed).fields.map{|val| val.label }.should == ["Imprint", "Date captured", "Issuance"]
+      mods_display_imprint(@mixed).fields.map{|val| val.label }.should == ["Imprint:", "Date captured:", "Issuance:"]
     end
     it "should use the displayLabel when available" do
-       mods_display_imprint(@display_label).fields.map{|val| val.label }.should == ["TheLabel", "IssuanceLabel"]
+       mods_display_imprint(@display_label).fields.map{|val| val.label }.should == ["TheLabel:", "IssuanceLabel:"]
     end
   end
 
@@ -149,38 +149,38 @@ describe ModsDisplay::Imprint do
           fields = mods_display_imprint(@encoded_dates).fields
           fields.length.should == 4
           fields.find do |field|
-            field.label == "Imprint"
+            field.label == "Imprint:"
           end.values.should == ["2013"]
         end
         it "should handle month+year dates properly" do
           fields = mods_display_imprint(@encoded_dates).fields
           fields.length.should == 4
           fields.find do |field|
-            field.label == "Date captured"
+            field.label == "Date captured:"
           end.values.should == ["July 2013"]
         end
         it "should handle full dates properly" do
           fields = mods_display_imprint(@encoded_dates).fields
           fields.length.should == 4
           fields.find do |field|
-            field.label == "Date created"
+            field.label == "Date created:"
           end.values.should == ["July 10, 2013"]
         end
         it "should not try to handle dates we can't parse" do
           fields = mods_display_imprint(@encoded_dates).fields
           fields.length.should == 4
           fields.find do |field|
-            field.label == "Date modified"
+            field.label == "Date modified:"
           end.values.should == ["Jul. 22, 2013"]
         end
         it "should accept date configurations" do
           fields = mods_display_format_date_imprint(@encoded_dates).fields
           fields.length.should == 4
           fields.find do |field|
-            field.label == "Date created"
+            field.label == "Date created:"
           end.values.should == ["(2013) July, 10"]
           fields.find do |field|
-            field.label == "Date captured"
+            field.label == "Date captured:"
           end.values.should == ["July (2013)"]
         end
       end

data/spec/fields/language_spec.rb

@@ -18,7 +18,7 @@ describe ModsDisplay::Language do
       values = mods_display_language(@display_label).fields
       values.length.should == 1
       values.first.should be_a ModsDisplay::Values
-      values.first.label.should == "Lang"
+      values.first.label.should == "Lang:"
       values.first.values.should == ["English"]
     end
     it "should not return any non type='code' languageTerms from the XML" do

data/spec/fields/location_spec.rb

@@ -13,16 +13,16 @@ describe ModsDisplay::Note do
   end
   describe "label" do
     it "should have a default label" do
-      mods_display_location(@location).fields.first.label.should == "Location"
+      mods_display_location(@location).fields.first.label.should == "Location:"
     end
     it "should use the displayLabel attribute when one is available" do
-      mods_display_location(@display_label).fields.first.label.should == "Special Label"
+      mods_display_location(@display_label).fields.first.label.should == "Special Label:"
     end
     it "should handle the URL labels correctly" do
-      mods_display_location(@urls).fields.map{|f| f.label}.should == ["Location", "PURL"]
+      mods_display_location(@urls).fields.map{|f| f.label}.should == ["Location:", "PURL:"]
     end
     it "should use get a label from a list of translations" do
-      mods_display_location(@repository_label).fields.first.label.should == "Repository"
+      mods_display_location(@repository_label).fields.first.label.should == "Repository:"
     end
   end  
   describe "fields" do
@@ -30,13 +30,13 @@ describe ModsDisplay::Note do
       it "should link and use the displayLabel as text" do
         fields = mods_display_location(@urls).fields
         fields.length.should == 2
-        field = fields.find{|f| f.label == "Location"}
+        field = fields.find{|f| f.label == "Location:"}
         field.values.should == ["<a href='http://library.stanford.edu'>Stanford University Library</a>"]
       end
       it "should link the URL itself in the absence of a displayLabel on the url element" do
         fields = mods_display_location(@urls).fields
         fields.length.should == 2
-        field = fields.find{|f| f.label == "PURL"}
+        field = fields.find{|f| f.label == "PURL:"}
         field.values.should == ["<a href='http://purl.stanford.edu'>http://purl.stanford.edu</a>"]
       end
     end

data/spec/fields/name_spec.rb

@@ -32,13 +32,13 @@ describe ModsDisplay::Language do
   end
   describe "label" do
     it "should default Author/Creator when no role is available" do
-      mods_display_name(@name).fields.first.label.should == "Author/Creator"
+      mods_display_name(@name).fields.first.label.should == "Author/Creator:"
     end
     it "should label 'Author/Creator' for primary authors" do
-      mods_display_name(@primary_name).fields.first.label.should == "Author/Creator"
+      mods_display_name(@primary_name).fields.first.label.should == "Author/Creator:"
     end
     it "should apply contributor labeling to all non blank/author/creator roles" do
-      mods_display_name(@contributor).fields.first.label.should == "Contributor"
+      mods_display_name(@contributor).fields.first.label.should == "Contributor:"
     end
   end
   
@@ -70,7 +70,7 @@ describe ModsDisplay::Language do
     it "should collapse adjacent matching labels" do
       fields = mods_display_name(@collapse_label).fields
       fields.length.should == 1
-      fields.first.label.should == "Author/Creator"
+      fields.first.label.should == "Author/Creator:"
       fields.first.values.each do |val|
         ["John Doe", "Jane Doe"].should include val.to_s
       end
@@ -79,16 +79,16 @@ describe ModsDisplay::Language do
       fields = mods_display_name(@complex_labels).fields
 
       fields.length.should == 3
-      fields.first.label.should == "Author/Creator"
+      fields.first.label.should == "Author/Creator:"
       fields.first.values.length.should == 1
       fields.first.values.first.to_s.should == "John Doe"
       
-      fields[1].label.should == "Contributor"
+      fields[1].label.should == "Contributor:"
       fields[1].values.length.should == 1
       fields[1].values.first.name.should == "Jane Doe"
       fields[1].values.first.roles.should == ["lithographer"]
 
-      fields.last.label.should == "Author/Creator"
+      fields.last.label.should == "Author/Creator:"
       fields.last.values.length.should == 2
       fields.last.values.each do |val|
         ["Jane Dough", "John Dough"].should include val.to_s

data/spec/fields/note_spec.rb

@@ -15,16 +15,16 @@ describe ModsDisplay::Note do
   end
   describe "label" do
     it "should have a default label" do
-      mods_display_note(@note).fields.first.label.should == "Note"
+      mods_display_note(@note).fields.first.label.should == "Note:"
     end
     it "should use the displayLabel attribute when one is available" do
-      mods_display_note(@display_label).fields.first.label.should == "Special Label"
+      mods_display_note(@display_label).fields.first.label.should == "Special Label:"
     end
     it "should use get a label from a list of translations" do
-      mods_display_note(@sor_label).fields.first.label.should == "Statement of responsibility"
+      mods_display_note(@sor_label).fields.first.label.should == "Statement of responsibility:"
     end
     it "should use use the capitalized type attribute if one is present" do
-      mods_display_note(@type_label).fields.first.label.should == "Some other type"
+      mods_display_note(@type_label).fields.first.label.should == "Some other type:"
     end
   end
   
@@ -37,7 +37,7 @@ describe ModsDisplay::Note do
     it "should handle complex grouping" do
       fields = mods_display_note(@complex_label).fields
       fields.length.should == 3
-      fields.first.label.should == "Note"
+      fields.first.label.should == "Note:"
       fields.first.values.length == 2
       fields.first.values.should == ["Note Field", "2nd Note Field"]
       
@@ -45,7 +45,7 @@ describe ModsDisplay::Note do
       fields[1].values.length == 1
       fields[1].values.should == ["SoR"]
       
-      fields.last.label.should == "Note"
+      fields.last.label.should == "Note:"
       fields.last.values.length == 1
       fields.last.values.should == ["Another Note"]
     end

data/spec/fields/related_item_spec.rb

@@ -17,16 +17,16 @@ describe ModsDisplay::RelatedItem do
   end
   describe "label" do
     it "should default to Related Item" do
-      mods_display_item(@item).fields.first.label.should == "Related item"
+      mods_display_item(@item).fields.first.label.should == "Related item:"
     end
     it "should get the location label" do
-      mods_display_item(@location).fields.first.label.should == "Location"
+      mods_display_item(@location).fields.first.label.should == "Location:"
     end
     it "should get the reference label" do
-      mods_display_item(@reference).fields.first.label.should == "Referenced by"
+      mods_display_item(@reference).fields.first.label.should == "Referenced by:"
     end
     it "should get the displayLabel if available" do
-      mods_display_item(@display_label).fields.first.label.should == "Special Item"
+      mods_display_item(@display_label).fields.first.label.should == "Special Item:"
     end
   end
   describe "fields" do
@@ -54,7 +54,7 @@ describe ModsDisplay::RelatedItem do
     it "should collapse labels down into the same record" do
       fields = mods_display_item(@multi_items).fields
       fields.length.should == 1
-      fields.first.label.should == "Related item"
+      fields.first.label.should == "Related item:"
       fields.first.values.length.should == 2
       fields.first.values.first.should =~ /<a href=.*>Library<\/a>/ or
       fields.first.values.last.should =~ /<a href=.*>SDR<\/a>/

data/spec/fields/resource_type_spec.rb

@@ -13,12 +13,12 @@ describe ModsDisplay::ResourceType do
   it "should default to a label of 'Type of resource'" do
     fields = mods_display_resource_type(@type).fields
     fields.length.should == 1
-    fields.first.label.should == "Type of resource"
+    fields.first.label.should == "Type of resource:"
   end
   it "should use the displayLabel attribute when present" do
     fields = mods_display_resource_type(@display_label).fields
     fields.length.should == 1
-    fields.first.label.should == "Special label"
+    fields.first.label.should == "Special label:"
   end
   it "should capitalize the first letter of the values" do
     fields = mods_display_resource_type(@downcase).fields

data/spec/fields/sub_title_spec.rb

@@ -11,7 +11,7 @@ describe ModsDisplay::SubTitle do
   it "omit the main title and only return sub titles" do
     fields = mods_display_sub_title(@title).fields
     fields.length.should == 1
-    fields.first.label.should == "Title"
+    fields.first.label.should == "Title:"
     fields.first.values.should == ["Sub Title"]
     
   end

data/spec/fields/subject_spec.rb

@@ -46,11 +46,11 @@ describe ModsDisplay::Subject do
     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.label.should  == "Subject:"
       fields.first.values.should == [["A Subject", "Another Subject"], ["B Subject", "Another B Subject"]]
-      fields[1].label.should     == "Subject Heading"
+      fields[1].label.should     == "Subject Heading:"
       fields[1].values.should    == [["Jazz", "Japan", "History and criticism"]]
-      fields.last.label.should   == "Subject"
+      fields.last.label.should   == "Subject:"
       fields.last.values.should  == [["Bay Area", "Stanford"]]
     end
     it "should handle blank subjects properly" do

data/spec/fields/title_spec.rb

@@ -16,21 +16,21 @@ describe ModsDisplay::Title do
   end
   describe "labels" do
     it "should return a default label of Title if nothing else is available" do
-      mods_display_title(@title).fields.first.label.should == "Title"
+      mods_display_title(@title).fields.first.label.should == "Title:"
     end
     it "should return an appropriate label from the type attribute" do
-      mods_display_title(@alt_title).fields.first.label.should == "Alternative title"
+      mods_display_title(@alt_title).fields.first.label.should == "Alternative title:"
     end
     it "should return the label held in the displayLabel attribute of the titleInfo element when available" do
-      mods_display_title(@display_label).fields.first.label.should == "MyTitle"
+      mods_display_title(@display_label).fields.first.label.should == "MyTitle:"
     end
     it "should collapse adjacent identical labels" do
       fields = mods_display_title(@multi_label).fields
       fields.length.should == 4
-      fields[0].label.should == "Title"
-      fields[1].label.should == "Alternative title"
-      fields[2].label.should == "Uniform title"
-      fields[3].label.should == "Alternative title"
+      fields[0].label.should == "Title:"
+      fields[1].label.should == "Alternative title:"
+      fields[2].label.should == "Uniform title:"
+      fields[3].label.should == "Alternative title:"
       fields[3].values.should == ["Another Alt Title", "Yet Another Alt Title"]
     end
   end
@@ -39,7 +39,7 @@ describe ModsDisplay::Title do
       values = mods_display_title(@display_label).fields
       values.length.should == 1
       values.first.should be_a ModsDisplay::Values
-      values.first.label.should == "MyTitle"
+      values.first.label.should == "MyTitle:"
       values.first.values.should == ["Title"]
     end
   end

data/spec/integration/html_spec.rb

@@ -1,8 +1,10 @@
+# encoding: UTF-8
 require "spec_helper"
 
-def html_from_mods(xml)
+def html_from_mods(xml, locale=nil)
   model = TestModel.new
   model.modsxml = xml
+  I18n.locale = locale if locale
   TestController.new.render_mods_display(model)
 end
 
@@ -10,8 +12,24 @@ describe "HTML Output" do
   before(:all) do
     @multiple_titles = html_from_mods("<mods><titleInfo><title>Main Title</title></titleInfo><titleInfo type='alternative'><title>Alternate Title</title></titleInfo></mods>")
     @abstract = html_from_mods("<mods><abstract>Hey. I'm an abstract.</abstract></mods>")
+    mods = "<mods><titleInfo><title>Main Title</title></titleInfo><abstract>Hey. I'm an abstract.</abstract></mods>"
+    @mods = html_from_mods(mods)
+    @fr_mods = html_from_mods(mods, :fr)
+  end
+  after(:all) do
+    I18n.locale = :en
+  end
+  describe "i18n" do
+    it "should get the default english translations" do
+      expect(@mods.to_html).to match(/<dt title='Title'>Title:<\/dt>/)
+    end
+    it "should internationalize the labels when translations are available" do
+      expect(@fr_mods.to_html).to match(/<dt title='Résumé'>Résumé :<\/dt>/)
+    end
+    it "should get fallback to the default english translations if a translation is missing" do
+      expect(@fr_mods.to_html).to match(/<dt title='Title'>Title:<\/dt>/)
+    end
   end
-  
   describe "titles" do
     it "should include both titles it regular display" do
       @multiple_titles.to_html.should include("<dd>Main Title</dd>")

data/spec/spec_helper.rb

@@ -1,6 +1,12 @@
 require "mods_display"
 require "stanford-mods"
 
+# Load i18n test file.
+# We don't have any reliable translations yet so this
+# just make sure that we're handling i18n properly.
+I18n.load_path += Dir["#{File.expand_path('../..', __FILE__)}/spec/test_fr.yml"]
+I18n.backend.load_translations
+
 # This file was generated by the `rspec --init` command. Conventionally, all
 # specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
 # Require this file using `require "spec_helper"` to ensure that it is only

data/spec/test_fr.yml

@@ -0,0 +1,3 @@
+fr:
+  mods_display:
+    abstract: "Résumé :"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mods_display
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.3.0
 platform: ruby
 authors:
 - Jessie Keck
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-12-14 00:00:00.000000000 Z
+date: 2013-12-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: stanford-mods
@@ -24,6 +24,20 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: i18n
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -65,6 +79,7 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- config/locales/en.yml
 - lib/mods_display.rb
 - lib/mods_display/configuration.rb
 - lib/mods_display/configuration/access_condition.rb
@@ -137,6 +152,7 @@ files:
 - spec/integration/html_spec.rb
 - spec/integration/installation_spec.rb
 - spec/spec_helper.rb
+- spec/test_fr.yml
 homepage: https://github.com/sul-dlss/mods_display
 licenses: []
 metadata: {}
@@ -194,3 +210,4 @@ test_files:
 - spec/integration/html_spec.rb
 - spec/integration/installation_spec.rb
 - spec/spec_helper.rb
+- spec/test_fr.yml