updateメソッドで更新ができないを解決したい。

Question

実現したいこと

ここに実現したいことを箇条書きで書いてください。

• ▲▲機能を動作するようにする

updateメソッドで更新ができないを解決したい。

前提

railsえtodoアプリの開発練習しています。
更新機能を実装中にエラーが発生しました。

Started PATCH "/tasks/:id" for ::1 at 2023-06-10 21:29:23 +0900
Processing by TasksController#update as TURBO_STREAM
  Parameters: {"authenticity_token"=>"[FILTERED]", "task"=>{"title"=>"ｒfew"}, "commit"=>"更新", "id"=>":id"}
  Task Load (0.2ms)  SELECT "tasks".* FROM "tasks" WHERE "tasks"."id" = ? LIMIT ?  [["id", nil], ["LIMIT", 1]]
  ↳ app/controllers/tasks_controller.rb:25:in `update'
Completed 404 Not Found in 16ms (ActiveRecord: 0.2ms | Allocations: 1229)


  
ActiveRecord::RecordNotFound (Couldn't find Task with 'id'=:id):

該当のソースコード

class TasksController < ApplicationController

  def index
    @tasks = Task.all
  end

  def new

  end

  def create
    Task.create(title:params[:title])
    redirect_to "/tasks"
  end

  def edit
    @task = Task.find(params[:id])
  end

  def update
    @task = Task.find(params[:id])
    @task.update(title: params[:title])
    redirect_to "/tasks"
  end

  def destroy
    @task = Task.find(params[:id])
    @task.destroy
    redirect_to "/tasks"
  end
end

<h2>EDIT TASK</h2>
<%= form_with model: @task, url: "/tasks/:id", local: true do |f| %>
  <p><%= f.text_field :title %></p>
  <p><%= f.submit "更新" %></p>
<% end %>

Rails.application.routes.draw do
  get "tasks" => "tasks#index"
  get "tasks/new" => "tasks#new"
  post "tasks/create" => "tasks#create"
  get "tasks/:id/edit" => "tasks#edit"
  patch "tasks/:id" => "tasks#update"
  delete "tasks/:id" => "tasks#destroy"
end

試したこと

ルーティングのupdateへ渡すところを以下のように変更しましたが、更新できませんでした。

 変更前：
 patch "tasks/:id" => "tasks#update"

変更後：
 patch "tasks/:id" => "tasks#update", as: :update

それに伴いform_withのurlを以下のように変更

変更前：
<%= form_with model: @task, url: "/tasks/:id", local: true do |f| %>

変更後：
<%= form_with model: @task, url: update_path, local: true do |f| %>

ストロングパラメーターを設定してやっても見ましたが同じようなエラーです。

以下エラーのTURBO_STREAMが原因かを調べていましたが、正確に理解できませんでしたが、おそらくJavaScriptが動いているように理解しています。
しかし、form_withでlocal: trueにしているので問題ないと考えています。

Processing by TasksController#update as TURBO_STREAM

今の状態はupdateメソッドに値を渡しきれていないと考えています。
なぜなのか教えていただけますでしょうか。
ヒントや可能性でも思いついたら教えていただけますでしょうか。

よろしくお願いします。

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

ruby 3.2.2
Rails 7.0.5

Answer 1

ログを見ると1行目に以下のような記述があります。
Started PATCH "/tasks/:id"
Patch(UPDATE)をしているURLが /tasks/:id であるとわかります。

ここには本来、1,2,3といったデータベースに記録されているidが入ります。
今はidが入っておらず、:idという文字列になっています。
その結果、 Completed 404 Not Found in 16ms (ActiveRecord: 0.2ms | Allocations: 1229) という404エラーになっています。つまり:id番というデータが見つからなかったという結果です。

よって、今出ているエラーを解消するには、idを正しくしてあげるといいです。

Before

<%= form_with model: @task, url: "/tasks/:id", local: true do |f| %>

After

<%= form_with model: @task, url: "/tasks/#{@task.id}", local: true do |f| %>

のように、@task.idを渡してあげるなどが解決策の1つです。

routes.rbの書き方にもよりますが、urlの部分は
url: tasks_path(id: @task.id)のような書き方も考えられます。

Comments

[winterboum]

form_with model: @task, があるなら url: オプションはなくても良いです。
つけずに form_withに委ねたほうがトラブル起きにくいです。
url をつけるのは model: @task から自然に導かれるURL ではないところに飛ばしたいとき。