nodebb-plugin-ezoic-infinite 1.4.47 → 1.4.49

package/package.json

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

package/public/client.js

@@ -26,11 +26,6 @@
   usedPosts: new Set(),
   usedCategories: new Set(),
 
-  // wrappers currently on page (FIFO for recycling)
-  liveBetween: [],
-  liveMessage: [],
-  liveCategory: [],
-
   lastShowById: new Map(),
   pendingById: new Set(),
   definedIds: new Set(),
@@ -39,7 +34,6 @@
   timer: null,
 
   obs: null,
-  canShowAds: true,
   activeTimeouts: new Set(),
   lastScrollRun: 0,
   __scrollBound: false,
@@ -325,6 +319,56 @@
   return filled;
   }
 
+  // Appeler showAds() en batch selon recommandations Ezoic
+  // Au lieu de showAds(id1), showAds(id2)... faire showAds(id1, id2, id3...)
+  let batchShowAdsTimer = null;
+  const pendingShowAdsIds = new Set();
+
+  function scheduleShowAdsBatch(id) {
+  if (!id) return;
+
+  // CRITIQUE: Si cet ID a déjà été défini (sessionDefinedIds), le détruire d'abord
+  if (sessionDefinedIds.has(id)) {
+  try {
+  destroyPlaceholderIds([id]);
+  sessionDefinedIds.delete(id);
+  } catch (e) {}
+  }
+
+  // Throttle: ne pas rappeler le même ID trop vite
+  const now = Date.now(), last = state.lastShowById.get(id) || 0;
+  if (now - last < 3500) return;
+
+  // Ajouter à la batch
+  pendingShowAdsIds.add(id);
+
+  // Debounce: attendre 100ms pour collecter tous les IDs
+  clearTimeout(batchShowAdsTimer);
+  batchShowAdsTimer = setTimeout(() => {
+  if (pendingShowAdsIds.size === 0) return;
+
+  const idsArray = Array.from(pendingShowAdsIds);
+  pendingShowAdsIds.clear();
+
+  // Appeler showAds avec TOUS les IDs en une fois
+  try {
+  window.ezstandalone = window.ezstandalone || {};
+  window.ezstandalone.cmd = window.ezstandalone.cmd || [];
+  window.ezstandalone.cmd.push(function() {
+  if (typeof window.ezstandalone.showAds === 'function') {
+  // Appel batch: showAds(id1, id2, id3...)
+  window.ezstandalone.showAds(...idsArray);
+  // Tracker tous les IDs
+  idsArray.forEach(id => {
+  state.lastShowById.set(id, Date.now());
+  sessionDefinedIds.add(id);
+  });
+  }
+  });
+  } catch (e) {}
+  }, 100);
+  }
+
   function callShowAdsWhenReady(id) {
   if (!id) return;
 
@@ -357,10 +401,8 @@
   const el = document.getElementById(phId);
   if (el && el.isConnected) {
   // CRITIQUE: Vérifier que le placeholder est VISIBLE
-  
-  
+
   // Si on arrive ici, soit visible, soit timeout
-  
 
   // Si doCall() réussit, Ezoic est chargé et showAds a été appelé → sortir
   if (doCall()) {
@@ -368,23 +410,6 @@
   return;
   }
 
-  // Ezoic n'est pas encore chargé → attendre via cmd queue
-  state.pendingById.add(id);
-
-  window.ezstandalone = window.ezstandalone || {};
-  window.ezstandalone.cmd = window.ezstandalone.cmd || [];
-  window.ezstandalone.cmd.push(() => {
-  try {
-  if (typeof window.ezstandalone.showAds === 'function') {
-  state.pendingById.delete(id);
-  state.lastShowById.set(id, Date.now());
-  window.ezstandalone.showAds(id);
-  sessionDefinedIds.add(id);
-  }
-  } catch (e) {}
-  });
-  // cmd.push suffit - pas besoin de tick() qui crée des doublons
-  return;
   }
 
   if (attempts < 100) {
@@ -540,10 +565,6 @@
   state.usedTopics.clear();
   state.usedPosts.clear();
   state.usedCategories.clear();
-  state.liveBetween = [];
-  state.liveMessage = [];
-  state.liveCategory = [];
-
   state.lastShowById.clear();
   // CRITIQUE: Vider pendingById pour annuler tous les showAds en cours
   // Sinon Ezoic essaie d'accéder aux placeholders pendant que NodeBB vide le DOM
@@ -560,12 +581,12 @@
   try { clearTimeout(id); } catch (e) {}
   });
   state.activeTimeouts.clear();
-  
+
   // Vider aussi pendingById pour annuler les showAds en attente
   state.pendingById.clear();
 
   if (state.obs) { state.obs.disconnect(); state.obs = null; }
-  state.canShowAds = false;
+
   state.scheduled = false;
   clearTimeout(state.timer);
   state.timer = null;
@@ -675,10 +696,7 @@
   // CRITIQUE: Attendre 300ms avant de permettre l'insertion de nouveaux placeholders
   // pour laisser les anciens showAds() (en cours) se terminer ou échouer proprement
   // Sinon race condition: NodeBB vide le DOM pendant que Ezoic essaie d'accéder aux placeholders
-  state.canShowAds = false;
-  setTimeout(() => {
   state.canShowAds = true;
-  }, 2000);
   });
 
   $(window).on('action:category.loaded.ezoicInfinite', () => {
@@ -705,8 +723,8 @@
   }
 
   function bindScroll() {
-  if (state.__scrollBound) return;
-  state.__scrollBound = true;
+  if (state.lastScrollRun > 0) return;
+  state.lastScrollRun = Date.now();
   let ticking = false;
   window.addEventListener('scroll', () => {
   if (ticking) return;