destroyアクションが反応しない

Question

前提・実現したいこと

userを削除したいのですが
destroyアクションがうまく機能しません

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

エラーメッセージが表示されません

該当のソースコード

users_controller

class UsersController < ApplicationController
  include SessionsHelper


  def destroy
    user = User.find(params[:id])
    #binding.pry
    user.destroy
    redirect_to root_url
  end

drinks/index.html.erb

<%= link_to "退会する", user_path(current_user),method: :delete %>

current_userは自分で定義したヘルパーメソッドです

sessions_helper.rb

  # 現在ログインしているユーザーの情報を取得
  def current_user
    # DBの問い合わせの数を可能な限り小さくしたい
    # logged_in?メソッドでも使われてるし、、、
    if user_id = session[:user_id]
      # セッションがある場合
      # すなわちログインしてる時のみ

      # sessionにアクセスした結果を変数に
      # 入れておいてあとで使いまわした方が
      # 早くなる
      @current_user ||= User.find_by(id: user_id)
      # find_byでデータベースにクエリを投げる
      # ブラウザのセッションにあるuser_idをもとにUser定義

      # find_byの実行結果をインスタンス変数に保存する
      # ことで、1リクエスト内におけるデータベースへの
      # 問い合わせは最初の一回だけになり、 
      # 以後の呼び出しではインスタンス変数の結果を
      # 再利用する

      # すでに@current_userが存在する場合って何？
      # 一回current_userを実行したら、
      # @current_userがあるのでそれを使ってね
      
      # sessionのuser＿idがあるということは
      # 既にログインしてるといてDBにユーザーの情報があるはず。
      # だからsessionのuser_idをDBでfind_byかければいい
    elsif  (user_id = cookies.signed[:user_id])
      # sessionが張られてなかったらcookiesにあるかも
      user = User.find_by(id: user_id)
      if user && user.authenticated?(cookies[:remember_token])
        # nilガード
        # クッキーのuser_idとremember_tokenが一致してる
        log_in user
        @current_user = user
      end
    end
  end

user.rb

class User < ApplicationRecord
  attr_accessor :remember_token
  # remember_tokenというメソッドを作成
  # password,password_comfirmationみたいな
  # 変数のように扱える
  # u.remember_token的なことができる
  has_many :drinks, dependent: :delete_all
  before_save  { self.email = email.downcase }
  has_secure_password
  validates :nickname,  presence: true, length: { maximum: 50 }
  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+.[a-z]+\z/i
  validates :email, presence: true, length: { maximum: 255 },
                    format: { with: VALID_EMAIL_REGEX },
                    uniqueness: { case_sensitive: false }
  validates :password, presence: true, length: { minimum: 6 },allow_nil: true
  # ユーザー更新時に空のパスワードでも大丈夫
  # has_secure_passwordの方でpasswordの存在性を検証するから大丈夫
  
  

  # 渡された文字列のハッシュ値を返す
  def User.digest(string)
    cost = ActiveModel::SecurePassword.min_cost ? BCrypt::Engine::MIN_COST :
                                                  BCrypt::Engine.cost
    BCrypt::Password.create(string, cost: cost)
  end

  # ランダムなトークンを返す
  def User.new_token
    SecureRandom.urlsafe_base64
  end

  # 永続的にログインするためにトークンをDBに保存
  def remember
    self.remember_token = User.new_token
    # 何のトークンか分かりやすいから
    # remember_tokenって名前を作った
    update_attribute(:remember_digest,User.digest(remember_token))
    # ハッシュ化したトークンをremember_digestに保存
  end

  # 渡されたトークンがダイジェストと一致したらtrue
  # を返す
  def authenticated?(remember_token)
    return false if remember_digest.nil?
    # 二種類のブラウザを使用してログアウトした場合
    # cookiesのremember_tokenはあるけど、
    # サーバー側でremember_digestをnilにしてるから
    # nilに対して.is_password?とかやるとfor nil classエラーが起きちゃう
    # remember_digestがnilの場合はfalseを返して、処理を止める

    # 後置if文に当てはまる条件があれば処理を止めて！
    # って場合はreturnとかで明示的に書くとif ~ else ~ end 
    # とか書かなくて済む
    BCrypt::Password.new(remember_digest).is_password?(remember_token)
  end

  # ユーザーのログイン情報を破棄する
  def forget
    update_attribute(:remember_digest,nil)
  end
