nodebb-plugin-ezoic-infinite 1.4.64 → 1.4.66

package/package.json

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

package/public/client.js

@@ -8,7 +8,6 @@
   }, 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 = {
@@ -106,9 +105,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;
@@ -128,6 +125,22 @@
   } catch (e) {}
   }
 
+  function cleanupEmptyWrappers() {
+  try {
+  document.querySelectorAll('.ezoic-ad').forEach(wrapper => {
+  const ph = wrapper.querySelector('[id^="ezoic-pub-ad-placeholder-"]');
+  if (ph && ph.children.length === 0) {
+  // Placeholder vide après 3s = pub non chargée
+  setTimeout(() => {
+  if (ph.children.length === 0) {
+  wrapper.remove();
+  }
+  }, 3000);
+  }
+  });
+  } catch (e) {}
+  }
+
   function getRecyclable(liveArr) {
   const margin = 600;
   for (let i = 0; i < liveArr.length; i++) {
@@ -199,10 +212,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;
 
@@ -215,7 +226,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);
   }
   }
@@ -226,8 +236,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;
@@ -254,7 +262,6 @@
   };
 
   applyPatch();
-  // Si Ezoic n'est pas encore chargé, appliquer le patch via sa cmd queue
   if (!window.__nodebbEzoicPatched) {
   try {
   window.ezstandalone = window.ezstandalone || {};
@@ -267,7 +274,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) {}
@@ -295,7 +301,6 @@
   }
   });
   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) {}
   }
@@ -315,15 +320,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]);
@@ -338,7 +340,6 @@
   // Ajouter à la batch
   pendingShowAdsIds.add(id);
 
-  // Debounce: attendre 100ms pour collecter tous les IDs
   clearTimeout(batchShowAdsTimer);
   batchShowAdsTimer = setTimeout(() => {
   if (pendingShowAdsIds.size === 0) return;
@@ -388,19 +389,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;
@@ -466,7 +463,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;
   }
@@ -483,17 +479,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);
@@ -507,12 +500,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);
@@ -529,7 +520,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) {
@@ -550,8 +540,6 @@
   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) {}
   });
@@ -567,23 +555,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; }
@@ -600,7 +579,6 @@
   }
 
   async function runCore() {
-  // Attendre que canInsert soit true (protection race condition navigation)
   if (!state.canShowAds) {
   return;
   }
@@ -643,7 +621,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;
@@ -655,12 +632,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);
@@ -694,15 +668,11 @@
   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', () => {
@@ -732,7 +702,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;
@@ -742,7 +711,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;
@@ -773,7 +741,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,18 @@
-.ezoic-ad{height:auto !important; min-height:0 !important; padding:0 !important; margin:0.5rem 0;}
+.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;}
+
+/* Cacher placeholder vide pour éviter espace réservé */
+.ezoic-ad [id^="ezoic-pub-ad-placeholder-"]:empty {
+  display: none !important;
+  height: 0 !important;
+  margin: 0 !important;
+  padding: 0 !important;
+}
+
+/* Wrapper vide aussi caché */
+.ezoic-ad:has([id^="ezoic-pub-ad-placeholder-"]:empty) {
+  display: none !important;
+  height: 0 !important;
+  margin: 0 !important;
+}