nodebb-plugin-ezoic-infinite 1.4.39 → 1.4.41

package/package.json

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

package/public/client.js

@@ -39,6 +39,8 @@
   timer: null,
 
   obs: null,
+  canShowAds: true,
+  activeTimeouts: new Set(),
   lastScrollRun: 0,
   __scrollBound: false,
   };
@@ -325,10 +327,6 @@
 
   function callShowAdsWhenReady(id) {
   if (!id) return;
-  
-  if (window.location.pathname !== '/') {
-  console.log('[TIMING] callShowAdsWhenReady ID:', id);
-  }
 
   const now = Date.now(), last = state.lastShowById.get(id) || 0;
   if (now - last < 3500) return;
@@ -352,25 +350,19 @@
   if (state.pageKey !== startPageKey) return;
   // Abort if another concurrent call is already handling this id
   if (state.pendingById.has(id)) return;
-  
+
   attempts += 1;
   const el = document.getElementById(phId);
   if (el && el.isConnected) {
-  if (window.location.pathname !== '/') {
-  console.log('[TIMING] Placeholder', id, 'trouvé dans DOM, appel showAds');
-  }
-  
   // Double-check pageKey juste avant doCall (au cas où cleanup appelé entre-temps)
   if (state.pageKey !== startPageKey) {
-  if (window.location.pathname !== '/') {
-  console.log('[TIMING] ABORT ID', id, '- navigation détectée avant showAds');
-  }
+  // Note: les timeouts seront nettoyés par cleanup()
   return;
   }
-  
+
   // Si doCall() réussit, Ezoic est chargé et showAds a été appelé → sortir
   if (doCall()) {
-  state.pendingById.delete(id); // nettoyage au cas où
+  state.pendingById.delete(id);
   return;
   }
 
@@ -392,7 +384,10 @@
   return;
   }
 
-  if (attempts < 100) setTimeout(waitForPh, 50);
+  if (attempts < 100) {
+  const timeoutId = setTimeout(waitForPh, 50);
+  state.activeTimeouts.add(timeoutId);
+  }
   })();
   }
 
@@ -556,8 +551,13 @@
   // quand NodeBB vide le DOM lors de la navigation ajaxify
   // Les supprimer manuellement cause des problèmes avec l'état interne d'Ezoic
 
+  // CRITIQUE: Annuler TOUS les timeouts en cours pour éviter que les anciens
+  // showAds() continuent à s'exécuter après la navigation
+  state.activeTimeouts.forEach(id => clearTimeout(id));
+  state.activeTimeouts.clear();
+
   if (state.obs) { state.obs.disconnect(); state.obs = null; }
-  state.canInsert = false;
+  state.canShowAds = false;
   state.scheduled = false;
   clearTimeout(state.timer);
   state.timer = null;
@@ -571,10 +571,10 @@
 
   async function runCore() {
   // Attendre que canInsert soit true (protection race condition navigation)
-  if (!state.canInsert) {
+  if (!state.canShowAds) {
   return;
   }
-  
+
   patchShowAds();
 
   const cfg = await fetchConfig();
@@ -613,7 +613,6 @@
 
   enforceNoAdjacentAds();
 
-
   // If nothing inserted and list isn't in DOM yet (first click), retry a bit
   let count = 0;
   if (kind === 'topic') count = getPostContainers().length;
@@ -664,14 +663,14 @@
   $(window).on('action:ajaxify.end.ezoicInfinite', () => {
   state.pageKey = getPageKey();
   ensureObserver();
-  
+
   // 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.canInsert = false;
+  state.canShowAds = false;
   setTimeout(() => {
-  state.canInsert = true;
-  }, 300);
+  state.canShowAds = true;
+  }, 500);
   });
 
   $(window).on('action:category.loaded.ezoicInfinite', () => {