前提・発生している問題

初学者です。タスク管理のオリジナルアプリを作成しているのですが、Roomを分けてtaskを作成したあとに編集をできるようにしたところ、選択したtask以外のものの編集ページに遷移してしまいました。

#該当のソースコード
rooms_controller.rb

ruby
1class RoomsController < ApplicationController
2  before_action :authenticate_user!
3
4  def index
5    @rooms = Room.all.order(:id)
6  end
7
8  def new
9    @room = Room.new
10  end
11
12  def create
13    @room = Room.new(room_params)
14    if @room.save
15      redirect_to room_path(@room)
16    else
17      render :new
18    end
19  end
20
21  def show
22    @room = Room.find(params[:id])
23    @messages = @room.messages
24    @tasks = Task.where(room: @room).order("created_at ASC")
25  end
26
27  private
28  def room_params
29    params.require(:room).permit(:name)
30  end
31end

tasks_controller.rb

ruby
1class TasksController < ApplicationController
2  def new
3    @task = Task.new
4  end
5
6  def create
7    @room = Room.find(params[:room_id])
8    @task = Task.new(task_params)
9    if @task.save
10      redirect_to room_path(@room)
11    else
12      render :new
13    end
14  end
15  
16  def edit
17    binding.pry
18    @task = Task.find(params[:id])
19  end
20  
21  def update
22    @room = Room.find(params[:room_id])
23    @task = Task.find(params[:id])
24    if @task.update(task_params)
25      redirect_to room_path(@room)
26    else
27      render :edit
28    end 
29  end
30
31  private
32  def task_params
33      params.require(:task).permit(:title).merge(room_id: @room.id)
34  end
35end

show.html.erb

ruby
1<div class="topPage">
2  <div class="taskWrapper">
3    <% @tasks.each do |task| %>
4      <div class="task">
5        <div class="task_header">
6          <h2 class="task_header_title"><%= task.title %></h2>
7          <div class="task_header_action">
8            <%= link_to(content_tag(:i,'', class: 'fas fa-pen'), edit_room_task_path(task))%>
9            <%= link_to(content_tag(:i,'', class: 'fas fa-trash'), room_task_path(task), method: :delete ) %>
10          </div>
11        </div>
12      </div>
13    <% end %>
14    <div class='task_new'>
15      <%= link_to new_room_task_path(@room), class: "nav-link" do %>
16      <span class="mr-1">タスクを作成</span>
17      <% end %>
18    </div>
19  </div>
20</div>

試したこと

bindin.pryを使用し、Parameterを確認したところ<ActionController::Parameters {"controller"=>"tasks", "action"=>"edit", "room_id"=>"5", "id"=>"3"} permitted: false>となっており、room_idとidが求めている値と逆になっていることがわかりました。

原因と修正方法についてはわからず、教えて頂きたいです。
よろしくお願いします。

FKM

2020/12/21 09:03

RoomとTaskを制御しているマイグレーション用のテーブルはどうなっていますか。

行動規範の内容に同意します

回答1件

自己解決

editアクションにも@room = Room.find(params[:room_id])を追加し、show.html.erbのlink_toに引数を加えることで解決することができました。

ruby
1before
2<%= link_to(content_tag(:i,'', class: 'fas fa-pen'), edit_room_task_path(task))%>
3<%= link_to(content_tag(:i,'', class: 'fas fa-trash'), room_task_path(task), method: :delete ) %>
4
5after
6<%= link_to(content_tag(:i,'', class: 'fas fa-pen'), edit_room_task_path(@room, task))%>
7<%= link_to(content_tag(:i,'', class: 'fas fa-trash'), room_task_path(@room, task), method: :delete) %>