Refactor Manual Runs (#75)

* only create monitor when service is running

* simplify manual run

* beta version

addon.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
 <addon id="service.libraryautoupdate"
-    name="Library Auto Update" version="1.2.5" provider-name="robweber">
+    name="Library Auto Update" version="1.2.6~beta1" provider-name="robweber">
   <requires>
     <import addon="xbmc.python" version="3.0.0"/>
     <import addon="script.module.dateutil" version="2.7.3" />

manual.py

@@ -17,13 +17,5 @@
     # run the program
     utils.log("Update Library Manual Run...")
 
-    # trick the auto updater into resetting the last_run time
-    autoUpdate.last_run = 0
-    autoUpdate.writeLastRun()
-
     # update the schedules and evaluate them in manual override mode
-    autoUpdate.createSchedules(True)
     autoUpdate.evalSchedules(True)
-
-    # delete the monitor before exiting
-    del autoUpdate.monitor

resources/lib/service.py

@@ -20,13 +20,13 @@ class AutoUpdater:
 
     def __init__(self):
         utils.check_data_dir()  # in case this directory does not exist yet
-        self.monitor = UpdateMonitor(update_settings=self.createSchedules, after_scan=self.databaseUpdated)
         self.readLastRun()
 
         # force and update on startup to create the array
         self.createSchedules(True)
 
     def runProgram(self):
+        self.monitor = UpdateMonitor(update_settings=self.createSchedules, after_scan=self.databaseUpdated)
         # a one-time catch for the startup delay
         if(utils.getSettingInt("startup_delay") != 0):
             count = 0
@@ -70,7 +70,7 @@ def evalSchedules(self, manual=False):
             while count < len(self.schedules):
                 cronJob = self.schedules[count]
 
-                if(cronJob.next_run <= now):
+                if(cronJob.next_run <= now or manual):
                     if(not player.isPlaying() or utils.getSetting("run_during_playback") == "true"):
 
                         # check if run on idle is checked and screen is idle - disable this on manual run