nodebb-plugin-ezoic-infinite 1.7.32 → 1.7.34

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.32",
+  "version": "1.7.34",
   "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 v37
+ * NodeBB Ezoic Infinite Ads — client.js v36
  *
  * Historique des corrections majeures
  * ────────────────────────────────────
@@ -30,19 +30,21 @@
  *        – Toujours désactivé pour les posts : NodeBB virtualise les posts
  *          hors-viewport → faux-orphelins → bug réinjection en haut.
  *
- * v34  moveDistantWrap : déplace le nœud wrap existant au lieu de le supprimer
- *      et recréer. Ezoic garde une registry interne des placeholders — recréer
- *      un div avec le même id déclenche "already been defined" et bloque
- *      showAds(). En déplaçant le même nœud DOM, la registry reste valide.
+ * v34  moveDistantWrap — voir v38.
  *
- * v37  Fix wraps vides persistants : ez.refresh() sur les placeholders
- *      en DOM, connectés, non remplis et visibles (dans les marges IO).
- *      Appelé dans runCore() après injection, une fois par burst max.
+ * v40  Recyclage slots via destroyPlaceholders+define+displayMore avec délais.
+ *      Séquence : destroy → 300ms → define → 300ms → displayMore.
+ *      Testé manuellement : fonctionne. displayMore = API Ezoic infinite scroll.
+ *
+ * v38  Pool épuisé = fin de quota Ezoic par page-view. ez.refresh() interdit
+ *      sur la même page que ez.enable() — supprimé. moveDistantWrap supprimé :
+ *      déplacer un wrap "already defined" ne re-sert aucune pub. Pool épuisé →
+ *      break propre dans injectBetween. muteConsole : ajout warnings refresh.
  *
  * v36  Optimisations chemin critique (scroll → injectBetween) :
  *      – S.wrapByKey Map<anchorKey,wrap> : findWrap() passe de querySelector
  *        sur tout le doc à un lookup O(1). Mis à jour dans insertAfter,
- *        moveDistantWrap, dropWrap et cleanup.
+ *        dropWrap et cleanup.
  *      – wrapIsLive allégé : pour les voisins immédiats on vérifie les
  *        attributs du nœud lui-même sans querySelector global.
  *      – MutationObserver : matches() vérifié avant querySelector() pour
@@ -51,9 +53,6 @@
  * v35  Revue complète prod-ready :
  *      – initPools protégé contre ré-initialisation inutile (S.poolsReady).
  *      – muteConsole élargit à "No valid placeholders for loadMore".
- *      – moveDistantWrap appelle ezstandalone.refresh([id]) après déplacement
- *        pour forcer Ezoic à réactiver le placeholder sur son nouveau nœud.
- *      – A_SHOWN initialisé à '0' dans moveDistantWrap (évite NaN dans parseInt).
  *      – Commentaires et historique nettoyés.
  */
 (function nbbEzoicInfinite() {
@@ -216,7 +215,7 @@
     // Lookup O(1) dans le registre — vrai si le wrap EST encore dans le registre
     // et connecté au DOM (le registre est tenu à jour par insertAfter/dropWrap).
     if (S.wrapByKey.get(key) === wrap) return wrap.isConnected;
-    // Fallback : wrap déplacé (moveDistantWrap) ou registre pas encore à jour.
+    // Fallback : registre pas encore à jour ou wrap non enregistré.
     const colonIdx = key.indexOf(':');
     const klass    = key.slice(0, colonIdx);
     const anchorId = key.slice(colonIdx + 1);
@@ -292,19 +291,20 @@
   }
 
   /**
-   * Pool épuisé : déplace physiquement le wrap le plus loin au-dessus du
-   * viewport vers targetEl. On NE supprime PAS le placeholder — Ezoic garde
-   * une registry interne, et recréer un div avec le même id déclenche
-   * "already been defined" puis bloque showAds(). En déplaçant le même nœud,
-   * la registry reste valide. On appelle ensuite ezstandalone.refresh([id])
-   * pour forcer Ezoic à réactiver la pub sur sa nouvelle position.
-   *
-   * Priorité : wraps non remplis (pas de pub visible perdue), puis remplis.
+   * Pool épuisé : recycle un wrap loin au-dessus du viewport.
+   * Séquence avec délais (destroyPlaceholders est asynchrone) :
+   *   destroy([id]) → 300ms → define([id]) → 300ms → displayMore([id])
+   * displayMore = API Ezoic prévue pour l'infinite scroll.
+   * Priorité : wraps vides d'abord, remplis si nécessaire.
    */
-  function moveDistantWrap(klass, targetEl, newKey) {
-    const vh        = window.innerHeight || 800;
-    const threshold = -vh * 4;  // 4 viewports au-dessus = hors vue assurée
+  function recycleAndMove(klass, targetEl, newKey) {
+    const ez = window.ezstandalone;
+    if (typeof ez?.destroyPlaceholders !== 'function' ||
+        typeof ez?.define             !== 'function' ||
+        typeof ez?.displayMore        !== 'function') return null;
 
+    const vh = window.innerHeight || 800;
+    const threshold = -vh * 4;
     let bestEmpty = null, bestEmptyBottom = Infinity;
     let bestFilled = null, bestFilledBottom = Infinity;
 
@@ -322,7 +322,6 @@
 
     const best = bestEmpty ?? bestFilled;
     if (!best) return null;
-
     const id = parseInt(best.getAttribute(A_WRAPID), 10);
     if (!Number.isFinite(id)) return null;
 
@@ -332,21 +331,18 @@
       best.setAttribute(A_CREATED, String(ts()));
       best.setAttribute(A_SHOWN,   '0');
       best.classList.remove('is-empty');
+      const ph = best.querySelector(`#${PH_PREFIX}${id}`);
+      if (ph) ph.innerHTML = '';
       targetEl.insertAdjacentElement('afterend', best);
     });
-    // Mettre à jour le registre : ancienne clé → nouvelle clé
     if (oldKey && S.wrapByKey.get(oldKey) === best) S.wrapByKey.delete(oldKey);
     S.wrapByKey.set(newKey, best);
 
-    // Signaler à Ezoic que ce placeholder a bougé dans le DOM
-    try {
-      const ez = window.ezstandalone;
-      if (typeof ez?.refresh === 'function') {
-        (Array.isArray(ez.cmd) ? p => ez.cmd.push(p) : p => p())(() => {
-          try { ez.refresh([id]); } catch (_) {}
-        });
-      }
-    } catch (_) {}
+    // Délais requis : destroyPlaceholders est asynchrone en interne
+    const doDestroy = () => { try { ez.destroyPlaceholders([id]); } catch (_) {} setTimeout(doDefine,  300); };
+    const doDefine  = () => { try { ez.define([id]);              } catch (_) {} setTimeout(doDisplay, 300); };
+    const doDisplay = () => { try { ez.displayMore([id]);         } catch (_) {} };
+    try { (typeof ez.cmd?.push === 'function') ? ez.cmd.push(doDestroy) : doDestroy(); } catch (_) {}
 
     return { id, wrap: best };
   }
