nodebb-plugin-ezoic-infinite 1.6.13 → 1.6.14

package/package.json

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

package/public/client.js

@@ -1,61 +1,65 @@
 (function () {
   'use strict';
 
-function ezoicAnchorKeyFromEl(el) {
+function ezoicAnchorKeyFromNode(node) {
+  // Robustly extract an anchor id from a <li> topic or any descendant.
   try {
-    if (!el || !el.getAttribute) return '';
-    var tid = el.getAttribute('data-tid') || el.getAttribute('data-topic-id');
-    if (tid) return 'tid:' + String(tid);
-    var pid = el.getAttribute('data-pid') || el.getAttribute('data-id');
-    if (pid) return 'pid:' + String(pid);
-    var uid = el.getAttribute('data-uid');
-    if (uid) return 'uid:' + String(uid);
+    if (!node) return '';
+    var el = node;
+    // If node itself doesn't have ids, search descendants.
+    if (!(el.getAttribute && (el.getAttribute('data-tid') || el.getAttribute('data-topic-id') || el.getAttribute('data-pid') || el.getAttribute('data-id')))) {
+      if (el.querySelector) {
+        el = el.querySelector('[data-tid],[data-topic-id],[data-pid],[data-id]') || node;
+      }
+    }
+    if (el.getAttribute) {
+      var tid = el.getAttribute('data-tid') || el.getAttribute('data-topic-id');
+      if (tid) return 'tid:' + String(tid);
+      var pid = el.getAttribute('data-pid') || el.getAttribute('data-id');
+      if (pid) return 'pid:' + String(pid);
+    }
   } catch (e) {}
   return '';
 }
 
-function ezoicFindAnchorEl(anchorKey, scope) {
+function ezoicFindAnchorLi(anchorKey, listEl) {
   try {
     if (!anchorKey) return null;
-    var root = scope || document;
+    var root = listEl || document;
+    var el = null;
     if (anchorKey.indexOf('tid:') === 0) {
       var v = anchorKey.slice(4);
-      return root.querySelector('[data-tid="' + v + '"], [data-topic-id="' + v + '"]');
-    }
-    if (anchorKey.indexOf('pid:') === 0) {
+      el = root.querySelector('[data-tid="' + v + '"], [data-topic-id="' + v + '"]');
+    } else if (anchorKey.indexOf('pid:') === 0) {
       var p = anchorKey.slice(4);
-      return root.querySelector('[data-pid="' + p + '"], [data-id="' + p + '"]');
-    }
-    if (anchorKey.indexOf('uid:') === 0) {
-      var u = anchorKey.slice(4);
-      return root.querySelector('[data-uid="' + u + '"]');
+      el = root.querySelector('[data-pid="' + p + '"], [data-id="' + p + '"]');
     }
-  } catch (e) {}
-  return null;
+    if (!el) return null;
+    return el.closest ? (el.closest('li') || null) : null;
+  } catch (e) { return null; }
 }
 
-function ezoicReconcileHosts(scope) {
-  // Ensure each li.nodebb-ezoic-host stays directly after its anchor <li>.
-  // Prevents "all hosts under first topic" after NodeBB re-render/virtualize.
+function ezoicReconcileOrPurgeHosts(scope) {
+  // Reposition each host after its anchor li. If anchor doesn't exist (virtualized away),
+  // remove the host to prevent it from collecting at the top.
   try {
     var root = scope || document;
     var hosts = root.querySelectorAll('li.nodebb-ezoic-host[data-ezoic-anchor]');
     if (!hosts || !hosts.length) return;
 
-    hosts.forEach(function (host) {
+    hosts.forEach(function(host){
       try {
+        var listEl = host.parentElement;
+        if (!listEl) return;
         var anchorKey = host.getAttribute('data-ezoic-anchor') || '';
         if (!anchorKey) return;
-        var ul = host.parentElement;
-        if (!ul) return;
-
-        var anchor = ezoicFindAnchorEl(anchorKey, ul) || ezoicFindAnchorEl(anchorKey, document);
-        if (!anchor) return;
-
-        // Anchor is often inside the <li> topic; normalize to list item
-        var anchorLi = anchor.closest ? (anchor.closest('li') || anchor) : anchor;
-        if (!anchorLi || !anchorLi.parentElement) return;
 
+        var anchorLi = ezoicFindAnchorLi(anchorKey, listEl) || ezoicFindAnchorLi(anchorKey, document);
+        if (!anchorLi) {
+          // Anchor not in DOM right now => delete host to avoid "pile at top"
+          host.remove();
+          return;
+        }
         if (host.previousElementSibling !== anchorLi) {
           anchorLi.insertAdjacentElement('afterend', host);
         }
@@ -64,8 +68,6 @@ function ezoicReconcileHosts(scope) {
   } catch (e) {}
 }
 
-' + '';
-
 
 function ezoicInsertAfterWithUlGuard(target, wrap, kindClass) {
   try {
@@ -80,7 +82,8 @@ function ezoicInsertAfterWithUlGuard(target, wrap, kindClass) {
         host.setAttribute('role', 'listitem');
         host.style.listStyle = 'none';
         host.style.width = '100%';
-        try { var ak = ezoicAnchorKeyFromEl(target); if (ak) host.setAttribute('data-ezoic-anchor', ak); } catch(e) {}
+        try { var prev = wrap.previousElementSibling; var ak2 = ezoicAnchorKeyFromNode(prev || (wrap.parentElement && wrap.parentElement.previousElementSibling)); if (ak2) host.setAttribute('data-ezoic-anchor', ak2); } catch(e) {}
+        try { var ak = ezoicAnchorKeyFromNode(target); if (ak) host.setAttribute('data-ezoic-anchor', ak); } catch(e) {}
 
         // Insert host after the target listitem, then move the wrap inside.
         if (target.insertAdjacentElement) {
@@ -115,7 +118,7 @@ function ezoicRepairInvalidBetweenWraps() {
         host.setAttribute('role', 'listitem');
         host.style.listStyle = 'none';
         host.style.width = '100%';
-        try { var ak = ezoicAnchorKeyFromEl(target); if (ak) host.setAttribute('data-ezoic-anchor', ak); } catch(e) {}
+        try { var ak = ezoicAnchorKeyFromNode(target); if (ak) host.setAttribute('data-ezoic-anchor', ak); } catch(e) {}
         p.insertBefore(host, wrap);
         host.appendChild(wrap);
         try { wrap.style.width = '100%'; } catch (e) {}
@@ -1647,52 +1650,49 @@ if (window.jQuery) {
 // ===== /V12.2 =====
 
 
-// ===== V12.3 reconcile scheduler =====
+// ===== V12.4 reconcile+purge scheduler =====
 (function(){
-  var pending = false;
-  var last = 0;
-  var COOLDOWN = 180;
-
+  var pending=false, last=0, COOLDOWN=160;
   function schedule(scope){
-    var now = Date.now();
-    if (now - last < COOLDOWN) return;
+    var now=Date.now();
+    if (now-last<COOLDOWN) return;
     if (pending) return;
-    pending = true;
+    pending=true;
     requestAnimationFrame(function(){
-      pending = false;
-      last = Date.now();
-      try { ezoicReconcileHosts(scope); } catch(e) {}
+      pending=false; last=Date.now();
+      try { ezoicReconcileOrPurgeHosts(scope); } catch(e) {}
     });
   }
 
-  window.addEventListener('scroll', function(){ schedule(document); }, { passive: true });
-  window.addEventListener('resize', function(){ schedule(document); }, { passive: true });
+  window.addEventListener('scroll', function(){ schedule(document); }, { passive:true });
+  window.addEventListener('resize', function(){ schedule(document); }, { passive:true });
 
   if (window.jQuery) {
     window.jQuery(window).on('action:ajaxify.end action:infiniteScroll.loaded', function(){
       setTimeout(function(){ schedule(document); }, 0);
       setTimeout(function(){ schedule(document); }, 220);
-      setTimeout(function(){ schedule(document); }, 600);
+      setTimeout(function(){ schedule(document); }, 700);
     });
   }
 
-  // MutationObserver lightweight: only watch ULs for child list changes
+  // Observe only the main list containers if present to reduce overhead
   try {
     if (typeof MutationObserver !== 'undefined') {
       var mo = new MutationObserver(function(muts){
-        // if any li hosts are moved/added, reconcile
         for (var i=0;i<muts.length;i++){
-          var m = muts[i];
-          if (m.addedNodes && m.addedNodes.length) { schedule(m.target); break; }
-          if (m.removedNodes && m.removedNodes.length) { schedule(m.target); break; }
+          var m=muts[i];
+          if ((m.addedNodes && m.addedNodes.length) || (m.removedNodes && m.removedNodes.length)) {
+            schedule(m.target);
+            break;
+          }
         }
       });
-      document.querySelectorAll('ul,ol').forEach(function(list){
+      document.querySelectorAll('ul[component], ul.topic-list, ul.categories, ol[component]').forEach(function(list){
         try { mo.observe(list, {childList:true}); } catch(e) {}
       });
     }
   } catch(e){}
 
-  setTimeout(function(){ try { schedule(document); } catch(e) {} }, 0);
+  setTimeout(function(){ schedule(document); }, 0);
 })();
-// ===== /V12.3 =====
+// ===== /V12.4 =====