vendor/plugins/will_paginate @@ -4,3 +4,6 @@ [submodule "vendor/plugins/dataset"] path = vendor/plugins/dataset url = git://github.com/jgarber/dataset.git +[submodule "vendor/plugins/will_paginate"] + path = vendor/plugins/will_paginate + url = git://github.com/mislav/will_paginate.git .gitmodules @@ -3,7 +3,8 @@ class Admin::VersionsController < ApplicationController before_filter :add_chronicle_stylesheet, :only => [:index, :show, :summary, :diff] def index - @versions = Version.find(:all, :order => "created_at DESC") + per_page = Radiant::Config['chronicle.history.per_page'] + @versions = Version.paginate(:page => params[:page], :order => 'created_at DESC', :per_page => per_page) end def show app/controllers/admin/versions_controller.rb @@ -20,3 +20,5 @@ = link_to version.instance.title, polymorphic_path([:edit, :admin, version.instance]) %td= version.instance.updated_by ? version.instance.updated_by.name : "" %td= version.created_at + += will_paginate @versions \ No newline at end of file app/views/admin/versions/index.html.haml @@ -12,6 +12,32 @@ describe Admin::VersionsController do get 'index' response.should be_success end + + it "should have some versions" do + get 'index' + assigns['versions'].should_not be_nil + end + + describe "pagination" do + before do + @page = pages(:published) + 100.times do |i| + @page.title = i + @page.save + end + end + + it "should paginate" do + get 'index' + assigns['versions'].size.should == 30 + end + + it "should paginate according to config" do + Radiant::Config['chronicle.history.per_page'] = 50 + get 'index' + assigns['versions'].size.should == 50 + end + end end describe "GET 'show'" do spec/controllers/admin/versions_controller_spec.rb 8352ec68390f9ea9593574d3ef285966760a9ec9 Jason Garber jg@jasongarber.com http://github.com/jgarber/radiant-chronicle-extension/commit/e10336f9bd47985e476a4237de2518cd7865dd52 e10336f9bd47985e476a4237de2518cd7865dd52 2009-08-07T11:51:51-07:00 2009-08-07T11:51:51-07:00 Paginate the history page. Closes #20. cc5afec273b40cb4769c59b54f2157fb3abd054e Jason Garber jg@jasongarber.com