Use activerecord way of determining existing report

openSUSE · Sep 21, 2023 · b6caa48 · b6caa48
1 parent 7e5b381
commit b6caa48
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/api/app/components/reports_notice_component.html.haml b/src/api/app/components/reports_notice_component.html.haml
@@ -1,6 +1,6 @@
 .text-warning
   %i.fas.fa-flag
-  - if by_user
+  - if by_user?
     You reported this #{reportable.class.name.downcase}.
   - else
     This #{reportable.class.name.downcase} has #{pluralize(reports.count, 'report')}.
diff --git a/src/api/app/components/reports_notice_component.rb b/src/api/app/components/reports_notice_component.rb
@@ -8,8 +8,8 @@ def initialize(reportable:, user:)
     @user = user
   end
 
-  def by_user
-    !!Report.find_by(user:, reportable:)
+  def by_user?
+    Report.exists?(user:, reportable:)
   end
 
   def reports