remove JsSse::$closeBeforeUnload prop

js/src/JqueryPlugin/ServerEventPlugin.js

@@ -7,6 +7,7 @@ export default class AtkServerEventPlugin extends AbstractPlugin {
         const hasLoader = this.settings.showLoader;
 
         this.source = new EventSource(this.settings.url);
+
         if (hasLoader) {
             element.addClass('loading');
         }
@@ -17,22 +18,20 @@ export default class AtkServerEventPlugin extends AbstractPlugin {
 
         this.source.addEventListener('error', (e) => {
             if (e.eventPhase === EventSource.CLOSED) {
+                this.source.close();
+
                 if (hasLoader) {
                     element.removeClass('loading');
                 }
-                this.source.close();
             }
         });
 
         this.source.addEventListener('atkSseAction', (e) => {
             atk.apiService.atkProcessExternalResponse(JSON.parse(e.data));
         });
 
-        if (this.settings.closeBeforeUnload) {
-            window.addEventListener('beforeunload', (event) => {
-                this.source.close();
-            });
-        }
+        // prevent "The connection to http://xxx was interrupted while the page was loading." browser console warning
+        window.addEventListener('beforeunload', () => this.source.close());
     }
 
     stop() {
@@ -48,5 +47,4 @@ AtkServerEventPlugin.DEFAULTS = {
     url: null,
     urlOptions: {},
     showLoader: false,
-    closeBeforeUnload: false,
 };

public/js/atkjs-ui.js

@@ -1775,20 +1775,18 @@ class AtkServerEventPlugin extends _AbstractPlugin__WEBPACK_IMPORTED_MODULE_2__[
     });
     this.source.addEventListener('error', e => {
       if (e.eventPhase === EventSource.CLOSED) {
+        this.source.close();
         if (hasLoader) {
           element.removeClass('loading');
         }
-        this.source.close();
       }
     });
     this.source.addEventListener('atkSseAction', e => {
       atk__WEBPACK_IMPORTED_MODULE_1__["default"].apiService.atkProcessExternalResponse(JSON.parse(e.data));
     });
-    if (this.settings.closeBeforeUnload) {
-      window.addEventListener('beforeunload', event => {
-        this.source.close();
-      });
-    }
+
+    // prevent "The connection to http://xxx was interrupted while the page was loading." browser console warning
+    window.addEventListener('beforeunload', () => this.source.close());
   }
   stop() {
     this.source.close();
@@ -1800,8 +1798,7 @@ class AtkServerEventPlugin extends _AbstractPlugin__WEBPACK_IMPORTED_MODULE_2__[
 AtkServerEventPlugin.DEFAULTS = {
   url: null,
   urlOptions: {},
-  showLoader: false,
-  closeBeforeUnload: false
+  showLoader: false
 };
 
 /***/ }),

src/JsSse.php

@@ -18,9 +18,6 @@ class JsSse extends JsCallback
     /** @var bool Show Loader when doing SSE. */
     public $showLoader = false;
 
-    /** @var bool Add window.beforeunload listener for closing js EventSource. Off by default. */
-    public $closeBeforeUnload = false;
-
     /** @var \Closure(string): void|null Custom function for outputting (instead of echo). */
     public $echoFunction;
 
@@ -33,9 +30,6 @@ public function jsExecute(): JsBlock
         if ($this->showLoader) {
             $options['showLoader'] = $this->showLoader;
         }
-        if ($this->closeBeforeUnload) {
-            $options['closeBeforeUnload'] = $this->closeBeforeUnload;
-        }
 
         return new JsBlock([(new Jquery($this->getOwner() /* TODO element and loader element should be passed explicitly */))->atkServerEvent($options)]);
     }