nodebb-plugin-ezoic-infinite 1.6.92 → 1.6.93

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.6.92",
+  "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,7 +9,10 @@
   
   let config = null;
   let isInternalChange = false;
-  let activeIdsOnPage = new Set();
+  
+  // 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 triggerTimer = null;
 
@@ -28,24 +31,25 @@
     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();
 
-      // 1. Définir les IDs (requis par Ezoic avant l'affichage)
-      window.ezstandalone.define(idsToProcess);
-
-      if (!window.ezstandalone.enabled) {
-        // PREMIER CHARGEMENT
-        window.ezstandalone.enable();
-        window.ezstandalone.showAds(); // Appelle tout ce qui est présent
-      } else {
-        // NOUVEAU CONTENU (INFINITE SCROLL)
-        // La doc dit : ezstandalone.showAds(104, 105);
-        window.ezstandalone.showAds.apply(null, 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));
+      }
+
+      // 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.showAds.apply(null, allPending);
+      } catch (e) {
+        console.warn('[Ezoic] showAds failed', e);
       }
-      
-      // On garde trace pour le nettoyage futur
-      idsToProcess.forEach(id => activeIdsOnPage.add(id));
     });
   }
 
@@ -54,22 +58,8 @@
     if (isNaN(pid)) return;
     pendingIds.add(pid);
     
-    // On attend un peu pour grouper si plusieurs pubs arrivent en même temps
     clearTimeout(triggerTimer);
-    triggerTimer = setTimeout(triggerEzoic, 200);
-  }
-
-  // "Changing Pages" : Nettoyage complet lors de la navigation NodeBB
-  function onPageChangeStart() {
-    if (typeof window.ezstandalone !== 'undefined') {
-      window.ezstandalone.cmd.push(function() {
-        // On nettoie tout avant de charger la nouvelle page
-        if (typeof window.ezstandalone.destroyAll === 'function') {
-          window.ezstandalone.destroyAll();
-        }
-        activeIdsOnPage.clear();
-      });
-    }
+    triggerTimer = setTimeout(triggerEzoic, 150);
   }
 
   function redistribute() {
@@ -122,6 +112,7 @@
         const setup = (raw, kind) => {
           if (!raw) return;
           raw.split(/[\s,]+/).filter(Boolean).forEach(id => {
+            // 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;
@@ -145,9 +136,10 @@
       });
   }
 
-  // Événements NodeBB
-  window.addEventListener('action:ajaxify.start', onPageChangeStart);
+  // 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);
   });