Don't include stack backtraces on xml errors when importing traces

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

diff --git a/app/models/issue.rb b/app/models/issue.rb
index 00ee934f7a0fe430d1b54720fb85837e948206ab..a3f82ee40b11881bddf5d5226404f8209668f614 100644 (file)
--- a/app/models/issue.rb
+++ b/app/models/issue.rb
@@ -6,7 +6,7 @@
 #  reportable_type  :string           not null
 #  reportable_id    :integer          not null
 #  reported_user_id :integer
 #  reportable_type  :string           not null
 #  reportable_id    :integer          not null
 #  reported_user_id :integer

-#  status           :integer
+#  status           :enum             default("open"), not null

 #  assigned_role    :enum             not null
 #  resolved_at      :datetime
 #  resolved_by      :integer
 #  assigned_role    :enum             not null
 #  resolved_at      :datetime
 #  resolved_by      :integer


@@ -17,8 +17,10 @@
 #
 # Indexes
 #
 #
 # Indexes
 #

+#  index_issues_on_assigned_role                      (assigned_role)

 #  index_issues_on_reportable_type_and_reportable_id  (reportable_type,reportable_id)
 #  index_issues_on_reported_user_id                   (reported_user_id)
 #  index_issues_on_reportable_type_and_reportable_id  (reportable_type,reportable_id)
 #  index_issues_on_reported_user_id                   (reported_user_id)

+#  index_issues_on_status                             (status)

 #  index_issues_on_updated_by                         (updated_by)
 #
 # Foreign Keys
 #  index_issues_on_updated_by                         (updated_by)
 #
 # Foreign Keys


@@ -28,11 +30,11 @@
 #  issues_updated_by_fkey        (updated_by => users.id)
 #
 
 #  issues_updated_by_fkey        (updated_by => users.id)
 #
 

-class Issue < ActiveRecord::Base
+class Issue < ApplicationRecord

   belongs_to :reportable, :polymorphic => true
   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
 
   has_many :reports, :dependent => :destroy
   has_many :comments, :class_name => "IssueComment", :dependent => :destroy


@@ -42,20 +44,17 @@ class Issue < ActiveRecord::Base
   ASSIGNED_ROLES = %w[administrator moderator].freeze
   validates :assigned_role, :presence => true, :inclusion => ASSIGNED_ROLES
 
   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
 
   before_validation :set_reported_user
 

-  # Check if more statuses are needed
-  enum :status => %w[open ignored resolved]
-
-  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
 
   def read_reports

-    resolved_at.present? ? reports.where("updated_at < ?", resolved_at) : nil
+    resolved_at.present? ? reports.where(:updated_at => ...resolved_at) : nil

   end
 
   def unread_reports
   end
 
   def unread_reports

-    resolved_at.present? ? reports.where("updated_at >= ?", resolved_at) : reports
+    resolved_at.present? ? reports.where(:updated_at => resolved_at..) : reports

   end
 
   include AASM
   end
 
   include AASM


@@ -71,7 +70,7 @@ class Issue < ActiveRecord::Base
     event :resolve do
       transitions :from => :open, :to => :resolved
       after do
     event :resolve do
       transitions :from => :open, :to => :resolved
       after do

-        self.resolved_at = Time.now.getutc
+        self.resolved_at = Time.now.utc

       end
     end
 
       end
     end
 


@@ -93,9 +92,4 @@ class Issue < ActiveRecord::Base
                            reportable.user
                          end
   end
                            reportable.user
                          end
   end

-
-  def set_default_assigned_role
-    role = %w[Note].include?(reportable.class.name) ? "moderator" : "administrator"
-    self.assigned_role = role if assigned_role.blank?
-  end

 end
 end