Merge pull request #245 from adam111316/feature/CleanupForceVariable

Remove unused force variable from code and PEP8

CHANGES.md

@@ -20,11 +20,12 @@
 * Add Unknown status query to Episode Status Management
 * Fix Episode Status Management error popup from coming up when show is selected without expanding
 * Add BET network logo
-* Change redirect from home to showlistView on changing Layout on showlistView
+* Remove unused force variable from code and PEP8
 
 [develop changelog]
 * Fix traceback error when using the menu item Manage/Update Kodi
 * Change BET network logo filename to lower case
+* Change redirect from home to showlistView on changing Layout on showlistView
 
 ### 0.7.0 (2015-02-23 11:02:00 UTC)
 

sickbeard/autoPostProcesser.py

@@ -19,26 +19,21 @@
 import os.path
 
 import sickbeard
-
+from sickbeard import logger, processTV
-from sickbeard import logger
 from sickbeard import encodingKludge as ek
-from sickbeard import processTV
 
 
 class PostProcesser():
-    def run(self, force=False):
+    @staticmethod
+    def run():
         if not ek.ek(os.path.isdir, sickbeard.TV_DOWNLOAD_DIR):
-            logger.log(u"Automatic post-processing attempted but dir " + sickbeard.TV_DOWNLOAD_DIR + " doesn't exist",
+            logger.log(u"Automatic post-processing attempted but dir %s doesn't exist" % sickbeard.TV_DOWNLOAD_DIR,
                        logger.ERROR)
             return
 
         if not ek.ek(os.path.isabs, sickbeard.TV_DOWNLOAD_DIR):
-            logger.log(
+            logger.log(u'Automatic post-processing attempted but dir %s is relative '
-                u"Automatic post-processing attempted but dir " + sickbeard.TV_DOWNLOAD_DIR + " is relative (and probably not what you really want to process)",
+                       '(and probably not what you really want to process)' % sickbeard.TV_DOWNLOAD_DIR, logger.ERROR)
-                logger.ERROR)
             return
 
-        processTV.processDir(sickbeard.TV_DOWNLOAD_DIR)
+        processTV.processDir(sickbeard.TV_DOWNLOAD_DIR)
-
-    def __del__(self):
-        pass

sickbeard/properFinder.py

@@ -39,7 +39,7 @@ class ProperFinder():
     def __init__(self):
         self.amActive = False
 
-    def run(self, force=False):
+    def run(self):
 
         if not sickbeard.DOWNLOAD_PROPERS:
             return

sickbeard/scheduler.py

@@ -77,7 +77,7 @@ class Scheduler(threading.Thread):
                     if not self.silent:
                         logger.log(u"Starting new thread: " + self.name, logger.DEBUG)
 
-                    self.action.run(self.force)
+                    self.action.run()
                 except Exception, e:
                     logger.log(u"Exception generated in thread " + self.name + ": " + ex(e), logger.ERROR)
                     logger.log(repr(traceback.format_exc()), logger.DEBUG)

sickbeard/searchBacklog.py

@@ -187,7 +187,7 @@ class BacklogSearcher:
             myDB.action("UPDATE info SET last_backlog=" + str(when))
 
 
-    def run(self, force=False):
+    def run(self):
         try:
             self.searchBacklog()
         except:

sickbeard/searchRecent.py

@@ -37,7 +37,7 @@ class RecentSearcher():
         self.lock = threading.Lock()
         self.amActive = False
 
-    def run(self, force=False):
+    def run(self):
 
         self.amActive = True