datacite-mapping 0.3.0 → 0.4.0

data/lib/datacite/mapping/date_value.rb

@@ -59,6 +59,7 @@ module Datacite
 
       def <=>(other)
         return nil unless other.class == self.class
+
         %i[year month day hour minute sec nsec].each do |v|
           order = send(v) <=> other.send(v)
           return order if order.nonzero?
@@ -86,24 +87,28 @@ module Datacite
       def to_year(val)
         return val if val.is_a?(Integer)
         return val.year if val.respond_to?(:year)
+
         matchdata = val.to_s.match(/^[0-9]+/)
         matchdata[0].to_i if matchdata
       end
 
       def to_month(val)
         return val.month if val.respond_to?(:month)
+
         matchdata = val.to_s.match(/^[0-9]+-([0-9]{2})(?![0-9])/)
         matchdata[1].to_i if matchdata
       end
 
       def to_day(val)
         return val.day if val.respond_to?(:day)
+
         matchdata = val.to_s.match(/^[0-9]+-[0-9]{2}-([0-9]{2})(?![0-9])/)
         matchdata[1].to_i if matchdata
       end
 
       def to_datetime(val)
         return val if val.is_a?(DateTime)
+
         DateTime.parse(val.to_s)
       rescue ArgumentError
         nil
@@ -112,6 +117,7 @@ module Datacite
       def to_date(val)
         return val if val.is_a?(::Date)
         return ::Date.parse(val.iso8601) if val.respond_to?(:iso8601)
+
         ::Date.parse(val.to_s)
       rescue ArgumentError
         nil

data/lib/datacite/mapping/description.rb

@@ -19,6 +19,9 @@ module Datacite
       # @!parse TABLE_OF_CONTENTS = TableOfContents
       new :TABLE_OF_CONTENTS, 'TableOfContents'
 
+      # @!parse TECHNICAL_INFO = TechnicalInfo
+      new :TECHNICAL_INFO, 'TechnicalInfo'
+
       # @!parse OTHER = Other
       new :OTHER, 'Other'
 
@@ -93,9 +96,10 @@ module Datacite
         @language = value&.strip
       end
 
-      def value=(v)
-        new_value = v&.strip
+      def value=(a_value)
+        new_value = a_value&.strip
         raise ArgumentError, 'Value cannot be empty or nil' unless new_value && !new_value.empty?
+
         @value = new_value
       end
 

data/lib/datacite/mapping/empty_filtering_nodes.rb

@@ -8,6 +8,7 @@ module Datacite
     module EmptyNodeUtils
       def not_empty(element)
         return unless element
+
         text = element.text
         empty = text.nil? || text.strip.empty?
         warn "Ignoring empty element #{element}" if empty

data/lib/datacite/mapping/funding_reference.rb

@@ -15,6 +15,9 @@ module Datacite
       # @!parse CROSSREF_FUNDER = 'Crossref Funder ID'
       new :CROSSREF_FUNDER_ID, 'Crossref Funder ID'
 
+      # @!parse ROR = ROR
+      new :ROR, 'ROR'
+
       # @!parse OTHER = Other
       new :OTHER, 'Other'
     end
@@ -24,18 +27,21 @@ module Datacite
 
       # @param type [FunderIdentifierType] the identifier type. Cannot be nil.
       # @param value [String] the identifier value. Cannot be nil.
-      def initialize(type:, value:)
+      def initialize(type:, scheme_uri: nil, value:)
         self.type = type
+        self.scheme_uri = scheme_uri
         self.value = value
       end
 
       def value=(value)
         raise ArgumentError, 'Value cannot be empty or nil' unless value && !value.empty?
+
         @value = value
       end
 
       def type=(value)
         raise ArgumentError, 'Type cannot be nil' unless value
+
         @type = value
       end
 
@@ -47,6 +53,10 @@ module Datacite
       #   @return [FunderIdentifierType] the identifier type. Cannot be nil.
       typesafe_enum_node :type, '@funderIdentifierType', class: FunderIdentifierType
 
+      # @!attribute [rw] scheme_uri
+      #   @return [URI, nil] the URI of the identifier scheme. Optional.
+      uri_node :scheme_uri, '@schemeURI', default_value: nil
+
       # @!attribute [rw] value
       #   @return [String] the identifier value. Cannot be nil.
       text_node :value, 'text()'
