bug for auto update is fixed

electron-projects/ucap-webmessenger-electron/src/index.ts

@@ -45,7 +45,10 @@ import {
   NotificationType,
   NativePathName
 } from '@ucap-webmessenger/native';
-import { ElectronAppChannel } from '@ucap-webmessenger/electron-core';
+import {
+  ElectronAppChannel,
+  ElectronBrowserWindowChannel
+} from '@ucap-webmessenger/electron-core';
 
 import {
   autoUpdater,
@@ -297,6 +300,10 @@ app.on(ElectronAppChannel.Ready, () => {
     maximizable: false,
     onReady: () => {},
     onAcceptUpdate: () => {
+      if (!!autoUpdaterCancellationToken) {
+        log.info('downloadUpdate already');
+        return;
+      }
       log.info('OnAcceptUpdate');
       autoUpdaterCancellationToken = new CancellationToken();
       autoUpdater.downloadUpdate(autoUpdaterCancellationToken);
@@ -368,10 +375,15 @@ ipcMain.on(UpdaterChannel.Apply, (event: IpcMainEvent, ...args: any[]) => {
 
   if (semver.lt(app.getVersion(), ver)) {
     updateCheckResult = undefined;
-    autoUpdater.checkForUpdatesAndNotify().then(r => {
+    autoUpdater
-      log.debug('checkForUpdatesAndNotify.then');
+      .checkForUpdatesAndNotify()
-      updateCheckResult = r;
+      .then(r => {
-    });
+        updateCheckResult = r;
+      })
+      .catch(reason => {
+        log.error(reason);
+      })
+      .finally(() => {});
   }
 });
 
@@ -796,16 +808,17 @@ autoUpdater.on('update-downloaded', info => {
 
   updateWindowService.setDownloadComplete();
 
-  app.removeAllListeners('window-all-closed');
+  app.removeAllListeners(ElectronAppChannel.WindowAllClosed);
   const browserWindows = BrowserWindow.getAllWindows();
 
   // https://github.com/electron-userland/electron-builder/issues/1604#issuecomment-372091881
   browserWindows.forEach(browserWindow => {
-    browserWindow.removeAllListeners('close');
+    browserWindow.removeAllListeners(ElectronBrowserWindowChannel.Close);
-    browserWindow.removeAllListeners('closed');
+    browserWindow.removeAllListeners(ElectronBrowserWindowChannel.Closed);
   });
 
   setTimeout(() => {
+    updateWindowService.close();
     autoUpdater.quitAndInstall(true, true);
   }, 2000);
 });

projects/ucap-webmessenger-app/src/app/layouts/native/components/top-bar.component.html

@@ -466,11 +466,8 @@
 
 <mat-menu #informationMenu="matMenu">
   <ng-template matMenuContent>
-    <div
+    <div class="version-info-container menu-item">
-      class="version-info-container menu-item"
+      <div class="version-info-now" (click)="$event.stopPropagation()">
-      (click)="$event.stopPropagation()"
-    >
-      <div class="version-info-now">
         <span class="version-info-item">
           {{ 'information.installedVersion' | translate }}:<span
             class="info-content"
@@ -493,7 +490,7 @@
         </div>
 
         <div *ngIf="!checkingUpdateIsProcessing" style="display: flex;">
-          <span class="version-info-item">
+          <span class="version-info-item" (click)="$event.stopPropagation()">
             {{ 'information.latestVersion' | translate }}:<span
               class="info-content"
               >{{ checkingUpdateAppVersion }}</span

projects/ucap-webmessenger-app/src/app/layouts/native/components/top-bar.component.ts

@@ -562,9 +562,9 @@ export class TopBarComponent implements OnInit, OnDestroy {
 
   onClickApplyUpdate(event: Event) {
     // this.profileMenuTrigger.closeMenu();
-    setTimeout(() => {
+    this.store.dispatch(
-      this.nativeService.checkForUpdates(this.checkingUpdateAppVersion);
+      UpdateStore.applyUpdate({ currentVersion: this.checkingUpdateAppVersion })
-    }, 1000);
+    );
   }
 
   onIntegratedSearch(keyword: string) {

projects/ucap-webmessenger-app/src/app/store/setting/update/actions.ts

@@ -2,10 +2,15 @@ import { createAction, props } from '@ngrx/store';
 import { UpdateInfo } from '@ucap-webmessenger/native';
 
 export const existInstantUpdate = createAction(
-  '[Setting::Update] existInstantUpdate',
+  '[Setting::Update] Exist Instant Update',
   props<{ updateInfo: UpdateInfo }>()
 );
 
-export const applyInstantUpdate = createAction(
+export const applyUpdate = createAction(
-  '[Setting::Update] applyInstantUpdate'
+  '[Setting::Update] Apply Update',
+  props<{ currentVersion: string }>()
+);
+
+export const applyInstantUpdate = createAction(
+  '[Setting::Update] Apply Instant Update'
 );

projects/ucap-webmessenger-app/src/app/store/setting/update/effects.ts

@@ -1,11 +1,24 @@
 import { Injectable, Inject } from '@angular/core';
 import { createEffect, Actions, ofType } from '@ngrx/effects';
-import { applyInstantUpdate } from './actions';
+import { applyInstantUpdate, applyUpdate } from './actions';
 import { UCAP_NATIVE_SERVICE, NativeService } from '@ucap-webmessenger/native';
-import { tap } from 'rxjs/operators';
+import { tap, map } from 'rxjs/operators';
 
 @Injectable()
 export class Effects {
+  applyUpdate$ = createEffect(
+    () => {
+      return this.actions$.pipe(
+        ofType(applyUpdate),
+        map(action => action.currentVersion),
+        tap(currentVersion => {
+          this.nativeService.checkForUpdates(currentVersion);
+        })
+      );
+    },
+    { dispatch: false }
+  );
+
   applyInstantUpdate$ = createEffect(
     () => {
       return this.actions$.pipe(