Do it well this time.

errbit · Jan 8, 2014 · 910e64f · 910e64f
1 parent 472a6d4
commit 910e64f
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/app/interactors/outdated_prolem_clearer.rb b/app/interactors/outdated_prolem_clearer.rb
@@ -23,7 +23,7 @@ def nb_problem_outdated
   end
 
   def criteria
-    @criteria = (Time.new - Problem.last_notice_at) / 3600 / 24 > 7
+    @criteria = Problem.outdated
   end
 
   def repair_database

diff --git a/app/models/problem.rb b/app/models/problem.rb
@@ -47,6 +47,8 @@ class Problem
   scope :unresolved, where(:resolved => false)
   scope :ordered, order_by(:last_notice_at.desc)
   scope :for_apps, lambda {|apps| where(:app_id.in => apps.all.map(&:id))}
+  scope :outdated, where(:last_notice_at.lt => 1.week.ago)
+
 
   validates_presence_of :last_notice_at, :first_notice_at
 

diff --git a/lib/tasks/errbit/database.rake b/lib/tasks/errbit/database.rake
@@ -28,7 +28,7 @@ namespace :errbit do
     desc "Delete old errors from the database. (Useful for limited heroku databases)"
     task :clear_outdated => :environment do
       require 'outdated_problem_clearer'
-      puts "=== Cleared #{OutdatedProblemClearer.new.execute} resolved errors from the database."
+      puts "=== Cleared #{OutdatedProblemClearer.new.execute} outdated errors from the database."
     end
 
     desc "Regenerate fingerprints"