pagy 5.7.5 → 8.6.2

data/lib/pagy/backend.rb

@@ -1,8 +1,8 @@
-# See Pagy::Backend API documentation: https://ddnexus.github.io/pagy/api/backend
+# See Pagy::Backend API documentation: https://ddnexus.github.io/pagy/docs/api/backend
 # frozen_string_literal: true
 
 class Pagy
-  # Define a few generic methods to paginate an ORM collection out of the box,
+  # Define a few generic methods to paginate a collection out of the box,
   # or any collection by overriding pagy_get_items and/or pagy_get_vars in your controller
   # See also the extras if you need specialized methods to paginate Arrays or other collections
   module Backend
@@ -18,11 +18,23 @@ class Pagy
     # You may need to override the count call for non AR collections
     def pagy_get_vars(collection, vars)
       pagy_set_items_from_params(vars) if defined?(ItemsExtra)
-      vars[:count] ||= (count = collection.count(:all)).is_a?(Hash) ? count.size : count
-      vars[:page]  ||= params[vars[:page_param] || DEFAULT[:page_param]]
+      vars[:count] ||= pagy_get_count(collection, vars)
+      vars[:page]  ||= pagy_get_page(vars)
       vars
     end
 
+    # Get the count from the collection
+    def pagy_get_count(collection, vars)
+      count_args = vars[:count_args] || DEFAULT[:count_args]
+      (count     = collection.count(*count_args)).is_a?(Hash) ? count.size : count
+    end
+
+    # Get the page integer from the params
+    # Overridable by the jsonapi extra
+    def pagy_get_page(vars)
+      [params[vars[:page_param] || DEFAULT[:page_param]].to_i, 1].max
+    end
+
     # Sub-method called only by #pagy: here for easy customization of record-extraction by overriding
     # You may need to override this method for collections without offset|limit
     def pagy_get_items(collection, pagy)

data/lib/pagy/calendar/day.rb

@@ -1,28 +1,40 @@
-# See Pagy::Countless API documentation: https://ddnexus.github.io/pagy/api/calendar
+# See Pagy::Countless API documentation: https://ddnexus.github.io/pagy/docs/api/calendar
 # frozen_string_literal: true
 
 class Pagy # :nodoc:
   class Calendar # :nodoc:
-    # Calendar day subclass
-    class Day < Calendar
-      DEFAULT = { order:  :asc,      # rubocop:disable Style/MutableConstant
-                  format: '%Y-%m-%d' }
+    # Day unit subclass
+    class Day < Unit
+      DEFAULT = { size:   31,   # rubocop:disable Style/MutableConstant
+                  ends: false,
+                  order:  :asc,
+                  format: '%d' }
 
       protected
 
       # Setup the calendar variables
       def setup_unit_vars
         super
-        @initial = new_time(@starting.year, @starting.month, @starting.day)
-        @final   = new_time(@ending.year, @ending.month, @ending.day) + DAY
-        @pages   = @last = (@final - @initial).to_i / DAY
+        @initial = @starting.beginning_of_day
+        @final   = @ending.tomorrow.beginning_of_day
+        @last    = page_offset(@initial, @final)
         @from    = starting_time_for(@page)
-        @to      = @from + DAY
+        @to      = @from.tomorrow
       end
 
       # Starting time for the page
       def starting_time_for(page)
-        @initial + (offset_units_for(page) * DAY)
+        @initial.days_since(time_offset_for(page))
+      end
+
+      def page_offset_at(time)
+        page_offset(@initial, time.beginning_of_day)
+      end
+
+      private
+
+      def page_offset(time_a, time_b)  # remove in 6.0
+        (time_b.time - time_a.time).to_i / 1.day
       end
     end
   end

data/lib/pagy/calendar/month.rb

@@ -1,16 +1,42 @@
-# See Pagy::Countless API documentation: https://ddnexus.github.io/pagy/api/calendar
+# See Pagy::Countless API documentation: https://ddnexus.github.io/pagy/docs/api/calendar
 # frozen_string_literal: true
 
-require_relative 'month_mixin'
-
 class Pagy # :nodoc:
   class Calendar # :nodoc:
