Add a settings page to make it easier to modify

the autoconnect delay and avoid issues like #9

octoprint_portlister/__init__.py

@@ -18,7 +18,10 @@ def on_created(self, event):
 		if not event.is_directory:
 			self._parent.on_port_created(event.src_path)
 
-class PortListerPlugin(octoprint.plugin.StartupPlugin, octoprint.plugin.AssetPlugin, octoprint.plugin.SettingsPlugin):
+class PortListerPlugin(octoprint.plugin.StartupPlugin,
+                       octoprint.plugin.AssetPlugin,
+                       octoprint.plugin.TemplatePlugin,
+                       octoprint.plugin.SettingsPlugin):
 	def on_after_startup(self, *args, **kwargs):
 		self._logger.info("Port Lister %s %s" % (repr(args), repr(kwargs)))
 		event_handler = PortListEventHandler(self)
@@ -39,8 +42,9 @@ def on_port_created(self, port, *args, **kwargs):
 
 				# if autoconnect and the new port matches, try to connect
 				if self._settings.global_get_boolean(["serial", "autoconnect"]):
-					self._logger.info("autoconnect_delay %d", self._settings.get(["autoconnect_delay"]))
-					Timer(self._settings.get(["autoconnect_delay"]), self.do_auto_connect, [port]).start()
+					autoconnect_delay = self._settings.get_int(["autoconnect_delay"])
+					self._logger.info("autoconnect_delay {0}".format(autoconnect_delay))
+					Timer(autoconnect_delay, self.do_auto_connect, [port]).start()
 				else:
 					self._logger.info("Not autoconnecting because autoconnect is turned off.")
 			else:
@@ -56,7 +60,7 @@ def on_shutdown(self, *args, **kwargs):
 	def do_auto_connect(self, port, *args, **kwargs):
 		try:
 			self._logger.info("do_auto_connect")
-			(autoport, baudrate) = self._settings.global_get(["serial", "port"]), self._settings.global_get_int(["serial", "baudrate"])
+			(autoport, baudrate) = self._settings.global_get(["serial", "port"]), self._settings.global_get(["serial", "baudrate"])
 			if not autoport:
 				autoport = "AUTO"
 			if not port:

octoprint_portlister/static/js/portlister.js

@@ -3,17 +3,22 @@ $(function () {
         var self = this;
 
         self.connection = parameters[0];
+        self.settingsViewModel = parameters[1];
 
         self.onDataUpdaterPluginMessage = function(plugin, message) {
             if (plugin == "PortLister") {
                 self.connection.requestData();
             }
         }
+
+        self.onBeforeBinding = function() {
+            self.settings = self.settingsViewModel.settings;
+        };
     }
 
-    OCTOPRINT_VIEWMODELS.push([
-        PortListerViewModel,
-        ["connectionViewModel"],
-        []
-    ]);
+    OCTOPRINT_VIEWMODELS.push({
+        construct: PortListerViewModel,
+        dependencies: ["connectionViewModel", "settingsViewModel"],
+        elements: ["#settings_plugin_portlister"]
+    });
 });

octoprint_portlister/templates/portlister_settings.jinja2

@@ -0,0 +1,11 @@
+<form id="settings_plugin_portlister" class="form-horizontal">
+    <div class="control-group">
+        <label class="control-label">{{ _('Autoconnect delay:') }}</label>
+        <div class="controls" data-toggle="tooltip" title="{{ _('The port is often detected before the printer is ready to connect. Delay this number of seconds before trying.') }}">
+            <div class="input-append">
+                <input type="number" step="1" class="input-mini text-right" data-bind="value: settings.plugins.portlister.autoconnect_delay">
+                <span class="add-on">secs</span>
+            </div>
+        </div>
+    </div>
+</form>

setup.py

@@ -5,7 +5,7 @@
 plugin_identifier = "portlister"
 plugin_package = "octoprint_portlister"
 plugin_name = "OctoPrint-PortLister"
-plugin_version = "0.1.9"
+plugin_version = "0.1.10"
 plugin_description = """PortLister notices when printers get turned on and tells the list of ports to refresh"""
 plugin_author = "Mark Walker"
 plugin_author_email = "[email protected]"