activerecord-sqlserver-adapter 8.0.5 → 8.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 488cfb3fda58b07e4bacc2db733d766492f84d4d0af50ecd6e5548f066ac34b3
-  data.tar.gz: b64864d043dd035dde5413736d9fc9adb5d2bf1e364b25ef57ca2ea69bc386cf
+  metadata.gz: 34f478c68720d0070576ee92aeaaaf4d32ed24a9edf746d575ddc1b7633c02f4
+  data.tar.gz: 5c9bbd2821b3d0431d93e74b07dc1a729fa5a90ee213e2a53f93e8b8d67034fa
 SHA512:
-  metadata.gz: fe2eb7bbc39edbc855bba7403792e532a43cae178d1cd8943189142f61c4203b346cb0191366626422ec287fde5d77d174240bcdef37152d177acda8d1e5dad4
-  data.tar.gz: 88ae10b6aa5b67f2bbc6a760a2f5af7679422fea1d10086aa3bb6592aaeadc202a2657c48906e4bb8b237acd7ca87e57588de6f02590a20a32278c855777b5c8
+  metadata.gz: 50dd731f8f49995aed5492cda26255234bb0c4b941cd67c26f9d8e7ca3e3ffe13d9d6470f3769617604388d9c92d3df99b21a2169d3c1c9c8c75906fe9d6d7bc
+  data.tar.gz: 66461bd72eaa9fd3cfb8235c6566f199eefae9e716ac59ed00908e3ddd19f99ec467c5fe83a9d1bf3ed3a8f7642e35c6267e6f6168344d9e1339bb950bc79e24

data/.github/workflows/ci.yml

@@ -5,7 +5,7 @@ on: [push, pull_request]
 jobs:
   test:
     name: Run test suite