@@ -62,6 +72,7 @@ module Datacite
 
       def value=(value)
         raise ArgumentError, 'Value cannot be empty or nil' unless value && !value.empty?
+
         @value = value
       end
 
@@ -111,8 +122,10 @@ module Datacite
       def award_number_or_nil(value)
         return nil unless value
         return value if value.is_a?(AwardNumber)
+
         new_value = value.to_s.strip
         return nil if new_value.empty?
+
         AwardNumber.new(value: new_value)
       end
     end

data/lib/datacite/mapping/geo_location.rb

@@ -21,11 +21,12 @@ module Datacite
       # @param box [GeoLocationBox, nil] the latitude-longitude quadrangle containing the area where the data was gathered or about which the data is focused.
       # @param place [String, nil] the spatial region or named place where the data was gathered or about which the data is focused.
       # @param polygon [GeoLocationPolygon, nil] a drawn polygon area containing the area where the data was gathered or about which the data is focused.
-      def initialize(point: nil, box: nil, place: nil, polygon: nil)
+      def initialize(point: nil, box: nil, place: nil, polygon: nil, polygons: [])
         self.point = point
         self.box = box
         self.place = place
         self.polygon = polygon
+        self.polygons = polygons
       end
 
       def place=(value)
@@ -50,6 +51,10 @@ module Datacite
       # #   @return [GeoLocationPolygon] a drawn polygon area containing the area where the data was gathered or about which the data is focused.
       object_node :polygon, 'geoLocationPolygon', class: GeoLocationPolygon, default_value: nil
 
+      # @!attribute [rw] polygons
+      #   @return [Array<GeoLocationPolygon>] multiple polygons where data is contained
+      array_node :polygons, 'geoLocationPolygons', 'geoLocationPolygon', class: GeoLocationPolygon, default_value: []
+
       # @!attribute [rw] place
       #   @return [String, nil] the spatial region or named place where the data was gathered or about which the data is focused.
       text_node :place, 'geoLocationPlace', default_value: nil

data/lib/datacite/mapping/geo_location_box.rb

@@ -62,24 +62,28 @@ module Datacite
       def south_latitude=(value)
         raise ArgumentError, 'South latitude cannot be nil' unless value
         raise ArgumentError, "#{value} is not a valid south latitude" unless value >= -90 && value <= 90
+
         @south_latitude = value
       end
 
       def west_longitude=(value)
         raise ArgumentError, 'West longitude cannot be nil' unless value
         raise ArgumentError, "#{value} is not a valid west longitude" unless value >= -180 && value <= 180
+
         @west_longitude = value
       end
 
       def north_latitude=(value)
         raise ArgumentError, 'North latitude cannot be nil' unless value
         raise ArgumentError, "#{value} is not a valid north latitude" unless value >= -90 && value <= 90
+
         @north_latitude = value
       end
 
       def east_longitude=(value)
         raise ArgumentError, 'East longitude cannot be nil' unless value
         raise ArgumentError, "#{value} is not a valid east longitude" unless value >= -180 && value <= 180
+
         @east_longitude = value
       end
 
@@ -96,6 +100,7 @@ module Datacite
       #   {GeoLocationBox}
       def <=>(other)
         return nil unless other.class == self.class
+
         %i[south_latitude west_longitude north_latitude east_longitude].each do |c|
           order = send(c) <=> other.send(c)
           return order if order.nonzero?

data/lib/datacite/mapping/geo_location_node.rb

@@ -13,6 +13,7 @@ module Datacite
       def initialize(*args)
         raise 'No geometry class provided' unless @geom_class
         raise 'No coordinate elements provided' unless @coord_elements
+
         path, *myargs = super(*args)
         @path = ::XML::XXPath.new(path)
         myargs # rubocop:disable Lint/Void
@@ -23,13 +24,14 @@ module Datacite
       end
 
       def extract_attr_value(xml)
-        return from_text(xml) || from_children(xml)
+        from_text(xml) || from_children(xml)
       rescue StandardError => e
         raise e, "#{@owner}.#{@attrname}: Can't extract #{self.class} from #{xml}: #{e.message}"
       end
 
       def set_attr_value(xml, value)
