Merge remote-tracking branch 'upstream/pull/2951' into master

[rails.git] / app / jobs / trace_importer_job.rb

diff --git a/app/jobs/trace_importer_job.rb b/app/jobs/trace_importer_job.rb
index 6b8943aa20f2dc5dca20dfdd979429f6f0096bbc..1eddcb6f8e8e50d40f20fac1e6f470ff8518edb3 100644 (file)
--- a/app/jobs/trace_importer_job.rb
+++ b/app/jobs/trace_importer_job.rb
@@ -5,15 +5,15 @@ class TraceImporterJob < ApplicationJob
     gpx = trace.import
 
     if gpx.actual_points.positive?
     gpx = trace.import
 
     if gpx.actual_points.positive?

-      Notifier.gpx_success(trace, gpx.actual_points).deliver_later
+      UserMailer.gpx_success(trace, gpx.actual_points).deliver

     else
     else

-      Notifier.gpx_failure(trace, "0 points parsed ok. Do they all have lat,lng,alt,timestamp?").deliver_later
+      UserMailer.gpx_failure(trace, "0 points parsed ok. Do they all have lat,lng,alt,timestamp?").deliver

       trace.destroy
     end
       trace.destroy
     end

-  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_later
+  rescue StandardError => e
+    logger.info e.to_s
+    e.backtrace.each { |l| logger.info l }
+    UserMailer.gpx_failure(trace, e.to_s + "\n" + e.backtrace.join("\n")).deliver

     trace.destroy
   end
 end
     trace.destroy
   end
 end