handling downloads properly wrt electron accross app restarts (#2490)

RocketChat · Sep 1, 2022 · 27373b5 · 27373b5
1 parent 94391db
commit 27373b5
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/src/downloads/reducers/downloads.ts b/src/downloads/reducers/downloads.ts
@@ -6,7 +6,7 @@ import {
   DOWNLOAD_REMOVED,
   DOWNLOAD_UPDATED,
 } from '../actions';
-import { Download } from '../common';
+import { Download, DownloadStatus } from '../common';
 
 type DownloadsAction =
   | ActionOf<typeof APP_SETTINGS_LOADED>
@@ -20,8 +20,16 @@ export const downloads = (
   action: DownloadsAction
 ): Record<Download['itemId'], Download> => {
   switch (action.type) {
-    case APP_SETTINGS_LOADED:
-      return action.payload.downloads ?? {};
+    case APP_SETTINGS_LOADED: {
+      const initDownloads = action.payload.downloads ?? {};
+      Object.values(initDownloads).forEach((value) => {
+        if (value.state === 'progressing' || value.state === 'paused') {
+          value.state = 'cancelled';
+          value.status = DownloadStatus.CANCELLED;
+        }
+      });
+      return initDownloads ?? {};
+    }
 
     case DOWNLOAD_CREATED: {
       const download = action.payload;