geared_pagination 1.0.1 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c3e2956267a150a716db8afc9a1e0a9b03ac14906162faa283d5de88624b2f21
-  data.tar.gz: 85222f625262561aafae1025403375f21adc4fb3b78d750d68929183dcd2269b
+  metadata.gz: 6c0df2ffda60431042065c8239bc92687ff300db3d4e6ffaa0f711334004ce7b
+  data.tar.gz: 5dbc61434adb9fe75618fb3e0ab075eea9abf90867de2e3f104cf1435b10a91e
 SHA512:
-  metadata.gz: ebaba73472a04348a60e2e73b679dc4040f39e9b2193816bdda3c111c3a6dc9d16bcf944d4a4adf3c41b9636f6f4d194d21137dcf652f727e3223ae0576b21b1
-  data.tar.gz: 657c6f6d8d0ee877910fb0f394e5229928325af59c471aa48c8bcddb34c75a824a5c92a128bb60757d2178205c914e3aaf3c2c96adcff036fba0098a30c7e3fe
+  metadata.gz: 1a789e055a1a1a6686e64dbd2bffc91e39caca801dfaee071fe6a1afd9a445fc006b01b4df2917dd1befaafb26278331ddadce9d7f2c2714b42c4f98e394632b
+  data.tar.gz: 0d635bfa95bbedcb9e34111413d60d1fb486dcbc80d2a2567921525fc2170b8e637a4bd65ef9cc2235ed2446c9c3ecfa3060e6ce45cae662eed5c0ec6b785f16

data/.gitignore

@@ -1,3 +1,4 @@
+*.gem
 .byebug_history
 
 test/dummy/db/*.sqlite3

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    geared_pagination (1.0.1)
+    geared_pagination (1.0.2)
       activesupport (>= 5.0)
       addressable (>= 2.5.0)
 
@@ -89,7 +89,7 @@ GEM
     nio4r (2.5.2)
     nokogiri (1.10.9)
       mini_portile2 (~> 2.4.0)
-    public_suffix (4.0.5)
+    public_suffix (4.0.6)
     rack (2.2.2)
     rack-test (1.1.0)
       rack (>= 1.0, < 3)

data/geared_pagination.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name     = 'geared_pagination'
-  s.version  = '1.0.1'
+  s.version  = '1.0.2'
   s.authors  = 'David Heinemeier Hansson'
   s.email    = 'david@basecamp.com'
   s.summary  = 'Paginate Active Record sets at variable speeds'

data/lib/geared_pagination/portions/portion_at_cursor.rb

@@ -34,7 +34,7 @@ module GearedPagination
       end
 
       def orderings
-        orders.to_h { |order| [ order.attribute, order.direction ] }
+        orders.map { |order| [ order.attribute, order.direction ] }.to_h
       end
 
       def limit

data/lib/geared_pagination/portions/portion_at_offset.rb

@@ -17,7 +17,10 @@ module GearedPagination
     end
 
     def offset
-      (page_number - 1).times.sum { |index| ratios[index + 1] }
+      variable = [(page_number - 1), ratios.size - 1].min.times.sum { |index| ratios[index + 1] }
+      fixed = [page_number - ratios.size, 0].max * ratios.fixed
+
+      variable + fixed
     end
 
     def next_param(*)

data/lib/geared_pagination/ratios.rb

@@ -3,15 +3,23 @@ module GearedPagination
     DEFAULTS = [ 15, 30, 50, 100 ]
 
     def initialize(ratios = nil)
-      @ratios = Array(ratios || DEFAULTS)
+      @ratios = Array(ratios || DEFAULTS).map(&:to_i)
     end
 
     def [](page_number)
-      @ratios[page_number - 1] || @ratios.last
+      @ratios[page_number - 1] || fixed
     end
 
     def cache_key
       @ratios.join('-')
     end
+
+    def size
+      @ratios.size
+    end
+
+    def fixed
+      @ratios.last
+    end
   end
 end

data/test/controller_test.rb

@@ -14,6 +14,9 @@ class GearedPagination::ControllerTest < ActionController::TestCase
     assert_equal etag_for("placeholder", "page/1:1-2"), response.etag
     etag_before_gearing_change = response.etag
 
+    get :index, params: { page: 2, per_page: [ 1, 2 ] }
+    assert_equal etag_for("placeholder", "page/2:1-2"), response.etag
+
     get :index, params: { page: 1, per_page: [ 1, 2 ] }
     assert_equal etag_before_gearing_change, response.etag
 

data/test/portion_at_offset_test.rb

@@ -7,6 +7,9 @@ class GearedPagination::PortionAtOffsetTest < ActiveSupport::TestCase
     assert_equal 0, GearedPagination::PortionAtOffset.new(page_number: 1).offset
     assert_equal GearedPagination::Ratios::DEFAULTS.first, GearedPagination::PortionAtOffset.new(page_number: 2).offset
     assert_equal GearedPagination::Ratios::DEFAULTS.first + GearedPagination::Ratios::DEFAULTS.second, GearedPagination::PortionAtOffset.new(page_number: 3).offset
+    assert_equal 4.times.sum { |index| GearedPagination::Ratios::DEFAULTS[index] || GearedPagination::Ratios::DEFAULTS.last }, GearedPagination::PortionAtOffset.new(page_number: 5).offset
+    assert_equal 5.times.sum { |index| GearedPagination::Ratios::DEFAULTS[index] || GearedPagination::Ratios::DEFAULTS.last }, GearedPagination::PortionAtOffset.new(page_number: 6).offset
+    assert_equal 9.times.sum { |index| GearedPagination::Ratios::DEFAULTS[index] || GearedPagination::Ratios::DEFAULTS.last }, GearedPagination::PortionAtOffset.new(page_number: 10).offset
   end
 
   test "limit" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: geared_pagination
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - David Heinemeier Hansson
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-05-28 00:00:00.000000000 Z
+date: 2020-12-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -52,7 +52,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.12'
-description: 
+description:
 email: david@basecamp.com
 executables: []
 extensions: []
@@ -162,7 +162,7 @@ homepage: https://github.com/basecamp/geared_pagination
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -177,8 +177,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.0.3
+signing_key:
 specification_version: 4
 summary: Paginate Active Record sets at variable speeds
 test_files: