PostgreSQL がアソシエーションのないテーブルを参照する（Rails, 多対多)

Question

動作環境

• Rails 5.2.4.2
• PostgreSQL 12.2

前提

従業員の活動を記録するデータベースにおいて、従業員(employees) は 活動(activities)と多対多の関係にあります。

下記のようなUIで一気に活動に参加した従業員を記録したく、

次のようにコードを書きました。

作成したコード

Model
activitiesとemployeesで多対多の関係を定義し、中間テーブルをactivity_employeesとしています。

class Employee < ApplicationRecord
  has_many :activity_employees, dependent: :destroy
  has_many :activities, through: :activity_employees
end

class Activity < ApplicationRecord
  has_many :activity_employees, dependent: :destroy
  has_many :employees, through: :activity_employees
  accepts_nested_attributes_for :activity_employees, allow_destroy: true

　# 今回のエラーに関係ありそうなので入れています
　has_many :activity_logs, dependent: :destroy
  has_many :staffs, through: :activity_logs
  accepts_nested_attributes_for :activity_logs
end

class ActivityEmployee < ApplicationRecord
  belongs_to :activity
  belongs_to :employee
end

Controller
一気にデータを受け取れるよう、{employee_ids: []}をstrong paramaterに入れています。

class ActivitiesController < ApplicationController
  # 略
  def create
    @activity = @facility.activities.build(activity_params)
    if @activity.save
      # 処理
    else
      # 処理
    end
  end

  private
  def activity_params
    params.require(:activity).permit(
      :activity_type,
      # 略,
      {employee_ids: []},
      activity_logs_attributes: [:staff_id, :carried_out, :got_appointment])
  end
end

View

= form_with model: [@facility, @activity], class: "p-input-form", local: true do |f|

  = f.collection_check_boxes :employee_ids, @employees, :id, :name, include_hidden: false do |t|
    = t.label(class: "c-check-box__label mr-4") { t.check_box(class: "c-check-box") + t.text }

起こっている問題

上記のコードでデータを保存しようとすると、下記のエラーが出ます。

staffsテーブルにデータを保存しようとはしていないのに、なぜ、staffsテーブルの参照整合性を崩すというエラーが出るのかわかりません。

調べたこと

activity_paramsとして送っているデータの中身を調べました。

[1] pry(#<ActivitiesController>)> activity_params
=> <ActionController::Parameters {
  "activity_type"=>"event",
  # 中略
  "employee_ids"=>["33"], 
　"activity_logs_attributes"=><ActionController::Parameters {
    "0"=><ActionController::Parameters {
       "staff_id"=>"1", "carried_out"=>"false",
       "got_appointment"=>"false"
     } permitted: true>
   } permitted: true>
} permitted: true>

特にネストがおかしいところはなさそうです。
なぜ、アソシエーションがないはずのstaffsテーブルの参照性を崩すというエラーが出いるのでしょうか？考えられる点だけでもアドバイスいただけると嬉しいです。

どうぞよろしくお願いいたします。

Answer 1

databaseの方にfkの定義がされているような気がします。
rails db:schema:dump
して現在のDBの状況を取り出し、db/schema.rbを見せてください。

Comments

[okutamatanuki]

ありがとうございます！
昨晩、自分でもFKがstaff_idになっているのでは、と気づき調べておりました。
`rails db:schema:dump` を実施した後の、schemaファイルはこちらです。

```Ruby
create_table "activity_employees", force: :cascade do |t|
t.bigint "activity_id"
t.bigint "employee_id"
t.datetime "created_at", null: false
t.datetime "updated_at", null: false
t.index ["activity_id"], name: "index_activity_employees_on_activity_id"
t.index ["employee_id"], name: "index_activity_employees_on_employee_id"
end
```

マイグレーションファイルをさかのぼって調べたところ、過去にこのようなファイルを作成しており、

```ruby
class CreateActivityEmployees < ActiveRecord::Migration[5.2]
def change
create_table :activity_employees do |t|
t.references :staff, foreign_key: true
t.references :facility_employee, foreign_key: true
t.timestamps
end
end
end
```

その後、このように変更していました。

```ruby
class RenameStaffIdColumnToActivityEmployees < ActiveRecord::Migration[5.2]
def change
rename_column :activity_employees, :staff_id, :activity_id
end
end
```

この時に、参照する外部キーを変更していなかったので、引き続き`staffs`テーブルを参照し続けているのかと思い、以下のマイグレーションを実行したのですが

```ruby
class ChangeReferencesOfActivityEmployeesTable < ActiveRecord::Migration[5.2]
def change
remove_reference :activity_employees, :staff, foreign_key: true
add_reference :activity_employees, :activity, foreign_key: true
end
end
```

現在このようなエラーが出て解決に苦しんでいます。

```ruby
== 20200814003145 ChangeReferencesOfActivityEmployeesTable: migrating =========
-- remove_reference(:activity_employees, :staff, {:foreign_key=>true})
rails aborted!
StandardError: An error has occurred, this and all later migrations canceled:

Table 'activity_employees' has no foreign key for {:to_table=>"staffs", :column=>"staff_id"}
```
まずは外部キーを削除してから、再び追加する、というようなマイグレーションにしてみようと思います。

[okutamatanuki]

こちら、マークダウンが使えないのですね。。。失礼いたしました。
下記のようなマイグレーションを書いたら解決しました！
===============
class ChangeReferencesOfActivityEmployeesTable < ActiveRecord::Migration[5.2]
def change
remove_foreign_key :activity_employees, :staffs
add_foreign_key :activity_employees, :activities
end
end

===============

おかげさまで、こうかな？と思っていたことに確証を持てたので、ベストアンサーにさせてください。
ありがとうございます！