nodebb-plugin-ezoic-infinite 0.6.1 → 0.6.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "0.6.1",
+  "version": "0.6.3",
   "description": "Ezoic ads with infinite scroll using a pool of placeholder IDs",
   "main": "library.js",
   "license": "MIT",

package/public/client.js

@@ -1,22 +1,35 @@
-/* globals ajaxify */
 'use strict';
 
+/* globals ajaxify */
 window.ezoicInfiniteLoaded = true;
 
 let cachedConfig;
 let lastFetch = 0;
 let debounceTimer;
 
-let lastSignature = null;
 let inFlight = false;
 let rerunRequested = false;
 
-async function fetchConfig() {
-  if (cachedConfig && Date.now() - lastFetch < 10000) return cachedConfig;
-  const res = await fetch('/api/plugins/ezoic-infinite/config', { credentials: 'same-origin' });
-  cachedConfig = await res.json();
-  lastFetch = Date.now();
-  return cachedConfig;
+// Incremental state (prevents ads "jumping to the top")
+let pageKey = null;
+let injectedSlots = new Set(); // slotNumber per page
+let usedIds = new Set();       // ids currently injected per page
+
+function resetPageState() {
+  injectedSlots = new Set();
+  usedIds = new Set();
+}
+
+function currentPageKey() {
+  // Stable key per ajaxified page
+  try {
+    if (ajaxify && ajaxify.data) {
+      if (ajaxify.data.tid) return 'topic:' + ajaxify.data.tid;
+      if (ajaxify.data.cid) return 'category:' + ajaxify.data.cid;
+      if (ajaxify.data.template) return 'tpl:' + ajaxify.data.template + ':' + (ajaxify.data.url || window.location.pathname);
+    }
+  } catch (e) {}
+  return window.location.pathname;
 }
 
 function parsePool(raw) {
@@ -28,44 +41,35 @@ function parsePool(raw) {
   ));
 }
 
