fix: Curseforge page cannot load

Voxelum · May 20, 2022 · f861aa6 · f861aa6
1 parent 117acce
commit f861aa6
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/xmcl-runtime/lib/services/CurseForgeService.ts b/xmcl-runtime/lib/services/CurseForgeService.ts
@@ -30,18 +30,23 @@ export class CurseForgeService extends StatefulService<CurseforgeState> implemen
   }
 
   private async fetchOrGetFromCache<K extends string | number, V>(cacheName: string, cache: Record<K, V>, key: K, query: () => Promise<V>) {
-    if (!cache[key]) {
-      const timestamp = await getAddonDatabaseTimestamp({ userAgent: this.networkManager.agents.https })
+    const timestamp = await getAddonDatabaseTimestamp({ userAgent: this.networkManager.agents.https })
+    if (cache[key]) {
       if (new Date(timestamp) > new Date(this.projectTimestamp)) {
         const value = await query()
         this.projectTimestamp = timestamp
         cache[key] = value
         this.log(`Cache missed for ${key} in ${cacheName}`)
         return value
       }
+      this.log(`Cache hit for ${key} in ${cacheName}`)
+      return cache[key]
     }
-    this.log(`Cache hit for ${key} in ${cacheName}`)
-    return cache[key]
+    const value = await query()
+    this.projectTimestamp = timestamp
+    cache[key] = value
+    this.log(`Cache missed for ${key} in ${cacheName}`)
+    return value
   }
 
   @Singleton()