From 67796d8a5e01fa70f2a8de28ee08fe56454545ee Mon Sep 17 00:00:00 2001
From: Georges-Antoine Assi <georges.antoine.assi@gmail.com>
Date: Mon, 15 Jan 2024 09:36:24 -0500
Subject: [PATCH] Disable the pwa service worker

---
 backend/models/rom.py                       |  2 +-
 frontend/src/components/Details/Cover.vue   |  7 ++++++-
 frontend/src/components/Game/Card/Cover.vue |  9 ++++++---
 frontend/vite.config.js                     | 10 ++--------
 4 files changed, 15 insertions(+), 13 deletions(-)

diff --git a/backend/models/rom.py b/backend/models/rom.py
index eb686ec91..4589d5cba 100644
--- a/backend/models/rom.py
+++ b/backend/models/rom.py
@@ -101,7 +101,7 @@ class Rom(BaseModel):
     def file_size_bytes(self) -> int:
         return int(self.file_size * SIZE_UNIT_TO_BYTES[self.file_size_units or "B"])
 
-    @property
+    @cached_property
     def has_cover(self) -> bool:
         return (
             self.path_cover_s != DEFAULT_PATH_COVER_S
diff --git a/frontend/src/components/Details/Cover.vue b/frontend/src/components/Details/Cover.vue
index 1941f597a..e2b4d0b30 100644
--- a/frontend/src/components/Details/Cover.vue
+++ b/frontend/src/components/Details/Cover.vue
@@ -11,7 +11,12 @@ defineProps<{ rom: Rom }>();
     :loading="downloadStore.value.includes(rom.id) ? 'romm-accent-1' : false"
   >
     <v-img
-      :src="`/assets/romm/resources/${rom.path_cover_l || rom.merged_screenshots[0]}`"
+      :cover="!rom.has_cover"
+      :src="
+        !rom.has_cover && rom.merged_screenshots.length > 0
+          ? rom.merged_screenshots[0]
+          : `/assets/romm/resources/${rom.path_cover_l}`
+      "
       :lazy-src="`/assets/romm/resources/${rom.path_cover_s}`"
       :aspect-ratio="3 / 4"
     >
diff --git a/frontend/src/components/Game/Card/Cover.vue b/frontend/src/components/Game/Card/Cover.vue
index 34c4a4240..dcd4b87fe 100644
--- a/frontend/src/components/Game/Card/Cover.vue
+++ b/frontend/src/components/Game/Card/Cover.vue
@@ -73,12 +73,15 @@ function onTouchEnd() {
     />
     <v-hover v-slot="{ isHovering, props }" open-delay="800">
       <v-img
+        :cover="!rom.has_cover"
         :value="rom.id"
         :key="rom.id"
         v-bind="props"
-        :src="`/assets/romm/resources/${
-          rom.path_cover_l || rom.merged_screenshots[0]
-        }`"
+        :src="
+          !rom.has_cover && rom.merged_screenshots.length > 0
+            ? rom.merged_screenshots[0]
+            : `/assets/romm/resources/${rom.path_cover_l}`
+        "
         :lazy-src="`/assets/romm/resources/${rom.path_cover_s}`"
         :aspect-ratio="3 / 4"
       >
diff --git a/frontend/vite.config.js b/frontend/vite.config.js
index ea1ce367a..2ed5ba767 100644
--- a/frontend/vite.config.js
+++ b/frontend/vite.config.js
@@ -30,6 +30,7 @@ export default defineConfig(({ mode }) => {
         },
       }),
       VitePWA({
+        injectRegister: null,
         manifest: {
           icons: [
             {
@@ -40,15 +41,8 @@ export default defineConfig(({ mode }) => {
             },
           ],
         },
-        workbox: {
-          globPatterns: ["**/*.{js,css,html,ico,png,svg}"],
-          navigateFallbackDenylist: [
-            /\/assets\/romm\/library/,
-            /\/api\/platforms\/.*\/roms\/.*\/download/,
-          ],
-        },
         devOptions: {
-          enabled: false,
+          enabled: true,
           type: "module",
         },
       }),