pagy 5.6.2 → 5.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0a24ac286e97d1aacd7603d7ada3791006120004bf56c027320c4184b9488fc7
-  data.tar.gz: 63a018d758c2ffff334de22fc8fdd433cdffe74030ba46ca5c3cf5dc8a563a9c
+  metadata.gz: f567ea14d1f7bf5697ca416cf0cf1bc200986b804099f31aba5fd1ae6a946fa4
+  data.tar.gz: 8b09e574678ab6673c4e9d8fdc48eaf611657354d457da4856a70173d7194732
 SHA512:
-  metadata.gz: f5fd0f9dd9c8c8a151ea026e7e749ba6446fec53baf4bd9af276f8059f062d50d54b0d500e46762f5f4df8a3f89af475f4b07c7d8633f8194ee3da5d33d2b104
-  data.tar.gz: cd09f6222ce1d6db7c92984c9382d080d386ea862f66b3b7104721ea6cc6fc5e788714ce2aa623517c6ad9f17c6c95fd52fdcbcf2ba9a017c1485b262ce86405
+  metadata.gz: 2cf7845ce5b8d668cfa0de512bb6fb7ed676e3586167a9b95fdfd6eebb8b9f7bfd5cb688ab523ef1e7e512ce5f5bb379e305beb4531dd00c609e755620bd0004
+  data.tar.gz: f79e860f496368e67bf328b3359f5860162801600452482cf4b1c0b06439b20f9fb296d5593159410c1817fff4576c4e03ef526f265dbb5f9aa4d8877e9c558b

data/lib/config/pagy.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Pagy initializer file (5.6.2)
+# Pagy initializer file (5.6.3)
 # Customize only what you really need and notice that the core Pagy works also without any of the following lines.
 # Should you just cherry pick part of this file, please maintain the require-order of the extras
 

data/lib/javascripts/pagy.js

@@ -5,7 +5,7 @@
 // Container of the whole pagy stuff
 function Pagy(){}
 
-Pagy.version = '5.6.2'
+Pagy.version = '5.6.3'
 
 // Used by the waitForMe function
 Pagy.delay = 100

data/lib/pagy/calendar/day.rb

@@ -22,7 +22,7 @@ class Pagy # :nodoc:
 
       # Starting time for the page
       def starting_time_for(page)
-        @initial + (snap(page) * DAY)
+        @initial + (offset_units_for(page) * DAY)
       end
     end
   end

data/lib/pagy/calendar/month_mixin.rb

@@ -13,8 +13,8 @@ class Pagy
       def setup_unit_vars
         super
         @months  = self.class::MONTHS  # number of months in the unit
-        @initial = new_time(@starting.year, starting_month_including(@starting.month))
-        @final   = add_months_to(new_time(@ending.year, starting_month_including(@ending.month)), @months)
+        @initial = unit_starting_time_for(@starting)
+        @final   = add_months_to(unit_starting_time_for(@ending), @months)
         @pages   = @last = (months_in(@final) - months_in(@initial)) / @months
         @from    = starting_time_for(@page)
         @to      = add_months_to(@from, @months)
@@ -22,16 +22,18 @@ class Pagy
 
       # Starting time for the page
       def starting_time_for(page)
-        add_months_to(@initial, snap(page) * @months)
-      end
-
-      # Starting month of the unit including the passed month
-      def starting_month_including(month)
-        (@months * ((month - 1) / @months)) + 1  # remove 1 month for 0-11 calculations and add it back for 1-12 conversion
+        add_months_to(@initial, offset_units_for(page) * @months)
       end
 
       private
 
+      # Unit starting time for time
+      def unit_starting_time_for(time)
+        # remove 1 month for 0-11 calculations and add it back for 1-12 conversion
+        starting_month = (@months * ((time.month - 1) / @months)) + 1
+        new_time(time.year, starting_month)
+      end
+
       # Number of months in time
       def months_in(time)
         (time.year * 12) + time.month

data/lib/pagy/calendar/week.rb

@@ -15,8 +15,8 @@ class Pagy # :nodoc:
       def setup_unit_vars
         setup_vars(offset: 0)
         super
-        @initial = week_start(@starting)
-        @final   = week_start(@ending) + WEEK
+        @initial = unit_starting_time_for(@starting)
+        @final   = unit_starting_time_for(@ending) + WEEK
         @pages   = @last = (@final - @initial).to_i / WEEK
         @from    = starting_time_for(@page)
         @to      = @from + WEEK
@@ -24,15 +24,15 @@ class Pagy # :nodoc:
 
       # Starting time for the page
       def starting_time_for(page)
-        @initial + (snap(page) * WEEK)
+        @initial + (offset_units_for(page) * WEEK)
       end
 
       private
 
-      # Return the start of the week for time
-      def week_start(time)
-        start = time - (((time.wday - @offset) * DAY) % WEEK)
-        new_time(start.year, start.month, start.day)
+      # 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)
       end
     end
   end

data/lib/pagy/calendar/year.rb

@@ -22,7 +22,7 @@ class Pagy # :nodoc:
 
       # Starting time for the page
       def starting_time_for(page)
-        new_time(@initial.year + snap(page))
+        new_time(@initial.year + offset_units_for(page))
       end
     end
   end

data/lib/pagy/calendar.rb

@@ -39,11 +39,6 @@ class Pagy # :nodoc:
       localize(starting_time_for(page.to_i), opts)  # page could be a string
     end
 
-    # Period of the active page (used for nested units)
-    def active_period
-      [[@starting, @from].max, [@to - 1, @ending].min] # -1 sec: include only last unit day
-    end
-
     protected
 
     # Base class method for the setup of the unit variables
@@ -63,8 +58,9 @@ class Pagy # :nodoc:
       time.strftime(opts[:format])
     end
 
-    # Simple trick to snap the page to its ordered start, without actually reordering anything in the internal structure
-    def snap(page)
+    # 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
     end
 
@@ -73,6 +69,11 @@ class Pagy # :nodoc:
       Time.new(year, month, day, 0, 0, 0, @utc_offset)
     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
+
     class << self
       # Create a subclass instance by unit name (internal use)
       def create(unit, vars)

data/lib/pagy/extras/calendar.rb

@@ -12,12 +12,7 @@ class Pagy # :nodoc:
 
       private
 
-      # Take a collection and a conf Hash with keys in [:year, :quarter, :month: week, :day, :pagy: :active];
-      # The calendar is active by default, but it can be explicitly inactivated with `active: false`
-      # Return a hash with 3 items:
-      # 0. Array of pagy calendar unit objects
-      # 1. Pagy object
-      # 2. Array of results
+      # Take a collection and a conf Hash with keys in CONF_KEYS and return an array with 3 items: [calendar, pagy, results]
       def pagy_calendar(collection, conf)
         unless conf.is_a?(Hash) && (conf.keys - CONF_KEYS).empty? && conf.all? { |k, v| v.is_a?(Hash) || k == :active }
           raise ArgumentError, "keys must be in #{CONF_KEYS.inspect} and object values must be Hashes; got #{conf.inspect}"
@@ -25,8 +20,8 @@ class Pagy # :nodoc:
 
         conf[:pagy]          = {} unless conf[:pagy]  # use default Pagy object when omitted
         calendar, collection = pagy_setup_calendar(collection, conf) unless conf.key?(:active) && !conf[:active]
-        pagy, result         = send(conf[:pagy][:backend] || :pagy, collection, conf[:pagy])  # use backend: :pagy when omitted
-        [calendar, pagy, result]
+        pagy, results        = send(conf[:pagy][:backend] || :pagy, collection, conf[:pagy])  # use backend: :pagy when omitted
+        [calendar, pagy, results]
       end
 
       # Setup and return the calendar objects and the filtered collection
