Remove redundant presence validation on belongs_to

[rails.git] / app / models / report.rb

diff --git a/app/models/report.rb b/app/models/report.rb
index 477b4b1824842d37997e8cc2e05054dbf5dc4706..77a96764125bbc8e839d73773a3fbfa50a5d9aa9 100644 (file)
--- a/app/models/report.rb
+++ b/app/models/report.rb
@@ -1,4 +1,39 @@
-class Report < ActiveRecord::Base
+# == Schema Information
+#
+# Table name: reports
+#
+#  id         :integer          not null, primary key
+#  issue_id   :integer          not null
+#  user_id    :integer          not null
+#  details    :text             not null
+#  category   :string           not null
+#  created_at :datetime         not null
+#  updated_at :datetime         not null
+#
+# Indexes
+#
+#  index_reports_on_issue_id  (issue_id)
+#  index_reports_on_user_id   (user_id)
+#
+# Foreign Keys
+#
+#  reports_issue_id_fkey  (issue_id => issues.id)
+#  reports_user_id_fkey   (user_id => users.id)
+#
+
+class Report < ApplicationRecord

   belongs_to :issue, :counter_cache => true
   belongs_to :issue, :counter_cache => true

-  belongs_to :user, :class_name => "User", :foreign_key => :reporter_user_id
+  belongs_to :user
+
+  validates :details, :presence => true, :characters => true
+  validates :category, :presence => true
+
+  def self.categories_for(reportable)
+    case reportable.class.name
+    when "DiaryEntry", "DiaryComment" then %w[spam offensive threat other]
+    when "User" then %w[spam offensive threat vandal other]
+    when "Note" then %w[spam personal abusive other]
+    else %w[other]
+    end
+  end

 end
 end