nodebb-plugin-ezoic-infinite 1.7.8 → 1.7.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.8",
+  "version": "1.7.9",
   "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.4
+ * NodeBB Ezoic Infinite Ads — client.js v21
  *
  * Historique des corrections majeures
  * ────────────────────────────────────
@@ -96,7 +96,6 @@
   };
 
   let blockedUntil = 0;
-  let poolsReady   = false;  // true dès que les pools sont initialisés pour la page courante
   const ts        = () => Date.now();
   const isBlocked = () => ts() < blockedUntil;
   const isMobile  = () => { try { return window.innerWidth < 768; } catch (_) { return false; } };
@@ -129,11 +128,9 @@
   }
 
   function initPools(cfg) {
-    if (poolsReady) return;
     S.pools.topics     = parseIds(cfg.placeholderIds);
     S.pools.posts      = parseIds(cfg.messagePlaceholderIds);
     S.pools.categories = parseIds(cfg.categoryPlaceholderIds);
-    poolsReady = true;
   }
 
   // ── Page identity ──────────────────────────────────────────────────────────
@@ -182,9 +179,6 @@
 
   // ── Ancres stables ─────────────────────────────────────────────────────────
 
-  // Map anchorKey → wrap Element — évite un querySelector full-DOM à chaque injection
-  const wrapByKey = new Map();
-
   function stableId(klass, el) {
     const attr = KIND[klass]?.anchorAttr;
     if (attr) {
@@ -201,13 +195,13 @@
 
   const anchorKey = (klass, el) => `${klass}:${stableId(klass, el)}`;
 
-  const findWrap  = (key) => {
-    const w = wrapByKey.get(key);
-    // Vérifier que le wrap est toujours dans le DOM (il peut avoir été dropWrap'd)
-    if (w && w.isConnected) return w;
-    if (w) wrapByKey.delete(key); // nettoyage lazy
-    return null;
-  };
+  function findWrap(key) {
+    try {
+      return document.querySelector(
+        `.${WRAP_CLASS}[${A_ANCHOR}="${key.replace(/\\/g, '\\\\').replace(/"/g, '\\"')}"]`
+      );
+    } catch (_) { return null; }
+  }
 
   // ── Pool ───────────────────────────────────────────────────────────────────
 
@@ -246,21 +240,17 @@
     const w = makeWrap(id, klass, key);
     mutate(() => el.insertAdjacentElement('afterend', w));
     S.mountedIds.add(id);
-    wrapByKey.set(key, w);
     return w;
   }
 
   function dropWrap(w) {
     try {
-      // Unobserve AVANT w.remove() — le placeholder est encore dans le DOM
-      // à ce stade, ce qui est requis par l'IO. Guard instanceof uniquement
-      // (même logique que v20.3 qui fonctionnait).
+      // Unobserve avant remove — guard instanceof évite unobserve(null)
+      // qui corrompt l'état interne de l'IO (pubads error au scroll suivant)
       const ph = w.querySelector(`[id^="${PH_PREFIX}"]`);
       if (ph instanceof Element) S.io?.unobserve(ph);
       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 (_) {}
   }
@@ -493,6 +483,7 @@
 
   async function runCore() {
     if (isBlocked()) return 0;
+    patchShowAds();
 
     const cfg = await fetchConfig();
     if (!cfg || cfg.excluded) return 0;
@@ -567,9 +558,7 @@
 
   function cleanup() {
     blockedUntil = ts() + 1500;
-    poolsReady   = false;
     mutate(() => document.querySelectorAll(`.${WRAP_CLASS}`).forEach(dropWrap));
-    wrapByKey.clear();
     S.cfg          = null;
     S.pools        = { topics: [], posts: [], categories: [] };
     S.cursors      = { topics: 0, posts: 0, categories: 0 };
@@ -588,8 +577,6 @@
     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) {