From f109147e622327176b89be4d104b9b1c3f61ea8a Mon Sep 17 00:00:00 2001
From: Casper Lamboo <casperlamboo@gmail.com>
Date: Mon, 23 Apr 2018 12:27:16 +0200
Subject: [PATCH] only request doodle printers when adding or managing printers

---
 src/interface/Settings.js | 39 +++++++++++++++++++++++++++------------
 1 file changed, 27 insertions(+), 12 deletions(-)

diff --git a/src/interface/Settings.js b/src/interface/Settings.js
index 32d1e8e..4872299 100644
--- a/src/interface/Settings.js
+++ b/src/interface/Settings.js
@@ -25,7 +25,6 @@ import { Doodle3DManager } from 'doodle3d-api';
 
 const DOODLE_3D_MANAGER = new Doodle3DManager();
 DOODLE_3D_MANAGER.checkNonServerBoxes = false;
-DOODLE_3D_MANAGER.setAutoUpdate(true, 5000);
 
 const CONNECT_URL = 'http://connect.doodle3d.com/';
 
@@ -127,8 +126,8 @@ class Settings extends React.Component {
     }
 
     const eventListener = ({ boxes }) => this.setState({ wifiBoxes: boxes });
-    this.setState({ wifiBoxes: DOODLE_3D_MANAGER.boxes, eventListener });
     DOODLE_3D_MANAGER.addEventListener('boxeschanged', eventListener);
+    this.setState({ eventListener });
   }
 
   componentWillUnmount() {
@@ -361,22 +360,38 @@ class Settings extends React.Component {
 
   closeAddPrinterDialog = (override) => this.setAddPrinterDialog(false, override);
   openAddPrinterDialog = (override) => this.setAddPrinterDialog(true, override);
-  setAddPrinterDialog = (open, override = {}) => this.setState({
-    addPrinter: {
-      ip: '',
-      name: '',
-      printer: '',
-      error: null,
-      open,
-      ...override
+  setAddPrinterDialog = (open, override = {}) => {
+    if (open) {
+      DOODLE_3D_MANAGER.setAutoUpdate(true, 10000);
+    } else {
+      DOODLE_3D_MANAGER.setAutoUpdate(false);
     }
-  });
+    this.setState({
+      addPrinter: {
+        ip: '',
+        name: '',
+        printer: '',
+        error: null,
+        open,
+        ...override
+      }
+    });
+  };
 
   closeManagePrinterDialog = () => this.setManagePrinterDialog(false);
   openManagePrinterDialog = () => this.setManagePrinterDialog(true);
   setManagePrinterDialog = (open) => {
     const { localStorage: { active, printers } } = this.state;
-    if (!active) return this.setState({ managePrinter: { open: false } });
+    if (!active) {
+      DOODLE_3D_MANAGER.setAutoUpdate(false);
+      return this.setState({ managePrinter: { open: false } });
+    }
+
+    if (open) {
+      DOODLE_3D_MANAGER.setAutoUpdate(true, 10000);
+    } else {
+      DOODLE_3D_MANAGER.setAutoUpdate(false);
+    }
     this.setState({
       managePrinter: {
         open,