rdf-tabular 2.2.1 → 3.2.0

data/lib/rdf/tabular/format.rb

@@ -22,7 +22,7 @@ module RDF::Tabular
   # @example Obtaining serialization format file extension mappings
   #     RDF::Format.file_extensions    #=> {:csv => "text/csv"}
   #
-  # @see http://www.w3.org/TR/rdf-testcases/#ntriples
+  # @see https://www.w3.org/TR/rdf-testcases/#ntriples
   class Format < RDF::Format
     content_type     'text/csv;q=0.4',
                      extensions: [:csv, :tsv],
@@ -62,7 +62,7 @@ module RDF::Tabular
             raise ArgumentError, "Outputting Tabular JSON only allowed when input format is tabular." unless opts[:format] == :tabular
             out = opts[:output] || $stdout
             out.set_encoding(Encoding::UTF_8) if RUBY_PLATFORM == "java"
-            RDF::CLI.parse(argv, opts) do |reader|
+            RDF::CLI.parse(argv, **opts) do |reader|
               out.puts reader.to_json
             end
           end

data/lib/rdf/tabular/literal.rb

@@ -13,7 +13,7 @@ module RDF::Tabular
     ##
     # @param  [Object] value
     # @option options [String] :lexical (nil)
-    def initialize(value, options = {})
+    def initialize(value, **options)
       @datatype = options[:datatype] || DATATYPE
       @string   = options[:lexical] if options.has_key?(:lexical)
       if value.is_a?(String)

data/lib/rdf/tabular/metadata.rb

@@ -16,7 +16,7 @@ require 'yaml'  # used by BCP47, which should have required it.
 # * Return Column-level annotations
 # * Return row iterator with column information
 #
-# @author [Gregg Kellogg](http://greggkellogg.net/)
+# @author [Gregg Kellogg](https://greggkellogg.net/)
 module RDF::Tabular
   class Metadata
     include RDF::Util::Logger
@@ -136,15 +136,15 @@ module RDF::Tabular
     #   see `RDF::Util::File.open_file` in RDF.rb and {new}
     # @yield [Metadata]
     # @raise [IOError] if file not found
-    def self.open(path, options = {})
+    def self.open(path, **options)
       options = options.merge(
         headers: {
           'Accept' => 'application/ld+json, application/json'
         }
       )
       path = "file:" + path unless path =~ /^\w+:/
-      RDF::Util::File.open_file(path, options) do |file|
-        self.new(file, options.merge(base: path, filenames: path))
+      RDF::Util::File.open_file(path, **options) do |file|
+        self.new(file, **options.merge(base: path, filenames: path))
       end
     end
 
@@ -173,16 +173,16 @@ module RDF::Tabular
     # @option options [RDF::URI] :base
     #   The Base URL to use when expanding the document. This overrides the value of `input` if it is a URL. If not specified and `input` is not an URL, the base URL defaults to the current document URL if in a browser context, or the empty string if there is no document context.
     # @return [Metadata]
-    def self.for_input(input, options = {})
+    def self.for_input(input, **options)
       base = options[:base]
 
       # Use user metadata, if provided
       metadata = case options[:metadata]
       when Metadata then options[:metadata]
       when Hash
-        Metadata.new(options[:metadata], options.merge(reason: "load user metadata: #{options[:metadata].inspect}"))
+        Metadata.new(options[:metadata], **options.merge(reason: "load user metadata: #{options[:metadata].inspect}"))
       when String, RDF::URI
-        Metadata.open(options[:metadata], options.merge(filenames: options[:metadata], reason: "load user metadata: #{options[:metadata].inspect}"))
+        Metadata.open(options[:metadata], **options.merge(filenames: options[:metadata], reason: "load user metadata: #{options[:metadata].inspect}"))
       end
 
       # Search for metadata until found
@@ -191,13 +191,13 @@ module RDF::Tabular
       if !metadata && input.respond_to?(:links) && 
         link = input.links.find_link(%w(rel describedby))
         link_loc = RDF::URI(base).join(link.href).to_s
