X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/6c225bd01c53ad4cc3429e05c7bb71da3b0a4b7b..83d9f1f1642d71f3b590e7ee33b22e4c5da00c53:/app/models/issue.rb diff --git a/app/models/issue.rb b/app/models/issue.rb index d893a6e3a..54947be30 100644 --- a/app/models/issue.rb +++ b/app/models/issue.rb @@ -30,11 +30,11 @@ # issues_updated_by_fkey (updated_by => users.id) # -class Issue < ActiveRecord::Base +class Issue < ApplicationRecord belongs_to :reportable, :polymorphic => true - belongs_to :reported_user, :class_name => "User", :foreign_key => :reported_user_id - belongs_to :user_resolved, :class_name => "User", :foreign_key => :resolved_by - belongs_to :user_updated, :class_name => "User", :foreign_key => :updated_by + belongs_to :reported_user, :class_name => "User", :optional => true + belongs_to :user_resolved, :class_name => "User", :foreign_key => :resolved_by, :optional => true + belongs_to :user_updated, :class_name => "User", :foreign_key => :updated_by, :optional => true has_many :reports, :dependent => :destroy has_many :comments, :class_name => "IssueComment", :dependent => :destroy @@ -44,10 +44,9 @@ class Issue < ActiveRecord::Base ASSIGNED_ROLES = %w[administrator moderator].freeze validates :assigned_role, :presence => true, :inclusion => ASSIGNED_ROLES - before_validation :set_default_assigned_role before_validation :set_reported_user - scope :with_status, ->(issue_status) { where(:status => statuses[issue_status]) } + scope :with_status, ->(issue_status) { where(:status => issue_status) } scope :visible_to, ->(user) { where(:assigned_role => user.roles.map(&:role)) } def read_reports @@ -71,7 +70,7 @@ class Issue < ActiveRecord::Base event :resolve do transitions :from => :open, :to => :resolved after do - self.resolved_at = Time.now.getutc + self.resolved_at = Time.now.utc end end @@ -93,13 +92,4 @@ class Issue < ActiveRecord::Base reportable.user end end - - def set_default_assigned_role - if assigned_role.blank? - self.assigned_role = case reportable - when Note then "moderator" - else "administrator" - end - end - end end