use HashWithIndifferentAccess, instead of symbolize_keys

Author: Tilo Sloboda

json_api_client.gemspec

@@ -20,6 +20,7 @@ Gem::Specification.new do |s|
 
   s.add_development_dependency "webmock", '~> 3.5.1'
   s.add_development_dependency "mocha"
+  s.add_development_dependency "pry"
 
   s.license = "MIT"
 

lib/json_api_client/query/builder.rb

@@ -67,11 +67,11 @@ def last
       end
 
       def build(attrs = {})
-        klass.new @path_params.merge(attrs.symbolize_keys)
+        klass.new @path_params.merge(attrs.with_indifferent_access)
       end
 
       def create(attrs = {})
-        klass.create @path_params.merge(attrs.symbolize_keys)
+        klass.create @path_params.merge(attrs.with_indifferent_access)
       end
 
       def params

lib/json_api_client/resource.rb

@@ -152,6 +152,7 @@ def path(params = nil)
         parts = [resource_path]
         if params && _prefix_path.present?
           path_params = params.delete(:path) || params
+
           parts.unshift(_set_prefix_path(path_params.symbolize_keys))
         else
           parts.unshift(_prefix_path)
@@ -348,7 +349,7 @@ def _build_connection(rebuild = false)
     #
     # @param params [Hash] Attributes, links, and relationships
     def initialize(params = {})
-      params = params.symbolize_keys
+      params = params.with_indifferent_access
       @persisted = nil
       @destroyed = nil
       self.links = self.class.linker.new(params.delete(:links) || {})
@@ -538,7 +539,7 @@ def reset_request_select!(*resource_types)
     end
 
     def path_attributes
-      _belongs_to_params.merge attributes.slice( self.class.primary_key ).symbolize_keys
+      _belongs_to_params.merge attributes.slice( self.class.primary_key ).with_indifferent_access
     end
 
     protected

test/test_helper.rb

@@ -5,6 +5,7 @@
 require 'webmock/minitest'
 require 'mocha/minitest'
 require 'pp'
+require 'pry'
 
 # shim for ActiveSupport 4.0.x requiring minitest 4.2
 unless defined?(Minitest::Test)

test/unit/query_builder_test.rb

@@ -308,7 +308,8 @@ def test_build_propagate_only_path_params
     query = ArticleNested.where(author_id: '123', name: 'John')
     record = query.build
     assert_equal [], record.changed
-    assert_equal({author_id: '123'}, record.__belongs_to_params)
+    assert_equal(record.__belongs_to_params[:author_id], '123')
+    assert_equal(record.__belongs_to_params['author_id'], '123')
     assert_equal '123', record.author_id
     assert_equal [], record.relationships.changed
   end