nodebb-plugin-ezoic-infinite 1.7.25 → 1.7.27

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.25",
+  "version": "1.7.27",
   "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 v31
+ * NodeBB Ezoic Infinite Ads — client.js v33
  *
  * Historique des corrections majeures
  * ────────────────────────────────────
@@ -31,6 +31,11 @@
  *
  * v29  Fix ancrage topics : data-index → data-tid.
  *
+ * v32  Retour anchorAttr = data-index pour ezoic-ad-between (comme v20.1).
+ *      data-tid peut être absent sur li[component="category/topic"] → clés
+ *      invalides → wraps empilés sur pos0. data-index est toujours présent.
+ *      pruneOrphansBetween cherche li[data-index="N"] — cohérent avec v20.1.
+ *
  * v31  pruneOrphans réactivé UNIQUEMENT pour ezoic-ad-between (topics de catégorie).
  *      NodeBB NE virtualise PAS les topics dans une liste de catégorie — les ancres
  *      restent dans le DOM entre les scrolls. pruneOrphans est donc safe ici et
@@ -98,7 +103,7 @@
    */
   const KIND = {
     'ezoic-ad-message':    { sel: SEL.post,     baseTag: '',   anchorAttr: 'data-pid',   ordinalAttr: 'data-index' },
-    'ezoic-ad-between':    { sel: SEL.topic,    baseTag: 'li', anchorAttr: 'data-tid',   ordinalAttr: 'data-index' },
+    'ezoic-ad-between':    { sel: SEL.topic,    baseTag: 'li', anchorAttr: 'data-index', ordinalAttr: 'data-index' },
     'ezoic-ad-categories': { sel: SEL.category, baseTag: 'li', anchorAttr: 'data-cid',   ordinalAttr: null         },
   };
 
@@ -257,31 +262,30 @@
     return null;
   }
 
-  function recycleOrphanId(klass) {
-    // Quand le pool est épuisé : cherche un wrap orphelin (ancre hors DOM, non rempli)
-    // loin au-dessus du viewport et libère son ID.
+  function recycleDistantWrap(klass) {
+    // Quand le pool est épuisé : récupère le wrap le plus loin au-dessus du
+    // viewport (qu'il soit orphelin ou non) pour libérer son ID.
+    // Les topics/pubs à 4+ viewports au-dessus ne sont plus visibles — on peut
+    // recycler leur slot. Non-remplis en priorité, remplis si rien d'autre.
     const vh = window.innerHeight || 800;
-    const threshold = -vh * 3;
-    let best = null, bestBottom = Infinity;
+    const threshold = -vh * 4;
+
+    let bestEmpty = null, bestEmptyBottom = Infinity;
+    let bestFilled = null, bestFilledBottom = Infinity;
+
     document.querySelectorAll(`.${WRAP_CLASS}.${klass}`).forEach(wrap => {
-      if (wrap.getAttribute(A_CREATED) === null) return;
-      if (isFilled(wrap)) return;
-      const key = wrap.getAttribute(A_ANCHOR);
-      if (!key) return;
-      const colonIdx = key.indexOf(':');
-      const anchorId = key.slice(colonIdx + 1);
-      const cfg = KIND[klass];
-      if (!cfg) return;
-      try {
-        const found = document.querySelector(`${cfg.sel}[${cfg.anchorAttr}="${anchorId}"]`);
-        if (found?.isConnected) return; // ancre encore dans le DOM, pas orphelin
-      } catch (_) { return; }
       try {
         const rect = wrap.getBoundingClientRect();
         if (rect.bottom > threshold) return;
-        if (rect.bottom < bestBottom) { bestBottom = rect.bottom; best = wrap; }
+        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;
@@ -405,7 +409,7 @@
       if (findWrap(key)) continue;
 
       let id = pickId(poolKey);
-      if (!id) { id = recycleOrphanId(klass); if (!id) continue; }
+      if (!id) { id = recycleDistantWrap(klass); if (!id) continue; }
 
       const w = insertAfter(el, id, klass, key);
       if (w) { observePh(id); inserted++; }