add paging to places list and travels list

Gemfile

@@ -9,11 +9,12 @@ gem 'coffee-rails'
 gem 'dotenv-rails'
 gem 'elasticsearch-model'
 gem 'elasticsearch-rails'
+gem 'geocoder'
 gem 'gon'
 gem 'rails-footnotes', '~> 4.0'
-gem 'geocoder'
 gem 'jbuilder', '~> 2.0'
 gem 'jquery-rails'
+gem 'kaminari'
 gem 'omniauth'
 gem 'omniauth-github'
 gem 'omniauth-twitter'

Gemfile.lock

@@ -115,6 +115,9 @@ GEM
       thor (>= 0.14, < 2.0)
     json (1.8.2)
     jwt (1.2.1)
+    kaminari (0.16.2)
+      actionpack (>= 3.0.0)
+      activesupport (>= 3.0.0)
     kgio (2.9.3)
     launchy (2.4.3)
       addressable (~> 2.3)
@@ -305,6 +308,7 @@ DEPENDENCIES
   guard-minitest
   jbuilder (~> 2.0)
   jquery-rails
+  kaminari
   launchy
   minitest-slow_test
   minitest-sound

app/controllers/places_controller.rb

@@ -3,9 +3,11 @@ class PlacesController < ApplicationController
 
   def index
     @places = if params[:already]
-                Place.mine(current_user).order('updated_at DESC').already_went
+                Place.mine(current_user).order('updated_at DESC')
+                  .already_went.page(params[:page])
               else
-                Place.mine(current_user).order('updated_at DESC').not_gone
+                Place.mine(current_user).order('updated_at DESC')
+                  .not_gone.page(params[:page])
               end
   end
 

app/controllers/travels_controller.rb

@@ -3,7 +3,7 @@ class TravelsController < ApplicationController
   before_action :set_travel_by_owner, only: [:destroy]
 
   def index
-    @travels = Travel.belong(current_user).order('start_date DESC')
+    @travels = Travel.belong(current_user).order('start_date DESC').page(params[:page])
   end
 
   def show

app/models/place.rb

@@ -31,6 +31,8 @@ class Place < ActiveRecord::Base
   validates :latitude, presence: true, if: -> { address.present? }
   validates :longitude, presence: true, if: -> { address.present? }
 
+  paginates_per 10
+
   class << self
     def build(params, user)
       place = new(params)

app/models/travel.rb

@@ -40,6 +40,7 @@ class Travel < ActiveRecord::Base
   validates :owner_id, presence: true
   validates_with TravelDateValidator
 
+  paginates_per 10
 
   class << self
     def build(params, user)

app/views/kaminari/_first_page.html.slim

@@ -0,0 +1,10 @@
+/ Link to the "First" page
+  - available local variables
+    url          : url to the first page
+    current_page : a page object for the currently displayed page
+    total_pages  : total number of pages
+    per_page     : number of items to fetch per page
+    remote       : data-remote
+li
+  == link_to_unless current_page.first?, t('views.pagination.first').html_safe, url, :remote => remote
+'

app/views/kaminari/_gap.html.slim

@@ -0,0 +1,9 @@
+/ Non-link tag that stands for skipped pages...
+  - available local variables
+    current_page : a page object for the currently displayed page
+    total_pages  : total number of pages
+    per_page     : number of items to fetch per page
+    remote       : data-remote
+span.page.gap
+  == t('views.pagination.truncate').html_safe
+'

app/views/kaminari/_last_page.html.slim

@@ -0,0 +1,10 @@
+/ Link to the "Last" page
+  - available local variables
+    url          : url to the last page
+    current_page : a page object for the currently displayed page
+    total_pages  : total number of pages
+    per_page     : number of items to fetch per page
+    remote       : data-remote
+li
+	== link_to_unless current_page.last?, t('views.pagination.last').html_safe, url, :remote => remote
+'

app/views/kaminari/_page.html.slim

