Optimize AJAX auth for guests

Only do one check, once. Login will clear the local data anyway.
Tetrakern · Dec 9, 2024 · dae4940 · dae4940
1 parent f95ee9a
commit dae4940
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 9 deletions.
diff --git a/js/application.min.js b/js/application.min.js
diff --git a/js/complete.min.js b/js/complete.min.js
diff --git a/js/utility.min.js b/js/utility.min.js
diff --git a/src/js/application.js b/src/js/application.js
@@ -75,6 +75,14 @@ const FcnGlobals = {
 
   debounceRate: fictioneer_ajax.post_debounce_rate,
 
+  /**
+   * Whether AJAX authentication is enabled.
+   *
+   * @type {Boolean}
+   */
+
+  ajaxAuth: document.documentElement.dataset.fictioneerAjaxAuthValue ?? false,
+
   /**
    * Theme fonts.
    *
@@ -235,7 +243,10 @@ application.register('fictioneer', class extends Stimulus.Controller {
     }
 
     // Only update from server after some time has passed (e.g. 60 seconds)
-    if (FcnGlobals.ajaxLimitThreshold < currentUserData.lastLoaded || currentUserData.loggedIn === false) {
+    if (
+      (FcnGlobals.ajaxLimitThreshold < currentUserData.lastLoaded || currentUserData.loggedIn === false) &&
+      currentUserData.loggedIn !== 'pending'
+    ) {
       // Prepare event
       const event = new CustomEvent(
         'fcnUserDataReady',

diff --git a/src/js/utility.js b/src/js/utility.js
@@ -88,7 +88,7 @@ const FcnUtils = {
     const reset = {
       'lastLoaded': data.lastLoaded ?? 0,
       'timestamp': 0,
-      'loggedIn': 'pending',
+      'loggedIn': (FcnGlobals.ajaxAuth && data.loggedIn === false) ? false : 'pending',
       'follows': false,
       'reminders': false,
       'checkmarks': false,