lib/pagy.rb in pagy-0.4.1 vs lib/pagy.rb in pagy-0.4.2

@@ -5,11 +5,11 @@
 # items-per-page-limit, etc.), does some simple aritmetic and creates a very
 # small object (~3k) with all is needed for pagination and navigation.
 # Notice that it doesn't actually do any pagination, nor navigation... that is
 # done with a few helpers in the Pagy::Backend and Pagy::Frontend modules.
 
-class Pagy ; VERSION = '0.4.1'
+class Pagy ; VERSION = '0.4.2'
 
   autoload :Backend,  'pagy/backend'
   autoload :Frontend, 'pagy/frontend'
 
   class OutOfRangeError < StandardError; end
@@ -37,22 +37,22 @@
       instance_variable_set(:"@#{k}", @opts.delete(k))                    # set all the metrics variables
     end
     @pages   = @last = [(@count.to_f / @limit).ceil, 1].max               # cardinal and ordinal meanings
     (1..@last).cover?(@page) || raise(OutOfRangeError, "expected :page in 1..#{@last}; got #{@page.inspect}")
     @offset += @limit * (@page - 1)                                       # initial offset + offset for pagination
-    @limit   = @count % @limit if @page == @last                          # adjust limit for last page (for pre-limit(ed) collection)
-    @from    = @count.zero? ? 0 : @offset+1                               # page begins from item
+    @limit   = @count % @limit if @page == @last                          # adjust limit for last page (for pre-limit(ed) collections)
+    @from    = @count == 0 ? 0 : @offset+1                                # page begins from item
     @to      = @offset + @limit                                           # page ends to item
     @prev    = (@page-1 unless @page == 1)                                # nil if no prev page
     @next    = (@page+1 unless @page == @last)                            # nil if no next page
     @series  = []                                                         # e.g. [1, :gap, 7, 8, "9", 10, 11, :gap, 36]
     all      = (0..@last+1)                                               # page range with boundaries
     around   = ([@page-@before, 1].max .. [@page+@after, @last].min).to_a # before..after pages
     row      = (all.first(@initial+1) | around | all.last(@final+1)).sort # row of pages with boundaries
     row.each_cons(2) do |a, b|                                            # loop in consecutive pairs
-      if    a+1 == b; @series.push(a)                                     # no gap     -> no additions
-      elsif a+2 == b; @series.push(a, a+1)                                # 1 page gap -> fill with missing page
-      else            @series.push(a, :gap)                               # n page gap -> add :gap
+      if    a+1 == b ; @series.push(a)                                    # no gap     -> no additions
+      elsif a+2 == b ; @series.push(a, a+1)                               # 1 page gap -> fill with missing page
+      else             @series.push(a, :gap)                              # n page gap -> add :gap
       end                                                                 # skip the end-boundary (last+1)
     end
     @series.shift                                                         # remove the start-boundary (0)
     @series[@series.index(@page)] = @page.to_s                            # convert the current page to String
   end