-    # Calendar month subclass
-    class Month < Calendar
-      DEFAULT = { order:  :asc,      # rubocop:disable Style/MutableConstant
-                  format: '%Y-%m' }
-      MONTHS  = 1  # number of months in the unit
-      include MonthMixin
+    # Month unit subclass
+    class Month < Unit
+      DEFAULT = { size: 12,   # rubocop:disable Style/MutableConstant
+                  ends: false,
+                  order: :asc,
+                  format: '%b' }
+
+      protected
+
+      # Setup the calendar variables
+      def setup_unit_vars
+        super
+        @initial = @starting.beginning_of_month
+        @final   = @ending.next_month.beginning_of_month
+        @last    = (months_in(@final) - months_in(@initial))
+        @from    = starting_time_for(@page)
+        @to      = @from.next_month
+      end
+
+      # Starting time for the page
+      def starting_time_for(page)
+        @initial.months_since(time_offset_for(page))
+      end
+
+      def page_offset_at(time)
+        months_in(time.beginning_of_month) - months_in(@initial)
+      end
+
+      private
+
+      # Number of months in time
+      def months_in(time)
+        (time.year * 12) + time.month
+      end
     end
   end
 end

data/lib/pagy/calendar/quarter.rb

@@ -1,23 +1,49 @@
-# See Pagy::Countless API documentation: https://ddnexus.github.io/pagy/api/calendar
+# See Pagy::Countless API documentation: https://ddnexus.github.io/pagy/docs/api/calendar
 # frozen_string_literal: true
 
-require_relative 'month_mixin'
-
 class Pagy # :nodoc:
   class Calendar # :nodoc:
-    # Calendar quarter subclass
-    class Quarter < Calendar
-      DEFAULT = { order:  :asc,      # rubocop:disable Style/MutableConstant
-                  format: '%Y-Q%q' } # '%q' token
-      MONTHS  = 3  # number of months of the unit
-      include MonthMixin
+    # Quarter unit subclass
+    class Quarter < Unit
+      DEFAULT = { size:   4,      # rubocop:disable Style/MutableConstant
+                  ends: false,
+                  order:  :asc,
+                  format: 'Q%q' } # '%q' token
 
       # The label for any page, with the substitution of the '%q' token
       def label_for(page, opts = {})
         starting_time = starting_time_for(page.to_i)  # page could be a string
-        opts[:format] = (opts[:format] || @vars[:format]).gsub('%q') { (starting_time.month / 4) + 1 }
+        opts[:format] = (opts[:format] || @vars[:format]).gsub('%q') { (starting_time.month / 3.0).ceil }
         localize(starting_time, opts)
       end
+
+      protected
+
+      # Setup the calendar variables
+      def setup_unit_vars
+        super
+        @initial = @starting.beginning_of_quarter
+        @final   = @ending.next_quarter.beginning_of_quarter
+        @last    = (months_in(@final) - months_in(@initial)) / 3
+        @from    = starting_time_for(@page)
+        @to      = @from.next_quarter
+      end
+
+      # Starting time for the page
+      def starting_time_for(page)
+        @initial.months_since(time_offset_for(page) * 3)
+      end
+
+      def page_offset_at(time)
+        (months_in(time.beginning_of_quarter) - months_in(@initial)) / 3
+      end
+
+      private
+
+      # Number of months in time
+      def months_in(time)
+        (time.year * 12) + time.month
+      end
     end
   end
 end

data/lib/pagy/calendar/unit.rb