-function guessIsTopicPage() {
-  // Prefer ajaxify template, fallback to DOM presence
-  try {
-    if (ajaxify && ajaxify.data && ajaxify.data.template) {
-      return ['topic', 'topicEvents'].includes(ajaxify.data.template);
-    }
-  } catch (e) {}
-  return $('[component="post/content"]').length > 0;
+async function fetchConfig() {
+  if (cachedConfig && Date.now() - lastFetch < 10000) return cachedConfig;
+  const res = await fetch('/api/plugins/ezoic-infinite/config', { credentials: 'same-origin' });
+  cachedConfig = await res.json();
+  lastFetch = Date.now();
+  return cachedConfig;
 }
 
-function guessIsCategoryPage() {
-  try {
-    if (ajaxify && ajaxify.data && ajaxify.data.template) {
-      return ajaxify.data.template === 'category';
-    }
-  } catch (e) {}
+function isTopicPage() {
+  return $('[component="post/content"]').length > 0 || $('[component="post"][data-pid]').length > 0;
+}
+
+function isCategoryTopicListPage() {
   return $('li[component="category/topic"]').length > 0;
 }
 
 function getTopicPosts() {
-  if (!guessIsTopicPage()) return $();
-
-  // Harmony standard wrapper
   const $primary = $('[component="post"][data-pid]');
   if ($primary.length) return $primary.not('.ezoic-ad-post');
 
-  // Fallback: top-level data-pid that contains post content and isn't nested
-  const $top = $('[data-pid]').filter(function () {
+  return $('[data-pid]').filter(function () {
     const $el = $(this);
     const hasContent = $el.find('[component="post/content"]').length > 0;
     const nested = $el.parents('[data-pid]').length > 0;
     return hasContent && !nested;
-  });
-  return $top.not('.ezoic-ad-post');
+  }).not('.ezoic-ad-post');
 }
 
 function getCategoryTopicItems() {
-  if (!guessIsCategoryPage()) return $();
   return $('li[component="category/topic"]').not('.ezoic-ad-topic');
 }
 
@@ -73,114 +77,122 @@ function tagName($el) {
   return ($el && $el.length ? (($el.prop('tagName') || '').toUpperCase()) : '');
 }
 
-function removeExistingEzoicNodes() {
-  $('.ezoic-ad-post').remove();
-  $('.ezoic-ad-between').remove();
-  $('.ezoic-ad-topic').remove();
-}
-
-function computeWindowSlots(totalItems, interval, poolSize) {
-  const slots = Math.floor(totalItems / interval);
-  if (slots <= 0) return [];
-  const start = Math.max(1, slots - poolSize + 1);
-  const out = [];
-  for (let s = start; s <= slots; s++) out.push(s);
-  return out;
-}
-
-function makeWrapperLike($target, classes, innerHtml) {
+function makeWrapperLike($target, classes, innerHtml, attrs) {
   const t = tagName($target);
+  const attrStr = attrs ? ' ' + attrs : '';
   if (t === 'LI') {
-    return '<li class="' + classes + ' list-unstyled" data-ezoic-ad="1">' + innerHtml + '</li>';
+    return '<li class="' + classes + ' list-unstyled"' + attrStr + '>' + innerHtml + '</li>';
   }
-  return '<div class="' + classes + '" data-ezoic-ad="1">' + innerHtml + '</div>';
+  return '<div class="' + classes + '"' + attrStr + '>' + innerHtml + '</div>';
 }
 
-function insertBetweenGeneric($items, ids, interval, wrapperClass) {
-  const total = $items.length;
-  const slotsToRender = computeWindowSlots(total, interval, ids.length);
-  if (!slotsToRender.length) return [];
-
-  const activeIds = [];
-  for (let i = 0; i < slotsToRender.length; i++) {
-    const slotNumber = slotsToRender[i];
-    const id = ids[i];
-    const index = slotNumber * interval - 1;
-    const $target = $items.eq(index);
-    if (!$target.length) continue;
-
-    const html = makeWrapperLike($target, wrapperClass, '<div id="ezoic-pub-ad-placeholder-' + id + '"></div>');
-    $target.after(html);
-    activeIds.push(id);
+function pickNextId(pool) {
+  for (const id of pool) {
+    if (!usedIds.has(id)) return id;
   }
-  return activeIds;
+  return null;
 }
 
-function insertAdMessagesBetweenReplies($posts, ids, interval) {
-  const total = $posts.length;
-  const slotsToRender = computeWindowSlots(total, interval, ids.length);
-  if (!slotsToRender.length) return [];
-
-  const activeIds = [];
-  for (let i = 0; i < slotsToRender.length; i++) {
-    const slotNumber = slotsToRender[i];
-    const id = ids[i];
-    const index = slotNumber * interval - 1;
-    const $target = $posts.eq(index);
-    if (!$target.length) continue;
-
-    const inner = '<div class="content"><div id="ezoic-pub-ad-placeholder-' + id + '"></div></div>';
-    const html = makeWrapperLike($target, 'post ezoic-ad-post', inner);
-    $target.after(html);
-    activeIds.push(id);
-  }
-  return activeIds;
-}
-
-function ezoicRender(ids) {
+function callEzoic(ids) {
   if (!ids || !ids.length) return;
 
   window.ezstandalone = window.ezstandalone || {};
   window.ezstandalone.cmd = window.ezstandalone.cmd || [];
 
-  // Ezoic doc: showAds(101,102,103) within cmd.push for dynamic content citeturn0search0turn0search6
-  window.ezstandalone.cmd.push(function () {
-    try {
-      if (typeof window.ezstandalone.refreshAds === 'function') {
-        window.ezstandalone.refreshAds.apply(window.ezstandalone, ids);
-        return;
-      }
-    } catch (e) {}
-
+  const run = function () {
     try {
       if (typeof window.ezstandalone.showAds === 'function') {
         window.ezstandalone.showAds.apply(window.ezstandalone, ids);
-        return;
+        return true;
       }
     } catch (e) {}
+    return false;
+  };
 
-    // Last resort: call showAds() with no args = "all placeholders" citeturn0search0turn0search6
-    try {
-      if (typeof window.ezstandalone.showAds === 'function') {
-        window.ezstandalone.showAds();
-      }
-    } catch (e) {}
-  });
+  window.ezstandalone.cmd.push(function () { run(); });
+
+  // retry a few times (Ezoic can load late)
+  let tries = 0;
+  const maxTries = 6;
+  const timer = setInterval(function () {
+    tries++;
+    if (run() || tries >= maxTries) clearInterval(timer);
+  }, 800);
 }
 
-function computeSignature($posts, $topicItems) {
-  if ($posts.length) {
-    const lastPid = $posts.last().attr('data-pid') || '';
-    return 'topic:' + $posts.length + ':' + lastPid;
+function injectBetweenIncremental($items, pool, interval, wrapperClass) {
+  const total = $items.length;
+  const maxSlot = Math.floor(total / interval);
+  if (maxSlot <= 0) return [];
+
+  const newIds = [];
+
+  for (let slot = 1; slot <= maxSlot; slot++) {
+    if (injectedSlots.has(slot)) continue;
+
+    const index = slot * interval - 1;
+    const $target = $items.eq(index);
+    if (!$target.length) continue;
+
+    const id = pickNextId(pool);
+    if (!id) {
+      // pool exhausted: stop injecting further to avoid reusing ids and "jumping"
+      break;
+    }
+
+    const placeholder = '<div id="ezoic-pub-ad-placeholder-' + id + '"></div>';
+    const html = makeWrapperLike($target, wrapperClass, placeholder, 'data-ezoic-slot="' + slot + '" data-ezoic-id="' + id + '"');
+
+    $target.after(html);
+
+    injectedSlots.add(slot);
+    usedIds.add(id);
+    newIds.push(id);
   }
-  if ($topicItems.length) {
-    const lastTid = $topicItems.last().attr('data-tid') || '';
-    return 'category:' + $topicItems.length + ':' + lastTid;
+
+  return newIds;
+}
+
+function injectMessageIncremental($posts, pool, interval) {
+  const total = $posts.length;
+  const maxSlot = Math.floor(total / interval);
+  if (maxSlot <= 0) return [];
+
+  const newIds = [];
+
+  for (let slot = 1; slot <= maxSlot; slot++) {
+    if (injectedSlots.has(slot)) continue;
+
+    const index = slot * interval - 1;
+    const $target = $posts.eq(index);
+    if (!$target.length) continue;
+
+    const id = pickNextId(pool);
+    if (!id) break;
+
+    const inner = '<div class="content"><div id="ezoic-pub-ad-placeholder-' + id + '"></div></div>';
+    const html = makeWrapperLike($target, 'post ezoic-ad-post', inner, 'data-ezoic-slot="' + slot + '" data-ezoic-id="' + id + '"');
+
+    $target.after(html);
+
+    injectedSlots.add(slot);
+    usedIds.add(id);
+    newIds.push(id);
   }
-  return 'none';
+
+  return newIds;
 }
 
 async function refreshAds() {
+  // reset state when navigating (ajaxify)
+  const key = currentPageKey();
+  if (pageKey !== key) {
+    pageKey = key;
+    resetPageState();
+    // also cleanup any injected wrappers that may have been left by browser bfcache
+    $('.ezoic-ad-post, .ezoic-ad-between, .ezoic-ad-topic').remove();
+  }
+
   if (inFlight) { rerunRequested = true; return; }
   inFlight = true;
 
@@ -194,49 +206,47 @@ async function refreshAds() {
     const messagePool = parsePool(cfg.messagePlaceholderIds);
     const messageInterval = Math.max(1, parseInt(cfg.messageIntervalPosts, 10) || 3);
 
-    const $posts = getTopicPosts();
-    const $topicItems = getCategoryTopicItems();
+    const onTopic = isTopicPage();
+    const onCategory = !onTopic && isCategoryTopicListPage();
 
-    if (!$posts.length && !$topicItems.length) return;
+    const $posts = onTopic ? getTopicPosts() : $();
+    const $topicItems = onCategory ? getCategoryTopicItems() : $();
 
-    const signature = computeSignature($posts, $topicItems);
-    if (signature === lastSignature) return;
-    lastSignature = signature;
-
-    removeExistingEzoicNodes();
+    if (!$posts.length && !$topicItems.length) return;
 
-    const activeIds = [];
+    const newIds = [];
 
-    // Category pages: BETWEEN only
+    // Your rule:
+    // - Category topic list: BETWEEN only
+    // - Topic page: MESSAGE only
     if ($topicItems.length) {
       if (cfg.enableBetweenAds && betweenPool.length) {
-        activeIds.push(...insertBetweenGeneric($topicItems, betweenPool, betweenInterval, 'ezoic-ad-topic'));
+        newIds.push(...injectBetweenIncremental($topicItems, betweenPool, betweenInterval, 'ezoic-ad-topic'));
       }
-      ezoicRender(activeIds);
+      callEzoic(newIds);
       return;
     }
 
-    // Topic pages: MESSAGE only (as you requested)
     if ($posts.length) {
       if (cfg.enableMessageAds && messagePool.length) {
-        activeIds.push(...insertAdMessagesBetweenReplies($posts, messagePool, messageInterval));
+        newIds.push(...injectMessageIncremental($posts, messagePool, messageInterval));
       }
-      ezoicRender(activeIds);
+      callEzoic(newIds);
     }
   } finally {
     inFlight = false;
     if (rerunRequested) {
       rerunRequested = false;
-      setTimeout(refreshAds, 80);
+      setTimeout(refreshAds, 120);
     }
   }
 }
 
 function debounceRefresh() {
   clearTimeout(debounceTimer);
-  debounceTimer = setTimeout(refreshAds, 220);
+  debounceTimer = setTimeout(refreshAds, 180);
 }
 
 $(document).ready(debounceRefresh);
 $(window).on('action:ajaxify.end action:posts.loaded action:topic.loaded', debounceRefresh);
-setTimeout(debounceRefresh, 2200);
+setTimeout(debounceRefresh, 1800);