nodebb-plugin-ezoic-infinite 1.4.57 → 1.4.59

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.4.57",
+  "version": "1.4.59",
   "description": "Production-ready Ezoic infinite ads integration for NodeBB 4.x",
   "main": "library.js",
   "license": "MIT",
@@ -18,4 +18,4 @@
     "compatibility": "^4.0.0"
   },
   "private": false
-}
+}

package/public/client.js

@@ -63,7 +63,10 @@
 
     // Retry counters.
     poolWaitAttempts: 0,
-    awaitItemsAttempts: 0
+    awaitItemsAttempts: 0,
+
+    // Placeholders to destroy after ajaxify navigation settles.
+    toDestroyIds: []
   };
 
   function normalizeBool(v) {
@@ -263,8 +266,8 @@
     state.usedPosts.forEach(function (id) { ids.push(id); });
     state.usedCategories.forEach(function (id) { ids.push(id); });
 
-    // Only destroy placeholders that were filled at least once in this session.
-    destroyPlaceholderIds(ids);
+    // Return ids; destruction is performed after ajaxify.end settles.
+    return ids;
   }
 
   function patchShowAds() {
@@ -450,7 +453,9 @@
       var wrap = null;
 
       if (pick.recycled && pick.recycled.wrap) {
-        // Recycle: do not call ezstandalone.destroyPlaceholders() (can break ajaxify flows).
+        // Recycle: destroy the old placeholder definition (after ajaxify has settled)
+        // so Ezoic can define it again at the new location.
+        if (sessionDefinedIds.has(id)) destroyPlaceholderIds([id]);
 
         // Remove old wrapper.
         var oldWrap = pick.recycled.wrap;
@@ -461,7 +466,7 @@
         if (!wrap) continue;
 
         // Give Ezoic a moment after DOM insertion.
-        setTimeoutTracked(function () { callShowAdsWhenReady(id); }, 700);
+        setTimeoutTracked(function () { callShowAdsWhenReady(id); }, 250);
       } else {
         usedSet.add(id);
         wrap = insertAfter(el, id, kindClass, afterPos);
@@ -523,6 +528,9 @@
       state.obs = null;
     }
 
+    // Collect ids used on the page we are leaving.
+    state.toDestroyIds = destroyUsedPlaceholders();
+
     // NOTE: Do NOT call ezstandalone.destroyPlaceholders() during ajaxify navigation.
     // In practice this can leave Ezoic in a state where new placeholders stay "unused"
     // until a full page reload. We only remove our wrappers.
@@ -778,6 +786,15 @@
 
       // Delay gate to avoid racing NodeBB DOM swap vs Ezoic processing.
       setTimeoutTracked(function () {
+        // After navigation, destroy placeholders that were used on the previous page
+        // so Ezoic can re-define them on this new page.
+        try {
+          if (state.toDestroyIds && state.toDestroyIds.length) {
+            destroyPlaceholderIds(state.toDestroyIds);
+          }
+        } catch (e) {}
+        state.toDestroyIds = [];
+
         state.canShowAds = true;
         waitForEzoicThenRun();
       }, 300);