-        raise "Invalid value: expected #{geom_class} instance, was #{value || 'nil'}" unless value && value.is_a?(geom_class)
+        raise "Invalid value: expected #{geom_class} instance, was #{value || 'nil'}" unless value&.is_a?(geom_class)
+
         element = @path.first(xml, ensure_created: true)
 
         if datacite_3?

data/lib/datacite/mapping/geo_location_point.rb

@@ -42,12 +42,14 @@ module Datacite
       def latitude=(value)
         raise ArgumentError, 'Latitude cannot be nil' unless value
         raise ArgumentError, "#{value} is not a valid latitude" unless value >= -90 && value <= 90
+
         @latitude = value
       end
 
       def longitude=(value)
         raise ArgumentError, 'Longitude cannot be nil' unless value
         raise ArgumentError, "#{value} is not a valid longitude" unless value >= -180 && value <= 180
+
         @longitude = value
       end
 
@@ -64,6 +66,7 @@ module Datacite
       #   {GeoLocationPoint}
       def <=>(other)
         return nil unless other.class == self.class
+
         %i[latitude longitude].each do |c|
           order = send(c) <=> other.send(c)
           return order if order.nonzero?

data/lib/datacite/mapping/geo_location_polygon.rb

@@ -13,8 +13,9 @@ module Datacite
       # @param points [Array<GeoLocationPoint>] an array of points defining the polygon area.
       #   Per the spec, the array should contain at least four points, the first and last being
       #   identical to close the polygon.
-      def initialize(points:) # TODO: allow simple array of point args, array of hashes
+      def initialize(points:, in_polygon_point: nil) # TODO: allow simple array of point args, array of hashes
         self.points = points
+        self.in_polygon_point = in_polygon_point
         warn "Polygon should contain at least 4 points, but has #{points.size}" if points.size < 4
         warn "Polygon is not closed; last and first point should be identical, but were: [#{points[0]}], [#{points[-1]}]" unless points[0] == points[-1] || points.size <= 1
       end
@@ -25,6 +26,7 @@ module Datacite
 
       def <=>(other)
         return nil unless other.class == self.class
+
         points <=> other.points
       end
 
@@ -44,6 +46,13 @@ module Datacite
                  marshaller: (proc { |xml, value| marshal_point(xml, value) }),
                  unmarshaller: (proc { |xml| unmarshal_point(xml) })
 
+      # # @!attribute [rw] in_polygon_point
+      # #   @return [InPolygonPoint] a point within the target polygon
+      object_node :in_polygon_point, 'inPolygonPoint',
+                  default_value: nil,
+                  marshaller: (proc { |xml, value| marshal_point(xml, value) }),
+                  unmarshaller: (proc { |xml| unmarshal_point(xml) })
+
       use_mapping :datacite_3
 
       # TODO: does this ever get called?

data/lib/datacite/mapping/identifier.rb

@@ -5,7 +5,7 @@ require 'datacite/mapping/empty_filtering_nodes'
 
 module Datacite
   module Mapping
-    DOI_PATTERN = %r{10\.\S+/\S+}
+    DOI_PATTERN = %r{10\.\S+/\S+}.freeze
 
     # The persistent identifier that identifies the resource.
     #
@@ -26,19 +26,21 @@ module Datacite
         self.value = value
       end
 
-      def value=(v)
-        new_value = v&.strip
+      def value=(new_value)
+        new_value = new_value&.strip
         raise ArgumentError, 'Identifier must have a non-nil value' unless new_value
         raise ArgumentError, "Identifier value '#{new_value}' is not a valid DOI" unless new_value.match?(DOI_PATTERN)
+
         @value = new_value
       end
 
       # Sets the identifier type. Should only be called by the XML mapping engine.
-      # @param v [String]
+      # @param new_value [String]
       #   the identifier type (always 'DOI')
-      def identifier_type=(v)
-        raise ArgumentError, "Identifier type '#{v}' must be 'DOI'" unless DOI == v
-        @identifier_type = v
+      def identifier_type=(new_value)
+        raise ArgumentError, "Identifier type '#{new_value}' must be 'DOI'" unless DOI == new_value
+
+        @identifier_type = new_value
       end
 
       # Gets the identifiery type.
