resource ルーティングのネストで、不要なアクションを作らない方法はありますか？

Question

現在、railsで通知機能を実装していて、Routingの実装方法が分かりません。具体的には、notificationリソースにdestroy_allアクションをcollectionとして追加したいのですが、書き方が分かりません。次のように書いており、index以外の不要なnotificationリソースのアクションが生成されています。

  resources :notifications, only: :index
  resources :notifications do 
    collection do
      delete 'destroy_all'
    end
  end

また、ルーティングが原因なのか不明ですが、destroy_all メソッドを実行すると、図1のようなshowアクションを探索するエラーが発生します。コントローラーは図2のようにindexアクションへのリダイレクトを期待します。何故showアクションが探索されるのでしょうか。図3は該当するviewです。

Unknown action
The action 'show' could not be found for NotificationsController

class NotificationsController < ApplicationController
  before_action :sign_in_required, only: :index

	def index
      #current_userの投稿に紐づいた通知一覧
    	@notifications = current_user.passive_notifications
      #@notificationの中でまだ確認していない(indexに一度も遷移していない)通知のみ
    	@notifications.where(checked: false).each do |notification|
      		notification.update(checked: true)
        end
  	end

  	def destroy_all
      #通知を全削除
  		@notifications = current_user.passive_notifications.destroy_all
  		redirect_to notifications_path
  	end
end

<div class="grid grid-cols-1 justify-center bg-lime-100 px-3 py-3">
  <div class='bg-lime-200'>
    <% if @notifications.exists? %>
      <%= link_to destroy_all_notifications_path, method: :delete do %>
        <p>全て削除</p>
      <% end %>
      <%= render @notifications %>
    <% else %>
      <div class="flex w-full flex-col justify-center items-center mb-48">
        <span class ="relative">
          <div class="h-36 w-36 bg-white rounded-full -ml-1.5 "></div>
          <%= image_tag 'ojigi_animal_inu.png', class:' absolute top-6 left-4 z-10 w-24 h-28 p-4' %>
        </span>
        <p class="font-medium py-2">新着の通知はありません。</p>
      </div>
    <% end %>
  </div>
</div>

情報の不備などございましたらご指摘ください。よろしくお願いします。

Comments

[winterboum]

delete を使っていないなら、only に delete を追加して、そこで destroy_all すれば良いように思うのだが。routes を不必要に複雑にしてる。

Show を探してしまうのは http method が GET になってるから。
method で :delete になってますが 素のブラウザーは対応してくれない。
Rails 7ではどうやるのかわからんので、回答できない。。。。

Answer 1

resources doにもonly:を付けられます。

# resources doは単なるresources＋中身を処理する、ということになるので、
# 2行書く必要はない
resources :notifications, only: :index do

Comments

[chasu]

ご回答ありがとうございます。
わかりました。では、エラーの原因についてもお考えをうかがえますでしょうか？

[maisumakun]

method: :deleteには適用する条件がいくつかあります。

・Railsのバージョンはいくつですか？
・rails-ujsは適用していますか？

[chasu]

返信遅くなりまして申し訳ありません。
Rails 7.0.3.1
rails-ujsは使用しておりません。

[maisumakun]

Rails 7ではmethod: :deleteは適用できません。「turbo」について調べてみてください。

[chasu]

現在, Hotwire をフル活用してポートフォリオアプリを作っています。
button_to で代用したところうまくいきました。
ご丁寧に対応していただきありがとうございました。

[chasu]

また、ルーティングは最終的に次のように修正いたしました。
resources :notifications, only: :index
scope :notifications do
delete '/destroy_all', to: 'notifications#destroy_all', as: 'destroy_all_notifications'
end