simply_paginate 0.0.1 → 0.0.2

data/README.md

@@ -1,6 +1,6 @@
 # SimplyPaginate
 
-TODO: Write a gem description
+Simply paginate will do just that, give me a collection and you will be able to use a pagination logic (no extra html boilerplate or dependecies).
 
 ## Installation
 
@@ -28,7 +28,7 @@ collection = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10]
 
 paginator = SimplyPaginate::Paginator.new(collection)
 
-# create pages with max of 5 elements
+# create pages with max of 3 elements
 paginator.paginate 3
 
 # retrieve a certain page
@@ -38,8 +38,8 @@ paginator[0]
 paginator[0].next
 paginator[0].previous
 
-#you might want the first element on the 2nd page
-paginator[0].next.next.next.previous[0]
+#you might want the first element on the 3rd page
+paginator[0].next.next.next.previous.elements[0]
 
 #or maybe all of them
 paginator[0].next.next.next.previous.elements
@@ -47,10 +47,13 @@ paginator[0].next.next.next.previous.elements
 
 ## Currently working on:
 
-* Moving from a copy algorithm to a more performant pagination using offsets.
 * Adding support for common ORMs like: ActiveRecord, Sequel and DataMapper.
 * Testing...
 
+## Release 0.0.2 features:
+* Dropping the each_slice in order to use a more memory efficient index algorithm for paging.
+* Adding more tests to the page object.
+
 ## Contributing
 
 1. Fork it

data/lib/simply_paginate/page.rb

@@ -1,39 +1,37 @@
 module SimplyPaginate
   class Page
     attr_reader :next_page, :previous_page
-    
-    def initialize(collection, previous_page = nil, next_page = nil)
-      @collection = collection
+
+    def initialize(first, last, paginator, previous_page = nil, next_page = nil)
+      @first = first
+      @last = last
+      @paginator = paginator
       @next_page = next_page
       @previous_page = previous_page
     end
-    
-    def [](pos)
-      @collection[pos]
-    end
-    
+
     def current
       self
     end
-    
+
     def next
       @next_page
     end
-    
+
     def previous
       @previous_page
     end
-    
+
     def next=(v)
       @next_page = v
     end
-    
+
     def previous=(v)
       @previous_page = v
     end
-    
+
     def elements
-      @collection
+      @paginator.collection[@first..@last]
     end
   end
-end
+end

data/lib/simply_paginate/paginator.rb

@@ -1,28 +1,33 @@
 module SimplyPaginate
   class Paginator
-    attr_reader :pages
-    
+    attr_reader :pages, :collection
+
     def initialize(collection)
       @collection = collection
       @pages = []
     end
-    
+
     def paginate(limit)
       @pages = []
-            
-      @collection.each_slice(limit) do |slice|
-        page = Page.new(slice, @pages[@pages.count - 1])
-        
+      first = last = 0
+      count = limit - 1
+      final_index = @collection.count - 1
+
+      (@collection.count.to_f / limit.to_f).ceil.times do
+        last = (last <= final_index) ? first + count : final_index
+
+        page = Page.new(first, last, self, @pages[@pages.count - 1])
         @pages[@pages.count - 1].next = page unless @pages[@pages.count - 1].nil?
-        
+
         @pages << page
+        first = last + 1
       end
-      
+
       self
     end
-    
+
     def [](pos)
       @pages[pos]
     end
   end
-end
+end

data/lib/simply_paginate/version.rb

@@ -1,3 +1,3 @@
 module SimplyPaginate
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/spec/page_spec.rb

@@ -1,5 +1,58 @@
 require 'spec_helper'
 
 describe SimplyPaginate::Page do
+  before do
+    @paginator = SimplyPaginate::Paginator.new([1, 2, 3, 4, 5, 6, 7, 8, 9, 10]).paginate 5
+    @first_page = SimplyPaginate::Page.new(0, 4, @paginator)
+    @last_page = SimplyPaginate::Page.new(5, 9, @paginator, @first_page)
+    @first_page.next = @last_page
 
-end
+    class SimplyPaginate::Page
+      def paginator
+        @paginator
+      end
+    end
+  end
+
+  it "must always have an instance of paginator" do
+    @first_page.paginator.wont_be_nil
+    @last_page.paginator.wont_be_nil
+
+    @first_page.next.paginator.wont_be_nil
+    @first_page.next.previous.paginator.wont_be_nil
+  end
+
+  it "must be able to get the elements" do
+    @first_page.elements.must_equal [1, 2, 3, 4, 5]
+    @last_page.elements.must_equal [6, 7, 8, 9, 10]
+
+    @first_page.next.elements.must_equal [6, 7, 8, 9, 10]
+    @last_page.previous.elements.must_equal [1, 2, 3, 4, 5]
+  end
+
+  describe "when having next and previous pages" do
+    before do
+      @paginator.paginate 2
+    end
+
+    it "must be able to go back and foward" do
+      @paginator[1].next.must_equal @paginator[2]
+      @paginator[1].previous.must_equal @paginator[0]
+    end
+  end
+
+  describe "when having previous and no next" do
+    it "must be able to go back " do
+      @last_page.next.must_be_nil
+      @last_page.previous.must_equal @first_page
+    end
+  end
+
+  describe "when having next and no previous" do
+    it "must be able to go back " do
+      @first_page.next.must_equal @last_page
+      @first_page.previous.must_be_nil
+    end
+  end
+
+end

data/spec/paginator_spec.rb

@@ -4,54 +4,54 @@ describe SimplyPaginate::Paginator do
   before do
     @paginator = SimplyPaginate::Paginator.new [1, 2, 3, 4, 5, 6, 7, 8, 9, 10]
   end
-  
+
   describe 'before paginating' do
     it 'must be able to paginate' do
       @paginator.paginate(3).pages.count.must_equal 4
     end
-    
+
     it 'must not be able to retrieve a certain page' do
       @paginator[0].must_be_nil
     end
-    
+
     it 'must not be able to retrieve all pages' do
       @paginator.pages.must_equal []
     end
   end
-  
+
   describe 'after paginating' do
     before do
       @paginator.paginate(5)
-      
-      @first_page = SimplyPaginate::Page.new([1, 2, 3, 4, 5], nil)
-      @last_page = SimplyPaginate::Page.new([6, 7, 8, 9, 10], @first_page, nil)
+
+      @first_page = SimplyPaginate::Page.new(0, 4, @paginator)
+      @last_page = SimplyPaginate::Page.new(5, 9, @paginator, @first_page, nil)
       @first_page.next = @last_page
-      
+
       @pages = [@first_page, @last_page]
     end
-    
+
     it 'must be able to paginate' do
       @paginator.paginate(3).pages.count.must_equal 4
     end
-    
+
     it 'must be able to retrieve a certain page' do
       @paginator[0].elements.must_equal @first_page.elements
       @paginator[0].previous.must_be_nil
       @paginator[0].next.wont_be_nil
     end
-    
+
     it 'must be able to retrieve all pages' do
-      @paginator.pages.count 2 
-      
+      @paginator.pages.count 2
+
       #test first element
       @paginator[0].elements.must_equal @first_page.elements
       @paginator[0].previous.must_be_nil
       @paginator[0].next.wont_be_nil
-      
+
       #test second element
       @paginator[1].elements.must_equal @last_page.elements
       @paginator[1].previous.wont_be_nil
       @paginator[1].next.must_be_nil
     end
   end
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: simply_paginate
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-22 00:00:00.000000000 Z
+date: 2013-04-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler