親モデルのページで表示されている子モデルのデータを、チェックボックスとボタンによって一括削除したい

Question

前提・実現したいこと

楽曲とその音源を登録するアプリを作っています。1つの曲に対し複数の演奏音源がつきます。親モデルはpostで、子モデルはrecordingです。

button_toで楽曲ページに表示されている子モデルのデータをチェックボックスを通して一括削除したいですが、IDの指定が上手くいきません。

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

任意のrecordingのデータのチェックボックスにチェックを入れてDeleteボタンを押すとこうなります。

Couldn't find all Posts with 'id': (1, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 34, 35, 36) (found 5 results, but was looking for 21).

  def delete_multiple
    @recording = Post.find(params[:id]).recording.where(id: params[:recording_ids])
    @recording.destroy_all
  end

該当のソースコード

Postモデル(親)

class Post < ApplicationRecord
  has_many :recordings, dependent: :destroy
  has_many :post_instruments
  accepts_nested_attributes_for :recordings, {allow_destroy: true}
end

Recordingモデル(子)

class Recording < ApplicationRecord
  belongs_to :post
end

ビュー(posts/show.html.erb)

<% @post.recordings.each do |recording| %>
  <%= check_box_tag "recording_ids[]", recording.id %>
<% end %>

<%= button_to "Delete", {action: "delete_multiple", id: @post.recording_ids,} method: :delete %>

Postコントローラ

  def delete_multiple
    @post= Post.find(params[:id]).recordings.where(id: params[:recording_ids]).destroy_all
    redirect_to('/posts/')
  end

ルーティング

Rails.application.routes.draw do

  resources :posts do
    resources :recordings

      collection do
        get :'index'
        delete :'delete_multiple'
      end
  end
end

試したこと

指定するべきIDはあくまでrecordingモデルのIDであって、postモデルのIDではないというのは分かるのですが、そこからどうすればいいのか分かりませんでした。

Recordingコントローラにも同じようなdelete=multipleモデルを作って、button_toにおいてコントローラをcontroller: :recordingsに指定しても、使われるのはpostコントローラの方のメソッドです。

そのほかbutton=toの{}の構成などを入れ替えてたりしました。

いくつかのウェブページも参考にしました。
https://stackoverflow.com/questions/28360663/delete-multiple-records-using-checkboxes-in-rails
https://stackoverflow.com/questions/22443064/rails-destroy-multiple-record-through-check-boxes

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

rails 6.0

Answer 1

大きな間違いが２つあります。

controllerに渡るのは PostのIDではなくRecodingのIDですから
Recording.where(id: params[:recording_ids]).destroy_all
です。

こちらは簡単なのですが。
<%= button_to "Delete",... id: @post.recording_ids,
これでは@postの全てのRecodingが削除されます。
checkしたものだけ削除ですから、
<%= check_box_tag "recording_ids[]", recording.id %>
ここの recording_ids を渡す必要があります。
それには二つの方法があります
0. button を押したら javascriptを呼んで、そこでパラメータを組み立てて送る
0. 全体を <form></form>で囲んで、buttonをsubmitにする

Comments

[kawa0tatsu]

遅くなりましたがありがとうございます。
とりあえず最初の間違いについては、教えて頂くまで親モデルのコントローラでも子モデルから書き始めても良いという発想が思い浮かびませんでした。

ただ、第2の間違いについては、なぜかbutton_toからはjavascriptの関数へのonclickのイベントが発生しません。それは別の質問にしようと思います。

1と2は別々の方法ですよね？

[kawa0tatsu]

やっと出来ました･･･。結局両方の方法を使ったら望み通りの挙動になりました。
こちらhttps://teratail.com/questions/62985 のページを参考にして、公式ドキュメントのform_withの項目https://apidock.com/rails/ActionView/Helpers/FormHelper/form_withを読んでやっと解決しました。

submitボタンだとわざわざonclickを指定しなくても良いのですね。

ヒントを教えてくださりありがとうございました。

ビュー
<%= form_with url: delete_multiple_posts_path, method: :delete do |f|%>
(中略)
<%= check_box_tag "checked_ids[]", recording.id, class: "delete" %>
(中略)
<%= f.submit "delete", id: "btn_submit" %>
<% end %>

<script language="javascript" type="text/javascript">

function call(){
var checks = document.getElementsByName("checked_ids[]");
var str = '';
for (i=0; i<checks.length; i++){
if(checks[i].checked === true){
if(str!==""){
str +=",";
}
str += checks[i].value;
}
}
}
</script>

コントローラ

def delete_multiple
Recording.where(id: params[:checked_ids]).destroy_all
redirect_to('/posts/')
end