Pylint alerts corrections as part of intervention experiment #81

src/lib/Config.py

@@ -34,7 +34,8 @@ def set_default_config_file(self):
 
         self.config['MAIN'] = {
             'version': '2.00',
-            'keyVersion': '1',  # Will be used to support legacy key versions if the algorithm changes
+            'keyVersion': '1',  # Will be used to support legacy key versions 
+                                # if the algorithm changes
             'salt': self.generate_random_salt(),
             'clipboardTTL': '15',
             'hideSecretTTL': '5',

src/modules/autocomplete.py

@@ -70,5 +70,3 @@ def get_input_autocomplete(message=''):
         return input(message).strip()
     except KeyboardInterrupt:
         return False
-    except Exception:  # Other Exception
-        return False

src/modules/misc.py

@@ -42,7 +42,7 @@ def create_directory_if_missing(dir_):
             return True
 
         return False
-    except Exception:
+    except (OSError, FileExistsError):
         import sys
 
         print()

src/tools/troubleshoot_db.py

@@ -78,7 +78,7 @@ def attempt_dummy_encrypted_db(db_path):
     connection.execute(text('CREATE TABLE foo (a int)'))
     connection.execute(text('INSERT INTO foo (a) VALUES (123)'))
     result_proxy = connection.execute(text('SELECT * FROM foo'))
-    return True if result_proxy.fetchall() == [(123,)] else False
+    return bool(result_proxy.fetchall() == [(123,)])
 
 
 def verify_if_dummy_db_is_encrypted(db_path):

src/views/categories.py

@@ -267,7 +267,7 @@ def main_menu():
         Categories menu
     """
 
-    while (True):
+    while True:
         # Clear screen
         clear_screen()
 

src/views/import_export.py

@@ -102,7 +102,8 @@ def import_from_json(path=None, rows=None):
 def import_items(rows):
     """
         Import items at the following format:
-        [{'name': '...', 'url': '...', 'login': '...', 'password': '...', 'notes': '...', 'category': '...'}]
+        [{'name': '...', 'url': '...', 'login': '...'
+        , 'password': '...', 'notes': '...', 'category': '...'}]
     """
 
     for row in rows:

src/views/menu.py

@@ -32,8 +32,6 @@ def get_input(message='', secure=False, lowercase=False, check_timer=True, non_l
             input_ = input_.lower()
     except KeyboardInterrupt:
         return False
-    except Exception:  # Other Exception
-        return False
 
     return input_
 

src/views/migration.py

@@ -84,7 +84,8 @@ def migrate(vault_path, config_path, new_vault_path=None):
     print()
     print('The migration is now complete!')
     print('Restart the application to use Vault 2.')
-    print('Your old vault is stored in `%s`. You can discard this file after ensuring that all your data was migrated properly.' % (vault_path))
+    print('Your old vault is stored in `%s`. You can discard this file after ensuring that all your data was migrated properly.' 
+          % (vault_path))
     print('Your new vault is stored in `%s`.' % (new_vault_path))
     print()
 

src/views/setup.py

@@ -39,7 +39,8 @@ def initialize(salt):
             print("Your vault has been created and encrypted with your master key.")
             print("Your unique salt is: %s " % (salt))
             print(
-                "Write it down. If you lose your config file you will need it to unlock your vault.")
+                "Write it down."
+                 + " If you lose your config file you will need it to unlock your vault.")
 
             return True