Ruby on Rails 外部キー制約によりデータが削除できません。

Question

###前提・実現したいこと
Rails4 PostgreSQL

現在、railsでdeviseを使用しログイン機能を実装し、ログインユーザーがユーザー同士、個別にメッセージのやり取りができるようにしました。

具体的なテーブル構成は

usersテーブル(ユーザ情報を格納する)
conversationsテーブル(会話情報を格納する)
messagesテーブル(会話の個々のメッセージを格納する)

このようにしました。

現在のコードではメッセージをやり取りしたユーザーが退会すると外部キー制約によりデータが削除できないというエラーが発生してしまいます。

どこを修正すればいいかアドバイスをお願いします。

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

メッセージのやり取りをしたユーザーが退会(ユーザーの削除)しようとすると以下のようなエラーが出てユーザーを削除することができません。

PG::ForeignKeyViolation at /users
ERROR:  update or delete on table "users" violates foreign key constraint "fk_rails_273a25a7a6" on table "messages"

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

class Message < ActiveRecord::Base
  belongs_to :conversation
  belongs_to :user, dependent: :destroy

  validates_presence_of :body, :conversation_id, :user_id
  def message_time
    created_at.try(:strftime,"%m/%d %R")
  end
end

conversation.rb

class Conversation < ActiveRecord::Base
  belongs_to :sender, foreign_key: :sender_id, class_name: 'User'
  belongs_to :recipient, foreign_key: :recipient_id, class_name: 'User'
  has_many :messages, dependent: :destroy
  validates_uniqueness_of :sender_id, scope: :recipient_id
  scope :between, -> (sender_id,recipient_id) do
    where("(conversations.sender_id = ? AND conversations.recipient_id =?) OR (conversations.sender_id = ? AND  conversations.recipient_id =?)", sender_id,recipient_id, recipient_id, sender_id)
  end

  def target_user(current_user)
    if sender_id == current_user.id
      User.find(recipient_id)
    elsif recipient_id == current_user.id
      User.find(sender_id)
    end
  end
end

今はこのようにコードを書いています。
他に必要なコードがあれば記載しますので、ご教授よろしくお願いします。

###補足・追記
Messagesテーブルのマイグレーションファイル

class CreateMessages < ActiveRecord::Migration
  def change
    create_table :messages do |t|
      t.text :body
      t.references :conversation, index: true, foreign_key: true
      t.references :user, index: true, foreign_key: true
      t.boolean :read, default: false

      t.timestamps null: false
    end
  end
end

Conversationsテーブルのマイグレーションファイル

class CreateConversations < ActiveRecord::Migration
  def change
    create_table :conversations do |t|
      t.integer :sender_id
      t.integer :recipient_id

      t.timestamps null: false
    end
  end
end

Answer 1

messagesテーブルの対象データを先にDELETEするか、FOREIGN KEYにON DELETE CASCADE付きで外部制約を付け直されては？

実務としてはユーザーが退会したらすぐDELETEして良いものか？悪質なユーザーの再入会の回避するために、論理削除としてDELETE_FLAGを立てたり、退会済み用のテーブルに情報を残すことが多いです。

Comments

[riamk]

ご回答ありがとうございます。
実務としてのご意見は勉強している中で触れることがない部分で、「退会済み用のテーブルに情報を残す」とかとても参考になります。
外部制約の付け方について調べてみながら、再度トライしてみます！

Answer 2

モデルの記述が問題と言うよりは、DBレベルでの問題な気がします。
マイグレーションファイルで、foreign_key制約はつけていますか？
どこに付けているかにも依りますが、そのあたり見せて頂けると原因が分かると思います。

Comments

[riamk]

ご回答ありがとうございます。
私のそもそもの視点が間違っていたのですね。
マイグレーションファイルの中身を追記しましたので、何かアドバイス頂けると嬉しいです。
よろしくお願いします。

[Miracle]

`t.references :user, index: true, foreign_key: true`
この記述が、ユーザーを削除出来ない原因かと思います。
messagesテーブルのuser_idカラムにかけているforeign_keys制約が、ユーザーの削除を防いでいる感じですね。
userモデルに、user has many messagesというリレーションが定義されていると思いますが、そこでdependent のオプションに delete_allとかを付けたら、解決できるかもです。
http://blog.sanojimaru.com/post/18426241191/activerecordhasmany%E3%81%AEdependent%E3%82%AA%E3%83%97%E3%82%B7%E3%83%A7%E3%83%B3

ただ、データの整合性を保つのがforeign_keyをつける目的なので、その意味ではOrlofskyさんが仰られているような実装方法のほうが良いかと思います。

また、別の話になりますが、messageモデルに書かれている
`belongs_to :user, dependent: :destroy` のdependent destroyオプションは付ける必要が無さそうですね。

[riamk]

ご教授ありがとうございます。
Miracleさんのアドバイスを参考にもう一度foreign_keyについて学習し直してチャレンジしてみようと思います。

messageモデルのdependentオプションは色々と試していてそのままになってしまっていました。
ご指摘ありがとうございます。