nodebb-plugin-ezoic-infinite 1.7.30 → 1.7.33

package/package.json

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

package/public/client.js

@@ -30,15 +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.
+ *
+ * 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
@@ -47,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() {
@@ -212,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);
@@ -287,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');
@@ -458,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++;
       }
     }
@@ -721,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}`,
     ];