nodebb-plugin-ezoic-infinite 1.6.81 → 1.6.83

package/library.js

@@ -2,66 +2,58 @@
 
 const meta = require.main.require('./src/meta');
 const groups = require.main.require('./src/groups');
-const db = require.main.require('./src/database');
 
 const SETTINGS_KEY = 'ezoic-infinite';
 const plugin = {};
 
 async function getSettings() {
-  const settings = await meta.settings.get(SETTINGS_KEY);
-  return settings || {};
-}
-
-async function isUserExcluded(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);
+    return await meta.settings.get(SETTINGS_KEY) || {};
 }
 
 plugin.init = async ({ router, middleware }) => {
-  const renderAdmin = async (req, res) => {
-    const settings = await getSettings();
-    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' : ''
-    });
-  };
-
-  router.get('/admin/plugins/ezoic-infinite', middleware.admin.buildHeader, renderAdmin);
-  router.get('/api/admin/plugins/ezoic-infinite', renderAdmin);
-
-  router.get('/api/plugins/ezoic-infinite/config', async (req, res) => {
-    const settings = await getSettings();
-    const excluded = await isUserExcluded(req.uid, settings.excludedGroups);
-    res.json({
-      excluded,
-      enableBetweenAds: settings.enableBetweenAds === 'on',
-      showFirstTopicAd: settings.showFirstTopicAd === 'on',
-      placeholderIds: settings.placeholderIds || '',
-      intervalPosts: settings.intervalPosts || 10,
-      enableMessageAds: settings.enableMessageAds === 'on',
-      showFirstMessageAd: settings.showFirstMessageAd === 'on',
-      messagePlaceholderIds: settings.messagePlaceholderIds || '',
-      messageIntervalPosts: settings.messageIntervalPosts || 10,
+    const renderAdmin = async (req, res) => {
+        const settings = await getSettings();
+        const db = require.main.require('./src/database');
+        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,
+            enableBetweenAds_checked: settings.enableBetweenAds === 'on' ? 'checked' : '',
+            enableMessageAds_checked: settings.enableMessageAds === 'on' ? 'checked' : ''
+        });
+    };
+
+    router.get('/admin/plugins/ezoic-infinite', middleware.admin.buildHeader, renderAdmin);
+    router.get('/api/admin/plugins/ezoic-infinite', renderAdmin);
+
+    router.get('/api/plugins/ezoic-infinite/config', async (req, res) => {
+        const settings = await getSettings();
+        let excluded = false;
+        if (req.uid && settings.excludedGroups) {
+            const groupsList = Array.isArray(settings.excludedGroups) ? settings.excludedGroups : [settings.excludedGroups];
+            excluded = await groups.isMemberOfGroups(req.uid, groupsList);
+        }
+
+        res.json({
+            excluded,
+            enableBetweenAds: settings.enableBetweenAds === 'on',
+            showFirstTopicAd: settings.showFirstTopicAd === 'on',
+            placeholderIds: settings.placeholderIds || '',
+            intervalPosts: settings.intervalPosts || 10,
+            enableMessageAds: settings.enableMessageAds === 'on',
+            showFirstMessageAd: settings.showFirstMessageAd === 'on',
+            messagePlaceholderIds: settings.messagePlaceholderIds || '',
+            messageIntervalPosts: settings.messageIntervalPosts || 10,
+        });
     });
-  });
 };
 
 plugin.addAdminNavigation = async (header) => {
-  header.plugins.push({
-    route: '/plugins/ezoic-infinite',
-    icon: 'fa-ad',
-    name: 'Ezoic Infinite'
-  });
-  return header;
+    header.plugins.push({ route: '/plugins/ezoic-infinite', icon: 'fa-ad', name: 'Ezoic Infinite' });
+    return header;
 };
 
 module.exports = plugin;

package/package.json

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

package/public/client.js

