nodebb-plugin-ezoic-infinite 1.6.53 → 1.6.54

package/package.json

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

package/public/client.js

@@ -1,68 +1,27 @@
 (function () {
   'use strict';
 
-  // --- 1. GESTION DU SCROLL ---
-  let lastScrollY = 0;
-  let scrollDir = 1; 
-  try {
-    lastScrollY = window.scrollY || 0;
-    window.addEventListener('scroll', () => {
-      const y = window.scrollY || 0;
-      const d = y - lastScrollY;
-      if (Math.abs(d) > 4) {
-        scrollDir = d > 0 ? 1 : -1;
-        lastScrollY = y;
-      }
-    }, { passive: true });
-  } catch (e) {}
-
   const WRAP_CLASS = 'nodebb-ezoic-wrap';
-  const PLACEHOLDER_PREFIX = 'ezoic-pub-ad-placeholder-';
   const PINNED_ATTR = 'data-ezoic-pinned';
+  const PROCESSED_ATTR = 'data-ezoic-seen'; // Nouveau : marqueur de sujet
+  const PLACEHOLDER_PREFIX = 'ezoic-pub-ad-placeholder-';
 
   let config = null;
   let usedIds = new Set();
   let scheduleTimer = null;
 
-  // --- 2. SÉLECTEURS DE THÈMES (Harmony/Persona) ---
   function getTopicList() {
-    return document.querySelector('.category [component="category"], .categories [component="categories"], [component="topic/list"], .topic-list');
-  }
-
-  function getPostList() {
-    return document.querySelector('[component="topic"], [component="post/list"]');
-  }
-
-  // --- 3. FIX ANTI-PILEUP / ANTI-REMONTÉE ---
-  function pruneOrphans(ul) {
-    if (!ul) return;
-    const wraps = ul.querySelectorAll('.' + WRAP_CLASS);
-    wraps.forEach(wrap => {
-      // Si c'est la pub après le 1er sujet, on la force à rester collée au 1er LI
-      if (wrap.getAttribute(PINNED_ATTR) === 'true') {
-        const firstLi = ul.querySelector('li[data-tid], li:first-child');
-        if (firstLi && wrap.previousElementSibling !== firstLi) {
-          firstLi.after(wrap);
-        }
-        return;
-      }
-      // Si on remonte, on supprime les pubs qui perdent leur "ancre" (le LI précédent)
-      // NodeBB détruit les LI du haut pour la performance, on doit suivre le mouvement
-      if (scrollDir === -1 && (!wrap.previousElementSibling || wrap.previousElementSibling.tagName !== 'LI')) {
-        wrap.remove();
-      }
-    });
+    return document.querySelector('[component="topic/list"], [component="category"], [component="categories"], .topic-list');
   }
 
-  // --- 4. LOGIQUE EZOIC (Pools & Refresh) ---
-  function getNextId(poolStr) {
-    if (!poolStr) return null;
-    const ids = poolStr.split(/[\s,]+/).filter(Boolean);
+  function getNextId(pool) {
+    if (!pool) return null;
+    const ids = pool.split(/[\s,]+/).filter(Boolean);
     for (const id of ids) {
-      const num = id.replace(/[^\d]/g, '');
-      if (num && !usedIds.has(num)) {
-        usedIds.add(num);
-        return num;
+      const n = id.replace(/[^\d]/g, '');
+      if (n && !usedIds.has(n)) {
+        usedIds.add(n);
+        return n;
       }
     }
     return null;
@@ -79,92 +38,87 @@
         } else {
           window.ezstandalone.refresh();
         }
-      } catch (e) { console.warn('[ezoic] refresh error', id, e); }
+      } catch (e) { console.warn('[ezoic] err', e); }
     }
   }
 