@@ -0,0 +1,11 @@
+/ Link showing page number
+  - available local variables
+    page         : a page object for "this" page
+    url          : url to this page
+    current_page : a page object for the currently displayed page
+    total_pages  : total number of pages
+    per_page     : number of items to fetch per page
+    remote       : data-remote
+li class="#{'active' if page.current?}"
+  == link_to page, url, {:remote => remote, :rel => page.next? ? 'next' : page.prev? ? 'prev' : nil}
+'

app/views/kaminari/_paginator.html.slim

@@ -0,0 +1,19 @@
+/ The container tag
+  - available local variables
+    current_page : a page object for the currently displayed page
+    total_pages  : total number of pages
+    per_page     : number of items to fetch per page
+    remote       : data-remote
+    paginator    : the paginator that renders the pagination tags inside
+
+== paginator.render do
+  ul.pagination
+    == first_page_tag unless current_page.first?
+    == prev_page_tag unless current_page.first?
+    - each_page do |page|
+      - if page.left_outer? || page.right_outer? || page.inside_window?
+        == page_tag page
+      - elsif !page.was_truncated?
+        == gap_tag
+    == next_page_tag unless current_page.last?
+    == last_page_tag unless current_page.last?

app/views/places/index.html.slim

@@ -39,3 +39,4 @@ section.page-block.space-top.space-bottom
                     = link_to "参考URL#{i + 1}", url, target: '_blank'
                 br
                 = simple_format(place.memo)
+      = paginate @places

app/views/travels/index.html.slim

@@ -25,3 +25,4 @@ section.page-block.space-top.space-bottom
                 = travel.formatted_schedule
                 br
                 = simple_format(travel.memo)
+      = paginate @travels

config/initializers/kaminari_config.rb

@@ -0,0 +1,10 @@
+Kaminari.configure do |config|
+  config.default_per_page = 10
+  # config.max_per_page = nil
+  config.window = 3
+  # config.outer_window = 0
+  # config.left = 0
+  # config.right = 0
+  # config.page_method_name = :page
+  # config.param_name = :page
+end

config/locales/ja.yml

@@ -216,3 +216,10 @@ ja:
       travel_photo:
         name: 名前
         photo_service_album_id: アルバム名
+  views:
+    pagination:
+      first: "« 先頭"
+      last: "最後 »"
+      previous: "‹ 前"
+      next: "次 ›"
+      truncate: "…"

test/fixtures/places.yml

@@ -33,3 +33,11 @@ shimogamo:
   longitude: 135.45462
   user: google
 
+<% 10.times.each do |i| %>
+places_<%= i %>:
+  name: $LABEL
+  address: 住所 $LABEL
+  latitude: 35.07182
+  longitude: 135.45462
+  user: google
+<% end %>

test/fixtures/travels.yml

@@ -9,8 +9,8 @@ kyoto:
 
 okayama:
   name: 岡山旅行
-  start_date: <%= 100.days.ago %>
-  end_date: <%= 95.days.ago %>
+  start_date: <%= 10.days.ago %>
+  end_date: <%= 5.days.ago %>
   memo: 過去の旅行
   owner: google
 
@@ -20,3 +20,12 @@ yamaguchi:
   end_date: <%= 15.days.since %>
   memo: 別ユーザの旅行
   owner: twitter
+
+<% 10.times.each do |i| %>
+travels_<%= i %>:
+  name: $LABEL
+  start_date: <%= 100.days.ago %>
+  end_date: <%= 95.days.ago %>
+  memo: $LABEL
+  owner: google
+<% end %>

test/integration/place_integration_test.rb

@@ -90,4 +90,12 @@ def setup
     assert_match '大麻比古神社', page.text
     assert_match '伏見稲荷大社', page.text
   end
+
+  test 'paging' do
+    assert_no_match 'places_9', page.text
+    click_link '最後'
+
+    assert_match 'places_9', page.text
+    assert_no_match 'places_0', page.text
+  end
 end

test/integration/travel_integration_test.rb

@@ -159,4 +159,12 @@ def setup
 
     assert_no_match 'バスで30分', page.text
   end
+
+  test 'paging' do
+    assert_no_match 'travels_9', page.text
+    click_link '最後'
+
+    assert_match 'travels_9', page.text
+    assert_no_match 'travels_0', page.text
+  end
 end