Formオブジェクトを用いた編集機能でupdateができません

Question

前提・実現したいこと

メッセージを画像や動画と共に投稿するアプリケーションを開発してます。
新規投稿機能、詳細機能、編集機能、削除機能と一通り実装した後
タグ機能を追加実装した際にFormオブジェクトを用いて実装した後
編集機能でNoMethodErrorが発生するようになりました。
編集を完了させるにはどうすればよろしいでしょうか？
よろしくお願いします。

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

NoMethodError in MessagesController#update
undefined method `update' for MessageTag:Class

該当のソースコード

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 = MessageTag.new
  end

  def create
    @message = MessageTag.new(message_params)
    if @message.valid?
      @message.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])
    @message = MessageTag.update
  end

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

  def search
    set_search
  end

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

  def search_message
    @m = Message.ransack(params[:q])
  end

  def set_search
    @search = Message.ransack(params[:q])
    @search_messages = @search.result(distinct: true).order(created_at: "DESC").includes(:user)
  end
end

message_tab.rb(Formオブジェクト)

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)

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

  def updateupdate
    @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)
    if MessageTagForm.update(message_id: message.id, tag_id: tag.id)
      return redirect_to root_path
    else
      render :edit
    end
  end
end

<div class="new-message">
  <h1>残したい想い</h1>
  <%= form_with model: @message, 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>

35: def update
    36:   @message = Message.find(params[:id])
 => 37:   binding.pry
    38:   @message = MessageTag.update(message_params)
    39: end

[1] pry(#<MessagesController>)> params
=> <ActionController::Parameters {"_method"=>"patch", "authenticity_token"=>"IpLwOPQlpeuAei8d+q0w6Q0kR57su7EmX9hk+ClskjYHMY13RNcdjwLDDqQv2Y+yxu59jGjfFS9gj7DMfuqLAQ==", "message"=>{"title"=>"アイウエオ", "message"=>"更新", "whom"=>"さしすせそ", "open_plan(1i)"=>"2022", "open_plan(2i)"=>"2", "open_plan(3i)"=>"2", "name"=>"更新"}, "commit"=>"想いを更新する", "controller"=>"messages", "action"=>"update", "id"=>"87"} permitted: false>
[2] pry(#<MessagesController>)> message_params
=> <ActionController::Parameters {"title"=>"アイウエオ", "whom"=>"さしすせそ", "message"=>"更新", "open_plan(1i)"=>"2022", "open_plan(2i)"=>"2", "open_plan(3i)"=>"2", "name"=>"更新", "user_id"=>1} permitted: true>

試したこと

message_tagにupdateが未定義となっているのでmessage_tagにupdateを定義しましたが更新できませんでした

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

ここにより詳細な情報を記載してください。

Answer 1

いくつか問題があります。
まず MessageTag には update が無いのは事実ですね。あるのは updateupdateです。

それをupdateになおしても params[:id] を使っていますが、modelにはparamsは伝わりません。引数で渡してください。

#MessagesController
 def update
    @message = MessageTag.update params

# MessageTag
 def update(params)
    @message = Message.find(params[:id])

MessageTag#updateの中身が正しいかは見ていませんが、これで「質問の内容」はクリア出来るでしょう

Answer 2

MessageTagをインスタンス化する必要があるのでは？
インスタンス化されてないからnilから呼び出し→NoMethodErrorと推察します。

updateならfindで対象データを持ってくるとかで。
Active Record の基礎#5.3 Update
【Rails】updateメソッドの使い方を徹底解説！

Comments

[shunxile]

>>m.ts10806さん
回答ありがとうございます。
message_tag.rbのupdateアクションでfindを用いてデータを持ってくるということですか？
いちようそちらにfindを追記して編集したのですが同じくエラーが出るんですが、、、l

[m.ts10806]

コードを追記してください。「書きました」という説明だけでは何も分かりません。

細かいですが「いちおう」ですね。

[shunxile]

>>m.ts10806さん
失礼しました。追記いたしました。