-  // --- 5. INJECTION ---
   function inject() {
     if (!config || config.excluded) return;
-
-    // A. Gestion des SUJETS (Liste de catégories/topics)
     const ul = getTopicList();
-    if (ul && config.enableBetweenAds) {
-      pruneOrphans(ul);
-      const items = Array.from(ul.children).filter(c => c.tagName === 'LI' && !c.classList.contains(WRAP_CLASS));
-      
-      // Pub 1 (Pinned)
-      if (config.showFirstTopicAd && !ul.querySelector(`[${PINNED_ATTR}="true"]`)) {
-        insertAd(items[0], getNextId(config.placeholderIds), true);
-      }
+    if (!ul) return;
 
-      // Intervalle (uniquement en scroll vers le bas)
-      if (scrollDir === 1) {
-        const interval = parseInt(config.intervalPosts, 10) || 5;
-        items.forEach((li, idx) => {
-          if ((idx + 1) % interval === 0 && idx > 0) {
-            if (!li.nextElementSibling?.classList.contains(WRAP_CLASS)) {
-              insertAd(li, getNextId(config.placeholderIds), false);
-            }
-          }
-        });
-      }
+    // 1. On récupère tous les LI qui ne sont pas des pubs
+    const items = Array.from(ul.children).filter(c => c.tagName === 'LI' && !c.classList.contains(WRAP_CLASS));
+    if (!items.length) return;
+
+    // 2. PUB N°1 (Après le 1er sujet)
+    if (config.showFirstTopicAd && !ul.querySelector(`[${PINNED_ATTR}="true"]`)) {
+      items[0].setAttribute(PROCESSED_ATTR, 'true');
+      insertAd(items[0], getNextId(config.placeholderIds), true);
     }
 
-    // B. Gestion des MESSAGES (Post list)
-    const postList = getPostList();
-    if (postList && config.enableMessageAds) {
-      const posts = Array.from(postList.querySelectorAll('[component="post"]')).filter(p => !p.closest('.' + WRAP_CLASS));
-      if (config.showFirstMessageAd && posts.length > 0 && !postList.querySelector('.ezoic-msg-first')) {
-          insertAd(posts[0], getNextId(config.messagePlaceholderIds), false, 'ezoic-msg-first');
+    // 3. PUBS SUIVANTES (Intervalle)
+    const interval = parseInt(config.intervalPosts, 10) || 5;
+    
+    // On ne traite que les éléments qui n'ont pas encore été "vus" par le script
+    items.forEach((li, idx) => {
+      const positionAbsolue = idx + 1;
+      
+      // Si on tombe sur un multiple de l'intervalle ET que le LI n'a pas déjà été marqué
+      if (positionAbsolue > 1 && positionAbsolue % interval === 0) {
+        // Vérification anti-doublon : est-ce que ce LI a déjà une pub juste après lui ?
+        const hasAdAfter = li.nextElementSibling && li.nextElementSibling.classList.contains(WRAP_CLASS);
+        
+        if (!hasAdAfter) {
+          insertAd(li, getNextId(config.placeholderIds), false);
+        }
       }
-      // Logique d'intervalle message ici si besoin (similaire aux topics)
-    }
+    });
   }
 
-  function insertAd(target, id, isPinned, extraClass = '') {
-    if (!target || !id) return;
+  function insertAd(target, id, isPinned) {
+    if (!id || !target) return;
+    
     const wrap = document.createElement('div');
-    wrap.className = `${WRAP_CLASS} ezoic-ad-between ${extraClass}`;
+    wrap.className = WRAP_CLASS + ' ezoic-ad-between';
     if (isPinned) wrap.setAttribute(PINNED_ATTR, 'true');
 
     const placeholder = document.createElement('div');
     placeholder.id = PLACEHOLDER_PREFIX + id;
     wrap.appendChild(placeholder);
 
+    // Insertion physique
     target.after(wrap);
-    callEzoic(id);
+    
+    // On force un petit délai pour laisser Ezoic voir le nouveau div dans le DOM
+    setTimeout(() => callEzoic(id), 50);
   }
 
-  // --- 6. INITIALISATION ---
-  async function fetchConfig() {
+  async function init() {
     try {
       const res = await fetch('/api/plugins/ezoic-infinite/config');
       config = await res.json();
-    } catch (e) {}
-  }
 
-  function schedule() {
-    if (scheduleTimer) clearTimeout(scheduleTimer);
-    scheduleTimer = setTimeout(inject, 200);
-  }
+      // Premier passage
+      inject();
 
-  function init() {
-    fetchConfig().then(() => {
-      schedule();
+      // Observation du DOM (pour le scroll infini)
       const ul = getTopicList();
       if (ul && typeof MutationObserver !== 'undefined') {
-        new MutationObserver((m) => {
-          if (m.some(record => record.addedNodes.length > 0)) schedule();
-        }).observe(ul, { childList: true });
+        const mo = new MutationObserver(() => {
+          if (scheduleTimer) clearTimeout(scheduleTimer);
+          scheduleTimer = setTimeout(inject, 300);
+        });
+        mo.observe(ul, { childList: true });
       }
-    });
 
-    if (window.jQuery) {
-      window.jQuery(window).on('action:ajaxify.end action:infiniteScroll.loaded', () => {
-        setTimeout(schedule, 400);
-      });
-    }
+      // Hooks NodeBB
+      if (window.jQuery) {
+        window.jQuery(window).on('action:ajaxify.end action:infiniteScroll.loaded', () => {
+          setTimeout(inject, 500);
+        });
+      }
+    } catch (e) { console.error('[ezoic] init fail', e); }
   }
 
   if (document.readyState === 'loading') document.addEventListener('DOMContentLoaded', init);

package/public/style.css

@@ -86,3 +86,15 @@ li.nodebb-ezoic-host { list-style: none; width: 100%; display: block; }
 li.nodebb-ezoic-host > .nodebb-ezoic-wrap.ezoic-ad-between { width: 100%; display: block; }
 /* ===== /V17 ===== */
 
+.nodebb-ezoic-wrap {
+  display: block;
+  width: 100%;
+  clear: both;
+  margin: 20px 0 !important; /* Ajoute un peu d'espace pour le visuel */
+  min-height: 50px; /* Évite que le bloc ne soit écrasé à 0px */
+}
+
+/* Si la pub est vide, on garde un petit espace pour éviter le saut au scroll up */
+.nodebb-ezoic-wrap:empty {
+  min-height: 1px;
+}