nodebb-plugin-ezoic-infinite 1.6.53 → 1.6.55

package/package.json

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

package/public/client.js

@@ -1,30 +1,16 @@
 (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) {}
-
+  // --- 1. VARIABLES ET CONFIG ---
   const WRAP_CLASS = 'nodebb-ezoic-wrap';
-  const PLACEHOLDER_PREFIX = 'ezoic-pub-ad-placeholder-';
   const PINNED_ATTR = 'data-ezoic-pinned';
-
+  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) ---
+  // --- 2. SÉLECTEURS DE THÈMES ---
   function getTopicList() {
     return document.querySelector('.category [component="category"], .categories [component="categories"], [component="topic/list"], .topic-list');
   }
@@ -33,28 +19,7 @@
     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();
-      }
-    });
-  }
-
-  // --- 4. LOGIQUE EZOIC (Pools & Refresh) ---
+  // --- 3. LOGIQUE DES POOLS D'ID ---
   function getNextId(poolStr) {
     if (!poolStr) return null;
     const ids = poolStr.split(/[\s,]+/).filter(Boolean);
@@ -68,6 +33,7 @@
     return null;
   }
 
+  // --- 4. APPEL EZOIC (REFRESH & SHOW) ---
   function callEzoic(id) {
     if (window.ezstandalone) {
       try {
@@ -83,27 +49,34 @@
     }
   }
 
-  // --- 5. INJECTION ---
+  // --- 5. INJECTION SÉCURISÉE (ANTI-PILEUP) ---
   function inject() {
     if (!config || config.excluded) return;
 
-    // A. Gestion des SUJETS (Liste de catégories/topics)
+    // A. LISTE DES SUJETS (Topics)
     const ul = getTopicList();
     if (ul && config.enableBetweenAds) {
-      pruneOrphans(ul);
+      // Nettoyage des pubs orphelines (celles qui n'ont plus de LI au dessus à cause du scroll up)
+      const wraps = ul.querySelectorAll('.' + WRAP_CLASS);
+      wraps.forEach(w => {
+        if (!w.previousElementSibling || w.previousElementSibling.tagName !== 'LI') w.remove();
+      });
+
       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 (items.length > 0) {
+        // Pub n°1 (Pinned)
+        if (config.showFirstTopicAd && !ul.querySelector(`[${PINNED_ATTR}="true"]`)) {
+          insertAd(items[0], getNextId(config.placeholderIds), true);
+        }
 
-      // Intervalle (uniquement en scroll vers le bas)
-      if (scrollDir === 1) {
+        // Pubs d'intervalles
         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)) {
+          const pos = idx + 1;
+          if (pos > 1 && pos % interval === 0) {
+            // Vérifie si ce LI a déjà une pub juste après lui pour éviter les doublons
+            const next = li.nextElementSibling;
+            if (!next || !next.classList.contains(WRAP_CLASS)) {
               insertAd(li, getNextId(config.placeholderIds), false);
             }
           }
@@ -111,14 +84,25 @@
       }
     }
 
-    // B. Gestion des MESSAGES (Post list)
+    // B. LISTE DES MESSAGES (Posts)
     const postList = getPostList();
     if (postList && config.enableMessageAds) {
-      const posts = Array.from(postList.querySelectorAll('[component="post"]')).filter(p => !p.closest('.' + WRAP_CLASS));
+      const posts = Array.from(postList.querySelectorAll('[component="post"]'));
       if (config.showFirstMessageAd && posts.length > 0 && !postList.querySelector('.ezoic-msg-first')) {
           insertAd(posts[0], getNextId(config.messagePlaceholderIds), false, 'ezoic-msg-first');
       }
-      // Logique d'intervalle message ici si besoin (similaire aux topics)
+      
+      const msgInterval = parseInt(config.messageIntervalPosts, 10) || 5;
+      if (msgInterval > 0) {
+          posts.forEach((post, idx) => {
+              const pPos = idx + 1;
+              if (pPos > 1 && pPos % msgInterval === 0) {
+                  if (!post.nextElementSibling || !post.nextElementSibling.classList.contains(WRAP_CLASS)) {
+                      insertAd(post, getNextId(config.messagePlaceholderIds), false);
+                  }
+              }
+          });
+      }
     }
   }
 
@@ -133,36 +117,45 @@
     wrap.appendChild(placeholder);
 
     target.after(wrap);
-    callEzoic(id);
+    
+    // On laisse le DOM respirer 50ms avant d'appeler Ezoic
+    setTimeout(() => callEzoic(id), 50);
   }
 
-  // --- 6. INITIALISATION ---
+  // --- 6. INITIALISATION ET HOOKS ---
   async function fetchConfig() {
     try {
       const res = await fetch('/api/plugins/ezoic-infinite/config');
       config = await res.json();
-    } catch (e) {}
+    } catch (e) { console.error('[ezoic] config fetch failed'); }
   }
 
   function schedule() {
     if (scheduleTimer) clearTimeout(scheduleTimer);
-    scheduleTimer = setTimeout(inject, 200);
+    scheduleTimer = setTimeout(inject, 250);
   }
 
   function init() {
     fetchConfig().then(() => {
       schedule();
-      const ul = getTopicList();
-      if (ul && typeof MutationObserver !== 'undefined') {
-        new MutationObserver((m) => {
-          if (m.some(record => record.addedNodes.length > 0)) schedule();
-        }).observe(ul, { childList: true });
-      }
+
+      // MutationObserver pour le scroll infini
+      const body = document.body;
+      const mo = new MutationObserver((mutations) => {
+        for (let m of mutations) {
+          if (m.addedNodes.length > 0) {
+            schedule();
+            break;
+          }
+        }
+      });
+      mo.observe(body, { childList: true, subtree: true });
     });
 
+    // Compatibilité jQuery/NodeBB Events
     if (window.jQuery) {
       window.jQuery(window).on('action:ajaxify.end action:infiniteScroll.loaded', () => {
-        setTimeout(schedule, 400);
+        setTimeout(schedule, 500);
       });
     }
   }

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;
+}