ruby on railsのgem"cocoon"でデータを保存した際にArgumentErrorが発生

Question

前提・実現したいこと

ruby on rails の環境でgem "cocoon"を使用して一対多の関係のモデルを同時に保存したい。

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

フォームに入力した内容が登録されない

Started POST "/goals" for 172.22.0.1 at 2021-06-14 03:07:07 +0000
Cannot render console from 172.22.0.1! Allowed networks: 127.0.0.1, ::1, 127.0.0.0/127.255.255.255
Processing by GoalsController#create as JS
  Parameters: {"utf8"=>"✓", "authenticity_token"=>"WRLgldSJ8fdtds5dQ2fIeVg8f8Pntonoe94WPzxHxwHVZSTL518WoJ4IbLBpl1yL7mmSIGvj60JxZCVpU5gDsA", "goal"=>{"content"=>"asdf", "start_date"=>"2021-06-14T12:06", "end_date"=>"2021-06-15T12:06", "result"=>"1", "tasks_attributes"=>{"0"=>{"content"=>"asdf", "action"=>"asdf", "start_date"=>"2021-06-14T12:06", "end_date"=>"2021-06-15T12:07", "status"=>"1", "_destroy"=>"false"}}}, "commit"=>"作成"}
   (0.5ms)  BEGIN
  ↳ app/controllers/goals_controller.rb:31
  User Load (0.7ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = $1 LIMIT $2  [["id", 1], ["LIMIT", 1]]
  ↳ app/controllers/goals_controller.rb:31
  CACHE User Load (0.0ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = $1 LIMIT $2  [["id", 1], ["LIMIT", 1]]
  ↳ app/controllers/goals_controller.rb:31
   (0.6ms)  ROLLBACK
  ↳ app/controllers/goals_controller.rb:31
保存失敗しました!!!!!!!!
Completed 500 Internal Server Error in 100ms (ActiveRecord: 5.0ms)


  
ArgumentError (wrong number of arguments (given 0, expected 2)):
  
app/controllers/goals_controller.rb:36:in `create'

該当のソースコード

[goal.rb]
class Goal < ApplicationRecord
  has_many :tasks, inverse_of: :goal
  accepts_nested_attributes_for :tasks, allow_destroy: true

  validates :user_id, presence: true
  validates :content, presence: true
  validates :result, presence: true
end

[task.rb]
class Task < ApplicationRecord
  belongs_to :goal

  validates :goal_id, presence: true
  validates :user_id, presence: true
  validates :content, presence: true
  validates :action, presence: true
  validates :status, presence: true
end

[goals_controller.rb]
  def new
    @goal = Goal.new
    @task = @goal.tasks.build
  end

  def create
    @goal = Goal.new(goal_params)

    if @goal.save
      logger.debug "保存成功しました!!!!!!!!"
      redirect_to "/", notice: "目標を作成しました"
    else
      logger.debug "保存失敗しました!!!!!!!!"
      logger.debug @goal.errors.full_message
      render new_goal_path
    end
  end

  private

  def goal_params
    params.require(:goal).permit(:content, :start_date, :end_date, :result,
                                  tasks_attributes: [:id, :content, :action, :start_date, :end_date, :status, :_destroy])
                          .merge(user_id: 1,
                                  tasks_attributes: [user_id: 1])# for develop
  end
end

ログイン機能は作成中のため、「user_id: 1」で固定値を入力しています。

[new.html.erb]
        <%= form_with model: @goal do |f| %>
          <div class="goal-table">
              <p> 目標：<%= f.text_field :content, placeholder: "達成したいこと" %> </p>
              <p> 開始日：<%= f.datetime_field :start_date %> </p>
              <p> 終了日：<%= f.datetime_field :end_date %> </p>
              <p> ステータス：<%= f.select :result, GoalState.all.collect { |a| [a.state, a.id] } %></p>
              <div class="under-line"></div>

              <%= f.fields_for :tasks do |t| %>
                <%= render "task_fields", f: t %>
              <% end %>

              <div id="detail-association-insertion-point"></div>

              <%= link_to_add_association "+課題を追加", f, :tasks,
              class: "btn-task-add btn btn-secondary",
              data: {
                association_insertion_node: '#detail-association-insertion-point',
                association_insertion_method: 'after'
              } %>
          </div>
          <%= f.submit "作成", class: "btn btn-primary" %>
          <%= link_to "キャンセル", goals_path, class: "btn btn-danger" %>
        <% end %>


[_task_fields.html.erb]
<div class="nested-fields">
  <div class="task-form-area">
    <div>
      <p> 課題：<%= f.text_field :content, placeholder: "目標に対する課題" %> </p>
      <p> アクション：<%= f.text_field :action, placeholder: "課題を解決するための行動" %> </p>
      <p> 開始日：<%= f.datetime_field :start_date %> </p>
      <p> 終了日：<%= f.datetime_field :end_date %> </p>
      <p> ステータス：<%= f.select :status, TaskState.all.collect { |a| [a.state, a.id] } %></p>
    </div>
    <div>
      <%= link_to_remove_association "課題を削除", f, class: "btn-task-delete btn btn-secondary" %>
    </div>
  </div>
</div>

疑問点

・（エラー内容と関連するか不明ですが、）ストロングパラメータの「merge」部分で関連するモデル(上記の場合はtask)の値を入力することは可能でしょうか？

Answer 1

とりあえず、出ているエラーメッセージを解消するには

logger.debug @goal.errors.full_message

logger.debug @goal.errors.full_messages.join("\n")

です

Comments

[genki0126]

ご指摘ありがとうございます。
ログ出力の処理を修正したところ、以下のログが表示されるようになりました。

Tasks goal can't be blank
Tasks content can't be blank
Tasks action can't be blank
Tasks status can't be blank

Taskへ入力値を渡せていないようなので、ストロングパラメータの辺りかパーシャルにしたフォームの連携部分が怪しそうな雰囲気ですが、参考にした記事と実装の差異がわからず詰まってしまいました。

参考にした記事：https://qiita.com/kmjooh/items/a1613531873a22fa7862

[asm]

なるほど、tasks_attributes: [{user_id: 1}]だけはblankになってないのがヒントですね
Hash#mergeは衝突した場合、引数の方の値で上書きします。

_task_fields.html.erbに<%= f.hidden_field :user_id, 1 %>と書いてしまった方が楽っぽいですね

[genki0126]

ご指摘の通り修正したところ、エラーログが「Tasks goal can't be blank」のみになりました。
このときにgoal_idのバリデーションを解除すると正常に登録できたため、goal_idが入力できていない状態ということがわかりました。

goal_idを入力するにはどのようにすればいいのでしょうか。
（cocoonのREADMEを見ても外部キーのカラムはxxx_attributes部分に追加されていないため、明示的に入力する処理は実装していませんでした。）

試しにストロングパラメータのtasks_attributesの部分にgoal_idを追加し、user_idと同じようにhidden_fieldでgoal_idを入力してみましたが、「Tasks goal can't be blank」のまま変化しませんでした。

[asm]

気になるのは
belongs_to :goalの時点でvalidates :goal_id, presence: trueと同様のvalidateがされるので
後者は不要だったはずです

[genki0126]

そうだったんですね。。。
外部キーのvalidateは削除して実装いたします。
ご協力ありがとうございました。