nodebb-plugin-ezoic-infinite 1.7.32 → 1.7.33

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.32",
+  "version": "1.7.33",
   "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,25 @@
  *        – 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.
+ * v39  Recyclage réel des slots via l'API Ezoic complète :
+ *      destroyPlaceholders([id]) → déplacement DOM → define([id]) → displayMore([id]).
+ *      destroyPlaceholders() efface le slot de la registry Ezoic, define() le
+ *      réenregistre comme neuf, displayMore() est l'API infinite-scroll d'Ezoic
+ *      (contrairement à showAds() qui est pour le chargement initial).
+ *      Priorité de recyclage : wraps vides en premier (pas de pub visible perdue),
+ *      puis wraps remplis si nécessaire. Seuil : 4 viewports au-dessus.
+ *
+ * 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 +57,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 +219,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);
@@ -291,67 +294,6 @@
     return null;
   }
 
-  /**
-   * 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.
-   */
-  function moveDistantWrap(klass, targetEl, newKey) {
-    const vh        = window.innerHeight || 800;
-    const threshold = -vh * 4;  // 4 viewports au-dessus = hors vue assurée
-
-    let bestEmpty = null, bestEmptyBottom = Infinity;
-    let bestFilled = null, bestFilledBottom = Infinity;
-
-    document.querySelectorAll(`.${WRAP_CLASS}.${klass}`).forEach(wrap => {
-      try {
-        const rect = wrap.getBoundingClientRect();
-        if (rect.bottom > threshold) return;
-        if (!isFilled(wrap)) {
-          if (rect.bottom < bestEmptyBottom) { bestEmptyBottom = rect.bottom; bestEmpty = wrap; }
-        } else {
-          if (rect.bottom < bestFilledBottom) { bestFilledBottom = rect.bottom; bestFilled = wrap; }
-        }
-      } catch (_) {}
-    });
-
-    const best = bestEmpty ?? bestFilled;
-    if (!best) return null;
-
-    const id = parseInt(best.getAttribute(A_WRAPID), 10);
-    if (!Number.isFinite(id)) return null;
-
-    const oldKey = best.getAttribute(A_ANCHOR);
-    mutate(() => {
-      best.setAttribute(A_ANCHOR,  newKey);
-      best.setAttribute(A_CREATED, String(ts()));
-      best.setAttribute(A_SHOWN,   '0');
-      best.classList.remove('is-empty');
-      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 (_) {}
-
-    return { id, wrap: best };
-  }
-
-  // ── Wraps DOM — création / suppression ────────────────────────────────────
 
   function makeWrap(id, klass, key) {
     const w = document.createElement('div');
@@ -462,14 +404,12 @@
 
       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);
+        // Pool épuisé : recycler un wrap distant via destroyPlaceholders+define+displayMore
+        const recycled = recycleAndMove(klass, el, key);
+        if (!recycled) break; // Aucun wrap recyclable → arrêt
         inserted++;
       }
     }
@@ -619,70 +559,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 +665,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}`,
     ];