-        md = Metadata.open(link_loc, options.merge(filenames: link_loc, reason: "load linked metadata: #{link_loc}"))
+        md = Metadata.open(link_loc, **options.merge(filenames: link_loc, reason: "load linked metadata: #{link_loc}"))
         if md
           # Metadata must describe file to be useful
           if md.describes_file?(base)
             metadata = md
           else
-            log_warn("Found metadata at #{link_loc}, which does not describe #{base}, ignoring", options)
+            log_warn("Found metadata at #{link_loc}, which does not describe #{base}, ignoring", **options)
           end
         end
       end
@@ -206,28 +206,30 @@ module RDF::Tabular
       # If we still don't have metadata, load the site-wide configuration file and use templates found there as locations
       if !metadata && base
         templates = site_wide_config(base)
-        log_debug("for_input", options) {"templates: #{templates.map(&:to_s).inspect}"}
+        log_debug("for_input", **options) {"templates: #{templates.map(&:to_s).inspect}"}
         locs = templates.map do |template|
           t = Addressable::Template.new(template)
-          RDF::URI(base).join(t.expand(url: base).to_s)
+          mapped = t.expand(url: base).to_s
+          mapped = RDF::URI.decode(mapped) if options[:decode_uri]
+          RDF::URI(base).join(mapped)
         end
-        log_debug("for_input", options) {"locs: #{locs.map(&:to_s).inspect}"}
+        log_debug("for_input", **options) {"locs: #{locs.map(&:to_s).inspect}"}
 
         locs.each do |loc|
           metadata ||= begin
-            md = Metadata.open(loc, options.merge(filenames: loc, reason: "load found metadata: #{loc}"))
+            md = Metadata.open(loc, **options.merge(filenames: loc, reason: "load found metadata: #{loc}"))
             # Metadata must describe file to be useful
             if md
               # Metadata must describe file to be useful
               if md.describes_file?(base)
                 md
               else
-                log_warn("Found metadata at #{loc}, which does not describe #{base}, ignoring", options)
+                log_warn("Found metadata at #{loc}, which does not describe #{base}, ignoring", **options)
                 nil
               end
             end
           rescue IOError
-            log_debug("for_input", options) {"failed to load found metadata #{loc}: #{$!}"}
+            log_debug("for_input", **options) {"failed to load found metadata #{loc}: #{$!}"}
             nil
           end
         end
@@ -236,8 +238,8 @@ module RDF::Tabular
       # Return either the merge or user- and found-metadata, any of these, or an empty TableGroup
       metadata = case
       when metadata then metadata
-      when base     then TableGroup.new({"@context" => "http://www.w3.org/ns/csvw", tables: [{url: base}]}, options)
-      else               TableGroup.new({"@context" => "http://www.w3.org/ns/csvw", tables: [{url: nil}]}, options)
+      when base     then TableGroup.new({"@context" => "http://www.w3.org/ns/csvw", tables: [{url: base}]}, **options)
+      else               TableGroup.new({"@context" => "http://www.w3.org/ns/csvw", tables: [{url: nil}]}, **options)
       end
 
       # Make TableGroup, if not already
@@ -246,7 +248,7 @@ module RDF::Tabular
 
     ##
     # @private
-    def self.new(input, options = {})
+    def self.new(input, **options)
       # Triveal case
       return input if input.is_a?(Metadata)
 
@@ -297,7 +299,7 @@ module RDF::Tabular
         end
 
       md = klass.allocate
-      md.send(:initialize, object, options)
+      md.send(:initialize, object, **options)
       md
     rescue ::JSON::ParserError
       raise Error, "Expected input to be a JSON Object"
@@ -314,11 +316,13 @@ module RDF::Tabular
     #   Context used for this metadata. Taken from input if not provided
     # @option options [RDF::URI] :base
     #   The Base URL to use when expanding the document. This overrides the value of `input` if it is a URL. If not specified and `input` is not an URL, the base URL defaults to the current document URL if in a browser context, or the empty string if there is no document context.