@@ -0,0 +1,106 @@
+# frozen_string_literal: true
+
+require 'active_support'
+require 'active_support/core_ext/time'
+require 'active_support/core_ext/date_and_time/calculations'
+require 'active_support/core_ext/numeric/time'
+require 'active_support/core_ext/integer/time'
+
+class Pagy # :nodoc:
+  class Calendar < Hash # :nodoc:
+    # Base class for time units subclasses (Year, Quarter, Month, Week, Day)
+    class Unit < Pagy
+      attr_reader :order, :from, :to
+
+      # Merge and validate the options, do some simple arithmetic and set a few instance variables
+      def initialize(vars)    # rubocop:disable Lint/MissingSuper
+        raise InternalError, 'Pagy::Calendar::Unit is a base class; use one of its subclasses' \
+            if instance_of?(Pagy::Calendar::Unit)
+
+        vars = self.class::DEFAULT.merge(vars)  # subclass specific default
+        normalize_vars(vars)                    # general default
+        setup_vars(page: 1)
+        setup_unit_vars
+        raise OverflowError.new(self, :page, "in 1..#{@last}", @page) if @page > @last
+
+        @prev = (@page - 1 unless @page == 1)
+        @next = @page == @last ? (1 if @vars[:cycle]) : @page + 1
+      end
+
+      # The label for the current page (it can pass along the I18n gem opts when it's used with the i18n extra)
+      def label(opts = {})
+        label_for(@page, opts)
+      end
+
+      # The label for any page (it can pass along the I18n gem opts when it's used with the i18n extra)
+      def label_for(page, opts = {})
+        opts[:format] ||= @vars[:format]
+        localize(starting_time_for(page.to_i), opts)  # page could be a string
+      end
+
+      protected
+
+      # The page that includes time
+      # In case of out of range time, the :fit_time option avoids the outOfRangeError
+      # and returns the closest page to the passed time argument (first or last page)
+      def page_at(time, **opts)
+        fit_time  = time
+        fit_final = @final - 1
+        unless time.between?(@initial, fit_final)
+          raise OutOfRangeError.new(self, :time, "between #{@initial} and #{fit_final}", time) unless opts[:fit_time]
+
+          if time < @final
+            fit_time = @initial
+            ordinal  = 'first'
+          else
+            fit_time = fit_final
+            ordinal  = 'last'
+          end
+          warn "Pagy::Calendar#page_at: Rescued #{time} out of range by returning the #{ordinal} page."
+        end
+        offset = page_offset_at(fit_time)   # offset starts from 0
+        @order == :asc ? offset + 1 : @last - offset
+      end
+
+      # Base class method for the setup of the unit variables (subclasses must implement it and call super)
+      def setup_unit_vars
+        raise VariableError.new(self, :format, 'to be a strftime format', @vars[:format]) unless @vars[:format].is_a?(String)
+        raise VariableError.new(self, :order, 'to be in [:asc, :desc]', @order) \
+        unless %i[asc desc].include?(@order = @vars[:order])
+
+        @starting, @ending = @vars[:period]
+        raise VariableError.new(self, :period, 'to be a an Array of min and max TimeWithZone instances', @vars[:period]) \
+        unless @starting.is_a?(ActiveSupport::TimeWithZone) \
+        && @ending.is_a?(ActiveSupport::TimeWithZone) && @starting <= @ending
+      end
+
+      # Apply the strftime format to the time (overridden by the i18n extra when localization is required)
+      def localize(time, opts)
+        time.strftime(opts[:format])
+      end
+
+      # Number of time units to offset from the @initial time, in order to get the ordered starting time for the page.
+      # Used in starting_time_for(page) where page starts from 1 (e.g. page to starting_time means subtracting 1)
+      def time_offset_for(page)
+        @order == :asc ? page - 1 : @last - page
+      end
+
+      # Period of the active page (used internally for nested units)
+      def active_period
+        [[@starting, @from].max, [@to - 1, @ending].min] # -1 sec: include only last unit day
+      end
+
+      # :nocov:
+      # This method must be implemented by the unit subclass
+      def starting_time_for(*)
+        raise NoMethodError, 'the starting_time_for method must be implemented by the unit subclass'
+      end
+
+      # This method must be implemented by the unit subclass
+      def page_offset_at(*)
+        raise NoMethodError, 'the page_offset_at method must be implemented by the unit subclass'
+      end
+      # :nocov:
+    end
+  end
+end

data/lib/pagy/calendar/week.rb

@@ -1,38 +1,38 @@
-# See Pagy::Countless API documentation: https://ddnexus.github.io/pagy/api/calendar
+# See Pagy::Countless API documentation: https://ddnexus.github.io/pagy/docs/api/calendar
 # frozen_string_literal: true
 
 class Pagy # :nodoc:
   class Calendar # :nodoc:
-    # Calendar week subclass
-    class Week < Calendar
-      DEFAULT = { order:  :asc,      # rubocop:disable Style/MutableConstant
-                  format: '%Y-%W',
-                  offset: 0 }
+    # Week unit subclass
+    class Week < Unit
+      DEFAULT  = { order:  :asc,      # rubocop:disable Style/MutableConstant
+                   format: '%Y-%W' }
 
       protected
 
       # Setup the calendar variables
       def setup_unit_vars
-        setup_vars(offset: 0)
         super
-        @initial = unit_starting_time_for(@starting)
-        @final   = unit_starting_time_for(@ending) + WEEK
-        @pages   = @last = (@final - @initial).to_i / WEEK
+        @initial = @starting.beginning_of_week
+        @final   = @ending.next_week.beginning_of_week
+        @last    = page_offset(@initial, @final)
         @from    = starting_time_for(@page)
