nodebb-plugin-ezoic-infinite 1.4.95 → 1.4.96

package/package.json

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

package/public/client.js

@@ -7,8 +7,12 @@
   categoryItem: 'li[component="categories/category"]',
   }, WRAP_CLASS = 'ezoic-ad';
   const PLACEHOLDER_PREFIX = 'ezoic-pub-ad-placeholder-', MAX_INSERTS_PER_RUN = 3;
+  
+  // FLAG GLOBAL: Bloquer Ezoic pendant navigation
+  let EZOIC_BLOCKED = false;
+  
+  // DEBUG: Vérifier que le plugin charge
 
-  // Nécessaire pour savoir si on doit appeler destroyPlaceholders avant recyclage.
   const sessionDefinedIds = new Set();
 
   const insertingIds = new Set(), state = {
@@ -106,9 +110,7 @@
 
   function destroyPlaceholderIds(ids) {
   if (!ids || !ids.length) return;
-  // Only destroy ids that were actually "defined" (filled at least once) to avoid Ezoic warnings.
   const filtered = ids.filter((id) => {
-  // Utiliser sessionDefinedIds (survit aux navigations) plutôt que state.definedIds
   try { return sessionDefinedIds.has(id); } catch (e) { return true; }
   });
   if (!filtered.length) return;
@@ -119,6 +121,11 @@
   window.ezstandalone.destroyPlaceholders(filtered);
   }
   } catch (e) {}
+
+  // Recyclage: libérer IDs après 100ms
+  setTimeout(() => {
+  filtered.forEach(id => sessionDefinedIds.delete(id));
+  }, 100);
   };
   try {
   window.ezstandalone = window.ezstandalone || {};
@@ -126,10 +133,63 @@
   if (window.ezstandalone && typeof window.ezstandalone.destroyPlaceholders === 'function') call();
   else window.ezstandalone.cmd.push(call);
   } catch (e) {}
+
+  // Recyclage: libérer IDs après 100ms
+  setTimeout(() => {
+  filtered.forEach(id => sessionDefinedIds.delete(id));
+  }, 100);
   }
 
+  // Nettoyer éléments Ezoic invisibles qui créent espace vertical
+  function cleanupInvisibleEzoicElements() {
+  try {
+  document.querySelectorAll('.ezoic-ad').forEach(wrapper => {
+  const ph = wrapper.querySelector('[id^="ezoic-pub-ad-placeholder-"]');
+  if (!ph) return;
+  
+  // ULTRA-AGRESSIF: Supprimer TOUT sauf placeholder
+  Array.from(wrapper.children).forEach(child => {
+  if (child === ph || child.contains(ph)) return;
+  if (child.id && child.id.startsWith('ezoic-pub-ad-placeholder-')) return;
+  child.remove();
+  });
   
-  // Nettoyer les wrappers vides (sans pub) pour éviter espaces verticaux
+  // Forcer wrapper collé
+  wrapper.style.height = 'auto';
+  wrapper.style.overflow = 'hidden';
+  wrapper.style.lineHeight = '0';
+  wrapper.style.fontSize = '0';
+  wrapper.style.margin = '0';
+  wrapper.style.padding = '0';
+  
+  // CRITIQUE: Forcer suppression margin sur TOUS les .ezoic-ad enfants
+  // Pour overrider les margin-top/bottom:15px !important d'Ezoic
+  wrapper.querySelectorAll('.ezoic-ad, span.ezoic-ad').forEach(ezSpan => {
+  ezSpan.style.setProperty('margin-top', '0', 'important');
+  ezSpan.style.setProperty('margin-bottom', '0', 'important');
+  ezSpan.style.setProperty('margin-left', '0', 'important');
+  ezSpan.style.setProperty('margin-right', '0', 'important');
+  });
+  
+  // Nettoyer aussi DANS le placeholder
+  if (ph) {
+  Array.from(ph.children).forEach(child => {
+  // Garder seulement iframe, ins, img
+  const tag = child.tagName;
+  if (tag !== 'IFRAME' && tag !== 'INS' && tag !== 'IMG' && tag !== 'DIV') {
+  child.remove();
+  }
+  });
+  }
+  });
+  } catch (e) {}
+  }
+  
+  // Lancer cleanup périodique toutes les 2s
+  setInterval(() => {
+  try { cleanupInvisibleEzoicElements(); } catch (e) {}
+  }, 2000);
+
   function cleanupEmptyWrappers() {
   try {
   document.querySelectorAll('.ezoic-ad').forEach(wrapper => {
@@ -140,7 +200,7 @@
   if (ph.children.length === 0) {
   wrapper.remove();
   }
-  }, 3000);
+  }, 1500);
   }
   });
   } catch (e) {}