@@ -462,14 +458,11 @@
 
       const id = pickId(poolKey);
       if (id) {
-        // Pool disponible : créer un nouveau wrap
         const w = insertAfter(el, id, klass, key);
         if (w) { observePh(id); inserted++; }
       } else {
-        // Pool épuisé : déplacer un wrap distant (préserve le nœud placeholder)
-        const moved = moveDistantWrap(klass, el, key);
-        if (!moved) continue;
-        observePh(moved.id);
+        const recycled = recycleAndMove(klass, el, key);
+        if (!recycled) break;
         inserted++;
       }
     }
@@ -619,70 +612,23 @@
       return injectBetween(klass, getItems(), interval, normBool(cfgShowFirst), poolKey);
     };
 
-    if (kind === 'topic') {
-      const n = exec(
-        'ezoic-ad-message', getPosts,
-        cfg.enableMessageAds, cfg.messageIntervalPosts, cfg.showFirstMessageAd, 'posts'
-      );
-      refreshStaleWraps();
-      return n;
-    }
+    if (kind === 'topic') return exec(
+      'ezoic-ad-message', getPosts,
+      cfg.enableMessageAds, cfg.messageIntervalPosts, cfg.showFirstMessageAd, 'posts'
+    );
 
     if (kind === 'categoryTopics') {
       pruneOrphansBetween();
-      const n = exec(
+      return exec(
         'ezoic-ad-between', getTopics,
         cfg.enableBetweenAds, cfg.intervalPosts, cfg.showFirstTopicAd, 'topics'
       );
-      refreshStaleWraps();
-      return n;
     }
 
-    const n2 = exec(
+    return exec(
       'ezoic-ad-categories', getCategories,
       cfg.enableCategoryAds, cfg.intervalCategories, cfg.showFirstCategoryAd, 'categories'
     );
-    refreshStaleWraps();
-    return n2;
-  }
-
-  /**
-   * Appelle ez.refresh() sur les placeholders en DOM, non remplis,
-   * dans la zone visible (marges IO). Cela force Ezoic à re-servir
-   * une pub sur des slots qu'il avait refusé lors du showAds() initial.
-   * Limité à MAX_REFRESH_PER_RUN appels par invocation.
-   */
-  const MAX_REFRESH_PER_RUN = 3;
-
-  function refreshStaleWraps() {
-    const ez = window.ezstandalone;
-    if (typeof ez?.refresh !== 'function') return;
-
-    const vh  = window.innerHeight || 800;
-    const margin = (isMobile() ? 3500 : 2500);
-    const top  = -margin;
-    const bot  = vh + margin;
-
-    const ids = [];
-    for (const [, wrap] of S.wrapByKey) {
-      if (ids.length >= MAX_REFRESH_PER_RUN) break;
-      if (!wrap.isConnected || isFilled(wrap)) continue;
-      // Vérifier que le wrap a déjà eu un showAds (A_SHOWN > 0)
-      const shown = parseInt(wrap.getAttribute(A_SHOWN) || '0', 10);
-      if (!shown) continue;
-      try {
-        const rect = wrap.getBoundingClientRect();
-        if (rect.top > bot || rect.bottom < top) continue;
-        const id = parseInt(wrap.getAttribute(A_WRAPID), 10);
-        if (Number.isFinite(id) && id > 0) ids.push(id);
-      } catch (_) {}
-    }
-
-    if (!ids.length) return;
-    try {
-      const doRefresh = () => { try { ez.refresh(ids); } catch (_) {} };
-      Array.isArray(ez.cmd) ? ez.cmd.push(doRefresh) : doRefresh();
-    } catch (_) {}
   }
 
   // ── Scheduler ──────────────────────────────────────────────────────────────
@@ -772,6 +718,8 @@
     const MUTED = [
       '[EzoicAds JS]: Placeholder Id',
       'No valid placeholders for loadMore',
+      'cannot call refresh on the same page',
+      'no placeholders are currently defined in Refresh',
       'Debugger iframe already exists',
       `with id ${PH_PREFIX}`,
     ];