nodebb-plugin-ezoic-infinite 1.7.10 → 1.7.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.10",
+  "version": "1.7.11",
   "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 (v20)
+ * NodeBB Ezoic Infinite Ads — client.js v22
  *
  * Correctifs critiques vs v19
  * ───────────────────────────
@@ -293,13 +293,17 @@
     const meta = KIND[klass];
     if (!meta) return;
 
-    const baseTag = meta.sel.split('[')[0]; // ex: "li", "[component=..." → "" (géré)
+    // baseTag déduit du sélecteur : 'li' pour topics/catégories, '' pour posts.
+    // Pour les posts (baseTag=''), on cherche juste [data-pid="X"] sans préfixe
+    // — c'est correct car data-pid est unique dans le DOM.
+    const baseTag = meta.sel.split('[')[0];
 
     document.querySelectorAll(`.${WRAP_CLASS}.${klass}`).forEach(w => {
       if (ts() - parseInt(w.getAttribute(A_CREATED) || '0', 10) < MIN_PRUNE_AGE_MS) return;
+      if (isFilled(w)) return; // jamais supprimer un wrap rempli
 
       const key = w.getAttribute(A_ANCHOR) ?? '';
-      const sid = key.slice(klass.length + 1); // extrait la partie après "kindClass:"
+      const sid = key.slice(klass.length + 1);
       if (!sid) { mutate(() => dropWrap(w)); return; }
 
       const anchorEl = document.querySelector(
@@ -346,17 +350,14 @@
   function ordinal(klass, el) {
     const di = el.getAttribute('data-index');
     if (di !== null && di !== '' && !isNaN(di)) return parseInt(di, 10);
-    // Fallback positionnel
-    try {
-      const tag = KIND[klass]?.sel?.split('[')?.[0] ?? '';
-      if (tag) {
-        let i = 0;
-        for (const n of el.parentElement?.querySelectorAll(`:scope > ${tag}`) ?? []) {
-          if (n === el) return i;
-          i++;
-        }
-      }
-    } catch (_) {}
+    // Fallback positionnel — filtre par sélecteur complet pour éviter le bug
+    // baseTag='' (posts) où `:scope > ` sans tag ne fonctionne pas.
+    const fullSel = KIND[klass]?.sel ?? '';
+    let i = 0;
+    for (const s of el.parentElement?.children ?? []) {
+      if (s === el) return i;
+      if (!fullSel || s.matches?.(fullSel)) i++;
+    }
     return 0;
   }