@@ -34,41 +29,34 @@ class Pagy # :nodoc:
         units      = Calendar::UNITS & conf.keys # get the units in time length desc order
         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}"
+          unit_page_param         = :"#{unit}_#{page_param}"
           conf[unit][:page_param] = unit_page_param
           conf[unit][:page]       = params[unit_page_param]
         end
         calendar = {}
-        period   = pagy_calendar_period(collection)
+        last_obj = 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 do |params|  # delete page_param from the sub-units
-                                  params_to_delete.each { |p| params.delete(p.to_s) } # Hash#except missing from 2.5 baseline
+                                  params_to_delete.each { |p| params.delete(p.to_s) } # Hash#except missing from ruby 2.5 baseline
                                   params
                                 end
-          conf[unit][:period] = period
-          calendar[unit]      = Calendar.send(:create, unit, conf[unit])
-          period              = calendar[unit].active_period # set the period for the next unit
+          conf[unit][:period] = last_obj&.send(:active_period) || pagy_calendar_period(collection)
+          calendar[unit]      = last_obj = Calendar.send(:create, unit, conf[unit])
         end
-        [calendar, pagy_calendar_filter(collection, calendar[units.last].from, calendar[units.last].to)]
+        [calendar, pagy_calendar_filter(collection, last_obj.from, last_obj.to)]
       end
 
-      # This method must be implemented by the application.
-      # It must return the the starting and ending local Time objects defining the calendar :period
+      # This method must be implemented by the application
       def pagy_calendar_period(*)
-        # return_period_array_using(collection)
-        raise NoMethodError, 'the pagy_calendar_period method must be implemented by the application and must return ' \
-                             'the starting and ending local Time objects array defining the calendar :period'
+        raise NoMethodError, 'the pagy_calendar_period method must be implemented by the application ' \
+                             '(see https://ddnexus.github.io/pagy/extras/calendar#pagy_calendar_periodcollection)'
       end
 
-      # This method must be implemented by the application.
-      # It receives the main collection and must return a filtered version of it.
-      # The filter logic must be equivalent to {storage_time >= from && storage_time < to}
+      # This method must be implemented by the application
       def pagy_calendar_filter(*)
-        # return_filtered_collection_using(collection, from, to)
-        raise NoMethodError, 'the pagy_calendar_filter method must be implemented by the application and must return the ' \
-                             'collection filtered by a logic equivalent to '\
-                             '{storage_time >= from && storage_time < to}'
+        raise NoMethodError, 'the pagy_calendar_filter method must be implemented by the application ' \
+                             '(see https://ddnexus.github.io/pagy/extras/calendar#pagy_calendar_filtercollection-from-to)'
       end
     end
   end

data/lib/pagy/extras/gearbox.rb

@@ -28,11 +28,11 @@ class Pagy # :nodoc:
       @pages = @last = (if @count > (sum = gearbox_items.sum)
                           [((@count - sum).to_f / gearbox_items.last).ceil, 1].max + gearbox_items.count
                         else
-                          pages    = 0
-                          reminder = @count
-                          while reminder.positive?
-                            pages    += 1
-                            reminder -= gearbox_items[pages - 1]
+                          pages     = 0
+                          remainder = @count
+                          while remainder.positive?
+                            pages     += 1
+                            remainder -= gearbox_items[pages - 1]
                           end
                           [pages, 1].max
                         end)

data/lib/pagy.rb

@@ -5,7 +5,7 @@ require 'pathname'
 
 # Core class
 class Pagy
-  VERSION = '5.6.2'
+  VERSION = '5.6.3'
 
   # Root pathname to get the path of Pagy files like templates or dictionaries
   def self.root

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pagy
 version: !ruby/object:Gem::Version
-  version: 5.6.2
+  version: 5.6.3
 platform: ruby
 authors:
 - Domizio Demichelis
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-11-29 00:00:00.000000000 Z
+date: 2021-11-30 00:00:00.000000000 Z
 dependencies: []
 description: Agnostic pagination in plain ruby. It does it all. Better.
 email: