pagy 5.6.0 → 5.6.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4a62cb4c6decfdc1013da6d14b38220aa361246637301a8d92b2d781440397bc
-  data.tar.gz: 2555195198def80d6ec002928da8cef07419a76da9a196d8971d50e7fb5095e2
+  metadata.gz: 5436ee5b008a1f9db93ba52363fbae559f6444600c5925fcd87541b606e2a8ad
+  data.tar.gz: 6159c1b56724cf1e10af6c15c2a141200d1795ed3da6fa6655cb1e7ee3f7056c
 SHA512:
-  metadata.gz: f665f63cb281c25d6fcf430c12888def46ea46911e45a566fd52cb4943763efe775b094015d3b658b8322ade23a9a43a5888c60f10471bd567f000d5a67352c7
-  data.tar.gz: 8c22ab2ad5efc0bd91b648d89217d1b71d77fbf669e1153b015954b79c2506c8f5ffe29d80e7d5988bea76b4bed35e7a76bb9c3657f60de0db3cde84b38e62e2
+  metadata.gz: 6d2e0aa3e7a4a9bfee7b4360b1d6ae35d931a1aa7fd4959436adf78cb43c153149ebb80455d9a63f7ef36558c5d673191fcd5ad10ce7e4be2e75ca42be645d05
+  data.tar.gz: ee07cd90d3908f52c879cebf1956ee5cb1c12739b21e4a0519e7a8432e3ead0304a2bca94ca6a9444035e72563e2de2dd4a5191c279183fad39d22b7703babae

data/lib/config/pagy.rb

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
-# Pagy initializer file (5.6.0)
-# Customize only what you really need and notice that Pagy works also without any of the following lines.
+# Pagy initializer file (5.6.4)
+# 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
 
 
 # Pagy DEFAULT Variables
 # See https://ddnexus.github.io/pagy/api/pagy#variables
-# All the Pagy::DEFAULT are set for all the Pagy instances but can be overridden per instance
-# by just passing them to Pagy.new|Pagy::Countless.new|Pagy::Calendar.new or the #pagy controller method
+# All the Pagy::DEFAULT are set for all the Pagy instances but can be overridden per instance by just passing them to
+# Pagy.new|Pagy::Countless.new|Pagy::Calendar::*.new or any of the #pagy* controller methods
 
 
 # Instance variables
@@ -40,24 +40,27 @@
 # See https://ddnexus.github.io/pagy/extras/array
 # require 'pagy/extras/array'
 
-# Calendar extra: Add pagination filtering by calendar time unit (Year, Month, Week Day)
+# Calendar extra: Add pagination filtering by calendar time unit (year, quarter, month, week, day)
 # See https://ddnexus.github.io/pagy/extras/calendar
 # require 'pagy/extras/calendar'
 # Default for each unit
-# Pagy::Calendar::Year::DEFAULT[:order]   = :asc        # Time direction of pagination
-# Pagy::Calendar::Year::DEFAULT[:format]  = '%Y'        # strftime format
+# Pagy::Calendar::Year::DEFAULT[:order]     = :asc        # Time direction of pagination
+# Pagy::Calendar::Year::DEFAULT[:format]    = '%Y'        # strftime format
 #
-# Pagy::Calendar::Month::DEFAULT[:order]  = :asc        # Time direction of pagination
-# Pagy::Calendar::Month::DEFAULT[:format] = '%Y-%m'     # strftime format
+# Pagy::Calendar::Quarter::DEFAULT[:order]  = :asc        # Time direction of pagination
+# Pagy::Calendar::Quarter::DEFAULT[:format] = '%Y-Q%q'    # strftime format
 #
-# Pagy::Calendar::Week::DEFAULT[:order]   = :asc        # Time direction of pagination
-# Pagy::Calendar::Week::DEFAULT[:format]  = '%Y-%W'     # strftime format
-# Pagy::Calendar::Week::DEFAULT[:offset]  = 0           # Day offset from Sunday (0: Sunday; 1: Monday;... 6: Saturday)
+# Pagy::Calendar::Month::DEFAULT[:order]    = :asc        # Time direction of pagination
+# Pagy::Calendar::Month::DEFAULT[:format]   = '%Y-%m'     # strftime format
 #
-# Pagy::Calendar::Day::DEFAULT[:order]    = :asc        # Time direction of pagination
-# Pagy::Calendar::Day::DEFAULT[:format]   = '%Y-%m-%d'  # strftime format
+# Pagy::Calendar::Week::DEFAULT[:order]     = :asc        # Time direction of pagination
+# Pagy::Calendar::Week::DEFAULT[:format]    = '%Y-%W'     # strftime format
+# Pagy::Calendar::Week::DEFAULT[:offset]    = 0           # Day offset from Sunday (0: Sunday; 1: Monday;... 6: Saturday)
 #
-# Uncomment the following block, if you need calendar localization without using the I18n extra
+# Pagy::Calendar::Day::DEFAULT[:order]      = :asc        # Time direction of pagination
+# Pagy::Calendar::Day::DEFAULT[:format]     = '%Y-%m-%d'  # strftime format
+#
+# Uncomment the following lines, if you need calendar localization without using the I18n extra
 # module LocalizePagyCalendar
 #   def localize(time, opts)
 #     ::I18n.l(time, **opts)
@@ -97,8 +100,8 @@
 # you must require the shared internal extra (BEFORE the metadata extra) ONLY if you need also the :sequels
 # require 'pagy/extras/shared'
 # require 'pagy/extras/metadata'
-# For performance reason, you should explicitly set ONLY the metadata you use in the frontend
-# Pagy::DEFAULT[:metadata] = %i[scaffold_url count page prev next last]   # example
+# For performance reasons, you should explicitly set ONLY the metadata you use in the frontend
+# Pagy::DEFAULT[:metadata] = %i[scaffold_url page prev next last]   # example
 
 # Searchkick extra: Paginate `Searchkick::Results` objects
 # See https://ddnexus.github.io/pagy/extras/searchkick
@@ -209,18 +212,18 @@
 #
 # load the "de", "en" and "es" built-in locales:
 # (the first passed :locale will be used also as the default_locale)
-# Pagy::I18n.load({locale: 'de'},
-#                 {locale: 'en'},
-#                 {locale: 'es'})
+# Pagy::I18n.load({ locale: 'de' },
+#                 { locale: 'en' },
+#                 { locale: 'es' })
 #
 # load the "en" built-in locale, a custom "es" locale,
 # and a totally custom locale complete with a custom :pluralize proc:
 # (the first passed :locale will be used also as the default_locale)
-# Pagy::I18n.load({locale: 'en'},
-#                 {locale: 'es', filepath: 'path/to/pagy-es.yml'},
-#                 {locale: 'xyz',  # not built-in
-#                  filepath: 'path/to/pagy-xyz.yml',
-#                  pluralize: lambda{ |count| ... } )
+# Pagy::I18n.load({ locale: 'en' },
+#                 { locale: 'es', filepath: 'path/to/pagy-es.yml' },
+#                 { locale: 'xyz',  # not built-in
+#                   filepath: 'path/to/pagy-xyz.yml',
+#                   pluralize: lambda{ |count| ... } )
 
 
 # I18n extra: uses the standard i18n gem which is ~18x slower using ~10x more memory

data/lib/javascripts/pagy.js

@@ -2,10 +2,10 @@
 
 // This code should be OK also with very old browsers
 
-// Container of the whole pagy stuff
+// Container for the whole pagy stuff
 function Pagy(){}
 
-Pagy.version = '5.6.0'
+Pagy.version = '5.6.4'
 
 // Used by the waitForMe function
 Pagy.delay = 100
@@ -54,6 +54,7 @@ Pagy.nav =
             else if (typeof(item) === 'string') { html += fill(tags.active, item, label) }
           }
           html += tags.after
+          this.innerHTML = ''
           this.insertAdjacentHTML('afterbegin', html)
           lastWidth = width
         }

data/lib/locales/ta.yml

@@ -0,0 +1,22 @@
+# :one_other pluralization (see https://github.com/ddnexus/pagy/blob/master/lib/locales/utils/p11n.rb)
+
+ta:
+  pagy:
+
+    item_name:
+      one: "பதிவு"
+      other: "பதிவுகள்"
+
+    nav:
+      prev: "‹ முந்தையது"
+      next: "அடுத்தது ›"
+      gap: "…"
+
+    info:
+      no_items: "%{item_name} கிடைக்கவில்லை"
+      single_page: "<b>%{count}</b> %{item_name} காட்டப்படுகின்றது"
+      multiple_pages: "மொத்தம் <b>%{count}</b> %{item_name}, காட்டப்படுபவை <b>%{from}-%{to}</b>"
+
+    combo_nav_js: "<label>%{pages}-இல் %{page_input}-வது பக்கம்</label>"
+
+    items_selector_js: "<label>ஒரு பக்கத்திற்கு %{items_input} %{item_name} காட்டு</label>"

data/lib/pagy/calendar/day.rb

@@ -16,13 +16,13 @@ class Pagy # :nodoc:
         @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
-        @from    = start_for(@page)
+        @from    = starting_time_for(@page)
         @to      = @from + DAY
       end
 
-      # Time for the page
-      def start_for(page)
-        @initial + (snap(page) * DAY)
+      # Starting time for the page
+      def starting_time_for(page)
+        @initial + (offset_units_for(page) * DAY)
       end
     end
   end

data/lib/pagy/calendar/month_mixin.rb

@@ -5,7 +5,7 @@ class Pagy
     # Mixin for month based unit periods
     # It is used for month and quarter, but you could use it to implement less common unit of 6, 4, 2 months
     # (see the https://ddnexus.github.io/pagy/api/calendar#custom-units sections for details).
-    # The class that includes it needs to set the MONTH duration for the unit and the usual DEFAULT.
+    # The including class must set the MONTHS duration for the unit and the usual DEFAULT.
     module MonthMixin
       protected
 
@@ -13,36 +13,36 @@ class Pagy
       def setup_unit_vars
         super
         @months  = self.class::MONTHS  # number of months in the unit
-        @initial = new_time(@starting.year, beginning_month(@starting.month))
-        @final   = add_to(new_time(@ending.year, beginning_month(@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    = start_for(@page)
-        @to      = add_to(@from, @months)
+        @from    = starting_time_for(@page)
+        @to      = add_months_to(@from, @months)
       end
 
-      # Time for the page
-      def start_for(page)
-        add_to(@initial, snap(page) * @months)
-      end
-
-      # Return the beginning month for the unit (e.g. quarter) that includes the month argument
-      def beginning_month(month)
-        (@months * ((month - 1) / @months)) + 1
+      # Starting time for the page
+      def starting_time_for(page)
+        add_months_to(@initial, offset_units_for(page) * @months)
       end
 
       private
 
-      # Months in time
+      # 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
       end
 
       # Add months to time
-      def add_to(time, months)
-        months += months_in(time)
-        year  = months / 12
-        month = months % 12
-        month.zero? ? new_time(year - 1, 12) : new_time(year, month)
+      def add_months_to(time, months)
+        months += months_in(time) - 1             # remove 1 month for 0-11 calculations
+        new_time(months / 12, (months % 12) + 1)  # add 1 month back for 1-12 conversion
       end
     end
   end

data/lib/pagy/calendar/quarter.rb

@@ -8,9 +8,16 @@ class Pagy # :nodoc:
     # Calendar quarter subclass
     class Quarter < Calendar
       DEFAULT = { order:  :asc,      # rubocop:disable Style/MutableConstant
-                  format: '%Y-Q%q' }
+                  format: '%Y-Q%q' } # '%q' token
       MONTHS  = 3  # number of months of the unit
       include MonthMixin
+
+      # 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 }
+        localize(starting_time, opts)
+      end
     end
   end
 end

data/lib/pagy/calendar/week.rb

@@ -15,24 +15,24 @@ 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    = start_for(@page)
+        @from    = starting_time_for(@page)
         @to      = @from + WEEK
       end
 
-      # Time for the page
-      def start_for(page)
-        @initial + (snap(page) * WEEK)
+      # Starting time for the page
+      def starting_time_for(page)
+        @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

@@ -16,13 +16,13 @@ class Pagy # :nodoc:
         @initial = new_time(@starting.year)
         @final   = new_time(@ending.year + 1)
         @pages   = @last = @final.year - @initial.year
-        @from    = start_for(@page)
+        @from    = starting_time_for(@page)
         @to      = new_time(@from.year + 1)
       end
 
-      # Time for the page
-      def start_for(page)
-        new_time(@initial.year + snap(page))
+      # Starting time for the page
+      def starting_time_for(page)
+        new_time(@initial.year + offset_units_for(page))
       end
     end
   end

data/lib/pagy/calendar.rb

@@ -29,21 +29,14 @@ class Pagy # :nodoc:
     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)
+    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)
+    def label_for(page, opts = {})
       opts[:format] ||= @vars[:format]
-      start = start_for(page.to_i)
-      opts[:format] = opts[:format].gsub('%q') { (start.month / 4) + 1 }
-      localize(start, **opts)
-    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
+      localize(starting_time_for(page.to_i), opts)  # page could be a string
     end
 
     protected
@@ -65,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
 
@@ -75,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

@@ -4,7 +4,7 @@
 require 'pagy/calendar'
 
 class Pagy # :nodoc:
-  # Paginate based on calendar periods (year month week day) plus the regular pagination
+  # Add pagination filtering by calendar unit (:year, :quarter, :month, :week, :day) to the regular pagination
   module CalendarExtra
     # Additions for the Backend module
     module Backend
@@ -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/frontend.rb

@@ -76,8 +76,8 @@ class Pagy
 
     # Similar to I18n.t: just ~18x faster using ~10x less memory
     # (@pagy_locale explicitly initialized in order to avoid warning)
