Merge pull request #225 from richmolj/master

Avoid deleting id from attributes

Author: richmolj

lib/graphiti/adapters/active_record.rb

@@ -282,8 +282,8 @@ def create(model_class, create_params)
 
       # (see Adapters::Abstract#update)
       def update(model_class, update_params)
-        instance = model_class.find(update_params.delete(:id))
-        instance.update_attributes(update_params)
+        instance = model_class.find(update_params.only(:id))
+        instance.update_attributes(update_params.except(:id))
         instance
       end
 

lib/graphiti/resource/persistence.rb

@@ -89,7 +89,8 @@ def create(create_params, meta = nil)
 
       def update(update_params, meta = nil)
         model_instance = nil
-        id = update_params.delete(:id)
+        id = update_params[:id]
+        update_params = update_params.except(:id)
 
         run_callbacks :persistence, :update, update_params, meta do
           run_callbacks :attributes, :update, update_params, meta do |params|

lib/graphiti/resource/remote.rb

@@ -19,7 +19,7 @@ def remote_url
       end
 
       def save(model, meta)
-        if meta[:attributes] == {} && meta[:method] == :update
+        if meta[:attributes].except(:id) == {} && meta[:method] == :update
           model
         else
           raise Errors::RemoteWrite.new(self.class)