fix: clone record using write_attribute because attr acessors may be …

…overloaded
marcosgz · Jul 12, 2024 · f2e5b43 · f2e5b43
1 parent 22fca2b
commit f2e5b43
Show file tree

Hide file tree

Showing 8 changed files with 12 additions and 9 deletions.
diff --git a/Gemfile.lock b/Gemfile.lock
@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    esse-active_record (0.3.3)
+    esse-active_record (0.3.4)
       activerecord (>= 4.2, < 8)
       esse (>= 0.3.0)
 

diff --git a/ci/Gemfile.rails-5.2.lock b/ci/Gemfile.rails-5.2.lock
@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    esse-active_record (0.3.3)
+    esse-active_record (0.3.4)
       activerecord (>= 4.2, < 8)
       esse (>= 0.3.0)
 

diff --git a/ci/Gemfile.rails-6.0.lock b/ci/Gemfile.rails-6.0.lock
@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    esse-active_record (0.3.3)
+    esse-active_record (0.3.4)
       activerecord (>= 4.2, < 8)
       esse (>= 0.3.0)
 

diff --git a/ci/Gemfile.rails-6.1.lock b/ci/Gemfile.rails-6.1.lock
@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    esse-active_record (0.3.3)
+    esse-active_record (0.3.4)
       activerecord (>= 4.2, < 8)
       esse (>= 0.3.0)
 

diff --git a/ci/Gemfile.rails-7.0.lock b/ci/Gemfile.rails-7.0.lock
@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    esse-active_record (0.3.3)
+    esse-active_record (0.3.4)
       activerecord (>= 4.2, < 8)
       esse (>= 0.3.0)
 

diff --git a/ci/Gemfile.rails-7.1.lock b/ci/Gemfile.rails-7.1.lock
@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    esse-active_record (0.3.3)
+    esse-active_record (0.3.4)
       activerecord (>= 4.2, < 8)
       esse (>= 0.3.0)
 

diff --git a/lib/esse/active_record/callbacks/indexing_on_update.rb b/lib/esse/active_record/callbacks/indexing_on_update.rb
@@ -19,8 +19,11 @@ def call(model)
         update_document(document)
         return true unless document.routing
 
-        prev_record = model.class.new(model.attributes.merge(model.previous_changes.transform_values(&:first))).tap(&:readonly!)
-        prev_document = repo.serialize(prev_record)
+        prev_record = model.class.new
+        model.attributes.merge(model.previous_changes.transform_values(&:first)).each do |key, value|
+          prev_record[key] = value
+        end
+        prev_document = repo.serialize(prev_record.tap(&:readonly!))
 
         return true unless prev_document
         return true if [prev_document.id, prev_document.routing].include?(nil)

diff --git a/lib/esse/active_record/version.rb b/lib/esse/active_record/version.rb
@@ -2,6 +2,6 @@
 
 module Esse
   module ActiveRecord
-    VERSION = '0.3.3'
+    VERSION = '0.3.4'
   end
 end