saveメソッドによるデータ保存ができない

Question

前提・実現したいこと

概要
Ruby（Ruby on Rails）で自身と相手で連絡が可能なアプリを作成しています。
（自身と任意の別ユーザーのプロジェクト（連絡板）を作成し、その中で連絡をとることができる機能）
その中で実現したいことは、
プロジェクトを新規作成した際に、中間テーブルに外部キーであるuser_id、project_idを併せて保存する
ということですがこの点で先へ進めない状態となっています。

設定背景
モデル　　　 ：user、project、pro_user（左記テーブルの中間テーブル）
導入したgem：devise（ユーザー管理）、pry-rails

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

発生している問題

• binding.pryでプロセスを止めフォームから送られたparamsの内容を確認。projectsテーブル及びpro_userテーブルへの保存に必要なデータ（プロジェクト名及びuser_id）は含まれていることをターミナル上で確認。
• 上記を確認した後、saveメソッドで保存の可否をターミナル上で確認したところ、以下のターミナルの表示通りでROLLBACKがされ保存できない状況となった。

エラーメッセージ

     6: def create
     7:   @project = Project.new(project_params)
 =>  8:   binding.pry
     9:   if @project.save
    10:     redirect_to root_path
    11:   else
    12:     render :new
    13:   end
    14: end

