active_record-cursor_paginator 0.3.1 → 0.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0e6079364fab39f26c9d63199730cc44ae0138c5c3d8452aa1b73219e0afa0e1
-  data.tar.gz: 36d9c3020d18dafe2905b18a6a50bebee06ab17caff4ff4abd4eb9bb965aa851
+  metadata.gz: 2b976c754e305df50a8c5eb4889bf98a6041a3e4c8fa73b7c6d6a4566068037a
+  data.tar.gz: 680ed7b82dd56e0c1a72db977350b701bdd9c90666053e7399f9dd9d280afae6
 SHA512:
-  metadata.gz: db459de8e2b52873f5ccb95e9bb7b957af9381a30fcfca97786189874bfacc8808f80fb939ea86d54c8f50cb5983b263089df16f4e7f2294387d41757cf723d1
-  data.tar.gz: a344ab46f80d38a5c43d890793031a31194d3f3b7dfe6b53258c94a77a08386470c7201e46f5d6833cd2f9e77c0f3fa940afb0306c951648193b1f04f0530ce9
+  metadata.gz: 7097e691c6a902bb42d0c2b8a7cb55a22d768b3d7bedefe344d05500efa6e665145d5e6ed8263bc3cbc313dd256bfab294a2651aa6fc1b230c910a8bdabd7abb
+  data.tar.gz: e1508a1aed8100c86f8f389a16b35996a4d633fb371a0e7a918ab728ee7e236f57b22df4d9d7ab6a1b8fe9785294bd0c5cc804f2f6d0e712d2ef4e88b746de84

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## [0.3.2] - 2026-06-03
+
+- Fix timezone offset in datetime cursor values being ignored in WHERE clause
+- Warn when a datetime cursor column is a join table alias, as type inference may be inaccurate
+
 ## [0.3.1] - 2026-03-06
 
 - Add support for Rails enum columns

data/README.md

@@ -65,6 +65,18 @@ ActiveSupport::JSON::Encoding.time_precision = 6
 
 ## Limitation
 
+### Datetime cursor values on join table columns
+
+When ordering by a datetime column from a joined table via an alias (e.g. `authors.created_at as author_created_at`), the column type is inferred from the root model. If the joined table's column type differs from the root model's column with the same name, timezone conversion in cursor values may be inaccurate.
+
+A warning is logged when this situation is detected:
+
+```
+[CursorPaginator] Cursor column 'author_created_at' resolves to 'authors.created_at'
+(join column). Type is inferred from Post and may be inaccurate.
+Timezone-aware datetime cursors on joined tables may not work correctly.
+```
+
 This library does not support the following order expressions
 
 ```ruby

data/lib/active_record/cursor_paginator/version.rb

@@ -2,6 +2,6 @@
 
 module ActiveRecord
   class CursorPaginator
-    VERSION = '0.3.1'
+    VERSION = '0.3.2'
   end
 end

data/lib/active_record/cursor_paginator.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require 'active_record'
+require 'set'
 require_relative 'cursor_paginator/version'
 
 module ActiveRecord
@@ -168,7 +169,7 @@ module ActiveRecord
       def cursor_for_record(record)
         unencoded_cursor = @fields.map do |field|
           field_name = field.keys.first
-          value = if record.class.defined_enums.key?(field_name.to_s)
+          value = if record.class.defined_enums.has_key?(field_name.to_s)
                     # For enum columns, get the raw integer value from the database
                     record.read_attribute_before_type_cast(field_name)
                   else
@@ -268,14 +269,43 @@ module ActiveRecord
       def build_filter_query(sorted_relation, op, current_field, prev_fields)
         relation = sorted_relation
         prev_fields.each do |col, val|
+          col_key = col
           col = @aliases[col] if @aliases.has_key? col
           col = qualify_field_if_needed(col)
-          relation = relation.where("#{col} = ?", val)
+          relation = relation.where("#{col} = ?", cast_cursor_value(col_key, val))
         end
         col, val = current_field
+        col_key = col
         col = @aliases[col] if @aliases.has_key? col
         col = qualify_field_if_needed(col)
-        relation.where("#{col} #{op} ?", val)
+        relation.where("#{col} #{op} ?", cast_cursor_value(col_key, val))
+      end
+
+      def cast_cursor_value(col_name, val)
+        return val unless val.is_a?(String)
+
+        resolved_expr = @aliases[col_name.to_s]
+        resolved_col = (resolved_expr || col_name.to_s).split('.').last
+        type = @relation.klass.type_for_attribute(resolved_col)
+
+        return val unless %i[datetime time timestamp].include?(type.type)
+
+        warn_join_column_cast(col_name, resolved_expr) if resolved_expr&.match?(/\A\w+\.\w+\z/)
+
+        type.cast(val)
+      end
+
+      def warn_join_column_cast(alias_name, expression)
+        @_warned_join_casts ||= Set.new
+        return if @_warned_join_casts.include?(alias_name)
+
+        @_warned_join_casts << alias_name
+
+        msg = "[CursorPaginator] Cursor column '#{alias_name}' resolves to '#{expression}' " \
+              "(join column). Type is inferred from #{@relation.klass.name} and may be " \
+              'inaccurate. Timezone-aware datetime cursors on joined tables may not work correctly.'
+        logger = ActiveRecord::Base.logger
+        logger ? logger.warn(msg) : Kernel.warn(msg)
       end
 
       # parse aliases from select values

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_record-cursor_paginator
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.3.2
 platform: ruby
 authors:
 - Shinichi Sugiyama
@@ -66,7 +66,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.7.2
+rubygems_version: 3.6.7
 specification_version: 4
 summary: cursor pagination for ActiveRecord
 test_files: []