kaminariでページネーションが機能しない。

Question

前提・実現したいこと

各ユーザーの詳細ページ(users/show.html.erb)に、以下の2点をタブで分けて表示しています。

1. そのユーザーの投稿一覧。
2. そのユーザーがいいねした投稿一覧。

一覧の表示はそれぞれ出来ており、ここにkaminariと言うgemを使ってページネーション機能を加えている段階です。

しかし「1 2 Next › Last »」と言う表示は現れるものの、指定した件数で一覧が区切られません。エラーは特に発生していません。

該当のソースコード

users_controller.rb

def show
  @user = User.find_by(id: params[:id])

  @posts_page = Post.where(user_id: @user.id).all.order(created_at: :desc).page(params[:page]).per(2) 
  @goings_page = Going.where(user_id: @user.id).all.order(created_at: :desc).page(params[:page]).per(2) 

一旦2件ごとにページを切り替える設定にしていますが、3件目も同列で表示されてしまいます。

users/show.html.erb

<!-- そのユーザーの投稿一覧。-->
<div class="panel">
  <% @user.posts.each do |post| %>
  <div class="index-item">
    <div class="index-left">
      <%= link_to post_path(post) do %>
        <img src="<%= "/post_images/#{post.post_image_name}" %>">
      <% end %>
    </div>
  </div>  
  <% end %>  
  <div class="pagination"><%= paginate @posts_page %></div>
</div>

<!-- そのユーザーがいいねした投稿一覧。-->
<div class="panel">
  <% @goings.each do |going| %>
  <% post = Post.find_by(id: going.post_id) %>
  <div class="index-item">
    <div class="index-left">
      <%= link_to post_path(post) do %>
        <img src="<%= "/post_images/#{post.post_image_name}" %>">
      <% end %>
    </div>
  </div>  
  <% end %>  
  <% end %>  
  <div class="pagination"><%= paginate @goings_page %></div>
</div>

試したこと

ほぼ同様の記述をusers/top.html.erbで試すとページネーションがなぜか上手く機能します。
ただ上記でwhere(user_id: @user.id)となっていた記述を、ここではwhere(user_id: @current_user.id)としています。

users_controller.rb

def top
  @user = User.find_by(id: params[:id])
  @goings = Going.where(user_id: @current_user.id).all.order(created_at: :desc).page(params[:page]).per(2)
end

トップページでは、ログイン中のユーザーのみ自分が「いいね」した投稿一覧を確認出来る様、@current_user.idで絞り込みをしています。
しかし今回実装中の各ユーザーの詳細ページ(users/show.html.erb)では、ログイン中のユーザーではなく各ユーザーの情報を見たいため。@user.idと記述しました。しかしページネーションが上手くいきません…。

どなたかご助言頂けますと有難いです。

補足情報（FW/ツールのバージョンなど）

ruby 2.6.4p104
RubyGems 3.0.3
Rails 5.2.3

Answer 1

eachブロックで、page().per()のかかっている変数を使っていないためだと考えられます。

<div class="panel">
  <%# @user.posts ではなく、@posts_page を使う %>
  <% @user.posts.each do |post| %>
    <div class="index-item">
      <div class="index-left">
        <%= link_to post_path(post) do %>
          <img src="<%= "/post_images/#{post.post_image_name}" %>">
        <% end %>
      </div>
    </div>  
  <% end %>  
  <div class="pagination"><%= paginate @posts_page %></div>
</div>

<!-- そのユーザーがいいねした投稿一覧。-->
<div class="panel">
  <%# @goings ではなく、@goings_page を使う %>
  <% @goings.each do |going| %>
    <% post = Post.find_by(id: going.post_id) %>
    <div class="index-item">
      <div class="index-left">
        <%= link_to post_path(post) do %>
          <img src="<%= "/post_images/#{post.post_image_name}" %>">
        <% end %>
      </div>
    </div>  
    <%# end <= この end おそらくいらないような気がします %>  
  <% end %>  
  <div class="pagination"><%= paginate @goings_page %></div>
</div>

@posts_pageや@goings_pageがせっかくページネートしているのに、それを使用せずにわざわざ別のデータを@user.postや@goingsで引っ張ってきているので、ページネートされた結果を無視しているのではないかと考えられます。

Comments

[punchan36]

仰る通りでした…。
繰り返し処理用に以下を定義し、
@posts = Post.where(user_id: @user.id)

ページネーション用に以下を別個に定義しておりました。
@posts_page = Post.where(user_id: @user.id).all.order(created_at: :desc).page(params[:page]).per(2)

定義を分ける意味はない為、両方以下の記述に統一させる事で無事解決致しました。
@posts = Post.where(user_id: @user.id).all.order(created_at: :desc).page(params[:page]).per(2)
<div class="pagination"><%= paginate @posts %></div>

有難うございました！