-    def pagy_t(key, **opts)
-      Pagy::I18n.t(@pagy_locale ||= nil, key, **opts)
+    def pagy_t(key, opts = {})
+      Pagy::I18n.t(@pagy_locale ||= nil, key, opts)
     end
   end
 end

data/lib/pagy/i18n.rb

@@ -10,12 +10,16 @@ class Pagy
 
     # Pluralization rules
     module P11n
-      # Utility variables
-      from0to1   = [0, 1].freeze
-      from2to4   = [2, 3, 4].freeze
-      from5to9   = [5, 6, 7, 8, 9].freeze
-      from11to14 = [11, 12, 13, 14].freeze
-      from12to14 = [12, 13, 14].freeze
+      # Pluralization variables
+      from0to1   = (0..1).to_a.freeze
+      from2to4   = (2..4).to_a.freeze
+      from3to10  = (3..10).to_a.freeze
+      from5to9   = (5..9).to_a.freeze
+      from11to14 = (11..14).to_a.freeze
+      from11to99 = (11..99).to_a.freeze
+      from12to14 = (12..14).to_a.freeze
+
+      from0to1_from5to9 = from0to1 + from5to9
 
       # Store the proc defining each pluralization RULE
       # Logic adapted from https://github.com/svenfuchs/rails-i18n
@@ -24,12 +28,12 @@ class Pagy
           lambda do |n = 0|
             mod100 = n % 100
             case
-            when n == 0                         then 'zero' # rubocop:disable Style/NumericPredicate
-            when n == 1                         then 'one'
-            when n == 2                         then 'two'
-            when (3..10).to_a.include?(mod100)  then 'few'
-            when (11..99).to_a.include?(mod100) then 'many'
-            else                                     'other'
+            when n == 0                      then 'zero' # rubocop:disable Style/NumericPredicate
+            when n == 1                      then 'one'
+            when n == 2                      then 'two'
+            when from3to10.include?(mod100)  then 'few'
+            when from11to99.include?(mod100) then 'many'
+            else                                  'other'
             end
           end,
 
@@ -68,10 +72,10 @@ class Pagy
             mod10  = n % 10
             mod100 = n % 100
             case
-            when n == 1                                                               then 'one'
-            when from2to4.include?(mod10) && !from12to14.include?(mod100)             then 'few'
-            when (from0to1 + from5to9).include?(mod10) || from12to14.include?(mod100) then 'many'
-            else                                                                           'other'
+            when n == 1                                                           then 'one'
+            when from2to4.include?(mod10) && !from12to14.include?(mod100)         then 'few'
+            when from0to1_from5to9.include?(mod10) || from12to14.include?(mod100) then 'many'
+            else                                                                       'other'
             end
           end,
 
@@ -149,7 +153,7 @@ class Pagy
     end
 
     # Translate and pluralize the key with the locale DATA
-    def t(locale, key, **opts)
+    def t(locale, key, opts = {})
       data, pluralize = DATA[locale]
       translation = data[key] || (opts[:count] && data[key += ".#{pluralize.call(opts[:count])}"]) \
                       or return %([translation missing: "#{key}"])

data/lib/pagy.rb

@@ -5,7 +5,7 @@ require 'pathname'
 
 # Core class
 class Pagy
-  VERSION = '5.6.0'
+  VERSION = '5.6.4'
 
   # Root pathname to get the path of Pagy files like templates or dictionaries
   def self.root
@@ -59,11 +59,11 @@ class Pagy
     series          = []
     start           = 1
     if (left_gap_end - left_gap_start).positive?
-      series.push(*start..(left_gap_start - 1), :gap)
+      series.push(*start...left_gap_start, :gap)
       start = left_gap_end + 1
     end
     if (right_gap_end - right_gap_start).positive?
-      series.push(*start..(right_gap_start - 1), :gap)
+      series.push(*start...right_gap_start, :gap)
       start = right_gap_end + 1
     end
     series.push(*start..@last)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pagy
 version: !ruby/object:Gem::Version
-  version: 5.6.0
+  version: 5.6.4
 platform: ruby
 authors:
 - Domizio Demichelis
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-11-26 00:00:00.000000000 Z
+date: 2021-12-03 00:00:00.000000000 Z
 dependencies: []
 description: Agnostic pagination in plain ruby. It does it all. Better.
 email:
@@ -46,6 +46,7 @@ files:
 - lib/locales/sv-SE.yml
 - lib/locales/sv.yml
 - lib/locales/sw.yml
+- lib/locales/ta.yml
 - lib/locales/tr.yml
 - lib/locales/uk.yml
 - lib/locales/zh-CN.yml