fix: graph colors now properly reset when swapping printers

Signed-off-by: Craig Bassett <[email protected]>

src/init.ts

@@ -120,5 +120,6 @@ export const appInit = async (apiConfig?: ApiConfig, hostConfig?: HostConfig): P
 
   // apiConfig could have empty strings, meaning we have no valid connection.
   await store.dispatch('init', { apiConfig, hostConfig })
+
   return { apiConfig, hostConfig, apiConnected }
 }

src/plugins/colorSet.ts

@@ -75,12 +75,11 @@ export class ColorSet {
    */
   reset (list: string, force = false): void {
     if (this.colorList[list] === undefined) return // not found
-    this.colorList[list].map(color => {
-      if (color.name && !force) {
-        // don't reset colors that have a name.
-        return color
-      } else {
-        return { ...color, used: false }
+    this.colorList[list].forEach(color => {
+      // don't reset colors that have a name, unless forced.
+      if (!color.name || force) {
+        color.used = false
+        delete color.name
       }
     })
   }

src/store/index.ts

@@ -50,7 +50,7 @@ export default new Vuex.Store<RootState>({
      */
     async reset ({ dispatch }, payload: string[]) {
       // Reset our color set.
-      Vue.$colorset.resetAll()
+      Vue.$colorset.forceResetAll()
 
       // Dispatch a reset for each registered module.
       const p: Promise<any>[] = []

todo.md

@@ -2,7 +2,6 @@
 
 ## Next Up
 
-- fix color swap when changing printers
 - add cpu time
 - clicking legend items when no chart throws error, fix.
 - sync instances across moonraker db