+    # @option options [Boolean] :decode_uri
+    #   Decode %-encodings in the result of a URI Template operation.
     # @option options [Boolean] :normalize normalize the object
     # @option options [Boolean] :validate Strict metadata validation
     # @raise [Error]
     # @return [Metadata]
-    def initialize(input, options = {})
+    def initialize(input, **options)
       @options = options.dup
 
       # Parent of this Metadata, if any
@@ -467,16 +471,16 @@ module RDF::Tabular
       object[:tableSchema] = case value
       when String
         link = context.base.join(value).to_s
-        md = Schema.open(link, @options.merge(parent: self, context: nil, normalize: true))
+        md = Schema.open(link, **@options.merge(parent: self, context: nil, normalize: true))
         md[:@id] ||= link
         md
       when Hash
-        Schema.new(value, @options.merge(parent: self, context: nil))
+        Schema.new(value, **@options.merge(parent: self, context: nil))
       when Schema
         value
       else
         log_warn "#{type} has invalid property 'tableSchema' (#{value.inspect}): expected a URL or object"
-        Schema.new({}, @options.merge(parent: self, context: nil))
+        Schema.new({}, **@options.merge(parent: self, context: nil))
       end
     end
 
@@ -491,7 +495,7 @@ module RDF::Tabular
       when object[:dialect] then object[:dialect]
       when parent then parent.dialect
       when is_a?(Table) || is_a?(TableGroup)
-        d = Dialect.new({}, @options.merge(parent: self, context: nil))
+        d = Dialect.new({}, **@options.merge(parent: self, context: nil))
         self.dialect = d unless self.parent
         d
       else
@@ -514,11 +518,11 @@ module RDF::Tabular
       @dialect = object[:dialect] = case value
       when String
         link = context.base.join(value).to_s
-        md = Metadata.open(link, @options.merge(parent: self, context: nil, normalize: true))
+        md = Metadata.open(link, **@options.merge(parent: self, context: nil, normalize: true))
         md[:@id] ||= link
         md
       when Hash
-        Dialect.new(value, @options.merge(parent: self, context: nil))
+        Dialect.new(value, **@options.merge(parent: self, context: nil))
       when Dialect
         value
       else
@@ -532,8 +536,8 @@ module RDF::Tabular
     # @raise [Error] if datatype is not valid
     def datatype=(value)
       val = case value
-      when Hash then Datatype.new(value, @options.merge(parent: self))
-      else           Datatype.new({base: value}, @options.merge(parent: self))
+      when Hash then Datatype.new(value, **@options.merge(parent: self))
+      else           Datatype.new({base: value}, **@options.merge(parent: self))
       end
 
       if val.valid? || value.is_a?(Hash)
@@ -564,7 +568,7 @@ module RDF::Tabular
     end
 
     ##
-    # Validate metadata, raising an error containing all errors detected during validation
+    # Validate metadata and content, raising an error containing all errors detected during validation
     # @raise [Error] Raise error if metadata has any unexpected properties
     # @return [self]
     def validate
@@ -872,7 +876,7 @@ module RDF::Tabular
           csv << data unless data.empty?
         end
       else
-        csv = ::CSV.new(input, csv_options)
+        csv = ::CSV.new(input, **csv_options)
         # Skip skipRows and headerRowCount
         skipped = (dialect.skipRows.to_i + dialect.headerRowCount)
         (1..skipped).each {csv.shift}
@@ -891,7 +895,7 @@ module RDF::Tabular
           next
         end
         number += 1
-        row = Row.new(data, self, number, number + skipped, @options)
+        row = Row.new(data, self, number, number + skipped, **@options)
         (self.object[:rows] ||= []) << row if @options[:validate] # Keep track of rows when validating
         yield(row)
       end
@@ -1036,13 +1040,13 @@ module RDF::Tabular
         end
         index = 0
         object_columns.all? do |cb|