@@ -217,10 +277,8 @@
   if (findWrap(kindClass, afterPos)) return null;
 
   // CRITICAL: Double-lock pour éviter race conditions sur les doublons
-  // 1. Vérifier qu'aucun autre thread n'est en train d'insérer cet ID
   if (insertingIds.has(id)) return null;
 
-  // 2. Vérifier qu'aucun placeholder avec cet ID n'existe déjà dans le DOM
   const existingPh = document.getElementById(`${PLACEHOLDER_PREFIX}${id}`);
   if (existingPh && existingPh.isConnected) return null;
 
@@ -233,7 +291,6 @@
   attachFillObserver(wrap, id);
   return wrap;
   } finally {
-  // Libérer le lock après 100ms (le temps que le DOM soit stable)
   setTimeout(() => insertingIds.delete(id), 50);
   }
   }
@@ -244,8 +301,6 @@
   }
 
   function patchShowAds() {
-  // Minimal safety net: batch showAds can be triggered by other scripts; split into individual calls.
-  // Also ensures the patch is applied even if Ezoic loads after our script.
   const applyPatch = () => {
   try {
   window.ezstandalone = window.ezstandalone || {}, ez = window.ezstandalone;
@@ -256,23 +311,50 @@
   const orig = ez.showAds;
 
   ez.showAds = function (arg) {
+  // CRITIQUE: Bloquer TOUS appels si navigation en cours
+  if (EZOIC_BLOCKED) {
+  return; // Ignorer complètement
+  }
+  
+  // Filtrer IDs dont placeholders n'existent pas
+  const filterValidIds = (ids) => {
+  return ids.filter(id => {
+  const phId = `ezoic-pub-ad-placeholder-${id}`;
+  const ph = document.getElementById(phId);
+  return ph && ph.isConnected;
+  });
+  };
+  
   if (Array.isArray(arg)) {
   const seen = new Set();
   for (const v of arg) {
   const id = parseInt(v, 10);
   if (!Number.isFinite(id) || id <= 0 || seen.has(id)) continue;
+  
+  // Vérifier existence avant appel
+  const phId = `ezoic-pub-ad-placeholder-${id}`;
+  const ph = document.getElementById(phId);
+  if (!ph || !ph.isConnected) continue;
+  
   seen.add(id);
   try { orig.call(ez, id); } catch (e) {}
   }
   return;
   }
+  
+  // Pour appels simples, vérifier aussi
+  if (typeof arg === 'number') {
+  const phId = `ezoic-pub-ad-placeholder-${arg}`;
+  const ph = document.getElementById(phId);
+  if (!ph || !ph.isConnected) return;
+  }
+  
   return orig.apply(ez, arguments);
   };
   } catch (e) {}
   };
 
   applyPatch();
-  // Si Ezoic n'est pas encore chargé, appliquer le patch via sa cmd queue
   if (!window.__nodebbEzoicPatched) {
   try {
   window.ezstandalone = window.ezstandalone || {};
@@ -285,7 +367,6 @@
   function markFilled(wrap) {
   try {
   if (!wrap) return;
-  // Disconnect the fill observer first (no need to remove+re-add the attribute)
   if (wrap.__ezoicFillObs) { wrap.__ezoicFillObs.disconnect(); wrap.__ezoicFillObs = null; }
   wrap.setAttribute('data-ezoic-filled', '1');
   } catch (e) {}
@@ -301,19 +382,18 @@
   if (!ph) return;
   // Already filled?
   if (ph.childNodes && ph.childNodes.length > 0) {
-  markFilled(wrap);
-  state.definedIds && state.definedIds.add(id); sessionDefinedIds.add(id);
+  markFilled(wrap); // Afficher wrapper
+  sessionDefinedIds.add(id);
   return;
   }
   const obs = new MutationObserver(() => {
   if (ph.childNodes && ph.childNodes.length > 0) {
-  markFilled(wrap);
-  try { state.definedIds && state.definedIds.add(id); sessionDefinedIds.add(id); } catch (e) {}
+  markFilled(wrap); // CRITIQUE: Afficher wrapper maintenant
+  try { sessionDefinedIds.add(id); } catch (e) {}
   try { obs.disconnect(); } catch (e) {}
   }
   });
   obs.observe(ph, { childList: true, subtree: true });
-  // Keep a weak reference on the wrapper so we can disconnect on recycle/remove
   wrap.__ezoicFillObs = obs;
   } catch (e) {}
   }
@@ -333,15 +413,12 @@
   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]);
