nodebb-plugin-ezoic-infinite 1.6.94 → 1.6.96

package/library.js

@@ -7,33 +7,24 @@ const db = require.main.require('./src/database');
 const SETTINGS_KEY = 'ezoic-infinite';
 const plugin = {};
 
-/**
- * Récupère les paramètres du plugin
- */
 async function getSettings() {
     return await meta.settings.get(SETTINGS_KEY);
 }
 
 /**
- * Vérifie si l'utilisateur actuel fait partie d'un groupe exclu
+ * Vérifie l'exclusion par groupe avec gestion du format NodeBB
  */
 async function isUserExcluded(uid, excludedGroups) {
     if (!uid || uid <= 0) return false;
-    if (!excludedGroups || !excludedGroups.length) return false;
+    if (!excludedGroups || (Array.isArray(excludedGroups) && excludedGroups.length === 0)) return false;
 
-    // S'assurer que excludedGroups est un tableau
     const excludedList = Array.isArray(excludedGroups) ? excludedGroups : [excludedGroups];
-    
-    // Récupérer les groupes de l'utilisateur
     const userGroups = await groups.getUserGroupsNames([uid]);
     
-    // Vérifier s'il y a une intersection entre les groupes de l'utilisateur et les exclus
+    // userGroups[0] contient le tableau des noms de groupes de l'utilisateur
     return excludedList.some(g => userGroups[0].includes(g));
 }
 
