CR

Author: bossanova808

resources/lib/cabertoss.py

@@ -20,7 +20,7 @@ def _vfs_join(base: str, name: str) -> str:
     return os.path.join(base, name)
 
 
-def gather_log_files():
+def gather_log_files() -> List[Tuple[str, str]]:
     """
     Gather a list of the standard Kodi log files (Kodi.log, Kodi.old.log) and the latest crash log, if there is one.
 
@@ -29,7 +29,7 @@ def gather_log_files():
 
     # Basic log files
     log_files = [('log', os.path.join(LOG_PATH, 'kodi.log'))]
-    if os.path.exists(os.path.join(LOG_PATH, 'kodi.old.log')):
+    if xbmcvfs.exists(os.path.join(LOG_PATH, 'kodi.old.log')):
         log_files.append(('oldlog', os.path.join(LOG_PATH, 'kodi.old.log')))
 
     # Can we find a crashlog?

resources/lib/store.py

@@ -13,8 +13,8 @@ class Store:
 
     # Static class variables, referred to elsewhere by Store.whatever
     # https://docs.python.org/3/faq/programming.html#how-do-i-create-static-class-data-and-static-class-methods
-    destination_path = None
-    crashlog_max_days = 3
+    destination_path: str = ''
+    crashlog_max_days: int = 3
 
     def __init__(self):
         """
@@ -35,5 +35,5 @@ def load_config_from_settings():
             Logger.info(f'Logs will be tossed to: {clean_log(Store.destination_path)}')
         else:
             Logger.warning('No path set to toss logs to.')
-        Store.crashlog_max_days = ADDON.getSetting('crashlog_max_days') or 3
+        Store.crashlog_max_days = int(ADDON.getSetting('crashlog_max_days')) or 3
         Logger.info(f'Crashlog max days: {Store.crashlog_max_days}')

resources/settings.xml

@@ -20,6 +20,11 @@
                 <setting id="crashlog_max_days" type="integer" label="32032" help="">
                     <level>0</level>
                     <default>3</default>
+                    <constraints>
+                        <minimum>0</minimum>
+                        <maximum>365</maximum>
+                        <step>1</step>
+                    </constraints>
                     <control type="edit" format="integer">
                         <heading>32032</heading>
                     </control>