nodebb-plugin-ezoic-infinite 1.6.80 → 1.6.81

package/library.js

@@ -8,12 +8,12 @@ const SETTINGS_KEY = 'ezoic-infinite';
 const plugin = {};
 
 async function getSettings() {
-  return await meta.settings.get(SETTINGS_KEY);
+  const settings = await meta.settings.get(SETTINGS_KEY);
+  return settings || {};
 }
 
 async function isUserExcluded(uid, excludedGroups) {
   if (!uid || !excludedGroups) return false;
-  // Gérer le format multiple de NodeBB
   const groupsList = Array.isArray(excludedGroups) ? excludedGroups : [excludedGroups];
   if (!groupsList.length) return false;
   return await groups.isMemberOfGroups(uid, groupsList);
@@ -29,6 +29,7 @@ plugin.init = async ({ router, middleware }) => {
     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.80",
+  "version": "1.6.81",
   "description": "Production-ready Ezoic infinite ads integration for NodeBB 4.x",
   "main": "library.js",
   "license": "MIT",

package/public/client.js

@@ -21,7 +21,7 @@
 
   let config = null;
   let isInternalChange = false;
-  let ezEnabled = false; // Flag pour savoir si ezstandalone.enable() a été appelé
+  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,6 +48,7 @@
     }
   }
 
+  // Anti-pillup : évite que deux pubs se suivent
   function decluster(container) {
     const wraps = Array.from(container.querySelectorAll(`.${WRAP_CLASS}`));
     wraps.forEach(wrap => {
@@ -58,6 +59,7 @@
     });
   }
 
+  // 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"]');
@@ -93,25 +95,28 @@
 
   function callEzoic(placeholderId) {
     if (typeof window.ezstandalone === 'undefined') return;
-    
     const pid = parseInt(placeholderId, 10);
+    if (isNaN(pid)) return;
+
     try {
-      if (!ezEnabled) {
+      if (!ezInitialized) {
+        // PREMIER APPEL : define + enable + display
         window.ezstandalone.define(pid);
         window.ezstandalone.enable();
         window.ezstandalone.display();
-        ezEnabled = true;
+        ezInitialized = true;
       } else {
+        // APPELS SUIVANTS (Scroll) : define + refresh
         window.ezstandalone.define(pid);
-        // Utiliser une petite pause pour être sûr que le DOM est prêt
+        // On attend que le DOM soit stable pour le refresh
         setTimeout(() => {
           if (document.getElementById('ezoic-pub-ad-placeholder-' + pid)) {
             window.ezstandalone.refresh();
           }
-        }, 100);
+        }, 150);
       }
     } catch (e) {
-      console.warn('[Ezoic-Infinite] Error refreshing ad:', e);
+      console.warn('[Ezoic-Infinite] Ez Error:', e.message);
     }
   }
 
@@ -178,7 +183,7 @@
         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');
@@ -196,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(() => {
+      // Sélecteurs pour NodeBB 4.x / Harmony
       const lists = document.querySelectorAll('[component="category"], .topic-list, [component="topic"], [component="category/topic/list"]');
       lists.forEach(redistribute);
-    }, 150); // Un peu plus de délai pour NodeBB 4.x
+    }, 200); 
   }
 
   function init() {
@@ -214,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,37 +1,35 @@
+/* Container principal */
 .nodebb-ezoic-wrap {
   display: block;
   width: 100%;
   margin: 20px 0 !important;
   padding: 0 !important;
   clear: both;
-  overflow: visible;
-  min-height: 50px; /* Aide Ezoic à détecter l'élément */
+  min-height: 50px; /* Important pour que l'ad-tester détecte le bloc */
 }
 
-/* Cache les pubs orphelines proprement */
+/* 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;
 }
 
-/* Neutralisation du style interne Ezoic pour éviter les énormes espaces blancs */
+/* Nettoyage des styles internes Ezoic */
 .nodebb-ezoic-wrap .ezoic-ad {
   margin: 0 auto !important;
   padding: 0 !important;
   min-height: 1px !important;
   height: auto !important;
-  line-height: normal !important;
 }
 
-/* Enlève les marges doubles si plusieurs pubs se suivent malgré le script */
+/* Neutralise les marges doubles */
 .nodebb-ezoic-wrap + .nodebb-ezoic-wrap {
   margin-top: 0 !important;
 }
 
-/* Assurer la visibilité pour l'IntersectionObserver d'Ezoic */
+/* Évite les débordements sur mobile */
 .ezoic-ad iframe {
-  max-width: 100%;
+  max-width: 100% !important;
 }