[Fix] Check for game updates before launch

electron/legendary/games.ts

@@ -521,7 +521,9 @@ class LegendaryGame extends Game {
     return res
   }
 
-  public async launch(launchArguments = ''): Promise<LaunchResult> {
+  public async launch(launchArguments: string): Promise<LaunchResult> {
+    console.log('launch', { launchArguments })
+
     const gameSettings =
       GameConfig.get(this.appName).config ||
       (await GameConfig.get(this.appName).getSettings())
@@ -572,6 +574,7 @@ class LegendaryGame extends Game {
           steamRuntime
         )
       }
+
       // These options are required on both Windows and Mac
       commandParts = [
         'launch',
@@ -623,7 +626,8 @@ class LegendaryGame extends Game {
         offlineFlag,
         ...wineFlag,
         ...winePrefixFlag,
-        launcherArgs
+        launcherArgs,
+        launchArguments
       ]
     }
     const command = getLegendaryCommand(commandParts, commandEnv, wrappers)

src/helpers/library.ts

@@ -249,43 +249,45 @@ type LaunchOptions = {
   t: TFunction<'gamepage'>
   launchArguments?: string
   runner: Runner
+  hasUpdate: boolean
 }
 
 const launch = async ({
   appName,
   t,
   launchArguments,
-  runner
-}: LaunchOptions): Promise<void> =>
-  ipcRenderer
+  runner,
+  hasUpdate
+}: LaunchOptions): Promise<void> => {
+  if (hasUpdate) {
+    const args = {
+      buttons: [t('gamepage:box.yes'), t('box.no')],
+      message: t('gamepage:box.update.message'),
+      title: t('gamepage:box.update.title')
+    }
+
+    const { response } = await ipcRenderer.invoke('openMessageBox', args)
+
+    if (response === 0) {
+      return updateGame(appName, runner)
+    }
+
+    return ipcRenderer.invoke('launch', {
+      appName,
+      runner,
+      launchArguments: runner === 'legendary' ? '--skip-version-check' : ''
+    })
+  }
+
+  return ipcRenderer
     .invoke('launch', { appName, launchArguments, runner })
     .then(async (err: string | string[]) => {
       if (!err) {
         return
       }
-
-      if (
-        typeof err === 'string' &&
-        err.includes('ERROR: Game is out of date')
-      ) {
-        const args = {
-          buttons: [t('gamepage:box.yes'), t('box.no')],
-          message: t('gamepage:box.update.message'),
-          title: t('gamepage:box.update.title')
-        }
-
-        const { response } = await ipcRenderer.invoke('openMessageBox', args)
-
-        if (response === 0) {
-          return updateGame(appName, runner)
-        }
-        await ipcRenderer.invoke('launch', {
-          appName,
-          runner,
-          launchArguments: '--skip-version-check'
-        })
-      }
+      return ipcRenderer.invoke('showErrorBox', ['Error', `${err}`])
     })
+}
 
 const updateGame = async (appName: string, runner: Runner): Promise<void> =>
   ipcRenderer.invoke('updateGame', appName, runner)

src/screens/Game/GamePage/index.tsx

@@ -536,7 +536,13 @@ export default function GamePage(): JSX.Element | null {
         await syncSaves(savesPath, appName)
         setIsSyncing(false)
       }
-      await launch({ appName, t, launchArguments, runner: gameInfo.runner })
+      await launch({
+        appName,
+        t,
+        launchArguments,
+        runner: gameInfo.runner,
+        hasUpdate
+      })
 
       if (autoSyncSaves) {
         setIsSyncing(true)

src/screens/Library/components/GameCard/index.tsx

@@ -361,7 +361,7 @@ const GameCard = ({
       return sendKill(appName, runner)
     }
     if (isInstalled) {
-      return launch({ appName, t, runner })
+      return launch({ appName, t, runner, hasUpdate })
     }
     return
   }

src/state/GlobalState.tsx

@@ -454,7 +454,8 @@ export class GlobalState extends PureComponent<Props> {
       )[0]
       if (!currentApp) {
         // Add finding a runner for games
-        return launch({ appName, t, runner })
+        const hasUpdate = this.state.gameUpdates?.includes(appName)
+        return launch({ appName, t, runner, hasUpdate })
       }
     })