nodebb-plugin-ezoic-infinite 1.7.1 → 1.7.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.1",
+  "version": "1.7.3",
   "description": "Production-ready Ezoic infinite ads integration for NodeBB 4.x",
   "main": "library.js",
   "license": "MIT",

package/public/client.js

@@ -261,11 +261,45 @@
     return w;
   }
 
+  /**
+   * Retire proprement un wrap du DOM.
+   *
+   * Sans précaution, retirer un wrap contenant un player vidéo Ezoic (wyvern.js)
+   * déclenche des erreurs async sur des nœuds détachés :
+   *   "Cannot read properties of null (reading 'paused')"
+   *   "Cannot read properties of null (reading 'offsetWidth')"
+   *   "Invalid target for null#trigger / null#on"
+   *
+   * On pause les media et on tente de notifier l'API wyvern avant remove().
+   */
   function dropWrap(w) {
     try {
+      // 1. Pause tous les media actifs avant détachement
+      try {
+        w.querySelectorAll('video, audio').forEach(m => {
+          try { if (!m.paused) m.pause(); } catch (_) {}
+        });
+      } catch (_) {}
+
+      // 2. Notifier l'API wyvern si disponible
+      try {
+        if (window.wyvern && typeof window.wyvern === 'object') {
+          w.querySelectorAll('[id^="ezoic-"],[class*="wyvern"],[class*="ezoic-video"]')
+            .forEach(n => { try { window.wyvern.destroy?.(n); } catch (_) {} });
+        }
+      } catch (_) {}
+
+      // 3. Unobserve (guard instanceof Element — unobserve(null) corrompt l'IO)
+      try {
+        const ph = w.querySelector(`[id^="${PH_PREFIX}"]`);
+        if (ph instanceof Element) S.io?.unobserve(ph);
+      } catch (_) {}
+
+      // 4. Libérer l'id du pool
       const id = parseInt(w.getAttribute(A_WRAPID), 10);
       if (Number.isFinite(id)) S.mountedIds.delete(id);
-      try { S.io?.unobserve(w.querySelector(`[id^="${PH_PREFIX}"]`)); } catch (_) {}
+
+      // 5. Retrait DOM
       w.remove();
     } catch (_) {}
   }
@@ -283,17 +317,31 @@
    *
    * On ne prune pas les wraps < MIN_PRUNE_AGE_MS (DOM pas encore stabilisé).
    */
