Prefer `destroy!` over `destroy` when not checking for return value.

2018-08-31 09:52:10 +08:00 · 2018-08-31 09:52:10 +08:00 · 93a64c4aa1
parent 91e14226a5
commit 93a64c4aa1
1 changed files with 1 additions and 1 deletions
--- a/plugin.rb
+++ b/plugin.rb
@ -140,7 +140,7 @@ SQL
         post_number: post.post_number
      )

-      notification.destroy if notification
+      notification.destroy! if notification

      DiscourseEvent.trigger(:unaccepted_solution, post)
    end