@@ -51,6 +53,7 @@ module Datacite
       def self.from_doi(doi_string)
         match = doi_string.match(DOI_PATTERN)
         raise ArgumentError, "'#{doi_string}' does not appear to contain a valid DOI" unless match
+
         Identifier.new(value: match[0])
       end
 
@@ -60,13 +63,12 @@ module Datacite
       fallback_mapping :datacite_3, :_default
     end
 
-    # Custom node to warn (but not blow up) if we read an XML `<resource/>` that's
+    # Custom node to allow (but ignore) if we read an XML `<resource/>` that's
     # missing its `<identifier/>`.
     class IdentifierNode < XML::Mapping::ObjectNode
       include EmptyNodeUtils
       def xml_to_obj(_obj, xml)
         return super if (element = has_element?(xml)) && not_empty(element)
-        warn 'Identifier not found; add a valid Identifier to the Resource before saving'
       end
 
       private

data/lib/datacite/mapping/module_info.rb

@@ -6,9 +6,9 @@ module Datacite
     NAME = 'datacite-mapping'
 
     # The version of this gem
-    VERSION = '0.3.0'
+    VERSION = '0.4.0'
 
     # The copyright notice for this gem
-    COPYRIGHT = 'Copyright (c) 2016 The Regents of the University of California'
+    COPYRIGHT = 'Copyright (c) 2019 The Regents of the University of California'
   end
 end

data/lib/datacite/mapping/name_identifier.rb

@@ -18,14 +18,16 @@ module Datacite
         self.value = value
       end
 
-      def scheme=(v)
-        raise ArgumentError, 'Scheme cannot be empty or nil' unless v && !v.empty?
-        @scheme = v
+      def scheme=(new_value)
+        raise ArgumentError, 'Scheme cannot be empty or nil' unless new_value && !new_value.empty?
+
+        @scheme = new_value
       end
 
-      def value=(v)
-        raise ArgumentError, 'Value cannot be empty or nil' unless v && !v.empty?
-        @value = v
+      def value=(new_value)
+        raise ArgumentError, 'Value cannot be empty or nil' unless new_value && !new_value.empty?
+
+        @value = new_value
       end
 
       root_element_name 'nameIdentifier'
@@ -33,9 +35,11 @@ module Datacite
       # @!attribute [rw] scheme
       #   @return [String] the name identifier scheme. Cannot be nil.
       text_node :scheme, '@nameIdentifierScheme'
+
       # @!attribute [rw] scheme_uri
       #   @return [URI, nil] the URI of the identifier scheme. Optional.
       uri_node :scheme_uri, '@schemeURI', default_value: nil
+
       # @!attribute [rw] value
       #   @return [String] the identifier value. Cannot be nil.
       text_node :value, 'text()'

data/lib/datacite/mapping/name_type.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+require 'xml/mapping'
+require 'datacite/mapping/read_only_nodes'
+
+module Datacite
+  module Mapping
+
+    # Controlled vocabulary of name types
+    class NameType < TypesafeEnum::Base
+      # @!parse ORGANIZATIONAL = Organizational
+      new :ORGANIZATIONAL, 'Organizational'
+
+      # @!parse PERSONAL = Personal
+      new :PERSONAL, 'Personal'
+    end
+  end
+end

data/lib/datacite/mapping/publisher.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require 'xml/mapping_extensions'
+
+module Datacite
+  module Mapping
+
+    # The type of the resource
+    class Publisher
+      include XML::Mapping
+
+      # Initializes a new {Publisher}
+      # @param language [String, nil] an IETF BCP 47, ISO 639-1 language code identifying the language.
+      # @param value [String] name of the publisher
+      def initialize(language: nil, value:)
+        self.language = language
+        self.value = value
+      end
+
+      def language=(value)
+        @language = value&.strip
+      end
+
+      def value=(value)
+        new_value = value&.strip
+        raise ArgumentError, 'Value cannot be empty or nil' unless new_value && !new_value.empty?
+
+        @value = new_value.strip
+      end
+
+      # @!attribute [rw] language
+      #   @return [String] an IETF BCP 47, ISO 639-1 language code identifying the language.
+      text_node :language, '@xml:lang', default_value: nil
+
+      # @!attribute [rw] value
+      #   @return [String] the title itself.
+      text_node :value, 'text()'
+
+      fallback_mapping :datacite_3, :_default
+    end
+  end
+end