From d0d36470bae570785280e31ae6a95f48fde21b49 Mon Sep 17 00:00:00 2001
From: Stefan Dej <meteyou@gmail.com>
Date: Sat, 1 Aug 2020 16:41:33 +0200
Subject: [PATCH] update loadingQGL to generic function report klippy state in
 browser console log

Signed-off-by: Stefan Dej <meteyou@gmail.com>
---
 src/components/panels/ControlPanel.vue | 10 ++++------
 src/store/actions.js                   |  8 ++++----
 src/store/index.js                     |  1 -
 src/store/mutations.js                 | 23 ++++++++++++++---------
 4 files changed, 22 insertions(+), 20 deletions(-)

diff --git a/src/components/panels/ControlPanel.vue b/src/components/panels/ControlPanel.vue
index b3c9bca4..61213f1b 100644
--- a/src/components/panels/ControlPanel.vue
+++ b/src/components/panels/ControlPanel.vue
@@ -61,7 +61,7 @@
 </template>
 
 <script>
-    import { mapState, mapGetters, mapMutations } from 'vuex'
+    import { mapState, mapGetters } from 'vuex'
     import Vue from "vue";
 
     export default {
@@ -74,19 +74,16 @@
                 loadingHomeX: false,
                 loadingHomeY: false,
                 loadingHomeZ: false,
+                loadingQGL: false,
                 loadingZTilt: false,
             }
         },
         computed: {
             ...mapState({
-                loadingQGL: state => state.socket.loadingQGL,
                 homedAxes: state => state.printer.toolhead.homed_axes,
                 config: state => state.config,
                 loadings: state => state.loadings,
             }),
-            ...mapMutations([
-                'setLoadingQGL',
-            ]),
             ...mapGetters([
                 'getMacros',
                 'is_printing'
@@ -115,7 +112,7 @@
             },
             doQGL() {
                 this.$store.commit('addGcodeResponse', "QUAD_GANTRY_LEVEL");
-                this.$store.commit('setLoadingQGL', true);
+              this.$store.commit('setLoading', { name: 'controlQGL' });
                 this.$socket.sendObj('post_printer_gcode_script', { script: "QUAD_GANTRY_LEVEL" }, "responseQGL");
             },
             doZtilt() {
@@ -141,6 +138,7 @@
                 this.loadingHomeX = loadings.includes('controlHomeX');
                 this.loadingHomeY = loadings.includes('controlHomeY');
                 this.loadingHomeZ = loadings.includes('controlHomeZ');
+                this.loadingQGL = loadings.includes('controlQGL');
                 this.loadingZTilt = loadings.includes('controlZTilt');
             }
         }
diff --git a/src/store/actions.js b/src/store/actions.js
index c26e0c30..81d9fba5 100644
--- a/src/store/actions.js
+++ b/src/store/actions.js
@@ -32,6 +32,8 @@ export default {
                 break;
 
             case 'notify_klippy_state_changed':
+                window.console.log('notify_klippy_state_changed');
+                window.console.log(data.params[0]);
                 commit('setKlippyStatus', data.params[0]);
                 break;
 
@@ -105,8 +107,6 @@ export default {
                 version: data.version
             });
 
-            commit('setKlippyStatus', 'ready');
-
             Vue.prototype.$socket.sendObj('get_printer_objects_list', {}, 'getObjectInfo');
             Vue.prototype.$socket.sendObj('get_printer_objects_status', { heaters: [] }, 'getHeatersInfo');
             Vue.prototype.$socket.sendObj('get_printer_objects_status', { configfile: ['config'] }, 'getPrinterConfig');
@@ -122,7 +122,7 @@ export default {
             });
             Vue.prototype.$socket.sendObj('get_directory', { path: 'gcodes' }, 'getDirectory');
             Vue.prototype.$socket.sendObj('get_printer_gcode_help', {}, 'getHelpList');
-        } else if (data !== undefined && !data.is_ready) commit('setKlippyStatus', 'disconnect');
+        }
 
         commit('setPrinterStatusDetails', data);
 
@@ -276,7 +276,7 @@ export default {
     },
 
     responseQGL({commit}) {
-        commit('setLoadingQGL', false);
+        commit('removeLoading', { name: 'controlQGL' });
     },
 
     responseZTilt({commit}) {
diff --git a/src/store/index.js b/src/store/index.js
index 8ac9bf15..11abde06 100644
--- a/src/store/index.js
+++ b/src/store/index.js
@@ -20,7 +20,6 @@ export default new Vuex.Store({
             loadingEmergencyStop: false,
             loadingPrintPause: false,
             loadingPrintResume: false,
-            loadingQGL: false,
             loadingRestart: false,
             loadingRestartFirmware: false,
             loadingRebootHost: false,
diff --git a/src/store/mutations.js b/src/store/mutations.js
index 0f35f736..45f377ae 100644
--- a/src/store/mutations.js
+++ b/src/store/mutations.js
@@ -139,7 +139,7 @@ export default {
         }
 
         // update target temp in temperature chart
-        if (index_target >= 0) {
+        if (index_target >= 0 && payload.value.target !== undefined) {
             for (let i = 0; i < ticks; i++) state.temperaturChart.datasets[index_target].data.push(payload.value.target.toFixed(1));
         }
 
@@ -443,17 +443,22 @@ export default {
     },
 
     setKlippyStatus(state, value) {
-        state.socket.klippy_state = value;
-        if (value === "disconnect") {
-            state.socket.is_ready = false;
-            state.socket.klippy_message = "";
-        } else if (value === "ready") Vue.prototype.$socket.sendObj('get_printer_info', {}, 'getKlipperInfo');
+        if (value !== undefined) {
+            state.socket.klippy_state = value;
+
+            if (value === "disconnect") {
+                state.socket.is_ready = false;
+                state.socket.klippy_message = "";
+            } else if (value === "ready") Vue.prototype.$socket.sendObj('get_printer_info', {}, 'getKlipperInfo');
+        }
     },
 
     setPrinterStatusDetails(state, data) {
-        state.socket.error_detected = data.error_detected;
-        state.socket.is_ready = data.is_ready;
-        state.socket.klippy_message = data.message;
+        if (data !== undefined && data.is_ready !== undefined) {
+            state.socket.error_detected = data.error_detected;
+            state.socket.is_ready = data.is_ready;
+            state.socket.klippy_message = data.message;
+        }
     },
 
     setLoadingGcodeUpload(state, value) {