counter_culture 3.11.1 → 3.11.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 593e3704f40322fade2c263f93bb6d88749b397d641586b30f4d616ee5c9ed0c
-  data.tar.gz: b183412de943dbe2390a4eef636ca8aae9fea6aeac6ac298177545d26b08413e
+  metadata.gz: c568fc84225757b834a008eb60dbadfe5dea148078c855fe76a2b5711b0c5b1e
+  data.tar.gz: 3abc94f197b60fb6778eadb98ad402e6b961155c492970111b85bbf1bd171237
 SHA512:
-  metadata.gz: c0a74d87a9d56ec4b19d7fe9a7aaf9bd2698d527387251a52b866e534117386005e830eef556b96bbc67122a125ebf942a4531be5b6f2d35a075b55b3133b052
-  data.tar.gz: 4cd891261ed5402cffa7eea851e2400f3b6d7ec8c4d617b36a3b0dd680e00f17b9b1a909a55bb94db1dbe636ca4b5d2d929d02df851e00869bb316a703fa8928
+  metadata.gz: adfd25e90ca300e1d6bc19296b343a8fbd9a272076e48591dddc513ffa4173dff10ef2d26cec6bd08bdfbc372ccf1b94e97cee7d7827205523efe0278d1bffea
+  data.tar.gz: 733126919913da347057de9c81cbbdf7b4b45e9ea1bbbecad12586a06432801e37b1c8833078cc93ce11fdc0ad3b201c5484822dcebc708f970d2106c022d35b

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 3.11.2 (July 10, 2025)
+
+Bugfixes:
+  - Fix regression for foreign_key_values by adding support for composite primary keys (#417)
+
 ## 3.11.1 (June 27, 2025)
 
 Bugfixes:

data/README.md

@@ -232,7 +232,7 @@ The ```:delta_column``` option supports all numeric column types, not just ```:i
 class Product < ActiveRecord::Base
   belongs_to :category
   counter_culture :category, foreign_key_values:
-      proc {|category_id| [category_id, Category.find_by_id(category_id).try(:parent_category).try(:id)] }
+      proc {|category_id| [category_id, Category.find_by(id: category_id)&.parent_category&.id] }
 end
 
 class Category < ActiveRecord::Base

data/lib/counter_culture/counter.rb

@@ -108,6 +108,15 @@ module CounterCulture
 
         primary_key = relation_primary_key(relation, source: obj, was: options[:was])
 
+        if foreign_key_values && id_to_change.is_a?(Enumerable) && id_to_change.count > 1 &&
+            Array.wrap(primary_key).count == 1
+          # To keep this feature backwards compatible, we need to accommodate a case where
+          # `foreign_key_values` returns an array, but we're _not_ using composite primary
+          # keys. In that case, we need to wrap the `id_to_change` in one more array to keep
+          # it compatible with the `.zip` call in `primary_key_conditions`.
+          id_to_change = [id_to_change]
+        end
+
         if Thread.current[:aggregate_counter_updates]
           Thread.current[:primary_key_map][klass] ||= primary_key
         end

data/lib/counter_culture/version.rb

@@ -1,3 +1,3 @@
 module CounterCulture
-  VERSION = '3.11.1'.freeze
+  VERSION = '3.11.2'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: counter_culture
 version: !ruby/object:Gem::Version
-  version: 3.11.1
+  version: 3.11.2
 platform: ruby
 authors:
 - Magnus von Koeller
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-06-27 00:00:00.000000000 Z
+date: 2025-07-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord