Formオブジェクトを用いた編集機能

Question

前提・実現したいこと

プログラミング初心者です。
画像や動画と共にメッセージを残すアプリケーションを開発中で
Formオブジェクトを用いた編集機能を実装中です。
Routing Error、paramsのmissing or empty、Couldn't find Message with 'id'=:id
とエラーを解消してきたのですが今回のエラーでどうすればいいか分かりません。
どなたかご教授お願いいたします。

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

エラーメッセージ
ActiveRecord::NotNullViolation in MessagesController#update
Mysql2::Error: Field 'name' doesn't have a default value

ターミナル
Started PATCH "/messages/100" for ::1 at 2021-01-30 14:16:51 +0900
Processing by MessagesController#update as HTML
  Parameters: {"authenticity_token"=>"gM9DoUlMJ8F5/sUiOA59e4RyRgv1GPe8lD2YgHNSFCKlbD7u+b6fpftH5JvtesIgT7h8GXF8U7Wraky0JNQNFQ==", "message"=>{"title"=>"qaz123", "message"=>"wsx", "whom"=>"edc", "open_plan(1i)"=>"2022", "open_plan(2i)"=>"2", "open_plan(3i)"=>"22", "name"=>"updateテスト"}, "commit"=>"想いを更新する", "id"=>"100"}
  User Load (0.9ms)  SELECT `users`.* FROM `users` WHERE `users`.`id` = 1 ORDER BY `users`.`id` ASC LIMIT 1
  Message Load (0.3ms)  SELECT `messages`.* FROM `messages` WHERE `messages`.`id` = 100 LIMIT 1
  ↳ app/controllers/messages_controller.rb:36:in `update'
  Tag Load (0.3ms)  SELECT `tags`.* FROM `tags` WHERE `tags`.`name` IS NULL LIMIT 1
  ↳ app/models/message_tag.rb:29:in `update'
   (0.2ms)  BEGIN
  ↳ app/models/message_tag.rb:29:in `update'
  Tag Exists? (0.2ms)  SELECT 1 AS one FROM `tags` WHERE `tags`.`name` IS NULL LIMIT 1
  ↳ app/models/message_tag.rb:29:in `update'
  Tag Create (0.5ms)  INSERT INTO `tags` (`created_at`, `updated_at`) VALUES ('2021-01-30 05:16:51.237704', '2021-01-30 05:16:51.237704')
  ↳ app/models/message_tag.rb:29:in `update'
   (0.1ms)  ROLLBACK
  ↳ app/models/message_tag.rb:29:in `update'
