link_toに追加したパラメータをビューからコントローラに渡せない

Question

よろしくお願いします。

Rails初学者です。
自作のアプリを作っており、ビューに記述したlink_toメソッドに追加したパラメータをコントローラに渡したいのですがうまくいきません。

酪農をやっていて、自作の牛群管理アプリを作っています。

搾乳牛管理画面は実際の牛舎内の配置と同じになっています。
セル内の数字は牛の個体識別番号を示し、クリックすると牛の詳細情報を閲覧できるようになっています。

このアプリで、牛の配置を変更する機能を作ろうとしています。

①ある牛（とりあえずここではAとします）の詳細画面の「移動」ボタンで移動のためのアクションへレンダリング
↓
②牛の移動のページで搾乳牛管理画面で表示されている画面と同様のものを表示し、Aの牛を移動させたい場所をクリックする。
↓
③搾乳牛管理画面へレンダリングされ、Aの牛は移動先に選んだ場所へ移動し、元々Aの牛が登録されていた場所には「新規作成」のリンクが作られる。

という流れで機能を作りたいと考えています。

牛の移動画面（moveアクション）

「牛を配置場所」を管理するテーブルと「牛の個体情報」を管理するテーブルは1対1の関連付けをしています。
牛の移動画面のlink_toメソッドにそれぞれの配置場所のパラメータを追加して、移動させたい場所をクリックすると牛の個体情報に関連付けられている配置場所のパラメータが「現在登録されている配置場所」から「移動させたい場所（link_toに追加されているパラメータ）」に書き換わるようにすればいいのかな、と思い機能を作ってみているのですが、link_toに追加したパラメータをコントローラに渡せないので配置場所の書き換えがうまくできません。
どのようにすればいいのか教えていただきたいです。

cows_controller.rb

class CowsController < ApplicationController

  def show
    
    @cow = Cow.find(params[:id])
  end
        
  def new
    @cow = Cow.new(birthday: Date.new(2010,1,1),stall_id: params[:stall_id])
  end
        
  def edit
    @cow = Cow.find(params[:id])
  end
        
  def cow_params
    params.require(:cow).permit(:number, :name, :birthday, :status, :stall_id)
  end
        
  def create
    @cow = Cow.new(cow_params)
    
       if @cow.save
         redirect_to @cow, notice: "登録しました。"
       else
         render "new"
       end
  end
        
  def update
    @cow = Cow.find(params[:id])
    @cow.assign_attributes(params[:cow])
      if @cow.save
        redirect_to @cow, notice: "更新しました"
      else
        render "edit"
      end
  end
        
  def destroy
    @cow = Cow.find(params[:id])
    @cow.destroy
    redirect_to :stalls, notice: "削除しました。"
  end
  
  
  def move
    
    @cow = Cow.find(params[:id])
    
    @account = current_account
    
    @stall = @account.stalls.includes(:cow).order("stallnumber")    
   
  end
  
  def moved
    
    byebug

    @cow = Cow.find(params[:id])
    
    @account = current_account
      
    @cow.stall_id = params[:stallnumber]
    
    @cow.save
    
    redirect_to :stalls, notice: "変更しました。"
  end  
  
end

牛の配置場所　マイグレーションスクリプト

class CreateStalls < ActiveRecord::Migration[5.2]
  def change
    create_table :stalls do |t|
      t.integer :stallnumber, null: false

      t.timestamps
    end
  end
end

牛の個体情報　マイグレーションスクリプト

class CreateCows < ActiveRecord::Migration[5.2]
  def change
    create_table :cows do |t|
      t.references :stall,null: false
      t.integer :number, null: false
      t.string :name, null: false
      t.date :birthday, null: false
      t.integer :status,null: false, default: 1
     


      t.timestamps
    end
  end
end

move.html.erb

