rails メッセージ 編集機能

Question

前提・実現したいこと

､、、
チャットのやり取りができるアプリの制作をしています。
自分の送ったチャットのみ編集消去ができるようにしたいのですが、
うまくいきません。
アドバイスお願いします。
､、、

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

param is missing or the value is empty: message

該当のソースコード

messages_controller

class MessagesController < ApplicationController
  before_action :set_group
  
  def index
    @message = Message.new
    @messages = @group.messages.includes(:user)
  end

  def create
    @message = @group.messages.new(message_params)
    if @message.save
      redirect_to group_messages_path(@group), notice: 'メッセージが送信されました'
    else
      @messages = @group.messages.includes(:user)
      flash.now[:alert] = 'メッセージを入力してください'
      render :index
    end
  end

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

  def update
    @message = current_user.messages.find(params[:id])
    @message.update(message_params)
    redirect_to group_messages_path(@group), notice:"メッセージを編集しました"
  end

  def destroy
    @message = current_user.messages.find(params[:id])
    @message.destroy(message_params)
    redirect_to group_messages_path(@group), notice:"投稿を削除しました。"
  end

  private

  def message_params
    params.require(:message).permit(:content, :image).merge(user_id: current_user.id)
  end

表示されるメッセージのhaml

.message
  .upper-message
    .upper-message__user-name
      = message.user.name
    .upper-message__date
      = message.created_at.strftime("%Y年%m月%d日 %H時%M分")
  .lower-message
    - if message.content.present?
      %p.lower-message__content
        = message.content
    = image_tag message.image.url, class: 'lower-message__image' if message.image.present?


  = form_for [@group, @message] do
    = label_tag "編集する"
    = text_field :content, placeholder: 'type a message'
    .form__mask
      = label :image, class: 'form__mask__image' do
        = icon('fas', 'image', class: 'icon')
        = file_field :image, class: 'hidden'
    = submit_tag

試したこと

message_paramがないということで
@message.update(message_params)など持たせようと思ったのですが
ダメでした。
初心者で勉強中です。
よろしくお願いします。

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

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

Comments

[s.k]

params のログを追記できますか？

[yamadaippei]

https://gyazo.com/da80c421d5b91617fefd9916951b502d
こちたでよろしいでしょうか？

今現在messages/indexにてメッセージの送信ができるのですが
今まで学習してきた編集のやり方だと編集ページにとび
編集できるうにしていたのですが
= link_to "編集", edit_message_path(@message.id)で
edit.html.hamlに移動する事ができませんでした。

なのでindexのページのまま編集できるようにしようと思ったのですが、
messageのcreateアクションが動いて普通にメッセージが送信されてしまいます。。。

[s.k]

まずやりたいことは、一覧ページでメッセージを削除することでいいんでしたよね？

[yamadaippei]

一覧ページにてメッセージの編集です。
もしくは、編集ページに移動しての編集です。

消去もこの後に調べて機能追加しようと考えています。

[yamadaippei]

messagesのコントローラーがこのような感じで
https://gyazo.com/e2d4b1b51a6d18c18130ee2b1d380b1a
メッセージの送信と表示ができているのですが、
そのメッセージそれぞれにまたformを追記してupdateしたいのですが
そのまま新しいメッセージが送られるというのが現状です。

Answer 1

ループでメッセージを表示して、そのメッセージそれぞれにフォームを用意しているということですが、
ループ処理された message ではなく @message が使われているのが新規で作成されてしまう原因だと思います。

@ を外してください。

  = form_for [@group, message] do
    = label_tag "編集する"
    = text_field :content, placeholder: 'type a message'
    .form__mask
      = label :image, class: 'form__mask__image' do
        = icon('fas', 'image', class: 'icon')
        = file_field :image, class: 'hidden'
    = submit_tag

Comments

[yamadaippei]

こちら返信ありがとうございます。
こちらアドバイスのように記述変更すると
編集するform内に元のメッセージの情報を表示して内容の書き換えができるようになりました。
エラー画面
https://gyazo.com/08e84a47ead01c7eeaf7ddc88499c5c8
routes.rb
https://gyazo.com/f573e807862b6bf831a326b67bf6fc5b

その後このようなエラーになるのですが、よければ解決の考え方を教えてもらえますでしょうか？

[s.k]

post は create で put が update です。

[yamadaippei]

なるほど！
今メソッドのpostとputの使い方がいまいち分からなく調べて、
method: :putとする事でメッセージの編集できるようになりました。
本当にありがとうございます！

Answer 2

これで削除できると思います。
message_params は必要ないです。

  def destroy
    @message = current_user.messages.find(params[:id])
    @message.destroy
    redirect_to group_messages_path(@group), notice:"投稿を削除しました。"
  end

Comments

[yamadaippei]

消去ボタンの追加にも答えてくださってありがとうございます。
コントローラーの記述は変更しました。
その後消去ボタンの追加をしたのですがやはり自分で試しに書いた時のように
messageのiodが取れませんでした。
こちらの返信もよければお願いいたします。

エラー画面
https://gyazo.com/4eb979d24a3b59249167d6cab6430f33

[s.k]

`@` を外すんです。

[yamadaippei]

返信ありがとうございます。
返信していただいたこと変更してもエラーが解決せず自分のコードの間違っっている部分を調べて考えていました。

今コードを変更した状態でもう少しのところだと思うのですが、
まだ回答いただけるのであればお願いできないでしょうか。
なんどもすみません。

エラー画面
https://gyazo.com/de6249519eb08f9f3e690a169ba5612a

書き換えたコントローラー
https://gyazo.com/4a33def14550774a525f4a1589866307

haml
https://gyazo.com/d278a64d54184a2af3105cc0189ecc5b

なんども返信いただいているのに解決できなくてほんとすみません。

[s.k]

ここらへんは別の質問で対応するので、質問を新たに作成してコメントにURLを貼ってください。
あとgyazo使うのは構わないのですが、コードをコピーしたり確認するとき別ページにいかなければならないのでなるべく質問に書きましょう。それと、この質問自体に対しては回答出てるのでベストアンサーをお願いします。一つの質問に対して、答えられるだけ答えさせたい気持ちはわからなくもないですが、回答者の気持ちを考えていただきたいのと質問1つに複数の問題を持ち込むのは避けた方がいいです。検索に引っかかった時にこの質問では何をしてるのか閲覧者がよくわからなくなってしまいます。

[yamadaippei]

了解しました。
始めたばかりで基本的な使い方がわかっていませんでした。
申し訳ありません。