+  /**
+   * Supprime les wraps dont l'élément-ancre n'est plus dans le DOM.
+   *
+   * Règle stricte : on ne supprime JAMAIS un wrap rempli (filled).
+   *   - Il peut contenir un player wyvern actif → .remove() déclenche des
+   *     erreurs async ("Cannot read 'paused'", "offsetWidth", "getChild"…).
+   *   - Le post-ancre peut être temporairement virtualisé par NodeBB puis
+   *     revenir — dans ce cas le wrap filled doit rester en place.
+   *
+   * Seuls les wraps VIDES dont l'ancre a disparu sont supprimés.
+   */
   function pruneOrphans(klass) {
     const meta = KIND[klass];
     if (!meta) return;
 
-    const baseTag = meta.sel.split('[')[0]; // ex: "li", "[component=..." → "" (géré)
+    const baseTag = meta.sel.split('[')[0];
 
     document.querySelectorAll(`.${WRAP_CLASS}.${klass}`).forEach(w => {
       if (ts() - parseInt(w.getAttribute(A_CREATED) || '0', 10) < MIN_PRUNE_AGE_MS) return;
 
+      // Ne jamais retirer un wrap qui contient du contenu (player potentiellement actif)
+      if (isFilled(w)) return;
+
       const key = w.getAttribute(A_ANCHOR) ?? '';
-      const sid = key.slice(klass.length + 1); // extrait la partie après "kindClass:"
+      const sid = key.slice(klass.length + 1);
       if (!sid) { mutate(() => dropWrap(w)); return; }
 
       const anchorEl = document.querySelector(
@@ -308,11 +356,12 @@
   /**
    * Deux wraps adjacents = situation anormale → supprimer le moins prioritaire.
    * Priorité : filled > en grâce (fill en cours) > vide.
-   * Ne supprime jamais un wrap dont showAds() date de moins de FILL_GRACE_MS.
+   *
+   * Règle de sécurité wyvern : on ne supprime JAMAIS un wrap rempli.
+   * Si les deux wraps adjacents sont remplis, on laisse en place (edge case rare).
    */
   function decluster(klass) {
     for (const w of document.querySelectorAll(`.${WRAP_CLASS}.${klass}`)) {
-      // Grace sur le wrap courant : on le saute entièrement
       const wShown = parseInt(w.getAttribute(A_SHOWN) || '0', 10);
       if (wShown && ts() - wShown < FILL_GRACE_MS) continue;
 
@@ -321,10 +370,16 @@
         if (!prev.classList?.contains(WRAP_CLASS)) { prev = prev.previousElementSibling; continue; }
 
         const pShown = parseInt(prev.getAttribute(A_SHOWN) || '0', 10);
-        if (pShown && ts() - pShown < FILL_GRACE_MS) break; // précédent en grâce → rien
+        if (pShown && ts() - pShown < FILL_GRACE_MS) break;
+
+        const wFilled = isFilled(w);
+        const pFilled = isFilled(prev);
 
-        if      (!isFilled(w))    mutate(() => dropWrap(w));
-        else if (!isFilled(prev)) mutate(() => dropWrap(prev));
+        // Ne jamais retirer un wrap rempli (player actif potentiel)
+        if      (!wFilled && !pFilled) mutate(() => dropWrap(w));    // les deux vides → retirer le courant
+        else if (!wFilled &&  pFilled) mutate(() => dropWrap(w));    // courant vide, précédent rempli → retirer le courant
+        else if ( wFilled && !pFilled) mutate(() => dropWrap(prev)); // courant rempli, précédent vide → retirer le précédent
+        // les deux remplis → rien (on ne touche pas)
         break;
       }
     }
@@ -389,7 +444,7 @@
       S.io = new IntersectionObserver(entries => {
         for (const e of entries) {
           if (!e.isIntersecting) continue;
-          S.io?.unobserve(e.target);
+          if (e.target instanceof Element) S.io?.unobserve(e.target);
           const id = parseInt(e.target.getAttribute('data-ezoic-id'), 10);
           if (Number.isFinite(id) && id > 0) enqueueShow(id);
         }
@@ -586,6 +641,16 @@
 
   function cleanup() {
     blockedUntil = ts() + 1500;
+
+    // Pause tous les media dans nos wraps AVANT de les retirer du DOM.
+    // Empêche wyvern.js de continuer d'exécuter ses callbacks async sur des
+    // nœuds détachés (erreurs "Cannot read 'paused'", "offsetWidth"…).
+    try {
+      document.querySelectorAll(`.${WRAP_CLASS} video, .${WRAP_CLASS} audio`).forEach(m => {
+        try { if (!m.paused) m.pause(); } catch (_) {}
+      });
+    } catch (_) {}
+
     mutate(() => document.querySelectorAll(`.${WRAP_CLASS}`).forEach(dropWrap));
     S.cfg          = null;
     S.pools        = { topics: [], posts: [], categories: [] };
@@ -637,12 +702,30 @@
   }
 
   function ensureTcfLocator() {
+    // Le CMP utilise une iframe nommée __tcfapiLocator pour router les
+    // postMessage TCF. En navigation ajaxify, NodeBB peut retirer cette
+    // iframe du DOM (vidage partiel du body), ce qui provoque :
+    //   "Cannot read properties of null (reading 'postMessage')"
+    //   "Cannot set properties of null (setting 'addtlConsent')"
+    // Solution : la recrée immédiatement si elle disparaît, via un observer.
     try {
       if (!window.__tcfapi && !window.__cmp) return;
-      if (document.getElementById('__tcfapiLocator')) return;
-      const f = document.createElement('iframe');
-      f.style.display = 'none'; f.id = f.name = '__tcfapiLocator';
-      (document.body || document.documentElement).appendChild(f);
+
+      const inject = () => {
+        if (document.getElementById('__tcfapiLocator')) return;
+        const f = document.createElement('iframe');
+        f.style.display = 'none'; f.id = f.name = '__tcfapiLocator';
+        (document.body || document.documentElement).appendChild(f);
+      };
+
+      inject();
+
+      // Observer dédié — si quelqu'un retire l'iframe, on la remet.
+      if (!window.__nbbTcfObs) {
+        window.__nbbTcfObs = new MutationObserver(() => inject());
+        window.__nbbTcfObs.observe(document.documentElement,
+          { childList: true, subtree: true });
+      }
     } catch (_) {}
   }