lib/lolcommits/plugins/lolsrv.rb in lolcommits-0.5.4 vs lib/lolcommits/plugins/lolsrv.rb in lolcommits-0.5.5

@@ -8,11 +8,11 @@
     def initialize(runner)
       super
       self.options << 'server'
     end
 
-    def run
+    def run_postcapture
       return unless valid_configuration?
       fork { sync }
     end
 
     def configured?
@@ -30,40 +30,33 @@
         end
       end
     end
 
     def existing_lols
-      begin
-        lols = JSON.parse(
-        RestClient.get(configuration['server'] + '/lols'))
-        lols.map { |lol| lol['sha'] }
-      rescue => e
-        log_error(e, "ERROR: existing lols could not be retrieved #{e.class} - #{e.message}")
-        return nil
-      end
+      lols = JSON.parse(
+      RestClient.get(configuration['server'] + '/lols'))
+      lols.map { |lol| lol['sha'] }
+    rescue => e
+      log_error(e, "ERROR: existing lols could not be retrieved #{e.class} - #{e.message}")
+      return nil
     end
 
     def upload(file, sha)
-      begin
-        RestClient.post(
-          configuration['server'] + '/uplol',
-          :lol => File.new(file),
-          :url => self.runner.url + sha,
-          :repo => self.runner.repo,
-          :date => File.ctime(file),
-          :sha => sha)
-      rescue => e
-        log_error(e, "ERROR: Upload of lol #{sha} FAILED #{e.class} - #{e.message}")
-        return
-      end
+      RestClient.post(configuration['server'] + '/uplol',
+                      :lol => File.new(file),
+                      :url => self.runner.url + sha,
+                      :repo => self.runner.repo,
+                      :date => File.ctime(file),
+                      :sha => sha)
+    rescue => e
+      log_error(e, "ERROR: Upload of lol #{sha} FAILED #{e.class} - #{e.message}")
     end
 
-    def log_error(e, message)
-      debug message
-      debug e.backtrace
-    end
-
     def self.name
       'lolsrv'
+    end
+
+    def self.runner_order
+      :postcapture
     end
   end
 end