Combine site permissions declarations

[rails.git] / lib / daemons / gpx_import.rb

diff --git a/lib/daemons/gpx_import.rb b/lib/daemons/gpx_import.rb
index f597c87959127f4893203f987cc11840ab30b4c7..a0344b58c95f9495c35537fbfcbddd49b900dff7 100755 (executable)
--- a/lib/daemons/gpx_import.rb
+++ b/lib/daemons/gpx_import.rb
@@ -1,7 +1,7 @@
 #!/usr/bin/env ruby
 
-#You might want to change this
-#ENV["RAILS_ENV"] ||= "development"
+# You might want to change this
+# ENV["RAILS_ENV"] ||= "development"
 
 require File.dirname(__FILE__) + "/../../config/environment"
 
@@ -9,28 +9,28 @@ terminated = false
 
 logger = ActiveRecord::Base.logger
 
-while(true) do
+loop do
   ActiveRecord::Base.logger.info("GPX Import daemon wake @ #{Time.now}.")
 
-  Trace.find(:all, :conditions => ['inserted = ?', false]).each do |trace|
-    Signal.trap("TERM") do 
+  Trace.find(:all, :conditions => { :inserted => false, :visible => true }, :order => "id").each do |trace|
+    Signal.trap("TERM") do
       terminated = true
     end
 
     begin
       gpx = trace.import
 
-      if gpx.actual_points > 0
-        Notifier::deliver_gpx_success(trace, gpx.actual_points)
+      if gpx.actual_points.positive?
+        Notifier.gpx_success(trace, gpx.actual_points).deliver
       else
+        Notifier.gpx_failure(trace, "0 points parsed ok. Do they all have lat,lng,alt,timestamp?").deliver
         trace.destroy
-        Notifier::deliver_gpx_failure(trace, '0 points parsed ok. Do they all have lat,lng,alt,timestamp?')
       end
-    rescue Exception => ex
-      logger.info ex
-      ex.backtrace.each {|l| logger.info l }
+    rescue StandardError => ex
+      logger.info ex.to_s
+      ex.backtrace.each { |l| logger.info l }
+      Notifier.gpx_failure(trace, ex.to_s + "\n" + ex.backtrace.join("\n")).deliver
       trace.destroy
-      Notifier::deliver_gpx_failure(trace, ex.to_s + "\n" + ex.backtrace.join("\n"))
     end
 
     Signal.trap("TERM", "DEFAULT")
@@ -38,5 +38,22 @@ while(true) do
     exit if terminated
   end
 
-  sleep 15.minutes
+  Trace.find(:all, :conditions => { :visible => false }, :order => "id").each do |trace|
+    Signal.trap("TERM") do
+      terminated = true
+    end
+
+    begin
+      trace.destroy
+    rescue StandardError => ex
+      logger.info ex.to_s
+      ex.backtrace.each { |l| logger.info l }
+    end
+
+    Signal.trap("TERM", "DEFAULT")
+
+    exit if terminated
+  end
+
+  sleep 5.minutes.value
 end