nodebb-plugin-ezoic-infinite 1.4.94 → 1.4.95

package/package.json

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

package/public/client.js

@@ -8,6 +8,7 @@
   }, WRAP_CLASS = 'ezoic-ad';
   const PLACEHOLDER_PREFIX = 'ezoic-pub-ad-placeholder-', MAX_INSERTS_PER_RUN = 3;
 
+  // Nécessaire pour savoir si on doit appeler destroyPlaceholders avant recyclage.
   const sessionDefinedIds = new Set();
 
   const insertingIds = new Set(), state = {
@@ -105,7 +106,9 @@
 
   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;
@@ -116,11 +119,6 @@
   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 || {};
@@ -128,50 +126,10 @@
   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;
-  
-  // Supprimer TOUS les éléments après le placeholder rempli
-  // qui créent de l'espace vertical
-  let found = false;
-  Array.from(wrapper.children).forEach(child => {
-  if (child === ph || child.contains(ph)) {
-  found = true;
-  return;
-  }
-  
-  // Si élément APRÈS le placeholder
-  if (found) {
-  const rect = child.getBoundingClientRect();
-  const computed = window.getComputedStyle(child);
   
-  // Supprimer si:
-  // 1. Height > 0 mais pas de texte/image visible
-  // 2. Ou opacity: 0
-  // 3. Ou visibility: hidden
-  const hasContent = child.textContent.trim().length > 0 || 
-                    child.querySelector('img, iframe, video');
-  
-  if (!hasContent || computed.opacity === '0' || computed.visibility === 'hidden') {
-  child.remove();
-  }
-  }
-  });
-  });
-  } catch (e) {}
-  }
-
+  // Nettoyer les wrappers vides (sans pub) pour éviter espaces verticaux
   function cleanupEmptyWrappers() {
   try {
   document.querySelectorAll('.ezoic-ad').forEach(wrapper => {
@@ -182,7 +140,7 @@
   if (ph.children.length === 0) {
   wrapper.remove();
   }
-  }, 1500);
+  }, 3000);
   }
   });
   } catch (e) {}
@@ -259,8 +217,10 @@
   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;
 
@@ -273,6 +233,7 @@
   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);
   }
   }
@@ -283,6 +244,8 @@
   }
 
   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;
@@ -309,6 +272,7 @@
   };
 
   applyPatch();
+  // Si Ezoic n'est pas encore chargé, appliquer le patch via sa cmd queue
   if (!window.__nodebbEzoicPatched) {
   try {
   window.ezstandalone = window.ezstandalone || {};
@@ -321,6 +285,7 @@
   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) {}
@@ -336,18 +301,19 @@
   if (!ph) return;
   // Already filled?
   if (ph.childNodes && ph.childNodes.length > 0) {
-  markFilled(wrap); // Afficher wrapper
-  sessionDefinedIds.add(id);
+  markFilled(wrap);
+  state.definedIds && state.definedIds.add(id); sessionDefinedIds.add(id);
   return;
   }
   const obs = new MutationObserver(() => {
   if (ph.childNodes && ph.childNodes.length > 0) {
-  markFilled(wrap); // CRITIQUE: Afficher wrapper maintenant
-  try { sessionDefinedIds.add(id); } catch (e) {}
+  markFilled(wrap);
+  try { state.definedIds && state.definedIds.add(id); 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) {}
   }
@@ -367,12 +333,15 @@
   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]);
@@ -387,6 +356,7 @@
   // Ajouter à la batch
   pendingShowAdsIds.add(id);
 
+  // Debounce: attendre 100ms pour collecter tous les IDs
   clearTimeout(batchShowAdsTimer);
   batchShowAdsTimer = setTimeout(() => {
   if (pendingShowAdsIds.size === 0) return;
@@ -410,11 +380,6 @@
   }
   });
   } catch (e) {}
-  
-  // CRITIQUE: Nettoyer éléments invisibles APRÈS que pubs soient chargées
-  setTimeout(() => {
-  cleanupInvisibleEzoicElements();
-  }, 800); // 1.5s pour laisser Ezoic charger
   }, 100);
   }
 
@@ -441,15 +406,19 @@
   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;
@@ -515,6 +484,7 @@
   const el = items[afterPos - 1];
   if (!el || !el.isConnected) continue;
 
+  // Prevent adjacent ads (DOM-based, robust against virtualization)
   if (isAdjacentAd(el) || isPrevAd(el)) {
   continue;
   }
@@ -531,14 +501,17 @@
 
   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);
@@ -552,10 +525,12 @@
   }
 
   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);
@@ -572,6 +547,7 @@
   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) {
@@ -592,6 +568,8 @@
   function cleanup() {
   destroyUsedPlaceholders();
 
+  // 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 => {
   try { el.remove(); } catch (e) {}
   });
@@ -607,14 +585,23 @@
   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; }
@@ -631,6 +618,7 @@
   }
 
   async function runCore() {
+  // Attendre que canInsert soit true (protection race condition navigation)
   if (!state.canShowAds) {
   return;
   }
@@ -673,6 +661,7 @@
 
   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;
@@ -684,9 +673,12 @@
   }
 
   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);
@@ -720,11 +712,15 @@
   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.canShowAds = true;
   });
 
   $(window).on('action:category.loaded.ezoicInfinite', () => {
   ensureObserver();
+  // category.loaded = infinite scroll, Ezoic déjà chargé normalement
   waitForContentThenRun();
   });
   $(window).on('action:topics.loaded.ezoicInfinite', () => {
@@ -754,6 +750,7 @@
   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;
@@ -763,6 +760,7 @@
   }, { 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;
@@ -793,6 +791,7 @@
   })();
   }
 
+  // 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,10 +1,3 @@
-.ezoic-ad {
-  height: auto !important;
-  padding: 0 !important;
-  margin: 0 !important;
-}
-
-.ezoic-ad * {
-  margin: 0 !important;
-  padding: 0 !important;
-}
+.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;}