-        @to      = @from + WEEK
+        @to      = @from.next_week
       end
 
       # Starting time for the page
       def starting_time_for(page)
-        @initial + (offset_units_for(page) * WEEK)
+        @initial.weeks_since(time_offset_for(page))
+      end
+
+      def page_offset_at(time)
+        page_offset(@initial, time.beginning_of_week)
       end
 
       private
 
-      # Unit starting time for time
-      def unit_starting_time_for(time)
-        starting_time = time - (((time.wday - @offset) * DAY) % WEEK)
-        new_time(starting_time.year, starting_time.month, starting_time.day)
+      def page_offset(time_a, time_b)  # remove in 6.0
+        (time_b.time - time_a.time).to_i / 1.week
       end
     end
   end

data/lib/pagy/calendar/year.rb

@@ -1,11 +1,13 @@
-# See Pagy::Countless API documentation: https://ddnexus.github.io/pagy/api/calendar
+# See Pagy::Countless API documentation: https://ddnexus.github.io/pagy/docs/api/calendar
 # frozen_string_literal: true
 
 class Pagy # :nodoc:
   class Calendar # :nodoc:
-    # Calendar year subclass
-    class Year < Calendar
-      DEFAULT = { order:  :asc,      # rubocop:disable Style/MutableConstant
+    # Year unit subclass
+    class Year < Unit
+      DEFAULT = { size:   10,   # rubocop:disable Style/MutableConstant
+                  ends: false,
+                  order:  :asc,
                   format: '%Y' }
 
       protected
@@ -13,16 +15,20 @@ class Pagy # :nodoc:
       # Setup the calendar variables
       def setup_unit_vars
         super
-        @initial = new_time(@starting.year)
-        @final   = new_time(@ending.year + 1)
-        @pages   = @last = @final.year - @initial.year
+        @initial = @starting.beginning_of_year
+        @final   = @ending.next_year.beginning_of_year
+        @last    = @final.year - @initial.year
         @from    = starting_time_for(@page)
-        @to      = new_time(@from.year + 1)
+        @to      = @from.next_year
       end
 
       # Starting time for the page
       def starting_time_for(page)
-        new_time(@initial.year + offset_units_for(page))
+        @initial.years_since(time_offset_for(page))
+      end
+
+      def page_offset_at(time)
+        time.beginning_of_year.year - @initial.year
       end
     end
   end

data/lib/pagy/calendar.rb

@@ -1,87 +1,79 @@
-# See Pagy::Countless API documentation: https://ddnexus.github.io/pagy/api/calendar
+# See Pagy::Countless API documentation: https://ddnexus.github.io/pagy/docs/api/calendar
 # frozen_string_literal: true
 
-require 'pagy'
+require_relative '../pagy'
+require_relative 'calendar/unit'
 
 class Pagy # :nodoc:
-  # Base class for time units subclasses (Year, Quarter, Month, Week, Day)
-  class Calendar < Pagy
+  # Calendar class
+  class Calendar < Hash
+    # Specific out of range error
+    class OutOfRangeError < VariableError; end
+
     # List of units in desc order of duration. It can be used for custom units.
     UNITS = %i[year quarter month week day]  # rubocop:disable Style/MutableConstant
-    DAY   = 60 * 60 * 24  # One day in seconds
-    WEEK  = DAY * 7       # One week in seconds
-
-    attr_reader :order
-
-    # Merge and validate the options, do some simple arithmetic and set a few instance variables
-    def initialize(vars) # rubocop:disable Lint/MissingSuper
-      raise InternalError, 'Pagy::Calendar is a base class; use one of its subclasses' if instance_of?(Pagy::Calendar)
-
-      vars = self.class::DEFAULT.merge(vars)  # subclass specific default
-      normalize_vars(vars)                    # general default
-      setup_vars(page: 1)
-      setup_unit_vars
-      setup_params_var
-      raise OverflowError.new(self, :page, "in 1..#{@last}", @page) if @page > @last
-
-      @prev = (@page - 1 unless @page == 1)
-      @next = @page == @last ? (1 if @vars[:cycle]) : @page + 1
-    end
-
-    # The label for the current page (it can pass along the I18n gem opts when it's used with the i18n extra)
-    def label(opts = {})
-      label_for(@page, opts)
-    end
 
-    # The label for any page (it can pass along the I18n gem opts when it's used with the i18n extra)
-    def label_for(page, opts = {})
-      opts[:format] ||= @vars[:format]
-      localize(starting_time_for(page.to_i), opts)  # page could be a string
-    end
-
-    protected
-
-    # Base class method for the setup of the unit variables (subclasses must implement it and call super)
-    def setup_unit_vars
-      raise VariableError.new(self, :format, 'to be a strftime format', @vars[:format]) unless @vars[:format].is_a?(String)
-      raise VariableError.new(self, :order, 'to be in [:asc, :desc]', @order) \
-            unless %i[asc desc].include?(@order = @vars[:order])
+    class << self
+      private
 
-      @starting, @ending = @vars[:period]
-      raise VariableError.new(self, :period, 'to be a an Array of min and max local Time instances', @vars[:period]) \
-            unless @starting.is_a?(Time) && @ending.is_a?(Time) && !@starting.utc? && !@ending.utc? && @starting <= @ending \
-                   && (@utc_offset = @starting.utc_offset) == @ending.utc_offset
-    end
+      # Create a unit subclass instance by using the unit name (internal use)
+      def create(unit, vars)
+        raise InternalError, "unit must be in #{UNITS.inspect}; got #{unit}" unless UNITS.include?(unit)
 
-    # Apply the strftime format to the time (overridden by the i18n extra when localization is required)
-    def localize(time, opts)
-      time.strftime(opts[:format])
-    end
+        name    = unit.to_s
+        name[0] = name[0].capitalize
+        Object.const_get("Pagy::Calendar::#{name}").new(vars)
+      end
 
-    # Number of units to offset from the @initial time, in order to get the ordered starting time for the page.
-    # Used in starting_time_for(page) with a logic equivalent to: @initial + (offset_units_for(page) * unit_time_length)
-    def offset_units_for(page)
-      @order == :asc ? page - 1 : @pages - page
+      # Return calendar, from, to
+      def init(conf, period, params)
+        new.send(:init, conf, period, params)
+      end
     end
 
-    # Create a new local time at the beginning of the day
-    def new_time(year, month = 1, day = 1)
-      Time.new(year, month, day, 0, 0, 0, @utc_offset)
+    # Return the current time of the smallest time unit shown
+    def showtime
+      self[@units.last].from
     end
 
-    # Period of the active page (used internally for nested units)
-    def active_period
-      [[@starting, @from].max, [@to - 1, @ending].min] # -1 sec: include only last unit day
+    private
+
+    # Create the calendar
+    def init(conf, period, params)
+      @conf  = Marshal.load(Marshal.dump(conf))  # store a copy
+      @units = Calendar::UNITS & @conf.keys # get the units in time length desc order
+      raise ArgumentError, 'no calendar unit found in pagy_calendar @configuration' if @units.empty?
+
+      @period     = period
+      @params     = params
+      @page_param = conf[:pagy][:page_param] || DEFAULT[:page_param]
+      @units.each do |unit|  # set all the :page_param vars for later deletion
+        unit_page_param         = :"#{unit}_#{@page_param}"
+        conf[unit][:page_param] = unit_page_param
+        conf[unit][:page]       = @params[unit_page_param]
+      end
+      calendar = {}
+      object   = nil
+      @units.each_with_index do |unit, index|
+        params_to_delete    = @units[(index + 1), @units.size].map { |sub| conf[sub][:page_param] } + [@page_param]
+        conf[unit][:params] = lambda { |up| up.except(*params_to_delete.map(&:to_s)) } # rubocop:disable Style/Lambda
+        conf[unit][:period] = object&.send(:active_period) || @period
+        calendar[unit]      = object = Calendar.send(:create, unit, conf[unit])
+      end
+      [replace(calendar), object.from, object.to]
     end
 
-    class << self
-      # Create a subclass instance by unit name (internal use)
-      def create(unit, vars)
-        raise InternalError, "unit must be in #{UNITS.inspect}; got #{unit}" unless UNITS.include?(unit)
-
-        name    = unit.to_s
-        name[0] = name[0].capitalize
-        Object.const_get("Pagy::Calendar::#{name}").new(vars)
+    # Return the calendar object at time
+    def calendar_at(time, **opts)
+      conf        = Marshal.load(Marshal.dump(@conf))
+      page_params = {}
+      @units.inject(nil) do |object, unit|
+        conf[unit][:period] = object&.send(:active_period) || @period
+        conf[unit][:page]   = page_params[:"#{unit}_#{@page_param}"] \
+                            = Calendar.send(:create, unit, conf[unit]).send(:page_at, time, **opts)
+        conf[unit][:params] ||= {}
+        conf[unit][:params].merge!(page_params)
+        Calendar.send(:create, unit, conf[unit])
       end
     end
   end

data/lib/pagy/console.rb

@@ -1,8 +1,8 @@
-# See Pagy::Console API documentation: https://ddnexus.github.io/pagy/api/console
+# See Pagy::Console API documentation: https://ddnexus.github.io/pagy/docs/api/console
 # frozen_string_literal: true
 
-require 'pagy'  # so you can require just the extra in the console
-require 'pagy/extras/standalone'
+require_relative '../pagy'  # so you can require just the extra in the console
+require_relative 'extras/standalone'
 
 class Pagy
   # Provide a ready to use pagy environment when included in irb/rails console

data/lib/pagy/countless.rb

@@ -1,9 +1,9 @@
-# See Pagy::Countless API documentation: https://ddnexus.github.io/pagy/api/countless
+# See Pagy::Countless API documentation: https://ddnexus.github.io/pagy/docs/api/countless
 # frozen_string_literal: true
 
-require 'pagy'
+require_relative '../pagy'
 
-class Pagy
+class Pagy # :nodoc:
   # No need to know the count to paginate
   class Countless < Pagy
     # Merge and validate the options, do some simple arithmetic and set a few instance variables
@@ -11,27 +11,32 @@ class Pagy
       normalize_vars(vars)
       setup_vars(page: 1, outset: 0)
       setup_items_var
-      setup_params_var
-      @offset = (@items * (@page - 1)) + @outset
+      setup_offset_var
     end
 
     # Finalize the instance variables based on the fetched size
     def finalize(fetched_size)
       raise OverflowError.new(self, :page, "to be < #{@page}", @page) if fetched_size.zero? && @page > 1
 
-      @pages = @last = (fetched_size > @items ? @page + 1 : @page)
-      @in    = [fetched_size, @items].min
-      @from  = @in.zero? ? 0 : @offset - @outset + 1
-      @to    = @offset - @outset + @in
-      @prev  = (@page - 1 unless @page == 1)
-      @next  = @page == @last ? (1 if @vars[:cycle]) : @page + 1
+      @last = fetched_size > @items ? @page + 1 : @page
+      @last = vars[:max_pages] if vars[:max_pages] && @last > vars[:max_pages]
+      raise OverflowError.new(self, :page, "in 1..#{@last}", @page) if @page > @last
+
+      @in   = [fetched_size, @items].min
+      @from = @in.zero? ? 0 : @offset - @outset + 1
+      @to   = @offset - @outset + @in
+      @prev = (@page - 1 unless @page == 1)
+      @next = @page == @last ? (1 if @vars[:cycle]) : @page + 1
       self
     end
+  end
 
+  module SeriesOverride # :nodoc:
     # Override the original series.
     # Return nil if :countless_minimal is enabled
-    def series(*)
+    def series(**)
       super unless @vars[:countless_minimal]
     end
   end
+  prepend SeriesOverride
 end

data/lib/pagy/exceptions.rb

@@ -10,7 +10,7 @@ class Pagy
       @pagy     = pagy
       @variable = variable
       @value    = value
-      super "expected :#{@variable} #{description}; got #{@value.inspect}"
+      super("expected :#{@variable} #{description}; got #{@value.inspect}")
     end
   end
 

data/lib/pagy/extras/arel.rb

@@ -1,4 +1,4 @@
-# See the Pagy documentation: https://ddnexus.github.io/pagy/extras/arel
+# See the Pagy documentation: https://ddnexus.github.io/pagy/docs/extras/arel
 # frozen_string_literal: true
 
 class Pagy # :nodoc:
@@ -16,7 +16,7 @@ class Pagy # :nodoc:
     def pagy_arel_get_vars(collection, vars)
       pagy_set_items_from_params(vars) if defined?(ItemsExtra)
       vars[:count] ||= pagy_arel_count(collection)
-      vars[:page]  ||= params[vars[:page_param] || DEFAULT[:page_param]]
+      vars[:page]  ||= pagy_get_page(vars)
       vars
     end