nodebb-plugin-ezoic-infinite 1.4.65 → 1.4.67

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.4.65",
+  "version": "1.4.67",
   "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;
@@ -119,6 +116,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 +128,37 @@
   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 => {
+  // Supprimer éléments invisibles après le placeholder
+  wrapper.querySelectorAll('*').forEach(el => {
+  if (el.id && el.id.startsWith('ezoic-pub-ad-placeholder-')) return;
+
+  const rect = el.getBoundingClientRect();
+  const computed = window.getComputedStyle(el);
+
+  // Élément invisible mais prend de l'espace
+  if (
+  (computed.display !== 'none' && computed.visibility !== 'hidden') &&
+  (rect.height === 0 || computed.opacity === '0') &&
+  el.children.length === 0
+  ) {
+  el.remove();
+  }
+  });
+  });
+  } catch (e) {}
   }
 
-  
-  // Nettoyer les wrappers vides (sans pub) pour éviter espaces verticaux
   function cleanupEmptyWrappers() {
   try {
   document.querySelectorAll('.ezoic-ad').forEach(wrapper => {
@@ -217,10 +246,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 +260,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 +270,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;
@@ -272,7 +296,6 @@
   };
 
   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 +308,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 +323,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 +354,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,7 +374,6 @@
   // Ajouter à la batch
   pendingShowAdsIds.add(id);
 
-  // Debounce: attendre 100ms pour collecter tous les IDs
   clearTimeout(batchShowAdsTimer);
   batchShowAdsTimer = setTimeout(() => {
   if (pendingShowAdsIds.size === 0) return;
@@ -406,19 +423,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 +497,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 +513,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 +534,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 +554,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) {
@@ -568,8 +574,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) {}
   });
@@ -585,23 +589,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 +613,6 @@
   }
 
   async function runCore() {
-  // Attendre que canInsert soit true (protection race condition navigation)
   if (!state.canShowAds) {
   return;
   }
@@ -661,7 +655,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 +666,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);
@@ -712,15 +702,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', () => {
@@ -750,7 +736,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 +745,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 +775,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,10 @@
-.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 {
+  height: auto !important;
+  padding: 0 !important;
+  margin: 0 !important;
+}
+
+.ezoic-ad * {
+  margin: 0 !important;
+  padding: 0 !important;
+}