From 8a4a087a001ba1acc7a25f3bcf3413043de0afbb Mon Sep 17 00:00:00 2001 From: David Heinemeier Hansson Date: Fri, 11 Apr 2025 17:00:08 +0200 Subject: [PATCH] Fine to let destroying something destroyed be a no-op Would be an edge-casey race condition, but its possible --- app/controllers/cards/comments/reactions_controller.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/controllers/cards/comments/reactions_controller.rb b/app/controllers/cards/comments/reactions_controller.rb index 392179db3..55e673b21 100644 --- a/app/controllers/cards/comments/reactions_controller.rb +++ b/app/controllers/cards/comments/reactions_controller.rb @@ -17,7 +17,7 @@ class Cards::Comments::ReactionsController < ApplicationController def destroy @reaction = @comment.reactions.find(params[:id]) - @reaction.destroy! + @reaction.destroy broadcast_remove end