Use .merge! instead of .merge when merging defaults with provided options.

Author: jperville

lib/json/ld/api.rb

@@ -87,7 +87,7 @@ class API
     # @yieldparam [API]
     # @raise [JsonLdError]
     def initialize(input, context, options = {}, &block)
-      @options = {compactArrays: true, rename_bnodes: true}.merge(options)
+      @options = {compactArrays: true, rename_bnodes: true}.merge!(options)
       @options[:validate] = true if @options[:processingMode] == "json-ld-1.0"
       @options[:documentLoader] ||= self.class.method(:documentLoader)
       @namer = options[:unique_bnodes] ? BlankNodeUniqer.new : (@options[:rename_bnodes] ? BlankNodeNamer.new("b") : BlankNodeMapper.new)
@@ -98,7 +98,7 @@ def initialize(input, context, options = {}, &block)
       @value = case input
       when Array, Hash then input.dup
       when IO, StringIO
-        @options = {base: input.base_uri}.merge(@options) if input.respond_to?(:base_uri)
+        @options = {base: input.base_uri}.merge!(@options) if input.respond_to?(:base_uri)
 
         # if input impelements #links, attempt to get a contextUrl from that link
         content_type = input.respond_to?(:content_type) ? input.content_type : "application/json"
@@ -113,7 +113,7 @@ def initialize(input, context, options = {}, &block)
       when String
         remote_doc = @options[:documentLoader].call(input, @options)
 
-        @options = {base: remote_doc.documentUrl}.merge(@options)
+        @options = {base: remote_doc.documentUrl}.merge!(@options)
         context_ref = remote_doc.contextUrl
 
         case remote_doc.document
@@ -322,7 +322,7 @@ def self.frame(input, frame, options = {})
         requireAll:     true,
         omitDefault:    false,
         documentLoader: method(:documentLoader)
-      }.merge(options)
+      }.merge!(options)
 
       framing_state = {
         graphs:       {'@default' => {}, '@merged' => {}},
@@ -471,7 +471,7 @@ def self.toRdf(input, options = {}, &block)
     # @return [Object, Hash]
     #   If a block is given, the result of evaluating the block is returned, otherwise, the expanded JSON-LD document
     def self.fromRdf(input, options = {}, &block)
-      options = {useNativeTypes: false}.merge(options)
+      options = {useNativeTypes: false}.merge!(options)
       result = nil
 
       API.new(nil, nil, options) do |api|

lib/json/ld/context.rb

@@ -990,7 +990,7 @@ def compact_iri(iri, options = {})
     # @raise [RDF::ReaderError] if the iri cannot be expanded
     # @see http://json-ld.org/spec/latest/json-ld-api/#value-expansion
     def expand_value(property, value, options = {})
-      options = {useNativeTypes: false}.merge(options)
+      options = {useNativeTypes: false}.merge!(options)
       depth(options) do
         debug("expand_value") {"property: #{property.inspect}, value: #{value.inspect}"}
 

lib/json/ld/expand.rb

@@ -15,7 +15,7 @@ module Expand
     #   Ensure output objects have keys ordered properly
     # @return [Array, Hash]
     def expand(input, active_property, context, options = {})
-      options = {ordered: true}.merge(options)
+      options = {ordered: true}.merge!(options)
       debug("expand") {"input: #{input.inspect}, active_property: #{active_property.inspect}, context: #{context.inspect}"}
       result = case input
       when Array

lib/json/ld/utils.rb

@@ -118,7 +118,7 @@ def compare_values(v1, v2)
     #   true to allow duplicates, false not to (uses
     #     a simple shallow comparison of subject ID or value).
     def add_value(subject, property, value, options = {})
-      options = {property_is_array: false, allow_duplicate: true}.merge(options)
+      options = {property_is_array: false, allow_duplicate: true}.merge!(options)
 
       if value.is_a?(Array)
         subject[property] = [] if value.empty? && options[:property_is_array]