[Rails] ログインしている時と、していない時の表示

Question

###前提・実現したいこと
rails4 ローカル環境
deviseを使用してログイン機能を実装しました。

そこで、トップページとなるtop/inedex.html.erbで<% if user_signed_in? %>を使い、
ログインしてない場合はログイン用のページ
ログインしている場合は自分のトップページ
となるようにしました。

ログインしている場合のトップページで自分のタイムラインを表示して、kaminariを導入してページネーション機能をつけました。

###発生している問題・エラーメッセージ
ページネーション機能をつけて、ログイン時は正常に表示されるのですが、ログアウトすると以下のようなエラーになってしまいます。

NoMethodError at /
undefined method `page' for nil:NilClass

###該当のソースコード
top_controller.rb

class TopController < ApplicationController
  def index
    @timelines = current_user.try(:timeline).page(params[:page]).per(25).order(created_at: :desc)
  end
end

views/top/index.html.erb

<% if user_signed_in? %>

省略

<h4>タイムライン</h4>
<% @timelines.each do |timeline| %>
 <div class="row timeline-list">
  <%= link_to user_path(timeline.user_id) do %>
   <%= profile_img(timeline.user) %>
  <% end %>
  <p class="name"><b><%= timeline.user.name %></b><span class="right-list"><%= time_ago_in_words timeline.created_at %>前</span></p>
  <%= link_to timeline_path(timeline) do %>
   <p class="timeline-content"><%= timeline.content %></p>
  <% end %>
 </div>
<% end %>
<%= paginate(@timelines) %>

<% else %>

省略

<%= link_to 'ログイン', new_user_session_path %>
<%= link_to 'サインアップ', new_user_registration_path %>

<% end %>

ログアウト時に、pageがnilとなってしまうためエラーになっていると解釈しているのですが、どのように解決すればいいのかが分かりません。

どう修正すればいいか教えて頂ける嬉しいです。
よろしくお願いします。

Answer 1

undefined method `page' for nil:NilClass
はkaminariのライブラリの中で起きています
この場合@timelinesがnilになっていることに起因していると思います。

まあ、それはおいておいて、おそらく根本的な原因は
logout時のリダイレクトが正常に動作していないことが原因だと思います。
controllers/application_controller.rbという作った覚えのない
ファイルがありますので
def after_sign_out_path_for(resource)
methodを定義してそこに、ログインしていない状態で表示されるページへのpath
を指定してください。

application_controller.rb

def after_sign_out_path_for(resource)
  not_logined_path
end

not_logined_pathはログインしていない状態で表示されるページへのpathです。

Comments

[riamk]

mokeさん、いつもご回答ありがとうございます。
def after_sign_out_path_for(resource)
root_path
end
としてログアウトしてみましたが、エラーは同じように起こってしまいました。

ログインしていない状態もログインしている状態もどちらもtop/indexをトップページにしていて、top/indexの中身を<% if user_signed_in? %>でログイン、非ログイン時で表示させている状況です。

[moke]

ああ、そうですかよく読んでなかったみたいです
ではこう書き換えましょうか
def index
@timelines = current_user.try(:timeline).page(params[:page]).per(25).order(created_at: :desc)　if user_signed_in?
end

[riamk]

教えて頂いたように書き換えてみたところ
syntax error, unexpected tIDENTIFIER, expecting keyword_end
...).order(created_at: :desc)　if user_signed_in?
... ^

とエラーになってしまいました。
なんとかエラーを自分で解決しようと検索して調べてみましたがどうすることもできませんでした。

[riamk]

ご迷惑をお掛けしました。

@timelines = current_user.try(:timeline).page(params[:page]).per(25).order(created_at: :desc)if user_signed_in?
と、if〜の前のスペースを消してみたところ正常に表示できました！

今回も丁寧にご指導して頂きありがとうございました！