nodebb-plugin-ezoic-infinite 1.6.91 → 1.6.93

package/package.json

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

package/public/client.js

@@ -9,8 +9,12 @@
   
   let config = null;
   let isInternalChange = false;
+  
+  // Registre global pour ne JAMAIS redéfinir un ID déjà vu par Ezoic
+  window.ezoicDefinedIds = window.ezoicDefinedIds || new Set();
+  
   let pendingIds = new Set();
-  let refreshTimer = null;
+  let triggerTimer = null;
 
   function getPool() {
     let p = document.getElementById(POOL_ID);
@@ -27,18 +31,24 @@
     if (typeof window.ezstandalone === 'undefined' || pendingIds.size === 0) return;
 
     window.ezstandalone.cmd.push(function() {
-      const idsToProcess = Array.from(pendingIds);
+      const allPending = Array.from(pendingIds);
       pendingIds.clear();
 
-      // On enregistre les IDs
-      window.ezstandalone.define(idsToProcess);
+      // Filtrer pour ne garder que les IDs JAMAIS définis auparavant
+      const newIdsToDefine = allPending.filter(id => !window.ezoicDefinedIds.has(id));
+
+      if (newIdsToDefine.length > 0) {
+        window.ezstandalone.define(newIdsToDefine);
+        newIdsToDefine.forEach(id => window.ezoicDefinedIds.add(id));
+      }
 
-      // On appelle UNIQUEMENT refresh. 
-      // Si Ezoic n'est pas "enabled", il ignorera l'appel sans erreur bloquante.
+      // La doc dit : showAds(id1, id2) pour le nouveau contenu
+      // On appelle showAds sur tous les IDs qu'on vient d'injecter, 
+      // qu'ils soient nouveaux ou recyclés.
       try {
-          window.ezstandalone.refresh();
+        window.ezstandalone.showAds.apply(null, allPending);
       } catch (e) {
-          console.debug('[Ezoic] Refresh deferred');
+        console.warn('[Ezoic] showAds failed', e);
       }
     });
   }
@@ -47,29 +57,28 @@
     const pid = parseInt(id, 10);
     if (isNaN(pid)) return;
     pendingIds.add(pid);
-
-    clearTimeout(refreshTimer);
-    refreshTimer = setTimeout(triggerEzoic, 300);
+    
+    clearTimeout(triggerTimer);
+    triggerTimer = setTimeout(triggerEzoic, 150);
   }
 
   function redistribute() {
     if (!config || config.excluded) return;
 
-    // 1. Accueil (Catégories)
-    const categoryItems = document.querySelectorAll('.category-item, [component="categories/category"]');
-    // 2. Liste Topics
-    const topicItems = document.querySelectorAll('li[component="category/topic"]');
-    // 3. Messages
-    const postItems = document.querySelectorAll('[component="post"]');
+    const selectors = {
+      'home': '.category-item, [component="categories/category"]',
+      'topic-list': 'li[component="category/topic"]',
+      'message': '[component="post"]'
+    };
 
-    if (categoryItems.length > 0 && config.enableCategoryAds) {
-      process(Array.from(categoryItems), 'home', config.intervalCategories, config.showFirstCategoryAd);
+    if (config.enableCategoryAds) {
+      process(Array.from(document.querySelectorAll(selectors.home)), 'home', config.intervalCategories, config.showFirstCategoryAd);
     }
-    if (topicItems.length > 0 && config.enableBetweenAds) {
-      process(Array.from(topicItems), 'topic-list', config.intervalPosts, config.showFirstTopicAd);
+    if (config.enableBetweenAds) {
+      process(Array.from(document.querySelectorAll(selectors['topic-list'])), 'topic-list', config.intervalPosts, config.showFirstTopicAd);
     }
-    if (postItems.length > 0 && config.enableMessageAds) {
-      process(Array.from(postItems), 'message', config.messageIntervalPosts, config.showFirstMessageAd);
+    if (config.enableMessageAds) {
+      process(Array.from(document.querySelectorAll(selectors.message)), 'message', config.messageIntervalPosts, config.showFirstMessageAd);
     }
   }
 
@@ -79,14 +88,16 @@
       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 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);
+          setTimeout(() => { isInternalChange = false; }, 50);
         }
       }
     });
@@ -101,17 +112,21 @@
         const setup = (raw, kind) => {
           if (!raw) return;
           raw.split(/[\s,]+/).filter(Boolean).forEach(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}"></div>`;
-            pool.appendChild(d);
+            // Créer le placeholder dans le pool s'il n'existe pas encore physiquement
+            if (!pool.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}"></div>`;
+              pool.appendChild(d);
+            }
           });
         };
         setup(config.categoryPlaceholderIds, 'home');
         setup(config.placeholderIds, 'topic-list');
         setup(config.messagePlaceholderIds, 'message');
+        
         redistribute();
 
         const observer = new MutationObserver(() => {
@@ -121,7 +136,12 @@
       });
   }
 
-  window.addEventListener('action:ajaxify.end', () => setTimeout(redistribute, 500));
+  // Navigation NodeBB (Ajaxify)
+  window.addEventListener('action:ajaxify.end', () => {
+    // On ne détruit RIEN au changement de page pour éviter les erreurs "already defined"
+    // On se contente de ré-injecter les placeholders du pool
+    setTimeout(redistribute, 500);
+  });
 
   if (document.readyState === 'loading') {
     document.addEventListener('DOMContentLoaded', init);