@@ -356,30 +433,52 @@
   // Ajouter à la batch
   pendingShowAdsIds.add(id);
 
-  // Debounce: attendre 100ms pour collecter tous les IDs
   clearTimeout(batchShowAdsTimer);
   batchShowAdsTimer = setTimeout(() => {
+  // CRITIQUE: Vérifier que nous sommes toujours sur la même page
+  const currentPageKey = getPageKey();
+  if (state.pageKey && currentPageKey !== state.pageKey) {
+  pendingShowAdsIds.clear();
+  return; // Page a changé, annuler
+  }
+  
   if (pendingShowAdsIds.size === 0) return;
 
   const idsArray = Array.from(pendingShowAdsIds);
   pendingShowAdsIds.clear();
 
+  // CRITIQUE: Vérifier que placeholders existent encore
+  const validIds = idsArray.filter(id => {
+  const ph = document.getElementById(`${PLACEHOLDER_PREFIX}${id}`);
+  return ph && ph.isConnected;
+  });
+  
+  if (validIds.length === 0) return;
+
   // Appeler showAds avec TOUS les IDs en une fois
   try {
+  // CRITIQUE: Re-patcher AVANT chaque appel pour être sûr
+  patchShowAds();
+  
   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);
+  window.ezstandalone.showAds(...validIds);
   // Tracker tous les IDs
-  idsArray.forEach(id => {
+  validIds.forEach(id => {
   state.lastShowById.set(id, Date.now());
   sessionDefinedIds.add(id);
   });
   }
   });
   } catch (e) {}
+  
+  // CRITIQUE: Nettoyer éléments invisibles APRÈS que pubs soient chargées
+  setTimeout(() => {
+  cleanupInvisibleEzoicElements();
+  }, 1500); // 1.5s pour laisser Ezoic charger
   }, 100);
   }
 