@@ -1,237 +1,119 @@
 (function () {
-  'use strict';
+    'use strict';
 
-  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) {}
+    if (window.ezInfiniteInjected) return;
+    window.ezInfiniteInjected = true;
 
-  const WRAP_CLASS = 'nodebb-ezoic-wrap';
-  const POOL_ID = 'nodebb-ezoic-placeholder-pool';
-  const MAX_INSERTS_PER_RUN = 8;
-
-  let config = null;
-  let isInternalChange = false;
-  let ezInitialized = false; // Flag crucial pour éviter l'erreur de "refresh"
-
-  function withInternalDomChange(fn) {
-    isInternalChange = true;
-    try { fn(); } finally { isInternalChange = false; }
-  }
-
-  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 releaseWrapNode(wrap) {
-    if (!wrap || !wrap.parentNode) return;
-    const pool = getPool();
-    wrap.classList.remove('ez-orphan-hidden');
-    if (wrap.parentNode !== pool) {
-      pool.appendChild(wrap);
-    }
-  }
-
-  // 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;
-      if (next && next.classList.contains(WRAP_CLASS)) {
-        withInternalDomChange(() => releaseWrapNode(next));
-      }
-    });
-  }
-
-  // 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"]');
-    const itemSet = new Set(items);
-
-    wraps.forEach(wrap => {
-      if (wrap.parentElement && wrap.parentElement.id === POOL_ID) return;
-      let hasNeighbor = false;
-      let prev = wrap.previousElementSibling;
-      for (let i = 0; i < 3 && prev; i++) {
-        if (itemSet.has(prev)) { hasNeighbor = true; break; }
-        prev = prev.previousElementSibling;
-      }
-      if (!hasNeighbor) {
-        let next = wrap.nextElementSibling;
-        for (let i = 0; i < 3 && next; i++) {
-          if (itemSet.has(next)) { hasNeighbor = true; break; }
-          next = next.nextElementSibling;
-        }
-      }
-
-      if (!hasNeighbor) {
-        if (scrollDir === -1) {
-          withInternalDomChange(() => releaseWrapNode(wrap));
-        } else {
-          wrap.classList.add('ez-orphan-hidden');
+    const WRAP_CLASS = 'nodebb-ezoic-wrap';
+    const POOL_ID = 'nodebb-ezoic-placeholder-pool';
+    
+    let config = null;
+    let isInternalChange = false;
+    let ezEnabled = false;
+
+    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);
         }
-      } else {
-        wrap.classList.remove('ez-orphan-hidden');
-      }
-    });
-  }
-
-  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);
+        return p;
     }
-  }
 