-    runs-on: ubuntu-20.04 # TODO: Change back to 'ubuntu-latest' when https://github.com/microsoft/mssql-docker/issues/899 resolved.
+    runs-on: ubuntu-latest
 
     env:
       COMPOSE_FILE: compose.ci.yaml

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+## v8.0.6
+
+#### Fixed
+
+- [#1318](https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/1318) Reverse order of values when upserting
+- [#1321](https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/1321) Fix SQL statement to calculate `updated_at` when upserting
+
 ## v8.0.5
 
 #### Added

data/VERSION

@@ -1 +1 @@
-8.0.5
+8.0.6

data/lib/active_record/connection_adapters/sqlserver/database_statements.rb

@@ -159,6 +159,8 @@ module ActiveRecord
 
 
         def build_sql_for_merge_insert(insert:, insert_all:, columns_with_uniqueness_constraints:) # :nodoc:
+          insert_all.inserts.reverse! if insert.update_duplicates?
+
           sql = <<~SQL
             MERGE INTO #{insert.model.quoted_table_name} WITH (UPDLOCK, HOLDLOCK) AS target
             USING (
@@ -583,7 +585,7 @@ module ActiveRecord
         def build_sql_for_recording_timestamps_when_updating(insert:)
           insert.model.timestamp_attributes_for_update_in_model.filter_map do |column_name|
             if insert.send(:touch_timestamp_attribute?, column_name)
-              "target.#{quote_column_name(column_name)}=CASE WHEN (#{insert.updatable_columns.map { |column| "(COALESCE(target.#{quote_column_name(column)}, 'NULL') = COALESCE(source.#{quote_column_name(column)}, 'NULL'))" }.join(" AND ")}) THEN target.#{quote_column_name(column_name)} ELSE #{high_precision_current_timestamp} END,"
+              "target.#{quote_column_name(column_name)}=CASE WHEN (#{insert.updatable_columns.map { |column| "(source.#{quote_column_name(column)} = target.#{quote_column_name(column)} OR (source.#{quote_column_name(column)} IS NULL AND target.#{quote_column_name(column)} IS NULL))" }.join(" AND ")}) THEN target.#{quote_column_name(column_name)} ELSE #{high_precision_current_timestamp} END,"
             end
           end.join
         end

data/test/cases/insert_all_test_sqlserver.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+require "cases/helper_sqlserver"
+require "models/book"
+require "models/sqlserver/recurring_task"
+
+class InsertAllTestSQLServer < ActiveRecord::TestCase
+  # Test ported from the Rails `main` branch that is not on the `8-0-stable` branch.
+  def test_insert_all_only_applies_last_value_when_given_duplicate_identifiers
+    skip unless supports_insert_on_duplicate_skip?
+
+    Book.insert_all [
+                      { id: 111, name: "expected_new_name" },
+                      { id: 111, name: "unexpected_new_name" }
+                    ]
+    assert_equal "expected_new_name", Book.find(111).name
+  end
+
+  # Test ported from the Rails `main` branch that is not on the `8-0-stable` branch.
+  def test_upsert_all_only_applies_last_value_when_given_duplicate_identifiers
+    skip unless supports_insert_on_duplicate_update? && !current_adapter?(:PostgreSQLAdapter)
+
+    Book.create!(id: 112, name: "original_name")
+
+    Book.upsert_all [
+                      { id: 112, name: "unexpected_new_name" },
+                      { id: 112, name: "expected_new_name" }
+                    ]
+    assert_equal "expected_new_name", Book.find(112).name
+  end
+
+  test "upsert_all recording of timestamps works with mixed datatypes" do
+    task = RecurringTask.create!(
+      key: "abcdef",
+      priority: 5
+    )
+
+    RecurringTask.upsert_all([{
+                                id: task.id,
+                                priority: nil
+                              }])
+
+    assert_not_equal task.updated_at, RecurringTask.find(task.id).updated_at
+  end
+end

data/test/models/sqlserver/recurring_task.rb

@@ -0,0 +1,3 @@
+class RecurringTask < ActiveRecord::Base
+  self.table_name = "recurring_tasks"
+end

data/test/schema/sqlserver_specific_schema.rb

@@ -360,4 +360,12 @@ ActiveRecord::Schema.define do
       name varchar(255)
     )
   TABLE_IN_OTHER_SCHEMA_USED_BY_MODEL
+
+  create_table "recurring_tasks", force: true do |t|
+    t.string :key
+    t.integer :priority, default: 0
+
+    t.datetime2 :created_at
+    t.datetime2 :updated_at
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-sqlserver-adapter
 version: !ruby/object:Gem::Version
-  version: 8.0.5
+  version: 8.0.6
 platform: ruby
 authors:
 - Ken Collins
@@ -15,7 +15,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-03-20 00:00:00.000000000 Z
+date: 2025-05-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -164,6 +164,7 @@ files:
 - test/cases/helper_sqlserver.rb
 - test/cases/in_clause_test_sqlserver.rb
 - test/cases/index_test_sqlserver.rb
+- test/cases/insert_all_test_sqlserver.rb
 - test/cases/json_test_sqlserver.rb
 - test/cases/lateral_test_sqlserver.rb
 - test/cases/migration_test_sqlserver.rb
@@ -205,6 +206,7 @@ files:
 - test/models/sqlserver/quoted_table.rb
 - test/models/sqlserver/quoted_view_1.rb
 - test/models/sqlserver/quoted_view_2.rb
+- test/models/sqlserver/recurring_task.rb
 - test/models/sqlserver/sst_memory.rb
 - test/models/sqlserver/sst_string_collation.rb
 - test/models/sqlserver/string_default.rb
@@ -239,8 +241,8 @@ licenses:
 - MIT
 metadata:
   bug_tracker_uri: https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/issues
-  changelog_uri: https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/blob/v8.0.5/CHANGELOG.md
-  source_code_uri: https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/tree/v8.0.5
+  changelog_uri: https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/blob/v8.0.6/CHANGELOG.md
+  source_code_uri: https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/tree/v8.0.6
 post_install_message:
 rdoc_options: []
 require_paths:
@@ -283,6 +285,7 @@ test_files:
 - test/cases/helper_sqlserver.rb
 - test/cases/in_clause_test_sqlserver.rb
 - test/cases/index_test_sqlserver.rb
+- test/cases/insert_all_test_sqlserver.rb
 - test/cases/json_test_sqlserver.rb
 - test/cases/lateral_test_sqlserver.rb
 - test/cases/migration_test_sqlserver.rb
@@ -324,6 +327,7 @@ test_files:
 - test/models/sqlserver/quoted_table.rb
 - test/models/sqlserver/quoted_view_1.rb
 - test/models/sqlserver/quoted_view_2.rb
+- test/models/sqlserver/recurring_task.rb
 - test/models/sqlserver/sst_memory.rb
 - test/models/sqlserver/sst_string_collation.rb
 - test/models/sqlserver/string_default.rb