Display error message to user if file or directory isn't selected during export

Author: nur557

src/modules/flightlog.js

@@ -10,9 +10,7 @@ export function exportFlightLogToCSVs(flightLog) {
   let paths = dialog.showOpenDialogSync({ properties: ["openDirectory"] });
 
   if (!paths) {
-    console.log("No directory selected for export.");
-    // TODO: after snackbar is merged print error message to user
-    return;
+    throw new Error("No directory selected for export.");
   }
 
   const userFolderPath = path.join(paths[0]);
@@ -23,9 +21,7 @@ export function exportFlightLogToCSVs(flightLog) {
   for (let flightLogSection of flightLogSections) {
     fs.writeFile(`${exportFolderPath}/${flightLogSection}.csv`, objectArrayToCSV(flightLogSection, flightLog[flightLogSection]), "utf8", function (err) {
       if (err) {
-        console.log("An error occurred while writing CSV object to file.");
-        // TODO: after snackbar is merged print error message to user
-        return console.log(err);
+        throw new Error(`An error occurred while writing CSV object to file.`);
       }
     });
   }
@@ -35,9 +31,7 @@ export function exportFlightLogChartsToHTML(flightLogChartsHTML) {
   let paths = dialog.showOpenDialogSync({ properties: ["openDirectory"] });
 
   if (!paths) {
-    console.log("No directory selected for export.");
-    return;
-    // TODO: after snackbar is merged print error message to user
+    throw new Error("No directory selected for export.");
   }
 
   const exportFolderPath = path.join(paths[0]);
@@ -56,9 +50,7 @@ export function exportFlightLogChartsToHTML(flightLogChartsHTML) {
 
   fs.writeFile(`${exportFolderPath}/${flightLogFilename}_plots_${formatDateTime(new Date())}.html`, flightLogHtmlDocument, 'utf8', function (err) {
     if (err) {
-      console.log("An error occurred while writing HTML Object to File.");
-      return console.log(err);
-      // TODO: after snackbar is merged print error message to user
+      throw new Error(`An error occurred while writing HTML Object to File.`);
     }
   });
 }

src/modules/ipc.js

@@ -75,13 +75,21 @@ export function subscribeListeners() {
   });
 
   ipcMain.on("EXPORT_FLIGHTLOG_CSVS", (event, flightLog) => {
-    exportFlightLogToCSVs(flightLog)
-    event.sender.send("EXPORT_FLIGHTLOG_CSVS");
+    try {
+      exportFlightLogToCSVs(flightLog)
+      event.sender.send("EXPORT_FLIGHTLOG_CSVS");
+    } catch (error) {
+      event.sender.send("EXPORT_FLIGHTLOG_CSVS", { error: error.message });
+    }
   });
 
   ipcMain.on("EXPORT_FLIGHTLOG_HTML", (event, flightLogChartsHTML) => {
-    exportFlightLogChartsToHTML(flightLogChartsHTML);
-    event.sender.send("EXPORT_FLIGHTLOG_HTML");
+    try {
+      exportFlightLogChartsToHTML(flightLogChartsHTML);
+      event.sender.send("EXPORT_FLIGHTLOG_HTML");
+    } catch (error) {
+      event.sender.send("EXPORT_FLIGHTLOG_HTML", { error: error.message });
+    }    
   });
 
   ipcMain.on("BOARD:RESET_CONFIG", () => {

src/modules/serial.js

@@ -165,8 +165,12 @@ function onData(data) {
   if (currentCommand === "dump") {
     if (data === "#Configuration dump") return;
     else if (data === "#End of configuration dump") {
-      saveDumpDataToFile(backupConfig.trim());
-      sendToRenderer("BOARD:DUMP");
+      try {
+        saveDumpDataToFile(backupConfig.trim());
+        sendToRenderer("BOARD:DUMP");
+      } catch (error) {
+        return sendToRenderer("BOARD:DUMP", { error: error.message });
+      }
       backupConfig = "";
     } else backupConfig += data + "\n";
 
@@ -294,6 +298,8 @@ function saveDumpDataToFile(data) {
   if (paths) {
     const file = path.join(paths[0], "backup_cats_config.txt");
     fs.writeFileSync(file, data);
+  } else {
+    throw new Error("No directory selected for backup.");
   }
 }
 

src/store/index.js

@@ -142,6 +142,9 @@ export default new Vuex.Store({
     showSuccessSnackbar({ commit }, message) {
       commit("SHOW_SNACKBAR", { message, color: "success" });
     },
+    showErrorSnackbar({ commit }, message) {
+      commit("SHOW_SNACKBAR", { message, color: "error" });
+    },
     hideSnackbar({ commit }) {
       commit("HIDE_SNACKBAR");
     },

src/views/Config.vue

@@ -419,9 +419,14 @@ export default {
   },
   mounted() {
     this.init();
-    window.renderer.on("BOARD:DUMP", () => {
+    window.renderer.on("BOARD:DUMP", (result) => {
       this.backupLoading = false
-      this.showSuccessSnackbar("Backup created!");
+      if (result?.error) {
+        this.showErrorSnackbar(`${result.error}`);
+        return;
+      } else {
+        this.showSuccessSnackbar("Backup created!");
+      }
     });
     window.renderer.on("BOARD:RESTORE", () => {
       this.restoreLoading = false;
@@ -441,7 +446,7 @@ export default {
     clearInterval(this.timer);
   },
   methods: {
-    ...mapActions(["setChangedTab", "showSuccessSnackbar"]),
+    ...mapActions(["setChangedTab", "showSuccessSnackbar", "showErrorSnackbar"]),
     init() {
       getConfigs();
       this.getInfo();

src/views/Home.vue

@@ -106,17 +106,27 @@ export default {
 
       if (el) makePlots(flightLog, el, this.useImperialUnits);
     });
-    window.renderer.on("EXPORT_FLIGHTLOG_CSVS", (flightLog) => {
+    window.renderer.on("EXPORT_FLIGHTLOG_CSVS", (result) => {
       this.exportButtonLoading = false;
-      this.showSuccessSnackbar("Flight log CSVs exported!");
+      if (result?.error) {
+        this.showErrorSnackbar(`${result.error}`);
+        return;
+      } else {
+        this.showSuccessSnackbar("Flight log CSVs exported!");
+      }
     });
-    window.renderer.on("EXPORT_FLIGHTLOG_HTML", (flightLog) => {
+    window.renderer.on("EXPORT_FLIGHTLOG_HTML", (result) => {
       this.exportButtonLoading = false;
-      this.showSuccessSnackbar("Flight log HTML plots exported!");
+      if (result?.error) {
+        this.showErrorSnackbar(`${result.error}`);
+        return;
+      } else {
+        this.showSuccessSnackbar("Flight log HTML plots exported!");
+      }
     });
   },
   methods: {
-    ...mapActions(["showSuccessSnackbar"]),
+    ...mapActions(["showSuccessSnackbar", "showErrorSnackbar"]),
     loadFlightLog(file) {
       this.loadButtonLoading = true;
       this.flightLog = null