[1] pry(#<ProjectsController>)> @project.save
   (0.4ms)  BEGIN
  ↳ (pry):1:in `create'
  User Exists? (0.4ms)  SELECT 1 AS one FROM `users` WHERE `users`.`email` = BINARY 't@t' AND `users`.`id` != 1 LIMIT 1
  ↳ (pry):1:in `create'
  User Exists? (0.6ms)  SELECT 1 AS one FROM `users` WHERE `users`.`email` = BINARY 's@s' AND `users`.`id` != 2 LIMIT 1
  ↳ (pry):1:in `create'
   (0.2ms)  ROLLBACK
  ↳ (pry):1:in `create'
=> false

お伺いしたいこと
原因と考えられる点やアクションプランがあればご助言いただきたく存じます。

• 以下に記述している試したこと（ターミナル上での原因探索等）を踏まえ、ビューファイルの記述不備が根本原因ではなく、モデルやデータベースに何らかの不備があると推測しています
• しかし、知識不足ゆえに確実な根本原因と言い切れないこと、また対策のアクションプランが手詰まりであるため、今回質問をさせていただきました。

該当のソースコード

routes.rb

Rails.application.routes.draw do
  devise_for :users
  root to: 'messages#index'
  resources :projects, only: [:new, :create]
end

projects_controller.rb

class ProjectsController < ApplicationController
  def new
    @project = Project.new
  end

  def create
    @project = Project.new(project_params)
    if @project.save
      redirect_to root_path
    else
      render :new
    end
  end

  private

  def project_params
    params.require(:project).permit(:name, user_ids:[])
  end
end

models > user.rb

class User < ApplicationRecord
 #アソシエーションを抜粋
  has_many :pro_users
  has_many :projects, through: :pro_users
end

models > project.rb

class Project < ApplicationRecord
  #アソシエーションを抜粋
  has_many :pro_users
  has_many :users, through: :pro_users, dependent: :destroy
end

models > pro_user.rb

class ProUser < ApplicationRecord
  belongs_to :user
  belongs_to :project
end

試したこと

• ターミナルでのエラー原因探索（★情報追加依頼の方にお示しした箇所）

[1][4][5]で示されている通り、user_idをもとにusersテーブルからデータの取得ができていないことが原因と考えられる

[2] pry(#<ProjectsController>)> params
=> <ActionController::Parameters {"authenticity_token"=>"gE3lWZX40UGHzTvg/BkYV2WqROr6GK/QpV1PE6XHuqJUpauKuzxCY35jNKBKDOGVZJdKJVNlMu7KdRLttHWmkA==", "project"=><ActionController::Parameters {"name"=>"お試し", "user_ids"=>["1", "2"]} permitted: false>, "commit"=>"Create Project", "controller"=>"projects", "action"=>"create"} permitted: false>
[3] pry(#<ProjectsController>)> project_params
=> <ActionController::Parameters {"name"=>"お試し", "user_ids"=>["1", "2"]} permitted: true>
[4] pry(#<ProjectsController>)> @project.errors
=> #<ActiveModel::Errors:0x00007fed1b9ac328
 @base=#<Project:0x00007fed1431d5b8 id: nil, name: "お試し", created_at: nil, updated_at: nil>,
 @details={:users=>[{:error=>:invalid}, {:error=>:invalid}]},
 @messages={:users=>["is invalid"]}>
[5] pry(#<ProjectsController>)> @project.errors.full_messages
=> ["Users is invalid"]

• rails db:migrateの不備を疑う（カラムの型や外部キー設定に不備）

改めてrails db:migrate:resetを実行→以下の通り設定ができているものの結果は変わらず

+------------+-------------+------+-----+---------+----------------+
| Field      | Type        | Null | Key | Default | Extra          |
+------------+-------------+------+-----+---------+----------------+
| id         | bigint(20)  | NO   | PRI | NULL    | auto_increment |
| user_id    | bigint(20)  | YES  | MUL | NULL    |                |
| project_id | bigint(20)  | YES  | MUL | NULL    |                |
| created_at | datetime(6) | NO   |     | NULL    |                |
| updated_at | datetime(6) | NO   |     | NULL    |                |
+------------+-------------+------+-----+---------+----------------+

• user_idsという配列のパラメータが複数あることが原因と仮定

ビューファイルをい1行コメントアウトすることで、現在のユーザーidがパラメータに含まれないようにしてから実行
→試行前と同じエラーが出現したため配列内のパラメータ数は根本原因ではない可能性が高いことが推測された

<div class='project-form__field--right'>
　<select name='project[user_ids][]'>
　　<option value=''>アサインメンバーを選択してください</option>
　　<% User.all.where.not(id: current_user.id). each do |user| %>
　　　<option value=<%=user.id%>><%= user.last_name %><%= user.first_name %></option>
　　<% end %>
　</select>
 #以下の1行をコメントアウト
　<input name='project[user_ids][]' type='hidden' value=<%=current_user.id>>
</div>

####試したことの追記（2020/9/4）

• if @prject.save!（@prject.errors）により、User is invalid がエラー内容として出現していたことから、Validationが邪魔をしているため保存ができていないことを疑った

→UserモデルのValidationを一時的にコメントアウトした結果、意図したデータ保存がされた（プロジェクトと同時に中間テーブルのデータも保存）
→根本原因はValidationであることが判明した。しかし、ValidationはUser登録の時に必須でありコンフリクトが生じてしまった。

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable

  # japanese_letters = /\A[ぁ-んァ-ン一-龥]+\z/
  # with_options presence: true do
  #   validates :last_name, format: { with: japanese_letters, message: 'is not full-width characters' }
  #   validates :first_name, format: { with: japanese_letters, message: 'is not full-width characters' }
  #   validates :email, uniqueness: { case_sensitive: true },
  #              format: { with: /\w+([-+.]\w+)*@\w+([-.]\w+)*/ }
  #   validates :password, length: {minimum: 6 },
  #              format: { with: /\A(?=.*[A-Za-z])(?=.*\d)[A-Za-z\d]+\z/, message: 'includes both half-width letters and half-width numbers' }
  #   validates :company, :password_confirmation
  # end
  
  has_many :pro_users
  has_many :projects, through: :pro_users
  has_many :messages
end

ターミナル

  Parameters: {"authenticity_token"=>"ZOq+q72nyTZK7+6DAc1vIWclC/OS2UauGGAJ+BjjsUWwAvB4k2NaFLNB4cO32JbjZhgFPDuk25B3SFQGCVGtdw==", "project"=>{"name"=>"お試し", "user_ids"=>["1", "2"]}, "commit"=>"Create Project"}
  User Load (0.3ms)  SELECT `users`.* FROM `users` WHERE `users`.`id` = 2 ORDER BY `users`.`id` ASC LIMIT 1
  User Load (0.4ms)  SELECT `users`.* FROM `users` WHERE `users`.`id` IN (1, 2)
  ↳ app/controllers/projects_controller.rb:7:in `create'
   (0.2ms)  BEGIN
  ↳ app/controllers/projects_controller.rb:8:in `create'
  Project Create (0.9ms)  INSERT INTO `projects` (`name`, `created_at`, `updated_at`) VALUES ('お試し', '2020-09-04 05:43:52.654047', '2020-09-04 05:43:52.654047')
  ↳ app/controllers/projects_controller.rb:8:in `create'
  ProUser Create (0.2ms)  INSERT INTO `pro_users` (`user_id`, `project_id`, `created_at`, `updated_at`) VALUES (1, 4, '2020-09-04 05:43:52.657678', '2020-09-04 05:43:52.657678')
  ↳ app/controllers/projects_controller.rb:8:in `create'
  ProUser Create (0.2ms)  INSERT INTO `pro_users` (`user_id`, `project_id`, `created_at`, `updated_at`) VALUES (2, 4, '2020-09-04 05:43:52.659268', '2020-09-04 05:43:52.659268')
  ↳ app/controllers/projects_controller.rb:8:in `create'
   (0.6ms)  COMMIT

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

• ruby '2.6.5'
• rails (6.0.3.2)

以上、よろしくお願いいたします。

Comments

[maisumakun]

ロールバックした直後に@project.errorsとして何かしらの情報は得られませんでしょうか？

[退会済みユーザー]

早々にご返信ありがとうございます。

binding.pryでプロセスを止めた後の操作という解釈でよろしいでしょうか。
相違がないようでしたら@project.errorsで得られた情報は、上記試したことの結果と重複いたしますが下記の通りでした。

=> #<ActiveModel::Errors:0x00007fed1b9ac328
@base=#<Project:0x00007fed1431d5b8 id: nil, name: "お試し", created_at: nil, updated_at: nil>,
@details={:users=>[{:error=>:invalid}, {:error=>:invalid}]},
@messages={:users=>["is invalid"]}>

この記述からuserデータが取得できていないのではと疑いましたが、次のアクションプランが思い浮かばない次第です。

[no1knows]

createアクションで、user_idsという配列のパラメータをそれぞれのusr_idに分けてからsaveする必要があるのではないでしょうか？

[退会済みユーザー]

ご助言ありがとうございます。

＞user_idsという配列のパラメータをそれぞれのusr_idに分けてからsave
「配列に複数のuser_idが含まれていることが原因の可能性がある」ということでしょうか。
「分けてからsave」の適切な記述がわかりませんでしたので、まずはビューファイルを修正しパラメータに含まれるuser_idが一つとなるように修正し、早速試してみました。（本文の試したことに詳細を追記）

結果としては、状況は変わらず@project.errorsにて以下のエラーが出現しております。
（ご助言の意図から外れた検証内容でしたら申し訳ございません。。）
@details={:users=>[{:error=>:invalid}
@messages={:users=>["is invalid"]}

Answer 1

if @project.save を 一時的に　 if @project.save! にするとsaveできない原因が表示されるので、それがあるとはっきりしますが、今ある情報からは
project には user_id, user_ids というattribute がないのにそれを設定しようとしている、というのが原因でしょう。

<select name='project[user_ids][]'を
<select name='project[pro_users_attributes][user_ids][]'
とし、
params.require(:project).permit(:name, user_ids:[]) を
params.require(:project).permit(:name, pro_users_attributes: {user_ids:[]})
にし、
class Project に
accepts_nested_attributes_for :pro_users を追加

すればできるかな。。。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
関連を持ったモデルをまとめて操作するために、accepts_nested_attributes_forを設定してpro_usersテーブルにuser_idを保存する方針ですね。

上記の通り修正し試行したところ、今まで登録されなかったprojectテーブルはデータが保存され、一方で中間テーブルであるpro_usersテーブルには保存がされなかったという状況です。

原因（エラー）をターミナルにて確認したところ「Unpermitted parameter: :pro_users_attributes」と出現しており、user_id自体が渡されていないことが原因と考えられるかと思われました。

今回の件で、user_idの渡され方又は渡された後の処理が対処すべき箇所だと見当をつけることができました。

[winterboum]

Unpermitted parameter:　がでましたか。。。おかしいな。
（私の書いたのも含め）typoがないかは一度確認しておいてください。

[退会済みユーザー]

ご返信ありがとうございます。
winterboum様の意図する検証が正しく実施できるように、まずは誤記の見直しをさせていただきます！

Answer 2

以下検証より解決できたため、当該質問をcloseさせていただきます。
今回コメント及び回答をいただいた皆さま、ご助言いただき有難うございました。

• 仮説

以下、UserモデルのValidationを削除（コメントアウト）することでプロジェクトテーブル及び中間テーブルのデータが保存された。
中間テーブルとUsersテーブルはbelongs_to/has_manyの関係であることから、中間テーブルのデータ保存時にUserモデルのvalidationが作用し、saveメソッドが中止されていたことが推察された。

• 検証・修正内容

validationを削除することはUser新規登録に差し支えるため不可。
そこで、validationのかかるタイミングをonオプションでUserの新規登録時のみに指定
以下の通り修正することで、User新規登録にも影響なく意図したデータ登録ができるようになった。

###修正前
Userモデル

class User < ApplicationRecord
#該当箇所抜粋
  japanese_letters = /\A[ぁ-んァ-ン一-龥]+\z/
  with_options presence: true, on: :registration do |registration|
    registration.validates :last_name, format: { with: japanese_letters, message: 'is not full-width characters' }
    registration.validates :first_name, format: { with: japanese_letters, message: 'is not full-width characters' }
    registration.validates :email, uniqueness: { case_sensitive: true },
               format: { with: /\w+([-+.]\w+)*@\w+([-.]\w+)*/ }
               registration.validates :password, length: {minimum: 6 },
               format: { with: /\A(?=.*[A-Za-z])(?=.*\d)[A-Za-z\d]+\z/, message: 'includes both half-width letters and half-width numbers' }
    registration.validates :company, :password_confirmation
  end
  
  has_many :pro_users
  has_many :projects, through: :pro_users
  has_many :messages
end

###修正後
Userモデル

class User < ApplicationRecord
#該当箇所抜粋
  japanese_letters = /\A[ぁ-んァ-ン一-龥]+\z/
  with_options presence: true, on: :registration do |registration|
    registration.validates :last_name, format: { with: japanese_letters, message: 'is not full-width characters' }
    registration.validates :first_name, format: { with: japanese_letters, message: 'is not full-width characters' }
    registration.validates :email, uniqueness: { case_sensitive: true },
                            format: { with: /\w+([-+.]\w+)*@\w+([-.]\w+)*/ }
    registration.validates :password, length: {minimum: 6 },
                            format: { with: /\A(?=.*[A-Za-z])(?=.*\d)[A-Za-z\d]+\z/,
                                      message: 'includes both half-width letters and half-width numbers' }
    registration.validates :company, :password_confirmation
  end

新たに生成しカスタマイズしたdeviseのcontroller

class Users::RegistrationsController < Devise::RegistrationsController
#変更箇所付近を抜粋
def create
    build_resource(sign_up_params)

    resource.save(context: :registration)　←save以降を追記
    yield resource if block_given?
    if resource.persisted?
（後略）
end