<% @page_title = "牛の移動" %>
<h1><%= @page_title %></h1>



 <% @half = current_account.all_stalls/2 %>
  <table class = "index" align = "left">



     <% @half.downto(1).to_a.each do |i|%>
     <% @line = @stall.find_by stallnumber: i %>
     <% if @line.cow.present? %>
       <% if @line.cow.status == "乾乳牛" %>
         <tr><td class = "dry"><%= link_to @line.cow.number, moved_cow_path(@line.cow),method: :patch,stallnumber: @line.stallnumber  %></td></tr>
       <% else %>
         <tr><td><%= link_to @line.cow.number, moved_cow_path(@line.cow),method: :patch, stallnumber: @line.stallnumber %></td></tr>
       <% end %>
     <% else %>
     <tr><td><%= link_to "新規作成" , moved_cow_path(@line),method: :patch,stallnumber: @line.stallnumber%></td></tr>
     <% end %>
     <% end %>
 
  </table>
  
  
  <table class = "index" align = "left">
     <% current_account.all_stalls.downto(@half + 1).to_a.each do |i|%>
     <% @line = @stall.find_by stallnumber: i %>
     
     
     
     <% if @line.cow.present? %>
     <% if @line.cow.status == "乾乳牛" %>
      <tr><td class = "dry"><%= link_to @line.cow.number,  moved_cow_path(@line.cow),method: :patch,stallnumber: @line.stallnumber %></td></tr>
       <% else %>
         <tr><td><%= link_to @line.cow.number,  moved_cow_path(@line.cow),method: :patch ,stallnumber: @line.stallnumber%></td></tr>
       <% end %>
     <% else %>
     <tr><td><%=link_to "新規作成" ,  moved_cow_path(@line),method: :patch,stallnumber: @line.stallnumber %></td></tr>
     <% end %>
     <% end %>
     
     
     
  </table>
 
 

routes.rb

Rails.application.routes.draw do
  root "top#index"
  
  resources :stalls
  resources :cows do
    resources :treatments
    get "move" , as: "move", on: :member
    patch "moved" , as: "moved", on: :member
  end
  resources :treatments
  resource :session, only: [:create,:destroy]
  resource :password, except: [:index]
  resources :accounts do
    resources :stalls
  end
end

よろしくお願いします。

2020.1.30追記

rails routesの結果（抜粋）

  treatments#destroy
                 move_cow GET    /cows/:id/move(.:format)                                                                 cows#move
                moved_cow PATCH  /cows/:id/moved(.:format)                                                                cows#moved
                     cows GET    /cows(.:format)                                                                          cows#index
                          POST   /cows(.:format)                                                                          cows#create
                  new_cow GET    /cows/new(.:format)                                                                      cows#new
                 edit_cow GET    /cows/:id/edit(.:format)                                                                 cows#edit
                      cow GET    /cows/:id(.:format)                                                                      cows#show
                          PATCH  /cows/:id(.:format)                                                                      cows#update
                          PUT    /cows/:id(.:format)                                                                      cows#update
                          DELETE /cows/:id(.:format)                                                                      cows#destroy
               treatments GET    /treatments(.:format)                                                                    treatments#index

2020.1.30 20:45追記

Comments

[no1knows]

質問とあまり関係がないのですが、牛の移動先は空いているところ（図でいうと新規作成）に移動させるだけという前提で良いのでしょうか？

[ayumu0622]

no1knows様
一応牛が今いる場所にも移動できるようにするつもりでいました
その場合はlink_toメソッドにdata-confirm属性をつけて元いた牛に上書きされることを確認した上で牛を移動できるように対応しようかなと考えていました

Answer 1

rails routes してみてください moved_cow_path ってありますか？
view の moved_cow_path　でエラーは出ない？

patch "moved" , as: "moved", on: :member　
とあるので、　moved_path(@line)ではないかと

あと
@cow = Cow.find(params[:id])
とありますが、渡ってくるのは line のid ですので、
@cow = Line.find(params[:id]).cow
では
@cow = Cow.find(params[:id])　としたいのなら
moved_path(@line.cow)

Comments

[ayumu0622]

ご回答ありがとうございます。
質問本文の方にrails routesの内容を抜粋して載せました。
moved_cow_pathは一応ありました。
viewのmoved_cow_pathの部分は特にエラーも起こらず牛の移動ができないこと以外はmovedアクションは動いています。

lineはviewの中で記述してある変数で、内容は牛の場所を管理するStallのレコードになっていますので、
@cow = Line.find(params[:id]).cowだとエラーが出てしまいます。
movedアクションで牛の配置場所を書き換える際に、書き換えられる牛のレコードも必要だと思ってupdateアクションを参考に@cow = Cow.find(params[:id])と記述したのですが必要なかったのでしょうか。
私の認識違いだったらすいません．．．

[shinoharat]

横から失礼します。
winterboum さんの回答にも書いてありますが、view 側を以下のように変更する必要があると思います。

moved_cow_path(@line)
↓
moved_cow_path(@line.cow)

引数には Cow のインスタンスを渡さないといけないです。
コントローラ側は

@cow = Cow.find(params[:id])

のままでOKです。

[winterboum]

ああ、確かに lineはモデルじゃないんですね。
shinoharatさんがコメントしていただけてますが、
引数には Cow のインスタンスを渡さないといけないのでmoved_cow_path(@line.cow)にしてください

[ayumu0622]

shinoharat様、ご回答ありがとうございます。
winterboum様も引き続きご回答ありがとうございます。
ご指摘いただいた通り変更して動作を確認してみたところActionController::UrlGenerationError in Cows#moveとエラーが出ました。
エラー画面を質問本文の方に追加しました。
missing required keys: [:id]となっているのですが@lineにあったstallモデルのレコードのidが必要、ということでしょうか？

[ayumu0622]

時間が空いてしまい申し訳ございません。
前回のコメントに書いたエラーはmove.html.erbの新規作成ボタンでのパスをmoved_cow_path(@line.cow)からmoved_cow_path(@line)から変更することで解決しました。
エラーは解消されましたが依然として牛の移動がうまくいきません。

movedアクションでデバッグを行うと@cowに「移動させたい場所（@line.cow）」の牛のデータが入っています。「移動させたい牛」のデータがmovedアクションに渡っていない、あるいは取り出せていないのだと思うのですが「移動させたい牛」のデータの渡し方、取り出し方がわからないです。
お時間あるときで構いませんのでご助力頂きたいです。

[winterboum]

載っているのは最終形のソースになってますか？

[ayumu0622]

すいません、move.html.erbを今現在のものに修正しました。

moved_cow_path(@line)
↓
moved_cow_path(@line.cow)
に変更しています。
新規作成の部分だけmoved_cow_path(@line)になっています。

Answer 2

いろいろと修正が必要なので、やり方の一つとして下記を参考にしてください。
https://github.com/xxxxxxxxx

概要

• 牛の登録と移動だけができるシステムとなっています。
• テーブルは、牛の個体場所と配置場所を一つにまとめました。
• CSSは設定してないので提示頂いたテーブルの形ではないですが、stall番号順にしました。
• DHHさん（Railsを作った人）が書くようなシンプルなコントローラーの書き方にしました。（参考）
• 牛が配置されているところに、移動することしかできません。（元いた牛のstall番号は変更していないので、おなじstall番号ができてしまいます。）

Comments

[ayumu0622]

ご回答ありがとうございます。
返信が遅くなってしまい申し訳ございません。

githubを扱うのが初めてなのですが、上記のURLのページにあるコードをダウンロードしてアプリを起動してみればよいのでしょうか．．．

[no1knows]

はい、そうです！
Railsのバージョンがruby '2.6.3'、railsのバージョンが6.0.2.1になっているので、そこだけ利用している環境に合わせていただく形になるかと思います。

[no1knows]

設定が大変かもしれないので、下記で動作確認してみてください。
https://xxxx.herokuapp.com/

[ayumu0622]

起動できました。

link_toを使ってやってみたいと思っていましたがno1knows様にご提示いただいたように牛の個体情報の編集画面で牛の場所（stall）を指定して牛の移動を行った方が簡潔に実装できそうな気がしてきました．．．
前回の質問でno1knows様にご指摘いただいていましたがきちんとレールに沿って機能を作った方がやりやすいことが回りまわって理解できました．．．
作っていただいたアプリを参考に機能の実装方法を見直してみたいと思います。
ありがとうございます。

[no1knows]

あぁ、ちなみに移動時はlink_toを利用しています。
controllers/cows/moves_controllerの中にありますのでこちらを参考にしていただければと思います。
あと上記githubのリンクなどは削除させていただきますね。

[ayumu0622]

了解いたしました。
ありがとうございました。

[ayumu0622]

すいません、頂いたアプリのについてなんですが、move_idはどこから出てきたものなんでしょうか．．．
回答が終わってからまた質問するのは大変恐縮なんですがお時間あるときに教えていただきたいです．．．

[ayumu0622]

すいません解決しました

[no1knows]

一応、補足として、このMovesコントローラーですが、最初はとっつきにくいのでCows_controllerの中にまとめてしまっても良いかと思います。

勉強のためにDHHの書くようなシンプルなコントローラー（index/show/new/edit/create/update/detroyだけ）で作るかたちにしています。
MovesコントローラーはCowsコントローラーから継承しているため、Cowsコントローラーのファイルの1行目がclass Cows::MovesController < ApplicationControllerとなっているのですが．．．

たぶんわかりやすいのでCowsコントローラーにまとめちゃってください。

[ayumu0622]

了解です
参考にさせていただきます
ありがとうございます…