Completed 500 Internal Server Error in 23ms (ActiveRecord: 2.5ms | Allocations: 13849)


  
ActiveRecord::NotNullViolation (Mysql2::Error: Field 'name' doesn't have a default value):
  
app/models/message_tag.rb:29:in `update'
app/controllers/messages_controller.rb:37:in `update'

該当のソースコード

messages_controller.rb
class MessagesController < ApplicationController
  before_action :authenticate_user!, except: [:index]
  before_action :search_message, only: [:index, :search]

  def index
    @q = Message.ransack(params[:q])
    @messages = Message.includes(:user).order("created_at DESC")
  end

  def new
    @message_tag = MessageTag.new
  end

  def create
    @message_tag = MessageTag.new(message_params)
    if @message_tag.valid?
      @message_tag.save
      return redirect_to root_path
    else
      render :new
    end
  end

  def show
    @message = Message.find(params[:id])
  end

  def edit
    @message = Message.find(params[:id])
    if current_user.id != @message.user.id
      redirect_to root_path
    end
  end

  def update
    @message = Message.find(params[:id])
    if @message_tag = MessageTag.new.update
      return redirect_to root_path
    else
      render :edit
    end
  end

  def destroy
    @message = Message.find(params[:id])
    if current_user.id == @message.user.id
      @message.destroy
      redirect_to root_path
    end
  end

  private
  def message_params
    params.require(:message_tag).permit(:title, :whom, :message, :open_plan, :name, :video, images: []).merge(user_id: current_user.id)
  end
end

message_tag.rb
class MessageTag

  include ActiveModel::Model
  include ActiveModel::Attributes
  include ActiveRecord::AttributeAssignment
  attr_accessor :title, :whom, :open_plan, :message, :images, :video, :name, :user_id

  with_options presence: true, length: { maximum: 50 } do
    validates :title
    validates :whom
  end
  
  with_options presence: true do
    validates :message, length: { maximum: 200 }
    validates :open_plan
    validates :user_id
    validates :name, length: { maximum: 20 }
  end

  def save
    @message = Message.create(title: title, whom: whom, open_plan: open_plan, message: message, images: images, video: video, user_id: user_id)
    tag = Tag.find_or_create_by(name: name)

    @message_tag = MessageTagForm.create(message_id: message.id, tag_id: tag.id)
  end

  def update
    @message = Message.update(title: title, whom: whom, open_plan: open_plan, message: message, images: images, video: video, user_id: user_id)
    tag = Tag.find_or_create_by(name: name)

    @message_tag = MessageTagForm.update(message_id: message.id, tag_id: tag.id)
  end
end

message.rb
class Message < ApplicationRecord
  belongs_to :user
  has_many_attached :images
  has_one_attached :video
  has_many :message_tag_forms, dependent: :destroy
  has_many :tags, through: :message_tag_forms

  def self.search(search, id)
    if search != ""
      Item.joins(:tags).where(tags: {name: "#{search}"} , user_id: id)
    else 
      Item.where(user_id: id)
    end
  end
end

tag.rb
class Tag < ApplicationRecord
  has_many :messages, through: :message_tag_forms
  has_many :message_tag_forms

  validates :name, uniqueness: true
end

class MessageTagForm < ApplicationRecord
  belongs_to :message
  belongs_to :tag
end

class CreateTags < ActiveRecord::Migration[6.0]
  def change
    create_table :tags do |t|
      t.string :name, null:false, uniqueness: true
      t.timestamps
    end
  end
end



class CreateMessages < ActiveRecord::Migration[6.0]
  def change
    create_table :messages do |t|
      t.string :title, null: false
      t.string :whom, null: false
      t.text :message, null: false
      t.date :open_plan, null: false
      t.string :encrypted_password
      t.references :user, null: false
      t.timestamps
    end
  end
end



class CreateMessageTagForms < ActiveRecord::Migration[6.0]
  def change
    create_table :message_tag_forms do |t|
      t.references :message, foreign_key: true
      t.references :tag, foreign_key: true
      t.timestamps
    end
  end
end

edit.html.erb
<div class="new-message">
  <h1>残したい想い</h1>
  <%= form_with model: @message, multipart: true, local: true do |f| %>
  <%= render 'shared/error_messages', model: f.object %>
  <div class="posting-form">
    <div class="form">
      タイトル
      <span class="indispensable">必須</span>
    </div>
    <%= f.text_field :title, class:"form-title", id:"form-title", placeholder:"タイトル名(必須 50文字以内)", maxlength:"50" %>
    <div class="form">
      メッセージ
      <span class="indispensable">必須</span>
    </div>
    <%= f.text_area :message, class:"form-message", id:"form-message", placeholder:"メッセージ内容(必須 200文字以内)例)20年後の子供の誕生日に見て欲しい動画です。" ,rows:"5", maxlength:"200" %>
    <div class="form">
      誰に対してか
      <span class="indispensable">必須</span>
    </div>
    <%= f.text_field :whom, class:"form-whom", id:"form-whom", placeholder:"誰に宛てたものか(必須 50文字以内)例)20年後の子どもたちへ", maxlength:"50" %>
    <div class="form">
      開封予定日
      <span class="indispensable">必須</span>
    </div>
    <div class="form-open-plan">
      <%= raw sprintf(
                  f.date_select(
                    :open_plan,
                    class: 'open-plan-object',
                    use_month_numbers: true,
                    prompt:'--',
                    start_year: (Time.now.year),
                    end_year: (Time.now.year + 100),
                    date_separator: '%s'),
                  "<p> 年 </p>", "<p> 月 </p>") + "<p> 日 </p>" %>
    </div>
    <div class="form-caution">
      ※※※画像か動画どちらか1つの投稿にしてください。※※※
    </div>
    <div class="image-form">
      <div class="form">
      届ける想い(画像)
      </div>
      <div class="click-upload">
        <p>クリックしてファイルをアップロード</p>
        <%= f.file_field :images, name: 'message_tag[images][]', id:"form-image" %>
        <div id="image-list"></div>
      </div>
    </div>
    <div class="video-form">
      <div class="form">
      届ける想い(動画)
      </div>
      <div class="click-upload">
        <p>クリックしてファイルをアップロード</p>
        <%= f.file_field :video, id:"form-video" %>
      </div>
    </div>
    <div class="tag-form">
    タグ
    </div>
    <%= f.text_field :name, id:"form-tag", placeholder:"誕生日", maxlength:"20"%>
  </div>
  <div class="message-btn-contents">
    <%= f.submit "想いを更新する" ,class:"message-btn" %>
    <%= link_to 'もどる', root_path, class:"back-btn" %>
  </div>
  <% end %>

</div>

Request
Parameters:

{"_method"=>"patch",
 "authenticity_token"=>"gM9DoUlMJ8F5/sUiOA59e4RyRgv1GPe8lD2YgHNSFCKlbD7u+b6fpftH5JvtesIgT7h8GXF8U7Wraky0JNQNFQ==",
 "message"=>{"title"=>"qaz123", "message"=>"wsx", "whom"=>"edc", "open_plan(1i)"=>"2022", "open_plan(2i)"=>"2", "open_plan(3i)"=>"22", "name"=>"updateテスト"},
 "commit"=>"想いを更新する",
 "id"=>"100"}

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

参考記事

https://teratail.com/questions/291532

https://qiita.com/Tiima/items/b14c73df98d0465cbb52

https://qiita.com/divclass123/items/34c4a5a7200238a0589c

不足情報があれば教えてください。随時追記させて頂きます。
よろしくお願いいたします

Answer 1

モデルもschemaも載ってないのでエスパーモードです。
「Messaveにはnameが必須なのにそれが入っていない」
というエラーです。
def message_params　と　Parameters:　をよく見比べてください。

Comments

[shunxile]

>> winterboumさん
失礼いたしました。モデルとDB内容追記いたしました。

[winterboum]

def message_params　と　Parameters:　をよく見比べてください。

[shunxile]

>> winterboumさん
:title、:message、:whom、:open_plan、:nameと不足なくparamsにあると思うのですが何が足らないのでしょうか？

[winterboum]

Paramaters では "message"=>{}です
def message_params　では
params.require(:message_tag)　です

[shunxile]

>> winterboumさん
params.require(:message_tag)を
params.require(:message)としましたが同様のエラーが発生してしまいます。
新規投稿時のParametersと比べると確かに新規投稿時は現在"message"の部分が"message_tag"となっているので最初のfindの呼び出しの時点で間違えているという事でしょうか？

[winterboum]

「同様」と「全く同じ」とでは状況が違います。
なので、エラーメッセージを厳密に確認してください。
その時の Parameters の確認も。
「新規投稿時のParametersと比べると確かに新規投稿時は現在"message"の部分が"message_tag"となっているので最初のfindの呼び出しの時点で間違えているという事でしょうか？」　⇐ 　ここ何を聞いているのか理解出来ないです。saveの話？findの話？それが互いに関連してます？

[shunxile]

>> winterboumさん
:messageに変更時エラーメッセージは全く一緒でした。以下はParametersです。
Parameters: {"authenticity_token"=>"3XI4FNf0H3K1yuM/nj08aNo6PN5h7qHXO8mrwAE7y1/40UVbZwanFjdzwoZLSYMzEfAGzOWKBd4Enn/0Vr3SaA==", "message"=>{"title"=>"qaz123", "message"=>"wsx", "whom"=>"edc", "open_plan(1i)"=>"2022", "open_plan(2i)"=>"2", "open_plan(3i)"=>"22", "name"=>"updateテスト"}, "message_tag"=>{"images"=>[#<ActionDispatch::Http::UploadedFile:0x00007fb3b0ff3780 @tempfile=#<Tempfile:/var/folders/pg/6dn2c_091hb4pfr45vz6w9fr0000gn/T/RackMultipart20210131-37437-1wjehwp.jpg>, @original_filename="717Z-iIrpgL._AC_SY879_.jpg", @content_type="image/jpeg", @headers="Content-Disposition: form-data; name=\"message_tag[images][]\"; filename=\"717Z-iIrpgL._AC_SY879_.jpg\"\r\nContent-Type: image/jpeg\r\n">]}, "commit"=>"想いを更新する", "id"=>"100"}

Parametersでは:message、def message_paramsでは:message_tagとご指摘頂いたので新規投稿は正常に動作ていたので新規投稿時のParametersではどのようになっているのだろうと確認したところ:message_tagとなっていました。
しかし現在の編集時のParametersでは:messageなのでupdateとcreate時で何か違いがあるのではないかと思いupdate内の@message=Message.find(params[:id])が間違えているのではないかと思い聞きました。

[winterboum]

new　と edit で動きが異なるとすると、viewに問題がありそうですね

[shunxile]

>> winterboumさん
そうですよね.....
追記しようとしましたが文字制限でできなかったので2つのviewページの違う箇所を抜粋します。
editとnewのhtml.erbを比べると違いがあるのが1箇所でした。
3行目
new:<%= form_with model: @message, url: messages_path, local: true do |f| %>
edit:<%= form_with model: @message, url: message_path(@message.id), method: :patch, multipart: true, local: true do |f| %>

[winterboum]

両方共、urlは不要です。@messageをみてRailsが適切に計らいます。methodも。
で、
new は @message = MessageTag.new
edit は @message = Message.find(params[:id])
違うものですね。
で、
MessageTag.new なら　@message_tag としないとcode読む他人は間違えます。

[shunxile]

>> winterboumさん
url・methodは削除済で正常に動作することも確認しました。ありがとうございます。
newは@message_tagに変更しました。
イメージでは@messageにMessage.findで編集したい情報を呼び出してFormオブジェクトを用いてupdateで更新すると思って
@message = Message.find(params[:id])
@message = Message.create(title: title, whom: whom, open_plan: open_plan, message: message, images: images, video: video, user_id: user_id)
tag = Tag.find_or_create_by(name: name)
@message_tag = MessageTagForm.create(message_id: message.id, tag_id: tag.id)
という流れにしたのですが違いましたでしょうか？
現状コード編集した内容に追記しましたがエラー内容もParameres:も変化がありませんでした。

[winterboum]

updateといいながらcreateしてるように見えますが

[shunxile]

>> winterboumさん
このエラーより前にupdateが未定義ですというエラーとなり定義してもエラー内容等全く変わらなかったので今のようになっていました。
現在以下のように修正しましたがエラー内容、Parameters:共に全く変化はありませんでした。
def update
@message = Message.update(title: title, whom: whom, open_plan: open_plan, message: message, images: images, video: video, user_id: user_id)
tag = Tag.find_or_create_by(name: name)

@message_tag = MessageTagForm.update(message_id: message.id, tag_id: tag.id)
end

tag = Tag.find_or_create_by(name: name)が間違っているのでしょうか？
tag = Tag.update(name: name)とするとエラー内容が
Mysql2::Error: Column 'name' cannot be nullと変化しました。

[winterboum]

読みにくい、質問欄を編集して

[shunxile]

>> winterboumさん
編集しました。

[winterboum]

同じことを何度も書かせないで。
def edit
@message = Message.find(params[:id])
だから
Parameters:が　{"message"=>{}}
になるので
def message_params で params.require(:message_tag) が nil になるの。

[shunxile]

>> winterboumさん
申し訳ありません。
つまりどのようにすればよろしいのでしょうか？

[winterboum]

案1-1 create_params　と　update_params　を作る
案1-2 message_params(:message), message_params(:message_tag) と使い分けるように直す
案2 edit でも message_tag を渡す。

どの案にしても「、paramsのmissing or empt」はなくなりますが、正しいMessageをupdateするというところでひと工夫が必要になりそう。
「Formオブジェクト」を使う気になったその原典に戻ってじっくり読みなおしたほうが良いですよ