nodebb-plugin-ezoic-infinite 1.8.39 → 1.8.40

package/package.json

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

package/public/client.js

@@ -109,7 +109,6 @@
     ezFlushTimer: null,
 
     // Lifecycle
-    enableCalled: false,   // ez.enable() called once per page
     runQueued:    false,
     burstActive:  false, burstDeadline: 0, burstCount: 0, lastBurstTs: 0,
   };
@@ -301,7 +300,14 @@
   }
 
   /**
-   * Flush: define all queued IDs, then enable (first time) or displayMore.
+   * Flush: define new IDs with Ezoic, then call displayMore.
+   *
+   * IMPORTANT: We NEVER call ez.enable() — Ezoic's sa.min.js calls it
+   * automatically on page load. Calling it again causes
+   * "Enable should only ever be called once" error.
+   *
+   * We also check ez.getSelectedPlaceholders() or equivalent to skip
+   * IDs that Ezoic already knows about (defined during initial enable).
    */
   function ezFlush() {
     state.ezFlushTimer = null;
@@ -318,34 +324,39 @@
     state.ezBatch.clear();
     if (!ids.length) return;
 
-    const isFirst = !state.enableCalled;
-
     ezCmd(() => {
       const ez = window.ezstandalone;
       if (!ez) return;
 
-      // define() registers the placeholder IDs with Ezoic
+      // Check which IDs Ezoic already knows about to avoid "already defined"
+      const alreadyDefined = new Set();
       try {
-        if (typeof ez.define === 'function') {
-          ez.define(...ids);
+        // ez.allPlaceholders is an array of IDs Ezoic has already seen
+        if (Array.isArray(ez.allPlaceholders)) {
+          for (const p of ez.allPlaceholders) alreadyDefined.add(Number(p));
         }
       } catch (_) {}
 
-      for (const id of ids) state.phState.set(id, 'defined');
+      const toDefine = ids.filter(id => !alreadyDefined.has(id));
+      // All IDs get displayMore, but only new ones get define
+      const toDisplay = ids;
 
-      if (isFirst) {
-        // First batch on this page: use enable() which triggers the initial ad request
-        state.enableCalled = true;
+      // define() only for truly new placeholders
+      if (toDefine.length) {
         try {
-          if (typeof ez.enable === 'function') {
-            ez.enable();
+          if (typeof ez.define === 'function') {
+            ez.define(...toDefine);
           }
         } catch (_) {}
-      } else {
-        // Subsequent batches: use displayMore() for infinite scroll
+      }
+
+      for (const id of ids) state.phState.set(id, 'defined');
+
+      // displayMore() for all IDs — this is the infinite scroll API
+      if (toDisplay.length) {
         try {
           if (typeof ez.displayMore === 'function') {
-            ez.displayMore(...ids);
+            ez.displayMore(...toDisplay);
           }
         } catch (_) {}
       }
@@ -738,7 +749,6 @@
     state.wrapsByClass.clear();
     state.kind = null;
     state.phState.clear();
-    state.enableCalled = false;
     state.burstActive  = false;
     state.runQueued    = false;
   }