🚧 Automatic Update checking in background implemented

motorlatitude · motorlatitude · commit 4edec99a10f5 · 2020-02-16T13:41:47.000Z
diff --git a/src/main/ipc/channel_types/PaletteChannel.js b/src/main/ipc/channel_types/PaletteChannel.js
@@ -12,7 +12,7 @@ class PaletteChannel extends Channel {
    * Creates an instance of PaletteChannel.
    * @param {{windowManager: WindowManager, store: ElectronStore, tray: ElectronTray, colorFormats: ColorFormats}} channelProps
    * @param {event} ipcEventObject ipc event object
-   * @param {{type: 'GET' | 'GET_ALL' | 'SAVE' | 'REMOVE_COLOR' | 'DELETE', args: *}} [ipcEventDataObject] the included data
+   * @param {{type: 'GET' | 'GET_ALL' | 'SAVE' | 'DELETE', args: *}} [ipcEventDataObject] the included data
    * @memberof PaletteChannel
    */
   constructor(channelProps, ipcEventObject, ipcEventDataObject) {
diff --git a/src/main/ipc/channel_types/SettingChannel.js b/src/main/ipc/channel_types/SettingChannel.js
@@ -21,7 +21,7 @@ class SettingChannel extends Channel {
       case "INSTALL_UPDATE":
         return this.installUpdate(autoUpdater);
       case "MODIFY_SETTING":
-        return this.modifySetting(ipcEventDataObject.args, appController);
+        return this.modifySetting(ipcEventDataObject.args, autoUpdater, appController);
       case "GET_SETTING":
         return this.getSetting(ipcEventDataObject.args);
       case "GET_ALL_SETTINGS":
@@ -52,7 +52,7 @@ class SettingChannel extends Channel {
    * Set or update a setting property
    * @param {{key: string, value: *}} args setting arguments
    */
-  modifySetting(args, appController) {
+  modifySetting(args, autoUpdater, appController) {
     const currentSettings = this.Store.get("settings", {}); //TODO: create default settings object
     currentSettings[args.key] = args.value;
     this.Store.set("settings", currentSettings);
@@ -62,6 +62,11 @@ class SettingChannel extends Channel {
       case "launchOnStartup":
         appController.setLoginItem(args.value);
         break;
+      case "autoCheckDownloadUpdates":
+        autoUpdater.stopCheckInterval();
+        if (args.value) {
+          autoUpdater.startCheckInterval();
+        }
       default:
         //log.log("Unhandled settings key", args.key);
         break;
diff --git a/src/main/resources/Updater.js b/src/main/resources/Updater.js
@@ -17,9 +17,53 @@ class Updater {
     this._Store = store;
     this._WindowManager = wm;
     this._configureEventListener();
+    this._AutoUpdaterInterval = null;
     autoUpdater.logger = log;
     autoUpdater.logger.transports.file.level = "info";
-    autoUpdater.autoDownload = false;
+    const settings = store.get("settings", {});
+    this._AutoCheck =
+      settings.autoCheckDownloadUpdates === undefined || settings.autoCheckDownloadUpdates === true ? true : false;
+    autoUpdater.autoDownload = this._AutoCheck;
+
+    // Attempt to start auto updating interval if settings permit
+    this.startCheckInterval();
+  }
+
+  /**
+   * Start the automatic update checking interval
+   *
+   * @memberof Updater
+   */
+  startCheckInterval() {
+    if (this._AutoCheck) {
+      log.info("Starting Automatic Update Checking Interval");
+      this._AutoUpdaterInterval = setInterval(() => {
+        const settings = this._Store.get("settings", {});
+        this._AutoCheck =
+          settings.autoCheckDownloadUpdates === undefined || settings.autoCheckDownloadUpdates === true ? true : false;
+        if (
+          this._AutoCheck &&
+          settings.lastUpdateCheck !== undefined &&
+          new Date().getTime() - settings.lastUpdateCheck >= 3600000
+        ) {
+          log.info("Auto Checking For Updates");
+          this.checkForUpdates();
+        }
+      }, 3600000);
+    }
+  }
+
+  /**
+   * Stop the automatic update checking interval
+   *
+   * @memberof Updater
+   */
+  stopCheckInterval() {
+    if (this._AutoUpdaterInterval !== null) {
+      log.info("Stopping Automatic Update Checking Interval");
+      clearInterval(this._AutoUpdaterInterval);
+      this._AutoUpdaterInterval = null;
+    }
   }
 
   /**
@@ -28,7 +72,6 @@ class Updater {
   async checkForUpdates() {
     return await autoUpdater.checkForUpdates().catch(err => {
       log.error(err);
-      reject(err);
     });
   }
 