-/**
- * Récupère la liste de tous les groupes pour l'affichage dans l'admin
- */
 async function getAllGroups() {
     let names = await db.getSortedSetRange('groups:createtime', 0, -1);
     if (!names || !names.length) {
@@ -46,9 +37,6 @@ async function getAllGroups() {
     return valid;
 }
 
-/**
- * Initialisation du plugin (Routes et API)
- */
 plugin.init = async ({ router, middleware }) => {
     async function render(req, res) {
         const settings = await getSettings();
@@ -57,10 +45,10 @@ plugin.init = async ({ router, middleware }) => {
         res.render('admin/plugins/ezoic-infinite', {
             title: 'Ezoic Infinite Ads',
             ...settings,
-            // On force les booléens pour les cases à cocher dans le template
-            enableCategoryAds_checked: settings.enableCategoryAds === 'on' || settings.enableCategoryAds === true ? 'checked' : '',
-            enableBetweenAds_checked: settings.enableBetweenAds === 'on' || settings.enableBetweenAds === true ? 'checked' : '',
-            enableMessageAds_checked: settings.enableMessageAds === 'on' || settings.enableMessageAds === true ? 'checked' : '',
+            // On s'assure que les checkbox sont bien cochées dans l'admin
+            enableCategoryAds_checked: settings.enableCategoryAds === 'on' ? 'checked' : '',
+            enableBetweenAds_checked: settings.enableBetweenAds === 'on' ? 'checked' : '',
+            enableMessageAds_checked: settings.enableMessageAds === 'on' ? 'checked' : '',
             allGroups,
         });
     }
@@ -68,40 +56,34 @@ plugin.init = async ({ router, middleware }) => {
     router.get('/admin/plugins/ezoic-infinite', middleware.admin.buildHeader, render);
     router.get('/api/admin/plugins/ezoic-infinite', render);
 
-    // API de configuration appelée par client.js
+    // API consommée par le client.js
     router.get('/api/plugins/ezoic-infinite/config', async (req, res) => {
         const settings = await getSettings();
-        
-        // Vérification de l'exclusion
-        const excludedGroups = settings.excludedGroups ? (Array.isArray(settings.excludedGroups) ? settings.excludedGroups : [settings.excludedGroups]) : [];
-        const excluded = await isUserExcluded(req.uid, excludedGroups);
+        const excluded = await isUserExcluded(req.uid, settings.excludedGroups);
 
         res.json({
             excluded,
-            // 1. Accueil (Catégories)
-            enableCategoryAds: settings.enableCategoryAds === 'on' || settings.enableCategoryAds === true,
-            showFirstCategoryAd: settings.showFirstCategoryAd === 'on' || settings.showFirstCategoryAd === true,
+            // Pool Accueil
+            enableCategoryAds: settings.enableCategoryAds === 'on',
+            showFirstCategoryAd: settings.showFirstCategoryAd === 'on',
             categoryPlaceholderIds: settings.categoryPlaceholderIds || "",
             intervalCategories: parseInt(settings.intervalCategories, 10) || 5,
 
-            // 2. Liste des Topics (Page catégorie)
-            enableBetweenAds: settings.enableBetweenAds === 'on' || settings.enableBetweenAds === true,
-            showFirstTopicAd: settings.showFirstTopicAd === 'on' || settings.showFirstTopicAd === true,
+            // Pool Topics (Catégories)
+            enableBetweenAds: settings.enableBetweenAds === 'on',
+            showFirstTopicAd: settings.showFirstTopicAd === 'on',
             placeholderIds: settings.placeholderIds || "",
             intervalPosts: parseInt(settings.intervalPosts, 10) || 10,
 
-            // 3. Messages (Dans un post)
-            enableMessageAds: settings.enableMessageAds === 'on' || settings.enableMessageAds === true,
-            showFirstMessageAd: settings.showFirstMessageAd === 'on' || settings.showFirstMessageAd === true,
+            // Pool Messages (Topics)
+            enableMessageAds: settings.enableMessageAds === 'on',
+            showFirstMessageAd: settings.showFirstMessageAd === 'on',
             messagePlaceholderIds: settings.messagePlaceholderIds || "",
             messageIntervalPosts: parseInt(settings.messageIntervalPosts, 10) || 10,
         });
     });
 };
 
-/**
- * Ajoute le lien dans le menu de navigation de l'administration
- */
 plugin.addAdminNavigation = async (header) => {
     header.plugins.push({
         route: '/plugins/ezoic-infinite',
@@ -111,12 +93,9 @@ plugin.addAdminNavigation = async (header) => {
     return header;
 };
 
-/**
- * Gère la sauvegarde des paramètres
- */
 plugin.onSettingsSet = async (data) => {
     if (data.plugin === 'ezoic-infinite') {
-        // Logique optionnelle à l'enregistrement
+        // Optionnel : purger un cache si nécessaire
     }
 };
 

package/package.json

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

package/public/client.js

@@ -5,76 +5,83 @@
   window.ezInfiniteInjected = true;
 
   const WRAP_CLASS = 'nodebb-ezoic-wrap';
+  const POOL_ID = 'nodebb-ezoic-placeholder-pool';
   let config = null;
   let isInternalChange = false;
-  let triggerTimer = null;
 
-  // On récupère les IDs déjà présents dans la page (générés par ton intégration de base)
-  // pour éviter de tenter de les redéfinir.
-  function redistribute() {
-    if (!config || config.excluded) return;
-
-    const selectors = {
-      'home': '.category-item, [component="categories/category"]',
-      'topic-list': 'li[component="category/topic"]',
-      'message': '[component="post"]'
-    };
-
-    if (config.enableCategoryAds) {
-      process(Array.from(document.querySelectorAll(selectors.home)), 'home', config.intervalCategories, config.showFirstCategoryAd);
-    }
-    if (config.enableBetweenAds) {
-      process(Array.from(document.querySelectorAll(selectors['topic-list'])), 'topic-list', config.intervalPosts, config.showFirstTopicAd);
-    }
-    if (config.enableMessageAds) {
-      process(Array.from(document.querySelectorAll(selectors.message)), 'message', config.messageIntervalPosts, config.showFirstMessageAd);
+  function getPool() {
+    let p = document.getElementById(POOL_ID);
+    if (!p) {
+      p = document.createElement('div');
+      p.id = POOL_ID;
+      p.style.display = 'none';
+      document.body.appendChild(p);
     }
+    return p;
   }
 
-  function process(items, kind, interval, showFirst) {
-    const int = parseInt(interval, 10) || 10;
-    const pool = document.getElementById('nodebb-ezoic-placeholder-pool');
-    if (!pool) return;
-
-    items.forEach((item, index) => {
-      const pos = index + 1;
-      const shouldHaveAd = (pos === 1 && showFirst) || (pos % int === 0);
-      const next = item.nextElementSibling;
+  // Cette fonction force Ezoic à remplir le placeholder
+  function forceFillAd(id) {
+    if (typeof window.ezstandalone === 'undefined') return;
 
-      // Si on doit mettre une pub et qu'il n'y en a pas déjà une après cet élément
-      if (shouldHaveAd && !(next && next.classList.contains(WRAP_CLASS))) {
-        const available = pool.querySelector(`.${WRAP_CLASS}[data-kind="${kind}"]`);
-        
-        if (available) {
-          isInternalChange = true;
-          item.parentNode.insertBefore(available, item.nextSibling);
-          
-          const placeholderId = available.getAttribute('data-placeholder-id');
-          refreshAd(placeholderId);
-          
-          setTimeout(() => { isInternalChange = false; }, 50);
+    window.ezstandalone.cmd.push(function() {
+      // 1. On s'assure que l'ID est bien dans la liste d'Ezoic
+      window.ezstandalone.define(parseInt(id, 10));
+      
+      // 2. On attend un micro-délai pour que le DOM soit stable
+      setTimeout(() => {
+        try {
+          // La doc précise que showAds avec IDs est le mode "Dynamic"
+          window.ezstandalone.showAds(parseInt(id, 10));
+          // On force un rafraîchissement spécifique si showAds ne suffit pas
+          if (window.ezstandalone.refresh) {
+             window.ezstandalone.refresh(parseInt(id, 10));
+          }
+        } catch (e) {
+          console.warn('[Ezoic] Fill failed for', id, e);
         }
-      }
+      }, 200);
     });
   }
 
-  function refreshAd(id) {
-    if (typeof window.ezstandalone === 'undefined') return;
+  function redistribute() {
+    if (!config || config.excluded) return;
 
-    clearTimeout(triggerTimer);
-    triggerTimer = setTimeout(() => {
-      window.ezstandalone.cmd.push(function() {
-        // Au lieu de define(), on utilise simplement showAds()
-        // Ezoic va détecter que le div avec cet ID a bougé et va le remplir.
-        try {
-          // On force l'affichage de cet ID spécifique
-          window.ezstandalone.showAds(parseInt(id, 10));
-          console.debug('[Ezoic] Displaying ID:', id);
-        } catch (e) {
-          console.error('[Ezoic] Error showing ID:', id, e);
+    const selectors = {
+      'home': '.category-item, [component="categories/category"]',
+      'topic-list': 'li[component="category/topic"]',
+      'message': '[component="post"]'
+    };
+
+    const process = (items, kind, interval, showFirst) => {
+      const int = parseInt(interval, 10) || 10;
+      const pool = getPool();
+
+      items.forEach((item, index) => {
+        const pos = index + 1;
+        const shouldHaveAd = (pos === 1 && showFirst) || (pos % int === 0);
+        const next = item.nextElementSibling;
+
+        if (shouldHaveAd && !(next && next.classList.contains(WRAP_CLASS))) {
+          const available = pool.querySelector(`.${WRAP_CLASS}[data-kind="${kind}"]`);
+          if (available) {
+            isInternalChange = true;
+            // On insère l'élément
+            item.parentNode.insertBefore(available, item.nextSibling);
+            
+            // On déclenche le remplissage publicitaire
+            const pid = available.getAttribute('data-placeholder-id');
+            forceFillAd(pid);
+            
+            setTimeout(() => { isInternalChange = false; }, 50);
+          }
         }
       });
-    }, 100);
+    };
+
+    if (config.enableCategoryAds) process(Array.from(document.querySelectorAll(selectors.home)), 'home', config.intervalCategories, config.showFirstCategoryAd);
+    if (config.enableBetweenAds) process(Array.from(document.querySelectorAll(selectors['topic-list'])), 'topic-list', config.intervalPosts, config.showFirstTopicAd);
+    if (config.enableMessageAds) process(Array.from(document.querySelectorAll(selectors.message)), 'message', config.messageIntervalPosts, config.showFirstMessageAd);
   }
 
   function init() {
@@ -82,27 +89,18 @@
       .then(r => r.json())
       .then(data => {
         config = data;
+        const pool = getPool();
         
-        // On crée un pool CACHÉ qui contient tous les placeholders déclarés dans l'admin
-        let pool = document.getElementById('nodebb-ezoic-placeholder-pool');
-        if (!pool) {
-          pool = document.createElement('div');
-          pool.id = 'nodebb-ezoic-placeholder-pool';
-          pool.style.display = 'none';
-          document.body.appendChild(pool);
-        }
-
         const setup = (raw, kind) => {
           if (!raw) return;
           raw.split(/[\s,]+/).filter(Boolean).forEach(id => {
             if (!document.getElementById(`ezoic-pub-ad-placeholder-${id}`)) {
-              const wrap = document.createElement('div');
-              wrap.className = WRAP_CLASS;
-              wrap.setAttribute('data-kind', kind);
-              wrap.setAttribute('data-placeholder-id', id);
-              // Structure exacte attendue par Ezoic
-              wrap.innerHTML = `<div id="ezoic-pub-ad-placeholder-${id}"></div>`;
-              pool.appendChild(wrap);
+              const d = document.createElement('div');
+              d.className = WRAP_CLASS;
+              d.setAttribute('data-kind', kind);
+              d.setAttribute('data-placeholder-id', id);
+              d.innerHTML = `<div id="ezoic-pub-ad-placeholder-${id}"></div>`;
+              pool.appendChild(d);
             }
           });
         };
@@ -120,7 +118,9 @@
       });
   }
 
-  window.addEventListener('action:ajaxify.end', () => setTimeout(redistribute, 500));
+  window.addEventListener('action:ajaxify.end', () => {
+    setTimeout(redistribute, 500);
+  });
 
   if (document.readyState === 'loading') {
     document.addEventListener('DOMContentLoaded', init);