Merge pull request #27517 from maclover7/jm-fix-27502

Compare deserialized values for `PostgreSQL::OID::Hstore` types

activerecord/CHANGELOG.md

+*   Compare deserialized values for `PostgreSQL::OID::Hstore` types when
+    calling `ActiveRecord::Dirty#changed_in_place?`
+
+    Fixes #27502.
+
+    *Jon Moss*
+
 *   Raise `ArgumentError` when passing an `ActiveRecord::Base` instance to `.find`
     and `.exists?`.
 

activerecord/lib/active_record/connection_adapters/postgresql/oid/hstore.rb

@@ -35,6 +35,14 @@ def accessor
             ActiveRecord::Store::StringKeyedHashAccessor
           end
 
+          # Will compare the Hash equivalents of +raw_old_value+ and +new_value+.
+          # By comparing hashes, this avoids an edge case where the order of
+          # the keys change between the two hashes, and they would not be marked
+          # as equal.
+          def changed_in_place?(raw_old_value, new_value)
+            deserialize(raw_old_value) != new_value
+          end
+
           private
 
             HstorePair = begin

activerecord/test/cases/adapters/postgresql/hstore_test.rb

@@ -171,6 +171,25 @@ def test_changes_in_place
       assert_not hstore.changed?
     end
 
+    def test_dirty_from_user_equal
+      settings = { "alongkey" => "anything", "key" => "value" }
+      hstore = Hstore.create!(settings: settings)
+
+      hstore.settings = { "key" => "value", "alongkey" => "anything" }
+      assert_equal settings, hstore.settings
+      refute hstore.changed?
+    end
+
+    def test_hstore_dirty_from_database_equal
+      settings = { "alongkey" => "anything", "key" => "value" }
+      hstore = Hstore.create!(settings: settings)
+      hstore.reload
+
+      assert_equal settings, hstore.settings
+      hstore.settings = settings
+      refute hstore.changed?
+    end
+
     def test_gen1
       assert_equal('" "=>""', @type.serialize(" " => ""))
     end