From 0ae93ea5b6564c33565cc74216bacf90547513b4 Mon Sep 17 00:00:00 2001 From: Janne Savolainen Date: Mon, 9 May 2022 12:50:44 +0300 Subject: [PATCH] Switch to using injectable for limiting side effect Co-authored-by: Mikko Aspiala Signed-off-by: Janne Savolainen --- .../features/quit-and-install-update.injectable.ts | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/src/main/electron-app/features/quit-and-install-update.injectable.ts b/src/main/electron-app/features/quit-and-install-update.injectable.ts index a36c390bc4..6b313e21b0 100644 --- a/src/main/electron-app/features/quit-and-install-update.injectable.ts +++ b/src/main/electron-app/features/quit-and-install-update.injectable.ts @@ -3,16 +3,18 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; -import { autoUpdater } from "electron-updater"; +import electronUpdaterInjectable from "./electron-updater.injectable"; const quitAndInstallUpdateInjectable = getInjectable({ id: "quit-and-install-update", - instantiate: () => () => { - autoUpdater.quitAndInstall(true, true); - }, + instantiate: (di) => { + const electronUpdater = di.inject(electronUpdaterInjectable); - causesSideEffects: true, + return () => { + electronUpdater.quitAndInstall(true, true); + }; + }, }); export default quitAndInstallUpdateInjectable;