Fixed bug with loading lock monitor's settings when it hasn't been used before.

Previously its Setting class could handle when self.data['import_paths'] wasn't set, but the Plugin class would get tripped up. To work around this, the default value for that key is set in Setting.__init__() now.

Author: zakv

blacs/plugins/lock_monitor/__init__.py

@@ -1295,6 +1295,11 @@ def __init__(self, data):
         logger.info("Setting initialized.")
         self.data = data
 
+        # self.data['import_paths'] won't exist if the plugin hasn't been used
+        # before or if the save data was deleted. In that case set it to an
+        # empty list.
+        self.data.setdefault('import_paths', [])
+
     # Create the page, return the page and an icon to use on the label.
     def create_dialog(self, notebook):
         """Create the settings tab for this plugin.
@@ -1321,8 +1326,7 @@ def create_dialog(self, notebook):
         # Fill out the table with the saved values. Do this before connecting
         # the callback below to avoid it interfering.
         self._pop_row(0)  # Get rid of row in UI file.
-        import_paths = self.data.get('import_paths', [])
-        for import_path in import_paths:
+        for import_path in self.data['import_paths']:
             self._append_row(import_path)
         self._append_row(self._NEW_ENTRY_TEXT)