Merge pull request #180 from SensCritique/release/1.5.2

Release 1.5.2 : Correction de la loop infini (le retour) sur PrimeVideo

manifest_chrome.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 3,
   "name": "SensCritique",
-  "version": "1.5.1",
+  "version": "1.5.2",
   "description": "Vous avez du mal à trouver des bons programmes sur vos plateformes de streaming ?",
   "icons": {
     "48": "images/logo-48.png",

manifest_firefox.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 2,
   "name": "SensCritique",
-  "version": "1.5.1",
+  "version": "1.5.2",
   "description": "La note SensCritique est une boussole pour arrêter de perdre votre temps, et s’invite sur Netflix, Prime Video et Disney+ pour vous permettre de choisir de manière éclairée vos programmes.",
   "icons": {
     "48": "images/logo-48.png",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "senscritique-extension",
-  "version": "1.5.1",
+  "version": "1.5.2",
   "description": "La note SensCritique est une boussole pour arrêter de perdre votre temps, et s’invite sur Netflix, Prime Video et Disney+ pour vous permettre de choisir de manière éclairée vos programmes.",
   "author": "SensCritique",
   "repository": {

src/dom/providers/Netflix.ts

@@ -60,14 +60,26 @@ export default class Netflix extends Manager {
     browserExtensionProducts: BrowserExtensionProduct[]
   ): Promise<void> {
     // Response from API with all browserExtensionProducts
-    browserExtensionProducts.forEach((browserExtensionProduct) => {
+    for (const browserExtensionProduct of browserExtensionProducts) {
       const hash = md5(browserExtensionProduct.platformId.toString())
+      const videoInfo = {
+        name: name,
+        redirect: await generateRedirectUrl(name),
+        id: '',
+        url: browserExtensionProduct.url,
+        type: browserExtensionProduct.type,
+        rating: browserExtensionProduct?.rating?.toString(),
+        hash,
+        platformId: browserExtensionProduct?.platformId,
+      }
+
+      this.cache.save(videoInfo)
       const platformId = browserExtensionProduct.platformId
       const cardElements = document.querySelectorAll(
         `.title-card a[href*="/watch/${platformId}"]`
       )
 
-      cardElements.forEach(async (cardElement) => {
+      for (const cardElement of cardElements) {
         const videoName = cardElement?.getAttribute('aria-label')
         const hashClass = 'senscritique_' + hash
         if (!cardElement.querySelector(`.${hashClass}`)) {
@@ -91,8 +103,8 @@ export default class Netflix extends Manager {
             platformId: browserExtensionProduct?.platformId,
           })
         }
-      })
-    })
+      }
+    }
   }
 
   refreshModalRating(): void {

src/dom/providers/PrimeVideo.ts

@@ -79,8 +79,22 @@ export default class PrimeVideo extends Manager {
 
   renderWallRatings(browserExtensionProducts: BrowserExtensionProduct[]): void {
     // Response from API with all browserExtensionProducts
-    browserExtensionProducts.forEach((browserExtensionProduct) => {
+    browserExtensionProducts.forEach(async (browserExtensionProduct) => {
+      // Note : Save in cache product information before rendering to prevent UI bugs
       const hash = md5(browserExtensionProduct.platformId.toString())
+      const videoInfo = {
+        name: name,
+        redirect: await generateRedirectUrl(name),
+        id: '',
+        url: browserExtensionProduct.url,
+        type: browserExtensionProduct.type,
+        rating: browserExtensionProduct?.rating?.toString(),
+        hash,
+        platformId: browserExtensionProduct?.platformId,
+      }
+
+      this.cache.save(videoInfo)
+
       const platformId = browserExtensionProduct.platformId
       const wallElements = document.querySelectorAll(
         `[data-testid="packshot"] a[href*="/detail/${platformId}"][role="button"]:not([class*="tst-"])`
@@ -105,7 +119,7 @@ export default class PrimeVideo extends Manager {
         ...legacyElements,
       ]
 
-      cardElements.forEach(async (cardElement: HTMLElement) => {
+      for (const cardElement of cardElements) {
         // Only keep carousel and wall elements (without episode URL)
         if (
           cardElement.getAttribute('href')?.match(`/detail/${platformId}.*`) ||
@@ -121,7 +135,7 @@ export default class PrimeVideo extends Manager {
         ) {
           const hashClass = 'senscritique_' + hash
           if (!cardElement.querySelector(`.${hashClass}`)) {
-            let name = cardElement.innerText
+            let name = (cardElement as HTMLElement).innerText
             const mainDiv = document.createElement('div')
             mainDiv.style.position = 'absolute'
             mainDiv.style.zIndex = '2'
@@ -148,7 +162,7 @@ export default class PrimeVideo extends Manager {
             })
           }
         }
-      })
+      }
     })
   }