nodebb-plugin-ezoic-infinite 1.7.6 → 1.7.7

package/package.json

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

package/public/client.js

@@ -1,5 +1,5 @@
 /**
- * NodeBB Ezoic Infinite Ads — client.js v21.2
+ * NodeBB Ezoic Infinite Ads — client.js v21.3
  *
  * Historique des corrections majeures
  * ────────────────────────────────────
@@ -252,10 +252,8 @@
 
   function dropWrap(w) {
     try {
-      // Unobserve seulement si le placeholder est encore dans le DOM.
-      // unobserve() sur un nœud détaché corrompt l'IO interne de pubads
-      // → "Failed to execute 'observe': parameter 1 is not of type 'Element'"
-      // sur tous les observe() suivants.
+      // Unobserve uniquement si le ph est encore connecté — un nœud détaché
+      // passé à unobserve() corrompt l'IO interne de pubads.
       const ph = w.querySelector(`[id^="${PH_PREFIX}"]`);
       if (ph instanceof Element && ph.isConnected) S.io?.unobserve(ph);
       const id = parseInt(w.getAttribute(A_WRAPID), 10);
@@ -570,14 +568,29 @@
     blockedUntil = ts() + 1500;
     poolsReady   = false;
 
-    // Déconnecter l'IO AVANT les dropWrap pour éviter tout unobserve parasite.
-    // disconnect() vide la liste interne des cibles observées — les références
-    // aux placeholders de la page courante sont effacées proprement.
-    // L'IO sera recréé à ajaxify.end via getIO().
-    try { S.io?.disconnect(); } catch (_) {}
-    S.io = null;
+    // Unobserve ciblé sur NOS placeholders avant de les retirer du DOM.
+    // On ne fait JAMAIS disconnect() — c'est un IO global partagé avec pubads ;
+    // disconnect() corrompt son état interne et provoque des erreurs sur tous
+    // les observe() suivants de Google ("parameter 1 is not of type 'Element'").
+    if (S.io) {
+      for (const id of S.mountedIds) {
+        try {
+          const ph = document.getElementById(`${PH_PREFIX}${id}`);
+          if (ph instanceof Element && ph.isConnected) S.io.unobserve(ph);
+        } catch (_) {}
+      }
+    }
+
+    mutate(() => document.querySelectorAll(`.${WRAP_CLASS}`).forEach(w => {
+      try {
+        const id = parseInt(w.getAttribute(A_WRAPID), 10);
+        if (Number.isFinite(id)) S.mountedIds.delete(id);
+        const key = w.getAttribute(A_ANCHOR);
+        if (key) wrapByKey.delete(key);
+        w.remove();
+      } catch (_) {}
+    }));
 
-    mutate(() => document.querySelectorAll(`.${WRAP_CLASS}`).forEach(dropWrap));
     wrapByKey.clear();
     S.cfg          = null;
     S.pools        = { topics: [], posts: [], categories: [] };
@@ -597,6 +610,8 @@
     if (S.domObs) return;
     const allSel = [SEL.post, SEL.topic, SEL.category];
     S.domObs = new MutationObserver(muts => {
+      // Ne rien faire pendant la navigation (cleanup posé blockedUntil)
+      // ou si c'est nous qui mutons le DOM.
       if (S.mutGuard > 0 || isBlocked()) return;
       for (const m of muts) {
         for (const n of m.addedNodes) {