-  function redistribute(container) {
-    if (!container || !config || config.excluded) return;
-    
-    pruneOrphanWraps();
-    decluster(container);
-
-    const isTopicList = container.getAttribute('component') === 'category' || container.classList.contains('topic-list');
-    const isPostList = container.getAttribute('component') === 'topic' || container.querySelectorAll('[component="post"]').length > 0;
-
-    let enabled = false;
-    let interval = 10;
-    let kind = '';
-    let items = [];
-    let showFirst = false;
-
-    if (isTopicList && config.enableBetweenAds) {
-      enabled = true;
-      interval = parseInt(config.intervalPosts, 10) || 10;
-      kind = 'between';
-      items = Array.from(container.querySelectorAll('[component="category/topic"]'));
-      showFirst = config.showFirstTopicAd;
-    } else if (isPostList && config.enableMessageAds) {
-      enabled = true;
-      interval = parseInt(config.messageIntervalPosts, 10) || 10;
-      kind = 'message';
-      items = Array.from(container.querySelectorAll('[component="post"]'));
-      showFirst = config.showFirstMessageAd;
+    function callEzoic(pid) {
+        if (typeof window.ezstandalone === 'undefined') return;
+        const id = parseInt(pid, 10);
+        try {
+            window.ezstandalone.define(id);
+            if (!ezEnabled) {
+                window.ezstandalone.enable();
+                window.ezstandalone.display();
+                ezEnabled = true;
+            } else {
+                // Délai pour s'assurer que le div est bien rendu par le navigateur
+                setTimeout(() => { window.ezstandalone.refresh(); }, 200);
+            }
+        } catch (e) { console.warn('[Ezoic] Error:', e); }
     }
 
-    if (!enabled || items.length === 0) return;
-
-    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;
-
-      const next = item.nextElementSibling;
-      const hasAdAfter = next && next.classList.contains(WRAP_CLASS);
-
-      if (shouldHaveAd && !hasAdAfter) {
-        const pool = getPool();
-        const available = pool.querySelector(`.${WRAP_CLASS}[data-kind="${kind}"]`);
-        if (available) {
-          withInternalDomChange(() => {
-            item.parentNode.insertBefore(available, item.nextSibling);
-            callEzoic(available.getAttribute('data-placeholder-id'));
-          });
-          inserts++;
+    function redistribute() {
+        if (!config || config.excluded) return;
+
+        // Sélecteurs spécifiques à NodeBB 4.x / Harmony
+        const topicItems = document.querySelectorAll('[component="category/topic"]');
+        const postItems = document.querySelectorAll('[component="post"]');
+
+        let items = [], kind = '', interval = 10, showFirst = false;
+
+        if (topicItems.length > 0 && config.enableBetweenAds) {
+            items = Array.from(topicItems);
+            kind = 'between';
+            interval = parseInt(config.intervalPosts, 10);
+            showFirst = config.showFirstTopicAd;
+        } else if (postItems.length > 0 && config.enableMessageAds) {
+            items = Array.from(postItems);
+            kind = 'message';
+            interval = parseInt(config.messageIntervalPosts, 10);
+            showFirst = config.showFirstMessageAd;
         }
-      }
-    });
-  }
 
-  function fetchConfig(cb) {
-    fetch('/api/plugins/ezoic-infinite/config')
-      .then(r => r.json())
-      .then(data => {
-        config = data;
-        const pool = getPool();
-        const setupPool = (idsRaw, kind) => {
-          if (!idsRaw) return;
-          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');
-              d.className = WRAP_CLASS + ' nodebb-ezoic-ad-' + kind;
-              d.setAttribute('data-kind', kind);
-              d.setAttribute('data-placeholder-id', id);
-              const inner = document.createElement('div');
-              inner.id = 'ezoic-pub-ad-placeholder-' + id;
-              inner.className = 'ezoic-ad';
-              d.appendChild(inner);
-              pool.appendChild(d);
+        if (items.length === 0) return;
+
+        items.forEach((item, index) => {
+            const pos = index + 1;
+            const shouldHaveAd = (pos === 1 && showFirst) || (pos % interval === 0);
+            
+            const next = item.nextElementSibling;
+            if (shouldHaveAd && !(next && next.classList.contains(WRAP_CLASS))) {
+                const pool = getPool();
+                const available = pool.querySelector(`.${WRAP_CLASS}[data-kind="${kind}"]`);
+                if (available) {
+                    isInternalChange = true;
+                    item.parentNode.insertBefore(available, item.nextSibling);
+                    callEzoic(available.getAttribute('data-placeholder-id'));
+                    setTimeout(() => { isInternalChange = false; }, 100);
+                }
             }
-          });
-        };
-        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);
-    }, 200); 
-  }
-
-  function init() {
-    fetchConfig(() => {
-      schedule();
-      if (typeof MutationObserver !== 'undefined') {
-        const mo = new MutationObserver((muts) => {
-          if (isInternalChange) return;
-          // 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 });
-      }
-    });
-  }
+    }
 
-  if (document.readyState === 'loading') {
-    document.addEventListener('DOMContentLoaded', init);
-  } else {
-    init();
-  }
+    function init() {
+        fetch('/api/plugins/ezoic-infinite/config')
+            .then(r => r.json())
+            .then(data => {
+                config = data;
+                const pool = getPool();
+                const setup = (raw, kind) => {
+                    if (!raw) return;
+                    raw.split(/[\s,]+/).filter(Boolean).forEach(id => {
+                        if (!document.querySelector(`[data-placeholder-id="${id}"]`)) {
+                            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}" class="ezoic-ad"></div>`;
+                            pool.appendChild(d);
+                        }
+                    });
+                };
+                setup(config.placeholderIds, 'between');
+                setup(config.messagePlaceholderIds, 'message');
+                
+                redistribute();
+
+                // On surveille les changements de page et l'infinite scroll
+                const observer = new MutationObserver(() => {
+                    if (!isInternalChange) redistribute();
+                });
+                observer.observe(document.body, { childList: true, subtree: true });
+            });
+    }
+
+    if (document.readyState === 'loading') {
+        document.addEventListener('DOMContentLoaded', init);
+    } else {
+        init();
+    }
 })();

package/public/style.css

@@ -1,35 +1,20 @@
-/* Container principal */
+/* Style du container injecté */
 .nodebb-ezoic-wrap {
-  display: block;
-  width: 100%;
-  margin: 20px 0 !important;
-  padding: 0 !important;
-  clear: both;
-  min-height: 50px; /* Important pour que l'ad-tester détecte le bloc */
+    display: block !important;
+    width: 100% !important;
+    margin: 30px 0 !important;
+    min-height: 100px; /* Force un espace pour qu'Ezoic puisse injecter */
+    clear: both;
 }
 
-/* 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;
-}
-
-/* Nettoyage des styles internes Ezoic */
-.nodebb-ezoic-wrap .ezoic-ad {
-  margin: 0 auto !important;
-  padding: 0 !important;
-  min-height: 1px !important;
-  height: auto !important;
-}
-
-/* Neutralise les marges doubles */
+/* ANTI-PILLUP : Interdit deux pubs consécutives */
 .nodebb-ezoic-wrap + .nodebb-ezoic-wrap {
-  margin-top: 0 !important;
+    display: none !important;
 }
 
-/* Évite les débordements sur mobile */
-.ezoic-ad iframe {
-  max-width: 100% !important;
+/* Alignement Harmony */
+[component="category/topic"] + .nodebb-ezoic-wrap,
+[component="post"] + .nodebb-ezoic-wrap {
+    border-top: 1px solid rgba(0,0,0,0.05);
+    padding: 15px 0;
 }

package/public/test.txt

@@ -1 +0,0 @@
-hi