end

Drink.rb

class Drink < ApplicationRecord
  belongs_to :user
  with_options presence: true do
    validates :name
    validates :explain
  end
end

:20201116073717_create_users.rb

class CreateUsers < ActiveRecord::Migration[6.0]
  def change
    create_table :users do |t|
      t.string :nickname
      t.string :email
      t.string :password_digest
   
      t.timestamps
    end
  end
end

/20201118162753_create_drinks.rb

class CreateDrinks < ActiveRecord::Migration[6.0]
  def change
    create_table :drinks do |t|
      t.string :name
      t.integer :price
      t.text :explain
      t.references :user,null: false,foreign_key: true 
      t.timestamps
    end
  end
end

routes.rb

Rails.application.routes.draw do
  root to: 'drinks#index'
  get    '/login',   to: 'sessions#new'
  post   '/login',   to: 'sessions#create'
  delete '/logout',  to: 'sessions#destroy'
  resources :users
  resources :drinks, only: [:index,:new,:show,:create,:destroy]
end

退会リンクをusers/showに移動しました

エラーも起きていませんし、@userの定義もしっかりできていて、@user.nameとかやってもしっかり表示できているので、リンクの指定はあってると思います

このログを見るとリンクを踏んだ後にrootに戻ってるように見えます

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

ruby 2.6.5
rails 6.0.3

Comments

[maisumakun]

モデルはどのようなコードですか？

（モデル単体でdestroyしてロールバックがかかるということは、モデルに入ったバリデーションが影響していることが考えられます）

[divclass123]

ありがとうございます、追加しましたのでご確認のほどよろしくお願いします

[maisumakun]

irbでuser.destroyを行って失敗した直後にuser.errorsを確認することはできますか？

[divclass123]

```
irb(main):003:0> user = User.first
Traceback (most recent call last):
5: from /Users/soichirohara/.rbenv/versions/2.6.5/bin/irb:23:in `<main>'
4: from /Users/soichirohara/.rbenv/versions/2.6.5/bin/irb:23:in `load'
3: from /Users/soichirohara/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/irb-1.0.0/exe/irb:11:in `<top (required)>'
2: from (irb):3
1: from (irb):3:in `rescue in irb_binding'
NameError (uninitialized constant User)
```

rails c　的な感じで実行したら上手くいかないので、どのようなコマンドを打つべきか教えて下されば幸いです。

[maisumakun]

すみません、単なるirbではなくてrails cから行ってください（アドバイスが悪かったです）。

[divclass123]

[2] pry(main)> u = User.first
(0.7ms) SET NAMES utf8mb4, @@SESSION.sql_mode = CONCAT(CONCAT(@@sql_mode, ',STRICT_ALL_TABLES'), ',NO_AUTO_VALUE_ON_ZERO'), @@SESSION.sql_auto_is_null = 0, @@SESSION.wait_timeout = 2147483
User Load (0.3ms) SELECT `users`.* FROM `users` ORDER BY `users`.`id` ASC LIMIT 1
=> #<User:0x00007fcf0c065aa8
id: 26,
nickname: "はらそう",
email: "[e-mail]",
created_at: Mon, 23 Nov 2020 21:33:48 UTC +00:00,
updated_at: Mon, 23 Nov 2020 21:33:48 UTC +00:00,
password_digest: [FILTERED],
remember_digest: nil>
[3] pry(main)> u.destroy
(3.9ms) BEGIN
Drink Load (0.3ms) SELECT `drinks`.* FROM `drinks` WHERE `drinks`.`user_id` = 26
Drink Destroy (7.2ms) DELETE FROM `drinks` WHERE `drinks`.`id` = 8
User Destroy (0.9ms) DELETE FROM `users` WHERE `users`.`id` = 26
(2.0ms) COMMIT
=> #<User:0x00007fcf0c065aa8
id: 26,
nickname: "はらそう",
email: "[e-mail]",
created_at: Mon, 23 Nov 2020 21:33:48 UTC +00:00,
updated_at: Mon, 23 Nov 2020 21:33:48 UTC +00:00,
password_digest: [FILTERED],
remember_digest: nil>

質問の内容とは違い、今度はしっかり削除できてるような気がしますが、
実際に自分で作ったアプリ上から削除すると上手くいきません、、