@@ -406,19 +505,15 @@
   const startPageKey = state.pageKey;
   let attempts = 0;
   (function waitForPh() {
-  // Abort if the user navigated away since this showAds was scheduled
   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) {
-  // 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()) {
   state.pendingById.delete(id);
   return;
@@ -484,7 +579,6 @@
   const el = items[afterPos - 1];
   if (!el || !el.isConnected) continue;
 
-  // Prevent adjacent ads (DOM-based, robust against virtualization)
   if (isAdjacentAd(el) || isPrevAd(el)) {
   continue;
   }
@@ -501,17 +595,14 @@
 
   let wrap = null;
   if (pick.recycled && pick.recycled.wrap) {
-  // Only destroy if Ezoic has actually defined this placeholder before
   if (sessionDefinedIds.has(id)) {
   destroyPlaceholderIds([id]);
   }
-  // Remove the old wrapper entirely, then create a fresh wrapper at the new position (same id)
   const oldWrap = pick.recycled.wrap;
   try { if (oldWrap && oldWrap.__ezoicFillObs) { oldWrap.__ezoicFillObs.disconnect(); } } catch (e) {}
   try { oldWrap && oldWrap.remove(); } catch (e) {}
   wrap = insertAfter(el, id, kindClass, afterPos);
   if (!wrap) continue;
-  // Attendre que le wrapper soit dans le DOM puis appeler showAds
   setTimeout(() => {
   callShowAdsWhenReady(id);
   }, 50);
@@ -525,12 +616,10 @@
   }
 
   liveArr.push({ id, wrap });
-  // If adjacency ended up happening (e.g. DOM shifts), rollback this placement.
   if (wrap && (
   (wrap.previousElementSibling && wrap.previousElementSibling.classList && wrap.previousElementSibling.classList.contains(WRAP_CLASS)) || (wrap.nextElementSibling && wrap.nextElementSibling.classList && wrap.nextElementSibling.classList.contains(WRAP_CLASS))
   )) {
   try { wrap.remove(); } catch (e) {}
-  // Put id back if it was newly consumed (not recycled)
   if (!(pick.recycled && pick.recycled.wrap)) {
   try { kindPool.unshift(id); } catch (e) {}
   usedSet.delete(id);
@@ -547,7 +636,6 @@
   for (let i = 0; i < ads.length; i++) {
   const ad = ads[i], prev = ad.previousElementSibling;
   if (prev && prev.classList && prev.classList.contains(WRAP_CLASS)) {
-  // Supprimer le wrapper adjacent au lieu de le cacher
   try {
   const ph = ad.querySelector && ad.querySelector(`[id^="${PLACEHOLDER_PREFIX}"]`);
   if (ph) {
@@ -566,11 +654,45 @@
   }
 
   function cleanup() {
-  destroyUsedPlaceholders();
+  // DEBUG: Vérifier que cleanup est appelé
+  // CRITIQUE: BLOQUER Ezoic immédiatement
+  EZOIC_BLOCKED = true;
+  
+  // Détruire TOUS les placeholders Ezoic AVANT de supprimer DOM
+  const allWrappers = document.querySelectorAll('.ezoic-ad');
+  const allIds = [];
+  allWrappers.forEach(wrapper => {
+  const ph = wrapper.querySelector('[id^="ezoic-pub-ad-placeholder-"]');
+  if (ph) {
+  const match = ph.id.match(/\d+/);
+  if (match) allIds.push(parseInt(match[0]));
+  }
+  });
+  
+  // CRITIQUE: Vider COMPLÈTEMENT sessionDefinedIds
+  // Pour éviter que d'anciens IDs soient encore en mémoire
+  
+  // CRITIQUE: Détruire AUSSI tous les IDs tracés dans state
+  // Pour annuler les anciens IDs qu'Ezoic a en mémoire
+  const trackedIds = [
+  ...Array.from(state.usedTopics),
+  ...Array.from(state.usedPosts),
+  ...Array.from(state.usedCategories)
+  ];
+  
+  const allIdsToDestroy = [...new Set([...allIds, ...trackedIds, ...Array.from(sessionDefinedIds)])];
+  sessionDefinedIds.clear(); // ✅ VIDER TOUT
+  
+  if (allIdsToDestroy.length > 0) {
+  destroyPlaceholderIds(allIdsToDestroy);
+  }
+  
+  // Annuler batch showAds en attente
+  pendingShowAdsIds.clear();
+  clearTimeout(batchShowAdsTimer);
 
-  // CRITIQUE: Supprimer TOUS les wrappers .ezoic-ad du DOM
-  // Sinon ils restent et deviennent "unused" sur la nouvelle page
-  document.querySelectorAll('.ezoic-ad').forEach(el => {
+  // Maintenant supprimer DOM
+  allWrappers.forEach(el => {
   try { el.remove(); } catch (e) {}
   });
 
@@ -585,23 +707,14 @@
   state.usedPosts.clear();
   state.usedCategories.clear();
   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
   state.pendingById.clear();
   state.definedIds.clear();
 
-  // NE PAS supprimer les wrappers Ezoic ici - ils seront supprimés naturellement
-  // 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 => {
   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; }
@@ -618,7 +731,11 @@
   }
 
   async function runCore() {
-  // Attendre que canInsert soit true (protection race condition navigation)
+  // CRITIQUE: Ne rien insérer si navigation en cours
+  if (EZOIC_BLOCKED) {
+  return;
+  }
+  
   if (!state.canShowAds) {
   return;
   }
@@ -661,7 +778,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;
   else if (kind === 'categoryTopics') count = getTopicItems().length;
@@ -673,12 +789,9 @@
   }
 
   if (inserted >= MAX_INSERTS_PER_RUN) {
-  // Plus d'insertions possibles ce cycle, continuer immédiatement
   setTimeout(arguments[0], 50);
   } else if (inserted === 0 && count > 0) {
   // Pool épuisé ou recyclage pas encore disponible.
-  // Réessayer jusqu'à 8 fois (toutes les 400ms) pour laisser aux anciens wrappers
-  // le temps de défiler hors écran et devenir recyclables.
   if (state.poolWaitAttempts < 8) {
   state.poolWaitAttempts += 1;
   setTimeout(arguments[0], 50);
@@ -710,17 +823,20 @@
 
   $(window).on('action:ajaxify.end.ezoicInfinite', () => {
   state.pageKey = getPageKey();
+  
+  // Débloquer Ezoic IMMÉDIATEMENT pour la nouvelle page
+  EZOIC_BLOCKED = false;
+  
   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.canShowAds = true;
+  
+  // CRITIQUE: Relancer insertion maintenant que navigation est terminée
+  scheduleRun();
   });
 
   $(window).on('action:category.loaded.ezoicInfinite', () => {
   ensureObserver();
-  // category.loaded = infinite scroll, Ezoic déjà chargé normalement
   waitForContentThenRun();
   });
   $(window).on('action:topics.loaded.ezoicInfinite', () => {
@@ -750,7 +866,6 @@
   window.requestAnimationFrame(() => {
   ticking = false;
   enforceNoAdjacentAds();
-  // Debounce scheduleRun - une fois toutes les 2 secondes max au scroll
   const now = Date.now();
   if (!state.lastScrollRun || now - state.lastScrollRun > 2000) {
   state.lastScrollRun = now;
@@ -760,7 +875,6 @@
   }, { passive: true });
   }
 
-  // Fonction qui attend que la page ait assez de contenu avant d'insérer les pubs
   function waitForContentThenRun() {
   const MIN_WORDS = 250;
   let attempts = 0;
@@ -791,7 +905,6 @@
   })();
   }
 
-  // Fonction qui attend que Ezoic soit vraiment chargé
   function waitForEzoicThenRun() {
   let attempts = 0;
   const maxAttempts = 50; // 50 × 200ms = 10s max

package/public/style.css

@@ -1,3 +1,7 @@
-.ezoic-ad{height:auto !important; padding:0 !important; margin: 0.25rem 0;}
-.ezoic-ad .ezoic-ad-inner{padding:0;margin:0;}
-.ezoic-ad .ezoic-ad-inner > div{padding:0;margin:0;}
+.ezoic-ad,
+.ezoic-ad *,
+span.ezoic-ad,
+span[class*="ezoic"] {
+  min-height: 0 !important;
+  min-width: 0 !important;
+}