Merge pull request #132 from featurist/master

Don't use `nil` as payload key.
reidmorrison · Jun 16, 2021 · f9f9f9c · f9f9f9c
2 parents aec7eaf + 014cfb8
commit f9f9f9c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/rails_semantic_logger/active_record/log_subscriber.rb b/lib/rails_semantic_logger/active_record/log_subscriber.rb
@@ -178,7 +178,7 @@ def render_bind_v6_1(attr, value)
           attr = nil
         end
 
-        [attr&.name, value]
+        [attr&.name || :nil, value]
       end
 
       def type_casted_binds_v5_0_3(binds, casted_binds)

diff --git a/test/active_record_test.rb b/test/active_record_test.rb
@@ -90,7 +90,7 @@ class ActiveRecordTest < Minitest::Test
           if Rails.version.to_f >= 6.1
             # Rails 6.1 dropped the bound column name
             # Can be removed once this PR is fixed: https://github.com/rails/rails/pull/41068
-            assert_equal [2, 3], binds[nil], -> { actual.ai }
+            assert_equal [2, 3], binds[:nil], -> { actual.ai }
           else
             assert_equal [2, 3], binds[:age], -> { actual.ai }
           end