求めるParametersが逆になります。

Question

前提・発生している問題

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

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

class RoomsController < ApplicationController
  before_action :authenticate_user!

  def index
    @rooms = Room.all.order(:id)
  end

  def new
    @room = Room.new
  end

  def create
    @room = Room.new(room_params)
    if @room.save
      redirect_to room_path(@room)
    else
      render :new
    end
  end

  def show
    @room = Room.find(params[:id])
    @messages = @room.messages
    @tasks = Task.where(room: @room).order("created_at ASC")
  end

  private
  def room_params
    params.require(:room).permit(:name)
  end
end

tasks_controller.rb

class TasksController < ApplicationController
  def new
    @task = Task.new
  end

  def create
    @room = Room.find(params[:room_id])
    @task = Task.new(task_params)
    if @task.save
      redirect_to room_path(@room)
    else
      render :new
    end
  end
  
  def edit
    binding.pry
    @task = Task.find(params[:id])
  end
  
  def update
    @room = Room.find(params[:room_id])
    @task = Task.find(params[:id])
    if @task.update(task_params)
      redirect_to room_path(@room)
    else
      render :edit
    end 
  end

  private
  def task_params
      params.require(:task).permit(:title).merge(room_id: @room.id)
  end
end

show.html.erb

<div class="topPage">
  <div class="taskWrapper">
    <% @tasks.each do |task| %>
      <div class="task">
        <div class="task_header">
          <h2 class="task_header_title"><%= task.title %></h2>
          <div class="task_header_action">
            <%= link_to(content_tag(:i,'', class: 'fas fa-pen'), edit_room_task_path(task))%>
            <%= link_to(content_tag(:i,'', class: 'fas fa-trash'), room_task_path(task), method: :delete ) %>
          </div>
        </div>
      </div>
    <% end %>
    <div class='task_new'>
      <%= link_to new_room_task_path(@room), class: "nav-link" do %>
      <span class="mr-1">タスクを作成</span>
      <% end %>
    </div>
  </div>
</div>

試したこと

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

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

Comments

[FKM]

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

Answer 1

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

before
<%= link_to(content_tag(:i,'', class: 'fas fa-pen'), edit_room_task_path(task))%>
<%= link_to(content_tag(:i,'', class: 'fas fa-trash'), room_task_path(task), method: :delete ) %>

after
<%= link_to(content_tag(:i,'', class: 'fas fa-pen'), edit_room_task_path(@room, task))%>
<%= link_to(content_tag(:i,'', class: 'fas fa-trash'), room_task_path(@room, task), method: :delete) %>