-          ca = non_virtual_columns[index] || Column.new({}, @options)
+          ca = non_virtual_columns[index] || Column.new({}, **@options)
           ta = ca.titles || {}
           tb = cb.titles || {}
           if !ca.object.has_key?(:name) && !cb.object.has_key?(:name) && ta.empty? && tb.empty?
             true
           elsif ca.object.has_key?(:name) && cb.object.has_key?(:name)
-            raise Error, "Columns don't match on name: #{ca.name}, #{cb.name}" unless ca.name == cb.name
+            raise Error, "Column #{index + 1} doesn't match on name: #{ca.name || 'no name'}, #{cb.name || 'no name'}" unless ca.name == cb.name
           elsif @options[:validate] || !ta.empty? && !tb.empty?
             # If validating, column compatibility requires strict match between titles
             titles_match = case
@@ -1066,10 +1070,10 @@ module RDF::Tabular
               true
             elsif !@options[:validate]
               # If not validating, columns don't match, but processing continues
-              log_warn "Columns don't match on titles: #{ca.titles.inspect} vs #{cb.titles.inspect}"
+              log_warn "Column #{index + 1} doesn't match on titles: #{Array(ta['und']).join(',').inspect} vs #{Array(tb['und']).join(',').inspect}"
               true
             else
-              raise Error, "Columns don't match on titles: #{ca.titles.inspect} vs #{cb.titles.inspect}"
+              raise Error, "Column #{index + 1} doesn't match on titles: #{Array(ta['und']).join(',').inspect} vs #{Array(tb['und']).join(',').inspect}"
             end
           end
           index += 1
@@ -1235,13 +1239,13 @@ module RDF::Tabular
     end
 
     # General setter for array properties
-    def set_array_value(key, value, klass, options={})
+    def set_array_value(key, value, klass, **options)
       object[key] = case value
       when Array
         value.map do |v|
           case v
           when Hash
-            klass.new(v, @options.merge(options).merge(parent: self, context: nil))
+            klass.new(v, **@options.merge(options).merge(parent: self, context: nil))
           else v
           end
         end
@@ -1282,11 +1286,11 @@ module RDF::Tabular
     class DebugContext
       include RDF::Util::Logger
     end
-    def self.log_debug(*args, &block)
-      DebugContext.new.log_debug(*args, &block)
+    def self.log_debug(*args, **options, &block)
+      DebugContext.new.log_debug(*args, **options, &block)
     end
-    def self.log_warn(*args)
-      DebugContext.new.log_warn(*args)
+    def self.log_warn(*args, **options)
+      DebugContext.new.log_warn(*args, **options)
     end
   end
 
@@ -1434,7 +1438,7 @@ module RDF::Tabular
       content['@context'] = object.delete(:@context) if object[:@context]
       ctx = @context
       remove_instance_variable(:@context) if instance_variables.include?(:@context)
-      tg = TableGroup.new(content, @options.merge(context: ctx, filenames: @filenames, base: base))
+      tg = TableGroup.new(content, **@options.merge(context: ctx, filenames: @filenames, base: base))
       @parent = tg  # Link from parent
       tg
     end
@@ -1489,7 +1493,7 @@ module RDF::Tabular
           number += 1
           case v
           when Hash