link_toの記述が間違ってるのかな、、と思ったのですが
ログでは

Started DELETE “/users/27” for ::1 at 2020-11-24 17:38:11 +0900
Processing by UsersController#destroy as HTML
Parameters: {“authenticity_token”=>“6XlOECeU2iBZQO559AgCQDtxvC0AgRzJQcgyfFuDfrL8t8xQvTxCN9Hz4MowyrDFhnqM55Ob1VfpjerBDPqFDg==“, “id”=>“27"}
User Load (0.3ms) SELECT users.* FROM users WHERE users.id = 27 LIMIT 1
↳ app/helpers/sessions_helper.rb:53:in `current_user’
Redirected to http://localhost:3000/
Filter chain halted as :correct_user rendered or redirected
Completed 302 Found in 2ms (ActiveRecord: 0.3ms | Allocations: 978)

このようにしっかりdestroyに飛ばせてるような気がします、、、

[maisumakun]

Rails内ではdrinkのdestroyに失敗しているようですが、（Userではなく）そちらではどうでしょうか？

[divclass123]

4] pry(main)> drink = Drink.first
Drink Load (0.6ms) SELECT `drinks`.* FROM `drinks` ORDER BY `drinks`.`id` ASC LIMIT 1
=> #<Drink:0x00007fcf02f82a90
id: 9,
name: "そうちゃんの投稿",
price: 0,
explain: "aaaa",
user_id: 1,
created_at: Tue, 24 Nov 2020 10:29:37 UTC +00:00,
updated_at: Tue, 24 Nov 2020 10:29:37 UTC +00:00>
[5] pry(main)> drink.destroy
(0.4ms) BEGIN
Drink Destroy (0.6ms) DELETE FROM `drinks` WHERE `drinks`.`id` = 9
(1.8ms) COMMIT
=> #<Drink:0x00007fcf02f82a90
id: 9,
name: "そうちゃんの投稿",
price: 0,
explain: "aaaa",
user_id: 1,
created_at: Tue, 24 Nov 2020 10:29:37 UTC +00:00,
updated_at: Tue, 24 Nov 2020 10:29:37 UTC +00:00>

こーゆーことでよろしいのでしょうか、、？

[maisumakun]

問題なく削除できていますね…ちょっとわからないです。

[divclass123]

ですよね、、、、いえいえ！ありがとうございます！

[tomtomtomtom]

migrationをお見せいただくことは可能でしょうか？
userとdrinkです。

[divclass123]

追加しました、よろしくお願いします！

[tomtomtomtom]

なるほど
ちょっと予測していたmigrationとは違い、間違いはないようでした。
そこで、divさんのこれまでの返答を更に深く読んでいると、
「Filter chain halted as :correct_user rendered or redirected
　Completed 302 Found」
という非常に興味ぶかい記述、、、

非常にお手間をおかけして誠に申し訳有りませんが、
routes.rbをお見せいただくことは可能でしょうか？

[divclass123]

ログの意味がよく分からないので非常に助かります、、、！
いえいえとんでもないです、情報を追加したのでよろしくお願いします！

[tomtomtomtom]

ちょっと調べてみました。
間違っているかもしれませんが、

resources :users
get '/login', to: 'sessions#new'
post '/login', to: 'sessions#create'
delete '/logout', to: 'sessions#destroy'
root to: 'drinks#index'
resources :drinks, only: [:index,:new,:show,:create,:destroy]

↑これでいかがでしょうか？

[divclass123]

解決できました、、、！原因はbefore_actionで何故かdestroyをフィルタリングしてたことが原因でした、、、、

[tomtomtomtom]

ご解決おめでとうございます^^
力になれず申し訳有りませんでした。

[divclass123]

いえいえとんでも無いです、助けて頂いてありがとうございました、、！
質問欄にbefore_actionを記述しておくべきでした。

Answer 1

 before_action :logged_in_user, only: [:index,:edit,:update,:destroy]
  before_action :correct_user, only: [:edit, :update,:destroy]

    def correct_user
      redirect_to(root_url) unless current_user?(@user)
    end

  def logged_in_user
    unless logged_in?
      store_location
      # ユーザーがいきたがってたページを記憶
      flash[:danger] = "Please log in"
      redirect_to login_url
    end
  end

<strong>原因はbefore_actionにdestroyを設定してた</strong>
:destroyを外してあげればok