中間テーブルの相手テーブルのバリデーションにより保存できない

Question

前提・実現したいこと

チャット機能のあるアプリを作成しています。
問題はroomsテーブルの保存時にusersのバリデーションにより保存できないという点です。
roomsテーブルとusersテーブルは中間テーブル(room_users)を挟んだ多対多の関係にあります。
usersの保存時には特に問題なく保存でき、バリデーションによる制限も意図した動作をしてくれます。(passwordに半角英数混合を強制させ、そのように動作します。)

usersのバリデーションを動作させた上でroomsを保存したいのですが、どのようにすれば解決できますでしょうか。ご教授お願い致します。

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

バリデーションに失敗しました: Usersは不正な値です

該当のソースコード

■views/rooms/new.html.erb

<div class="contents">
  <div class="main">
    <%= render "tasks/header"%>
    <%= form_with model:@room, local: true do |f|%>
      <div class="center">
        <div class="room-builder">
          <div class="user-selector">
            <div class="user-seach">
              ①相談する相手を選択してください
            </div>
            <div class="user-scroll">
              <% User.all.where.not(id: current_user.id).each do |user|%>
                <div class="user-bar" data-name=<%= user.nickname%> data-id=<%= user.id%>>
                  <%= user.nickname%>
                </div>
              <% end %>
            </div>
          </div>
          <div class="room-form">
            <div class="user-confirmation" name="room[user_ids][]">
              ②相談する相手はこの方でよろしいですか？
              <div class='user-confirmation-bar'>
              </div>
              <input name="room[user_ids][]" type="hidden" value=<%= current_user.id %>>
              <%# ユーザーがクリックした.user-barのdata-idをJavaScriptの操作で下記select-userのvalueにしています。 %>
              <input name="room[user_ids][]" type="hidden" class="select-user">
            </div>
            <div class="room-name">
              ③ルーム名を入力してください。<br>
              　何についての相談かがわかるような名前にしましょう。<br>
              　例）次の会議の予定はいつ？<br>
              <%= f.text_field :room_name, class:'room-name-input', placeholder: 'ルーム名を入力してください'%>
            </div>
            <div class="instructions">
              ④入力が出来たら『ルーム作成』ボタンを押してルームを作成しましょう。
            </div>
          </div>
        </div>
        <% if @room.errors.any? %>
          <div class="error-alert">
            <ul>
              <% @room.errors.full_messages.each do |message| %>
                <li class='error-message'><%= message %></li>
              <% end %>
            </ul>
          </div>
        <% end %>
      </div>
      <div class="footer">
        <%=f.submit 'ルーム作成', class:"add-task-btn" %>
      </div> 
    <% end %>
  </div>
  <%= render "tasks/side"%>
</div>

■models/user.rb

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
  has_many :room_users, dependent: :destroy
  has_many :rooms, through: :room_users, dependent: :destroy
  has_many :tasks, dependent: :destroy


  validates :nickname, presence: true, length: {in: 1..12 }

  # 問題のバリデーションです。下2行をコメントアウトすると正常に保存できます。
  PASSWORD_REGEX = /\A(?=.*?[a-z])(?=.*?[\d])[a-z\d]+\z/i.freeze
  validates_format_of :password, with: PASSWORD_REGEX, message: 'は半角英数混合で入力して下さい'
end

■models/room.rb

class Room < ApplicationRecord
  has_many :room_users, dependent: :destroy
  has_many :users, through: :room_users
  has_many :problems, dependent: :destroy

  validates :room_name, presence: true, length: {in: 1..10 }
end

■models/room_user.rb

class RoomUser < ApplicationRecord
  belongs_to :user
  belongs_to :room
end

試したこと

①user.rbの問題のバリデーションをコメントアウト→rooms、room_users共に正常に保存可能。（勿論usersにはバリデーションがかかっていない）
②user.rbの問題のバリデーションを一意性制約に変更→rooms、room_users共に同様のエラーにより保存されず。
③バリデーションをかけた上でusersの保存→保存可能。バリデーションはpasswordを英数混合に指定する事を意図しているが、その通りに動作する。
④roomsのcreateアクションにてbinding.pryを仕込んでみる。→paramsは意図した通りのパラメーターを送っている。

From: /Users/kintarou/projects/good-question/app/controllers/rooms_controller.rb:9 RoomsController#create:

     7: def create
     8:   @room = Room.new(room_params)
 =>  9:   binding.pry
    10:   if @room.save
    11:     redirect_to root_path
    12:   else
    13:     render :new
    14:   end
    15: end

[1] pry(#<RoomsController>)> @room
=> #<Room:0x00007f843bdd0330 id: nil, room_name: "次の会議の予定", created_at: nil, updated_at: nil>
[2] pry(#<RoomsController>)> Started GET "/cable" for ::1 at 2020-11-10 14:42:27 +0900
[2] pry(#<RoomsController>)> params
=> <ActionController::Parameters {"authenticity_token"=>"lqTqiP2G7lc/kKrguWt7DrNNPPYgsAcX4ISbx/kvFe51zWC+PUvWDr6TOdDTn9HHphM1zm1eAQxrvS5gHKXaKQ==", "room"=><ActionController::Parameters {"user_ids"=>["1", "2"], "room_name"=>"次の会議の予定"} permitted: false>, "commit"=>"ルーム作成", "controller"=>"rooms", "action"=>"create"} permitted: false>
[3] pry(#<RoomsController>)> @room.save!
ActiveRecord::RecordInvalid: バリデーションに失敗しました: Usersは不正な値です
from /Users/kintarou/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/activerecord-6.0.3.3/lib/active_record/validations.rb:80:in `raise_validation_error'
[4] pry(#<RoomsController>)>

⑤サーバー再起動・PC再起動→状況変わらず。

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

Rails 6.0.3.3
ruby 2.6.5p114 (2019-10-01 revision 67812) [x86_64-darwin19]

初めての質問で要領を得ない事、足りない情報あるかもしれませんが、ご容赦ください。
併せてご教授頂ければ幸いです。
よろしくお願い致します。

Answer 1

よくわかっていないので、ちょっと調べた程度の回答です。

バリデーションで困っているなら、belongs_to :user, validate: falseで回避できそうな気がしました。（API dock: belongs_to）

:validate
When set to true, validates new objects added to association when saving the parent object. false by default. If you want to ensure associated objects are revalidated on every update, use validates_associated.

ただvalidate:falseがdefaultなので。。。どうなんだろう

Comments

[kintarou-ryo]

■models/room.rb
has_many :users, through: ;room_users, validate: false
とすることで保存可能になりました！ありがとうございます！

私自身他テーブルのバリデーションは何も操作しなければ無関係という認識だったので、どこでusersのバリデーションがかかることになったのかは不明です。
もっと知識をつけようと思います。

[no1knows]

たしかに中間テーブルで他のテーブルのバリデーションが引っかかるのは変な気がしますね。。。
なにはともあれ解決してよかったです！

Answer 2

bin/rails consoleを起動して以下のコードを実行したら、保存できますか？

user_ids = [1, 2]
room = Room.new(room_name: "次の会議の予定")
room.users << User.where(id: user_ids)
room.save!

これで保存できるようであれば、このように書き換えたらうまくいくかもしれません。
上記のコードが動かない場合はこちらの予測が外れているので忘れてください。

def create
  user_ids = params[:room][:user_ids]  # => ["1", "2"]

  @room = Room.new(room_params)
  @room.users << User.where(id: user_ids)

  if @room.save
    redirect_to root_path
  else
    render :new
  end
end

Comments

[kintarou-ryo]

早速のご回答ありがとうございます！

以下、試してみました。

```
From: /Users/kintarou/projects/good-question/app/controllers/rooms_controller.rb:8 RoomsController#create:

7: def create
=> 8: binding.pry
9: @room = Room.new(room_params)
10: if @room.save
11: redirect_to root_path
12: else
13: render :new
14: end
15: end

[1] pry(#<RoomsController>)> user_ids = [1,2]
=> [1, 2]
[2] pry(#<RoomsController>)> room = Room.new(room_name: "次の会議の予定")
=> #<Room:0x00007f8446254488 id: nil, room_name: "次の会議の予定", created_at: nil, updated_at: nil>
[3] pry(#<RoomsController>)> room.users << User.where(id: user_ids)
User Load (1.7ms) SELECT `users`.* FROM `users` WHERE `users`.`id` IN (1, 2)
↳ (pry):6:in `create'
=> [#<User id: 1, nickname: "太郎", email: "[e-mail]", created_at: "2020-10-29 08:34:09", updated_at: "2020-10-29 08:34:09", phone_number: "09099999999">,
#<User id: 2, nickname: "次郎", email: "[e-mail]", created_at: "2020-10-29 08:34:37", updated_at: "2020-10-29 08:34:37", phone_number: "09099999999">]
[4] pry(#<RoomsController>)> room.save!
ActiveRecord::RecordInvalid: バリデーションに失敗しました: Usersは不正な値です
from /Users/kintarou/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/activerecord-6.0.3.3/lib/active_record/validations.rb:80:in `raise_validation_error'
[5] pry(#<RoomsController>)>
```

一度コードを変えてみましたがやはり「Usersは不正な値です」となります。
一度も試してない方法だったので参考になります。ありがとうございます。