acts_as_ranked_list 0.2.3 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 55fb6b1014091f8df47866752d9aedc4cdac1204955852b09c84d480e2769486
-  data.tar.gz: 34bc12ea565e8fd881a6401d9b5cc3ea4e83b86106abb6ee1ec1b7d4a19c1fac
+  metadata.gz: 3c665111b3d1da9b578d19b1c49203ae597e49a06e09f274e92cb7cecca03687
+  data.tar.gz: d4d682b766b048afcc9af37fabcb99e37e683a2d0924e32a2d4a3941a6c29f36
 SHA512:
-  metadata.gz: b283316886e10b9d469843c3814fda1143ea619feacd8f2415e8d323c1a21c41281b91b9407b53bdd43425192ccc5dd5aa29ec905030788404490dba844a8765
-  data.tar.gz: dd72d4345b6d37ee33f8f4513200c793865f52c8468949f9935962c77ab898ded2b7fffc43ad9b1616e6e3a55517dd72964bed168069de9c79c04699f61c8b9e
+  metadata.gz: cf228e285474e6ffdee7aeeb36c5a024ee897a640a8f9dbc486faf4e7b8fd431e2c11907137e1f15fec253e0ee4ec38e7b73cedf2883690b1482261032a9cd87
+  data.tar.gz: 922b5ad1fa695325e779a3a44db204b3457a84d0b5bcec539cb16c492c48abe3c82a92acf978984aa3d758846fc532fe1f9c829c41eed9fe4f764f6852414afa

data/CHANGELOG.md

@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ## [Unreleased]
 
+## [0.4.0] - 2024-02-12
+
+- Fixes sporadic bug on postgres with `pluck` higher items.
+
+## [0.3.0] - 2024-02-11 [YANKED]
+
+- Fixes individual items updating timestamps even when `touch_on_update` is false.
+
 ## [0.2.3] - 2023-12-31
 
 - Updates gem dependencies.

data/lib/acts_as_ranked_list/active_record/rank_column.rb

@@ -198,7 +198,9 @@ module ActsAsRankedList #:nodoc:
 
           define_method :set_rank_above do |item|
             higher_items = get_higher_items(2, "DESC", item.current_rank, true, true)
-            padded_array = pad_array(higher_items.pluck(rank_column), 0)
+            # TODO: Why is higher_items and lower_items sometimes need to be converted to array.
+            # Is it related to if items are not found? Otherwise it returns an error
+            padded_array = pad_array(higher_items.to_a.pluck(rank_column), 0)
             new_rank = padded_array.sum / 2
 
             with_persistence do
@@ -223,7 +225,7 @@ module ActsAsRankedList #:nodoc:
             # new_rank = ::ActiveRecord::Base.connection.execute(sql).to_a.first.values.first
 
             lower_items = get_lower_items(2, "ASC", item.current_rank, true, true)
-            padded_array = pad_array(lower_items.pluck(rank_column))
+            padded_array = pad_array(lower_items.to_a.pluck(rank_column))
             new_rank = padded_array.sum / 2
 
             with_persistence do
@@ -312,7 +314,17 @@ module ActsAsRankedList #:nodoc:
           define_method :with_persistence do |items = [self], &blk|
             blk.call
 
+            item_record_timestamps = items.map do |i|
+              i.record_timestamps?
+            end
+
+            items.each { |i| i.record_timestamps = touch_on_update }
+
             items.each(&:save!) unless skip_persistence?
+
+            items.each_with_index do |i, idx|
+              i.record_timestamps = item_record_timestamps[idx]
+            end
           end
 
           define_method :pad_array do |array, value = nil, size = 2|

data/lib/acts_as_ranked_list/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActsAsRankedList
-  VERSION = "0.2.3"
+  VERSION = "0.4.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: acts_as_ranked_list
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.4.0
 platform: ruby
 authors:
 - Farbafe
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-12-31 00:00:00.000000000 Z
+date: 2024-02-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails