MealPostのサブ項目としてFoodItemを追加する

[kudojp]

概要

• MealPostのサブ項目としてFoodItemを追加する
• MealPost投稿時にFoodItemを一品以上追加することを必須とする
• 各MealPostの詳細ページから、編集ができるようにする

仕様

MealPost投稿時

• MealPostを投稿する際には1つ以上のFoodItemを登録する
• FoodItemは以下の項目からなる
  • 食品名(string, not null)　(例)カレーライス, 餃子
  • 量(string, null許可) (例)一杯、４個、たくさん、300g
  • 総カロリー(unsigned int, null許可、単位はカロリー) (例)500

MealPostカード

Homeページのタイムライン、User詳細ページのMealPost一覧ページの各MealPostカードでは(とりあえず)FoodItemの詳細な一覧は表示ない。ただし、品目数とカロリーの合計値は表示する。

• カロリーに関しては、MealPostに所属する全てのFoodItemのカロリーが入力されている場合には540kcalと、全て入力されていない場合には、540+kcalと、一つも入力されていない場合には+kcalと表示する

MealPost詳細ページ

• MealPostの詳細ページではFoodItemの一覧が表示される。

設計

FoodItemsテーブルの作成

フィールド名	データ型	制約
id	unsigned int	PK
name	string	not null
amount	string
calory	int (unsigned)
meal_post_id	int (unsigned)	not null, FK: MealPost(id)

MealPostsテーブルの更新

• calories_sumというカラムを作る
  • カウンターカルチャーでMealPostが登録、変更された際には自動更新される。

更新後のテーブルは以下。

フィールド名	データ型	制約
id	unsigned int	PK
time	datetime	not null
user_id	unsigned int	FK: user(id), not null
num_items	unsigned int	not null
sum_calories	unsigned int
is_all_calory_sum	Boolean	not null

[kudojp]

MealPostsのUpdate(nestされたFoodItemsを含めて)がうまくいかない

ログ

app/controllers/meal_posts_contoroller.rb

class MealPostsController < ApplicationController
  before_action :authenticate_user!, only: %i[create destroy update]

  def create
    @new_meal_post = current_user.meal_posts.build(meal_post_params)

    # meal_post object used for rendering partial form after successfully creating meal_post
    @next_meal_post = current_user.meal_posts.new
    3.times { @next_meal_post&.food_items&.build }

    if @new_meal_post.save
      return respond_to do |format|
        # TODO: friendly forwardingを実装
        format.html { redirect_to root, notice: 'You successfully made a meal post.' }
        format.js
      end
    end

    respond_to do |format|
      # TODO: friendly forwardingを実装
      # TODO: errorを伝搬するなりして、alertをもう少しdescriptiveにする
      format.html { redirect_to root, alert: 'You could not make a meal post.' }
      format.js do
        render partial: 'meal_posts/create_failure', status: :bad_request
      end
    end
  end

  def update
    @meal_post = MealPost.find(params[:id])
    return redirect_to root_path, alert: 'You are not authorized to update the post' if @meal_post.user_id == current_user

    if @meal_post.update(meal_post_params)
      return respond_to do |format|
        # TODO: friendly forwardingを実装
        format.html { redirect_to root, notice: 'You have successfully updated a meal post.' }
        format.js
      end
    end

    respond_to do |format|
      # TODO: friendly forwardingを実装
      # TODO: errorを伝搬するなりして、alertをもう少しdescriptiveにする
      format.html { redirect_to root, alert: 'You could not make a meal post.' }
      format.js do
        render partial: 'meal_posts/create_failure', status: :bad_request
      end
    end
  end

  private

  def meal_post_params
    params.require(:meal_post).permit(:content, :time, food_items_attributes: %i[name amount calory _destroy])
  end

app/views/meal_posts/_edit.html.erb

<%= form_for(meal_post, html: { method: :patch }, remote: true) do |f| %>
  <div class="field">
    <%= f.text_field :content, placeholder: 'メニュー名' %>
  </div><br>

  <div class="fields">
    <%= f.fields_for :food_items do |fi_field| %>
      <%= render partial: 'meal_posts/food_item_fields', locals: {f: fi_field} %>
    <% end %>
    <div class='links'>
      <%= link_to_add_association '', f, :food_items, partial: 'meal_posts/food_item_fields', class: "fas fa-plus-circle" %>
    </div>
  </div><br>

  <div class="field">
    <%= f.text_field :time, class: 'datetimepicker', placeholder: '食べた日時を選択' %>
  </div>

  <!-- TODO: 画像もアップロードできるようにする -->

  <%= f.submit "投稿する" %>
<% end %>

これだとcreateはうまくいくが、updateはうまくいかない。

コンソールログ

create時

Started POST "/meal_posts" for ::1 at 2020-05-17 19:57:17 +0900
Processing by MealPostsController#create as JS
  Parameters: {"meal_post"=>{"content"=>"new post", "food_items_attributes"=>{"0"=>{"name"=>"カレー", "amount"=>"", "calory"=>"", "_destroy"=>"false"}, "1"=>{"name"=>"", "amount"=>"", "calory"=>"", "_destroy"=>"false"}, "2"=>{"name"=>"", "amount"=>"", "calory"=>"", "_destroy"=>"false"}}, "time"=>"2020-05-04 12:00"}, "commit"=>"投稿する"}
(以下略)

update時

ArgumentError (When assigning attributes, you must pass a hash as an argument, NilClass passed.):
  
app/controllers/meal_posts_controller.rb:50:in `update'
Started PATCH "/meal_posts/260" for ::1 at 2020-05-17 19:55:53 +0900
Processing by MealPostsController#update as JS
  Parameters: {"meal_post"=>{"content"=>"カレー", "food_items_attributes"=>{"0"=>{"name"=>"チョコ", "amount"=>"100", "calory"=>"", "_destroy"=>"1", "id"=>"225"}}, "time"=>"2020-04-26 12:00"}, "commit"=>"投稿する", "id"=>"260"}
  User Load (0.6ms)  SELECT "users".* FROM "users" WHERE "users"."id" = $1 ORDER BY "users"."id" ASC LIMIT $2  [["id", 1], ["LIMIT", 1]]
  MealPost Load (0.7ms)  SELECT "meal_posts".* FROM "meal_posts" WHERE "meal_posts"."id" = $1 ORDER BY "meal_posts"."time" DESC LIMIT $2  [["id", 260], ["LIMIT", 1]]
  ↳ app/controllers/meal_posts_controller.rb:30:in `update'
Unpermitted parameter: :id
   (0.1ms)  BEGIN
  ↳ app/controllers/meal_posts_controller.rb:33:in `update'
  User Load (0.2ms)  SELECT "users".* FROM "users" WHERE "users"."id" = $1 LIMIT $2  [["id", 1], ["LIMIT", 1]]
  ↳ app/controllers/meal_posts_controller.rb:33:in `update'
  FoodItem Load (0.3ms)  SELECT "food_items".* FROM "food_items" WHERE "food_items"."meal_post_id" = $1  [["meal_post_id", 260]]
  ↳ app/controllers/meal_posts_controller.rb:33:in `update'
   (0.1ms)  COMMIT
  ↳ app/controllers/meal_posts_controller.rb:33:in `update'
No template found for MealPostsController#update, rendering head :no_content
Completed 204 No Content in 64ms (ActiveRecord: 6.7ms | Allocations: 27869)

わかったこと

子モデルをnestする親モデルはcreate時とupdate時でparamsの中身が違うようだ。以下のようにブラウザからの送信時にはrequest bodyは同形式であるが、PATCHメソッドではRails側でこれを配列と解釈してparamsへと変換することができていない。(この質問と同じじ状況)
create

update

解決策

以下のようにstrong parameterを設定して解決。
app/controllers/meal_posts_contoroller.rb

  private

  def new_meal_post_params
    params.require(:meal_post).permit(:content, :time, food_items_attributes: %i[name amount calory _destroy])
  end

  def update_meal_post_params
    # patchメソッドだとRailsアプリケーションでrequest_bodyの配列をうまく解釈できない(hashになる)ため
    params[:meal_post][:food_items_attributes] = params[:meal_post][:food_items_attributes].values
    params.require(:meal_post).permit(:content, :time, food_items_attributes: %i[name amount calory _destroy id])
  end

[azumag]

モデルに定数化したいかも

Suggested change

	3.times { @new_meal_post&.food_items&.build }
	MealPost::DEFAULT_FOOD_NUM.times { @new_meal_post&.food_items&.build }

[azumag]

こちらも定数化すると二重管理を防げる