nodebb-plugin-ezoic-infinite 1.6.79 → 1.6.81

package/library.js

@@ -8,27 +8,30 @@ const SETTINGS_KEY = 'ezoic-infinite';
 const plugin = {};
 
 async function getSettings() {
-  return await meta.settings.get(SETTINGS_KEY);
-}
-
-async function getAllGroups() {
-  const names = await db.getSortedSetRange('groups:createtime', 0, -1);
-  const data = await groups.getGroupsData(names);
-  return data.filter(g => g && g.name).map(g => ({ name: g.name }));
+  const settings = await meta.settings.get(SETTINGS_KEY);
+  return settings || {};
 }
 
 async function isUserExcluded(uid, excludedGroups) {
-  if (!uid || !excludedGroups || !excludedGroups.length) return false;
-  return await groups.isMemberOfGroups(uid, excludedGroups);
+  if (!uid || !excludedGroups) return false;
+  const groupsList = Array.isArray(excludedGroups) ? excludedGroups : [excludedGroups];
+  if (!groupsList.length) return false;
+  return await groups.isMemberOfGroups(uid, groupsList);
 }
 
 plugin.init = async ({ router, middleware }) => {
   const renderAdmin = async (req, res) => {
     const settings = await getSettings();
-    const allGroups = await getAllGroups();
+    const names = await db.getSortedSetRange('groups:createtime', 0, -1);
+    const groupsData = await groups.getGroupsData(names);
+    const allGroups = groupsData.filter(g => g && g.name).map(g => ({ name: g.name }));
+
     res.render('admin/plugins/ezoic-infinite', {
       ...settings,
       allGroups,
+      // Helper pour le tpl
+      enableBetweenAds_checked: settings.enableBetweenAds === 'on' ? 'checked' : '',
+      enableMessageAds_checked: settings.enableMessageAds === 'on' ? 'checked' : ''
     });
   };
 

package/package.json

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

package/public/client.js

@@ -1,9 +1,8 @@
 (function () {
   'use strict';
 
-  // --- Gestion de la direction du scroll ---
   let lastScrollY = 0;
-  let scrollDir = 1; // 1 = down, -1 = up
+  let scrollDir = 1; 
   try {
     lastScrollY = window.scrollY || 0;
     window.addEventListener('scroll', () => {
@@ -22,6 +21,7 @@
 
   let config = null;
   let isInternalChange = false;
+  let ezInitialized = false; // Flag crucial pour éviter l'erreur de "refresh"
 
   function withInternalDomChange(fn) {
     isInternalChange = true;
@@ -40,7 +40,7 @@
   }
 
   function releaseWrapNode(wrap) {
-    if (!wrap) return;
+    if (!wrap || !wrap.parentNode) return;
     const pool = getPool();
     wrap.classList.remove('ez-orphan-hidden');
     if (wrap.parentNode !== pool) {
@@ -48,19 +48,18 @@
     }
   }
 
-  // --- ANTI-PILLUP : Supprime les pubs consécutives ---
+  // Anti-pillup : évite que deux pubs se suivent
   function decluster(container) {
     const wraps = Array.from(container.querySelectorAll(`.${WRAP_CLASS}`));
     wraps.forEach(wrap => {
       let next = wrap.nextElementSibling;
-      // Si deux blocs de pub se suivent sans contenu entre les deux
       if (next && next.classList.contains(WRAP_CLASS)) {
         withInternalDomChange(() => releaseWrapNode(next));
       }
     });
   }
 
-  // --- NETTOYAGE DES ORPHELINS (Virtualisation NodeBB) ---
+  // Nettoyage des pubs orphelines lors de la virtualisation
   function pruneOrphanWraps() {
     const wraps = document.querySelectorAll(`.${WRAP_CLASS}`);
     const items = document.querySelectorAll('[component="category/topic"], [component="post"]');
@@ -68,9 +67,7 @@
 
     wraps.forEach(wrap => {
       if (wrap.parentElement && wrap.parentElement.id === POOL_ID) return;
-
       let hasNeighbor = false;
-      // On vérifie la proximité immédiate (3 éléments) pour voir si le contenu est toujours là
       let prev = wrap.previousElementSibling;
       for (let i = 0; i < 3 && prev; i++) {
         if (itemSet.has(prev)) { hasNeighbor = true; break; }
@@ -85,7 +82,6 @@
       }
 
       if (!hasNeighbor) {
-        // En remontee (scroll UP), on recycle immédiatement pour éviter l'empilement visuel
         if (scrollDir === -1) {
           withInternalDomChange(() => releaseWrapNode(wrap));
         } else {
@@ -97,10 +93,36 @@
     });
   }
 
+  function callEzoic(placeholderId) {
+    if (typeof window.ezstandalone === 'undefined') return;
+    const pid = parseInt(placeholderId, 10);
+    if (isNaN(pid)) return;
+
+    try {
+      if (!ezInitialized) {
+        // PREMIER APPEL : define + enable + display
+        window.ezstandalone.define(pid);
+        window.ezstandalone.enable();
+        window.ezstandalone.display();
+        ezInitialized = true;
+      } else {
+        // APPELS SUIVANTS (Scroll) : define + refresh
+        window.ezstandalone.define(pid);
+        // On attend que le DOM soit stable pour le refresh
+        setTimeout(() => {
+          if (document.getElementById('ezoic-pub-ad-placeholder-' + pid)) {
+            window.ezstandalone.refresh();
+          }
+        }, 150);
+      }
+    } catch (e) {
+      console.warn('[Ezoic-Infinite] Ez Error:', e.message);
+    }
+  }
+
   function redistribute(container) {
     if (!container || !config || config.excluded) return;
     
-    // 1. On nettoie avant d'injecter
     pruneOrphanWraps();
     decluster(container);
 
@@ -132,7 +154,6 @@
     let inserts = 0;
     items.forEach((item, index) => {
       if (inserts >= MAX_INSERTS_PER_RUN) return;
-
       const pos = index + 1;
       let shouldHaveAd = (pos % interval === 0);
       if (pos === 1 && showFirst) shouldHaveAd = true;
@@ -146,14 +167,7 @@
         if (available) {
           withInternalDomChange(() => {
             item.parentNode.insertBefore(available, item.nextSibling);
-            // Appel Ezoic Standalone si présent
-            if (typeof window.ezstandalone !== 'undefined') {
-              const pid = available.getAttribute('data-placeholder-id');
-              try {
-                window.ezstandalone.define(parseInt(pid, 10));
-                window.ezstandalone.refresh();
-              } catch (e) {}
-            }
+            callEzoic(available.getAttribute('data-placeholder-id'));
           });
           inserts++;
         }
@@ -166,11 +180,10 @@
       .then(r => r.json())
       .then(data => {
         config = data;
-        // Initialisation du pool d'IDs
         const pool = getPool();
         const setupPool = (idsRaw, kind) => {
           if (!idsRaw) return;
-          const ids = idsRaw.split(/[\s,]+/).filter(Boolean);
+          const ids = idsRaw.split(/[\s,]+/).map(s => s.trim()).filter(Boolean);
           ids.forEach(id => {
             if (!document.querySelector(`[data-placeholder-id="${id}"]`)) {
               const d = document.createElement('div');
@@ -188,16 +201,17 @@
         setupPool(config.placeholderIds, 'between');
         setupPool(config.messagePlaceholderIds, 'message');
         cb();
-      });
+      }).catch(e => console.error('[Ezoic] Config load error', e));
   }
 
   let timer = null;
   function schedule() {
     if (timer) clearTimeout(timer);
     timer = setTimeout(() => {
-      const lists = document.querySelectorAll('[component="category"], .topic-list, [component="topic"]');
+      // Sélecteurs pour NodeBB 4.x / Harmony
+      const lists = document.querySelectorAll('[component="category"], .topic-list, [component="topic"], [component="category/topic/list"]');
       lists.forEach(redistribute);
-    }, 100);
+    }, 200); 
   }
 
   function init() {
@@ -206,7 +220,9 @@
       if (typeof MutationObserver !== 'undefined') {
         const mo = new MutationObserver((muts) => {
           if (isInternalChange) return;
-          schedule();
+          // On vérifie si un élément structurel a été ajouté
+          const shouldRun = muts.some(m => m.addedNodes.length > 0);
+          if (shouldRun) schedule();
         });
         mo.observe(document.body, { childList: true, subtree: true });
       }

package/public/style.css

@@ -1,24 +1,22 @@
-/* Container de base */
+/* Container principal */
 .nodebb-ezoic-wrap {
   display: block;
   width: 100%;
-  margin: 15px 0 !important;
+  margin: 20px 0 !important;
   padding: 0 !important;
   clear: both;
-  overflow: hidden;
-  min-height: 1px;
+  min-height: 50px; /* Important pour que l'ad-tester détecte le bloc */
 }
 
-/* Cache les pubs orphelines sans casser le flux du DOM */
+/* Cache les pubs orphelines sans les supprimer du DOM */
 .nodebb-ezoic-wrap.ez-orphan-hidden {
   display: none !important;
   height: 0 !important;
+  min-height: 0 !important;
   margin: 0 !important;
-  padding: 0 !important;
-  pointer-events: none;
 }
 
-/* Force la suppression des marges Ezoic internes pour garder le contrôle */
+/* Nettoyage des styles internes Ezoic */
 .nodebb-ezoic-wrap .ezoic-ad {
   margin: 0 auto !important;
   padding: 0 !important;
@@ -26,23 +24,12 @@
   height: auto !important;
 }
 
-/* Neutralisation du line-height pour éviter les décalages de 1px */
-.nodebb-ezoic-wrap,
-.nodebb-ezoic-wrap * {
-  line-height: 0 !important;
-  font-size: 0 !important;
-}
-
-/* Harmonisation pour Harmony/NodeBB 4 */
-[component="category"] .nodebb-ezoic-wrap,
-.topic-list .nodebb-ezoic-wrap {
-  border-bottom: 1px solid rgba(0,0,0,0.05);
-  background: transparent;
+/* Neutralise les marges doubles */
+.nodebb-ezoic-wrap + .nodebb-ezoic-wrap {
+  margin-top: 0 !important;
 }
 
-/* Empêche le saut visuel si Ezoic met du temps à charger */
-.nodebb-ezoic-wrap iframe {
-  display: block !important;
-  max-width: 100%;
-  margin: 0 auto;
+/* Évite les débordements sur mobile */
+.ezoic-ad iframe {
+  max-width: 100% !important;
 }