-            Column.new(v, @options.merge(
+            Column.new(v, **@options.merge(
               table: (parent if parent.is_a?(Table)),
               parent: self,
               context: nil,
@@ -1621,8 +1625,8 @@ module RDF::Tabular
     def name
       self[:name] || if titles && (ts = titles[context.default_language || 'und'] || titles[self.lang || 'und'])
         n = Array(ts).first
-        n0 = URI.encode(n[0,1], /[^a-zA-Z0-9]/).encode("utf-8")
-        n1 = URI.encode(n[1..-1], /[^\w\.]/).encode("utf-8")
+        n0 = RDF::URI.encode(n[0,1], /[^a-zA-Z0-9]/).encode("utf-8")
+        n1 = RDF::URI.encode(n[1..-1], /[^\w\.]/).encode("utf-8")
         "#{n0}#{n1}"
       end || "_col.#{number}"
     end
@@ -1783,12 +1787,12 @@ module RDF::Tabular
     # @option options [String] :lang, language to set in table, if any
     # @return [Metadata] Tabular metadata
     # @see http://w3c.github.io/csvw/syntax/#parsing
-    def embedded_metadata(input, metadata, options = {})
+    def embedded_metadata(input, metadata, **options)
       options = options.dup
       options.delete(:context) # Don't accidentally use a passed context
       # Normalize input to an IO object
       if input.is_a?(String)
-        return ::RDF::Util::File.open_file(input) {|f| embedded_metadata(f, metadata, options.merge(base: input.to_s))}
+        return ::RDF::Util::File.open_file(input) {|f| embedded_metadata(f, metadata, **options.merge(base: input.to_s))}
       end
 
       table = {
@@ -1826,7 +1830,7 @@ module RDF::Tabular
           row.xpath('th').map(&:content).each_with_index do |value, index|
             # Skip columns
             skipCols = skipColumns.to_i
-            next if index < skipCols
+            next if index < skipCols || value.to_s.empty?
 
             # Trim value
             value.lstrip! if %w(true start).include?(trim.to_s)
@@ -1837,11 +1841,11 @@ module RDF::Tabular
             column = columns[index - skipCols] ||= {
               "titles" => {lang => []},
             }
-            column["titles"][lang] << value
+            column["titles"][lang] << value if value
           end
         end
       else
-        csv = ::CSV.new(input, csv_options)
+        csv = ::CSV.new(input, **csv_options)
         (1..skipRows.to_i).each do
           value = csv.shift.join(delimiter)  # Skip initial lines, these form comment annotations
           # Trim value
@@ -1858,7 +1862,7 @@ module RDF::Tabular
           Array(row_data).each_with_index do |value, index|
             # Skip columns
             skipCols = skipColumns.to_i
-            next if index < skipCols
+            next if index < skipCols || value.to_s.empty?
 
             # Trim value
             value.lstrip! if %w(true start).include?(trim.to_s)
@@ -1876,7 +1880,7 @@ module RDF::Tabular
       log_debug("embedded_metadata") {"table: #{table.inspect}"}
       input.rewind if input.respond_to?(:rewind)
 
-      Table.new(table, options.merge(reason: "load embedded metadata: #{table['@id']}"))
+      Table.new(table, **options.merge(reason: "load embedded metadata: #{table['@id']}"))
     end
   end
 
@@ -1947,13 +1951,14 @@ module RDF::Tabular
   class Row
     # Class for returning values
     Cell = Struct.new(:table, :column, :row, :stringValue, :aboutUrl, :propertyUrl, :valueUrl, :value, :errors) do
-      def set_urls(mapped_values)
+      def set_urls(mapped_values, decode_uri)
         %w(aboutUrl propertyUrl valueUrl).each do |prop|
           # If the cell value is nil, and it is not a virtual column
           next if prop == "valueUrl" && value.nil? && !column.virtual
           if v = column.send(prop.to_sym)
             t = Addressable::Template.new(v)
             mapped = t.expand(mapped_values).to_s
+            mapped = RDF::URI.decode(mapped) if decode_uri
             # FIXME: don't expand here, do it in CSV2RDF
             url = row.context.expand_iri(mapped, documentRelative: true)
             self.send("#{prop}=".to_sym, url)
@@ -2026,7 +2031,7 @@ module RDF::Tabular
     # @param [Hash{Symbol => Object}] options ({})
     # @option options [Boolean] :validate check for PK/FK consistency
     # @return [Row]
-    def initialize(row, metadata, number, source_number, options = {})
+    def initialize(row, metadata, number, source_number, **options)
       @table = metadata
       @number = number
       @sourceNumber = source_number
@@ -2058,13 +2063,13 @@ module RDF::Tabular
 
         # create column if necessary
         columns[index - skipColumns] ||=
-          Column.new({}, options.merge(table: metadata, parent: metadata.tableSchema, number: index + 1 - skipColumns))
+          Column.new({}, **options.merge(table: metadata, parent: metadata.tableSchema, number: index + 1 - skipColumns))
 
         column = columns[index - skipColumns]
 
         @values << cell = Cell.new(metadata, column, self, value)
 
-        datatype = column.datatype || Datatype.new({base: "string"}, options.merge(parent: column))
+        datatype = column.datatype || Datatype.new({base: "string"}, **options.merge(parent: column))
         value = value.gsub(/\r\n\t/, ' ') unless %w(string json xml html anyAtomicType).include?(datatype.base)
         value = value.strip.gsub(/\s+/, ' ') unless %w(string json xml html anyAtomicType normalizedString).include?(datatype.base)
         # if the resulting string is an empty string, apply the remaining steps to the string given by the default property
@@ -2110,11 +2115,11 @@ module RDF::Tabular
       # Map URLs for row
       @values.each_with_index do |cell, index|
         mapped_values = map_values.merge(
-          "_name" => URI.decode(cell.column.name),
+          "_name" => CGI.unescape(cell.column.name),
           "_column" => cell.column.number,
           "_sourceColumn" => cell.column.sourceNumber
         )
-        cell.set_urls(mapped_values)
+        cell.set_urls(mapped_values, options[:decode_uri])
       end
     end
 

data/lib/rdf/tabular/reader.rb

@@ -5,7 +5,7 @@ module RDF::Tabular
   ##
   # A Tabular Data to RDF parser in Ruby.
   #
-  # @author [Gregg Kellogg](http://greggkellogg.net/)
+  # @author [Gregg Kellogg](https://greggkellogg.net/)
   class Reader < RDF::Reader
     format Format
     include RDF::Util::Logger
@@ -22,7 +22,7 @@ module RDF::Tabular
 
     ##
     # Writer options
-    # @see http://www.rubydoc.info/github/ruby-rdf/rdf/RDF/Writer#options-class_method
+    # @see https://www.rubydoc.info/github/ruby-rdf/rdf/RDF/Writer#options-class_method
     def self.options
       super + [
         RDF::CLI::Option.new(
@@ -43,6 +43,13 @@ module RDF::Tabular
           control: :checkbox,
           on: ["--no-prov"],
           description: "do not output optional provenance information.") {true},
+        RDF::CLI::Option.new(
+          symbol: :decode_uri,
+          datatype: TrueClass,
+          control: :checkbox,
+          on: ["--decode-uri"],
+          description: "decode %-encodings in the result of a URI Template operation."
+        )
       ]
     end
 
@@ -54,16 +61,18 @@ module RDF::Tabular
     #   or an Array used as an internalized array of arrays
     # @param  [Hash{Symbol => Object}] options
     #   any additional options (see `RDF::Reader#initialize`)
+    # @option options [Boolean] :decode_uri
+    #   Decode %-encodings in the result of a URI Template operation.
+    # @option options [Array<Hash>] :fks_referencing_table
+    #   When called with Table metadata, a list of the foreign keys referencing this table
     # @option options [Metadata, Hash, String, RDF::URI] :metadata user supplied metadata, merged on top of extracted metadata. If provided as a URL, Metadata is loade from that location
     # @option options [Boolean] :minimal includes only the information gleaned from the cells of the tabular data
     # @option options [Boolean] :noProv do not output optional provenance information
-    # @option optinons [Array<Hash>] :fks_referencing_table
-    #   When called with Table metadata, a list of the foreign keys referencing this table
     # @yield  [reader] `self`
     # @yieldparam  [RDF::Reader] reader
     # @yieldreturn [void] ignored
     # @raise [RDF::ReaderError] if the CSV document cannot be loaded
-    def initialize(input = $stdin, options = {}, &block)
+    def initialize(input = $stdin, **options, &block)
       super do
         # Base would be how we are to take this
         @options[:base] ||= base_uri.to_s if base_uri
@@ -71,7 +80,7 @@ module RDF::Tabular
         @options[:base] ||= input.path if input.respond_to?(:path)
         @options[:base] ||= input.filename if input.respond_to?(:filename)
         if RDF::URI(@options[:base]).relative? && File.exist?(@options[:base].to_s)
-          @options[:base] = "file:/#{File.expand_path(@options[:base])}"
+          @options[:base] = RDF::URI("file:/#{File.expand_path(@options[:base])}").normalize
         end
 
         log_debug("Reader#initialize") {"input: #{input.inspect}, base: #{@options[:base]}"}
@@ -89,7 +98,7 @@ module RDF::Tabular
           # If input is JSON, then the input is the metadata
           content_type = @input.respond_to?(:content_type) ? @input.content_type : ""
           if @options[:base] =~ /\.json(?:ld)?$/ || content_type =~ %r(application/(csvm\+|ld\+)?json)
-            @metadata = Metadata.new(@input, @options.merge(filenames: @options[:base]))
+            @metadata = Metadata.new(@input, filenames: @options[:base], **@options)
             # If @metadata is for a Table, turn it into a TableGroup
             @metadata = @metadata.to_table_group if @metadata.is_a?(Table)
             @metadata.normalize!
@@ -102,7 +111,7 @@ module RDF::Tabular
               def script.content_type; "application/csvm+json"; end
               log_debug("Reader#initialize") {"Process HTML script block"}
               @input = script
-              @metadata = Metadata.new(@input, @options.merge(filenames: @options[:base]))
+              @metadata = Metadata.new(@input, filenames: @options[:base], **@options)
               # If @metadata is for a Table, turn it into a TableGroup
               @metadata = @metadata.to_table_group if @metadata.is_a?(Table)
               @metadata.normalize!
@@ -119,7 +128,7 @@ module RDF::Tabular
             dialect.separator = "\t" if (input.content_type == "text/tsv" rescue nil)
             embed_options = @options.dup
             embed_options[:lang] = dialect_metadata.lang if dialect_metadata.lang
-            embedded_metadata = dialect.embedded_metadata(input, @options[:metadata], embed_options)
+            embedded_metadata = dialect.embedded_metadata(input, @options[:metadata], **embed_options)
 
             if (@metadata = @options[:metadata]) && @metadata.tableSchema
               @metadata.verify_compatible!(embedded_metadata)
@@ -136,7 +145,7 @@ module RDF::Tabular
           else
             # It's tabluar data. Find metadata and proceed as if it was specified in the first place
             @options[:original_input] = @input unless @options[:metadata]
-            @input = @metadata = Metadata.for_input(@input, @options).normalize!
+            @input = @metadata = Metadata.for_input(@input, **@options).normalize!
           end
 
           log_debug("Reader#initialize") {"input: #{input}, metadata: #{metadata.inspect}"}
@@ -186,7 +195,7 @@ module RDF::Tabular
               if options[:original_input] && !input.describes_file?(options[:base_uri])
                 table_resource = RDF::Node.new
                 add_statement(0, table_group, CSVW.table, table_resource) unless minimal?
-                Reader.new(options[:original_input], options.merge(
+                Reader.new(options[:original_input], **options.merge(
                     metadata: input.tables.first,
                     base: input.tables.first.url,
                     no_found_metadata: true,
@@ -205,7 +214,7 @@ module RDF::Tabular
                   end.flatten.compact
                   table_resource = table.id || RDF::Node.new
                   add_statement(0, table_group, CSVW.table, table_resource) unless minimal?
-                  Reader.open(table.url, options.merge(
+                  Reader.open(table.url, **options.merge(
                       metadata: table,
                       base: table.url,
                       no_found_metadata: true,
@@ -225,7 +234,7 @@ module RDF::Tabular
                 activity = RDF::Node.new
                 add_statement(0, table_group, RDF::Vocab::PROV.wasGeneratedBy, activity)
                 add_statement(0, activity, RDF.type, RDF::Vocab::PROV.Activity)
-                add_statement(0, activity, RDF::Vocab::PROV.wasAssociatedWith, RDF::URI("http://rubygems.org/gems/rdf-tabular"))
+                add_statement(0, activity, RDF::Vocab::PROV.wasAssociatedWith, RDF::URI("https://rubygems.org/gems/rdf-tabular"))
                 add_statement(0, activity, RDF::Vocab::PROV.startedAtTime, RDF::Literal::DateTime.new(start_time))
                 add_statement(0, activity, RDF::Vocab::PROV.endedAtTime, RDF::Literal::DateTime.new(Time.now))
 
@@ -421,9 +430,9 @@ module RDF::Tabular
 
       res = if io
         ::JSON::dump_default_options = json_state
-        ::JSON.dump(self.send(hash_fn, options), io)
+        ::JSON.dump(self.send(hash_fn, **options), io)
       else
-        hash = self.send(hash_fn, options)
+        hash = self.send(hash_fn, **options)
         ::JSON.generate(hash, json_state)
       end
 
@@ -443,7 +452,7 @@ module RDF::Tabular
     #
     # @param [Hash{Symbol => Object}] options
     # @return [Hash, Array]
-    def to_hash(options = {})
+    def to_hash(**options)
       # Construct metadata from that passed from file open, along with information from the file.
       if input.is_a?(Metadata)
         log_debug("each_statement: metadata") {input.inspect}
@@ -467,13 +476,13 @@ module RDF::Tabular
             table_group['tables'] = tables
 
             if options[:original_input] && !input.describes_file?(options[:base_uri])
-              Reader.new(options[:original_input], options.merge(
+              Reader.new(options[:original_input], **options.merge(
                   metadata:           input.tables.first,
                   base:               input.tables.first.url,
                   minimal:            minimal?,
                   no_found_metadata:  true,
               )) do |r|
-                case t = r.to_hash(options)
+                case t = r.to_hash(**options)
                 when Array then tables += t unless input.tables.first.suppressOutput
                 when Hash  then tables << t unless input.tables.first.suppressOutput
                 end
@@ -481,13 +490,13 @@ module RDF::Tabular
             else
               input.each_table do |table|
                 next if table.suppressOutput && !validate?
-                Reader.open(table.url, options.merge(
+                Reader.open(table.url, **options.merge(
                   metadata:           table,
                   base:               table.url,
                   minimal:            minimal?,
                   no_found_metadata:  true,
                 )) do |r|
-                  case t = r.to_hash(options)
+                  case t = r.to_hash(**options)
                   when Array then tables += t unless table.suppressOutput
                   when Hash  then tables << t unless table.suppressOutput
                   end
@@ -560,7 +569,7 @@ module RDF::Tabular
             co['@id'] = subject.to_s unless subject == 'null'
             prop = case cell.propertyUrl
             when RDF.type then '@type'
-            when nil then URI.decode(column.name) # Use URI-decoded name
+            when nil then CGI.unescape(column.name) # Use URI-decoded name
             else
               # Compact the property to a term or prefixed name
               metadata.context.compact_iri(cell.propertyUrl, vocab: true)

data/lib/rdf/tabular/uax35.rb

@@ -3,7 +3,7 @@ module RDF::Tabular
   ##
   # Utilities for parsing UAX35 dates and numbers.
   #
-  # @see http://www.unicode.org/reports/tr35
+  # @see https://www.unicode.org/reports/tr35
   module UAX35
 
     ##

data/lib/rdf/tabular.rb

@@ -1,14 +1,14 @@
 $:.unshift(File.expand_path("..", __FILE__))
-require 'rdf' # @see http://rubygems.org/gems/rdf
+require 'rdf' # @see https://rubygems.org/gems/rdf
 require 'csv'
 
 module RDF
   ##
   # **`RDF::Tabular`** is a Tabular/CSV extension for RDF.rb.
   #
-  # @see http://w3c.github.io/csvw/
+  # @see https://w3c.github.io/csvw/
   #
-  # @author [Gregg Kellogg](http://greggkellogg.net/)
+  # @author [Gregg Kellogg](https://greggkellogg.net/)
   module Tabular
     require 'rdf/tabular/format'
     autoload :Column,         'rdf/tabular/metadata'