ログアウトしたあとの画面推移でエラーがでる

Question

ログアウトしたあとは、以下のコードのように「新規登録、ログイン」が表示される画面に推移してほしいのですが、

<% if user_signed_in? %>
          <ul class="navbar-nav ml-auto">
            <li class="nav-item">
              <a class="nav-link js-scroll-trigger" href="#contact"><li><%= link_to 'マイページ',  "/users/#{current_user.id}" %></li></a>
            </li>
            <li class="nav-item">
              <a class="nav-link js-scroll-trigger" href="#contact"><li><%= link_to 'ログアウト', destroy_user_session_path, :method => :delete %></li></a>
            </li>
          </ul>
          	<% else %>
          	<ul class="navbar-nav ml-auto">
	            <li class="nav-item">
	              <a class="nav-link js-scroll-trigger" href="#team"><li><%= link_to '新規登録', new_user_registration_path %></li></a>
	            </li>
	            <li class="nav-item">
	              <a class="nav-link js-scroll-trigger" href="#contact"><li><%= link_to 'ログイン', new_user_session_path %></li></a>
	            </li>
          </ul>
          	<% end %>

　　　　　　　　　　　　　　　　　　　　　　　・
　　　　　　　　　　　　　　　　　　　　　　　・
　　　　　　　　　　　　　　　　　　　　　　　・
　　　　　　　　　　　　　　　　　　　　　　　・
<!-- Footer -->
    <footer>
      <div class="container">
        <div class="row">
          <div class="col-md-4">
            <span class="copyright">Copyright &copy; Your Website 2017</span>
          </div>
          <div class="col-md-4">
            <ul class="list-inline social-buttons">
              <li class="list-inline-item">
                <a href="#">
                  <i class="fa fa-twitter"></i>
                </a>
              </li>
              <li class="list-inline-item">
                <a href="#">
                  <i class="fa fa-facebook"></i>
                </a>
              </li>
              <li class="list-inline-item">
                <a href="#">
                  <i class="fa fa-linkedin"></i>
                </a>
              </li>
            </ul>
          </div>
          <div class="col-md-4">
            <ul class="list-inline quicklinks">
              <li class="list-inline-item">
                <%= link_to '曲一覧', songs_path %>
              </li>
              <li class="list-inline-item">
                <%= link_to 'マイページ',  "/users/#{current_user.id}" %>
              </li>
              <li class="list-inline-item">
                <%= link_to '曲を投稿する', new_song_path %>
              </li>
            </ul>
          </div>
        </div>
      </div>
    </footer>

このようなエラーがでます。

ログアウトしたあとにもマイページへ続く固有のcurrent_user.idが残った状態になっているのですが、何故でしょうか？

Comments

[maisumakun]

上のコードと、エラーメッセージ内で出ているコードが別物のようなのですが、エラーメッセージにある「layouts/application.html.erb」のコードも示していただけないでしょうか。

[renren643]

回答ありがとうございます。追記しました。「マイページ」へのリンクがヘッダーとフッターの二つあり、その両方でエラーがでます「マイページ」の「/users/#{current_user.id}」でエラーがでるようです。

Answer 1

エラー画面を見る限り、ヘッダーではなくフッターでエラーが出ていると思います。
何故エラーがでるのかですが、

ログアウトしたあとにもマイページへ続く固有のcurrent_user.idが残った状態になっているのですが、何故でしょうか？

logoutをしているのでcurrent_userはnil→nilにidなんてメソッドはないという風にエラーが出ています。
ヘッダーでは条件分岐していますので、フッターでも<% if user_signed_in? %>という風にサインインしているかどうかで条件分岐させて、サインアウトした状態で<%= link_to 'マイページ', "/users/#{current_user.id}" %>の行が実行されないようにすればいいと思います。