From a7d41f09eb65c2a84fb612b476395826d198d0d9 Mon Sep 17 00:00:00 2001
From: Tom Hughes <tom@compton.nu>
Date: Sat, 9 Jun 2007 23:34:50 +0000
Subject: [PATCH] Refactor GPX import code to make reimporting old traces
 easier.

---
 app/models/trace.rb       | 19 +++++++------------
 lib/daemons/gpx_import.rb | 16 +++++++++++++++-
 2 files changed, 22 insertions(+), 13 deletions(-)

diff --git a/app/models/trace.rb b/app/models/trace.rb
index d44d71027..4f21c8b25 100644
--- a/app/models/trace.rb
+++ b/app/models/trace.rb
@@ -9,6 +9,11 @@ class Trace < ActiveRecord::Base
   has_many :tags, :class_name => 'Tracetag', :foreign_key => 'gpx_id', :dependent => :delete_all
   has_many :points, :class_name => 'Tracepoint', :foreign_key => 'gpx_id', :dependent => :delete_all
 
+  def destroy
+    super
+    FileUtils.rm_f(trace_name, icon_picture_name, large_picture_name)
+  end
+
   def tagstring=(s)
     self.tags = s.split().collect {|tag|
       tt = Tracetag.new
@@ -135,21 +140,11 @@ class Trace < ActiveRecord::Base
         self.size = gpx.actual_points
         self.inserted = true
         self.save
-
-        Notifier::deliver_gpx_success(self, gpx.possible_points)
-      else
-        FileUtils.rm_f("/home/osm/gpx/#{id}.gpx")
-        self.destroy
-        Notifier::deliver_gpx_failure(self, '0 points parsed ok. Do they all have lat,lng,alt,timestamp?')
       end
 
       logger.info "done trace #{id}"
-    rescue Exception => ex
-      logger.info ex
-      ex.backtrace.each {|l| logger.info l }
-      FileUtils.rm_f("/home/osm/gpx/#{id}.gpx")
-      self.destroy
-      Notifier::deliver_gpx_failure(self, ex.to_s + ex.backtrace.join("\n") )
+
+      return gpx
     ensure
       FileUtils.rm_f(tempfile) if tempfile
     end
diff --git a/lib/daemons/gpx_import.rb b/lib/daemons/gpx_import.rb
index 3b7fb02e2..f597c8795 100755
--- a/lib/daemons/gpx_import.rb
+++ b/lib/daemons/gpx_import.rb
@@ -17,7 +17,21 @@ while(true) do
       terminated = true
     end
 
-    trace.import
+    begin
+      gpx = trace.import
+
+      if gpx.actual_points > 0
+        Notifier::deliver_gpx_success(trace, gpx.actual_points)
+      else
+        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 }
+      trace.destroy
+      Notifier::deliver_gpx_failure(trace, ex.to_s + "\n" + ex.backtrace.join("\n"))
+    end
 
     Signal.trap("TERM", "DEFAULT")
 
-- 
2.39.5