[Fix] Update for non token gated games (#889)

* fix update for non token gated games

* rm unused code in cancel download

* rm log

* fix library refresh

* add comment

* fix lint

* rm hp achievements submodule

Author: BrettCleary

src/backend/downloadmanager/downloadqueue.ts

@@ -1,7 +1,7 @@
 import { gameManagerMap, libraryManagerMap } from 'backend/storeManagers'
 import { TypeCheckedStoreBackend } from './../electron_store'
 import { logError, logInfo, LogPrefix, logWarning } from '../logger/logger'
-import { getFileSize, removeFolder } from '../utils'
+import { getFileSize } from '../utils'
 import { DMQueueElement, DMStatus, DownloadManagerState } from 'common/types'
 import { installQueueElement, updateQueueElement } from './utils'
 import { sendFrontendMessage } from '../main_window'
@@ -221,23 +221,10 @@ function cancelCurrentDownload({ removeDownloaded = false }) {
 
     const { runner } = currentElement!.params
     if (runner === 'hyperplay' && removeDownloaded) {
-      const { appName, gameInfo, channelName } = currentElement!.params
-      const { folder_name } = gameInfo
-      if (gameInfo.channels === undefined || channelName === undefined) {
-        console.error(
-          `Error when canceling current download channels ${gameInfo.channels} or channelName ${channelName} is undefined`
-        )
-        return
-      }
-      const releaseMeta = gameInfo.channels[channelName].release_meta
-
-      if (releaseMeta) {
-        const tempfolder = join(configFolder, 'hyperplay', '.temp', appName)
-        logInfo(`Removing ${tempfolder}...`, LogPrefix.DownloadManager)
-        callAbortController(appName)
-      } else if (folder_name) {
-        removeFolder(currentElement.params.path, folder_name)
-      }
+      const { appName } = currentElement!.params
+      const tempfolder = join(configFolder, 'hyperplay', '.temp', appName)
+      logInfo(`Removing ${tempfolder}...`, LogPrefix.DownloadManager)
+      callAbortController(appName)
     }
     currentElement = null
   }

src/frontend/helpers/library.ts

@@ -267,7 +267,13 @@ const launch = async ({
 }
 
 const updateGame = async (gameInfo: GameInfo) => {
-  const siweValues = await signSiweMessage()
+  const channelRequiresTokens =
+    gameInfo?.channels?.[gameInfo.install.channelName ?? ''].license_config
+      .tokens
+  let siweValues = undefined
+  if (channelRequiresTokens) {
+    siweValues = await signSiweMessage()
+  }
   return gameUpdateState.updateGame({ ...gameInfo, siweValues })
 }
 

src/frontend/state/libraryState.ts

@@ -114,6 +114,11 @@ class LibraryState {
   async refresh(library?: Runner | 'all', checkUpdates = true): Promise<void> {
     if (checkUpdates) {
       const hpUpdates = await window.api.checkGameUpdates(['hyperplay'])
+      /**
+       * Need to clear the gameUpdates array on refresh or updated games will still show
+       * as needing update after updating and entries will be added multiple times to the array
+       */
+      this.gameUpdates = []
       this.gameUpdates = [...this.gameUpdates, ...hpUpdates]
       window.api
         .checkGameUpdates(['legendary', 'gog', 'nile'])