ログアウトはcontrollerでpostではないのでしょうか？

前提・実現したいこと

現在簡単なsnsを作っているのですが、ログアウトしようとするとエラーが出てしまいます。少し前まではエラーが出ることなくログアウトすることができたので、今日何かを追加したせいでログアウトがうまくいかなくなってしまったのかと思うのですが、その原因が特定できていません。

発生している問題・エラーメッセージ

そもそも/logoutはgetではなくpostに指定しているのになぜgetを探そうとするのか？

No route matches [GET] "/logout"

該当のソースコード

<!DOCTYPE html>
<html>
  <head>
    <title>Konpeki</title>
    <%= csrf_meta_tags %>
    <%= csp_meta_tag %>
    <link href="https://use.fontawesome.com/releases/v5.6.1/css/all.css" rel="stylesheet">
    <%= stylesheet_link_tag 'application', media: 'all', 'data-turbolinks-track': 'reload' %>
    <%= javascript_pack_tag 'application', 'data-turbolinks-track': 'reload' %>
  </head>

  <body>
  <header>
    <div class="heading">
    <% if session[:user_id] %>
      <div class="header-left">
        <span >Members</span>
      </div>
      <% else %>
      <div class="header-left">
        <%=link_to("Members","/") %>
      </div>
      <% end %>
      <div class="header-right">
      <% if session[:user_id]%>
       
        <%=link_to("ユーザー一覧","/users/index") %>
        <%=link_to("投稿する","/posts/new") %>
        <%=link_to("投稿一覧","/posts/index") %>
        <%= link_to("ログアウト", "/logout", {method: :post}) %>
        <span><%=session[:user_id]%></span>


        <% else%>
         <div class="else">
        <%=link_to("ログイン","/login")%>
        <%=link_to("新規登録","/signup")%>
        <%=link_to("Membersとは","/")%>
        </div>
        <% end%>
        
      </div>
    </div>
  </header>
    <%= yield %>
  </body>
</html>

Rails.application.routes.draw do

  post"logout" =>"users#logout"
  post "login"=>"users#login"
  get"login" => "users#login_form"

  post"users/:id/update" => "users#update"
  get 'users/index'
  get '/'=> "home#top"
  get 'posts/index' => "posts#index"
  get "posts/new" =>"posts#new"
  post"posts/create" => "posts#create"
  get"posts/:id" => "posts#show"

 get"posts/:id/edit" =>"posts#edit"

 post"posts/:id/update" =>"posts#update"
 post"posts/:id/destroy" =>"posts#destroy"
  
 get "users/index"=>"users#index"
 get"signup" =>"users#new"
 post"users/create" => "users#create"
 get"users/:id" =>"users#show"
 
 get"users/new" => "users#new"
 get"users/:id/edit" => "users#edit"

 
end

def logout
    
    session[:user_id]= nil
    flash[:notice]="ログアウトしました"
    redirect_to ("/login")
  end

試したこと

ルーティングのpost"logout"をget"logoutにしたらログアウトはできたのですが、progateでsessionの値を変更するときはpostにすると習ったので、postの方がいいのかと考えています。また以前はpostでできたので、なぜ今できないのかを知りたいです。

行動規範の内容に同意します

回答1件

ベストアンサー

的外れかもしれませんが、
link_to("ログアウト", "/logout", {method: :post})
の {} を外して、
link_to("ログアウト", "/logout", method: :post)
にしたらどうなりますか？

この辺の記事を見ると、 {} なしで動きそうだったので、試してみてもいいかなと思いました
link_toはRailsの基本！これであなたも必ずlink_toが書ける！

投稿2019/11/03 01:43

siruku6

総合スコア1382

sn.jr

2019/11/04 05:54

ご回答ありがとうございます！エラーが出てしまいます、、

siruku6

2019/11/04 07:50 編集

かっこも外してこうするといかがでしょう？普段こう書いているので動きそうなのですが... link_to "ログアウト", "/logout", method: :post （link_to と "ログアウト" の間にはスペース必須です）他の方もこんな風に {} なしで書いています https://qiita.com/STHEXA/items/aa6a5af6cfbdc7b751a3#link_to%E3%83%A1%E3%82%BD%E3%83%83%E3%83%89%E3%81%A8%E3%81%AF それと、もしよかったらエラーの内容を省略せず教えてもらっても構いませんか？

sn.jr

2019/12/29 01:08

すっかり返答忘れていました！申し訳ございません！なんとか理解できました！

行動規範の内容に同意します