Run callable before flushing, revert default

Author: bencroker

src/config.php

@@ -28,7 +28,7 @@
          * Whether the session should be closed when the event stream begins.
          * This is useful to allow other requests to be processed while the event stream is being sent.
          */
-        'shouldCloseSession' => false,
+        'shouldCloseSession' => true,
 
         /**
          * The name of the signals variable that will be injected into Datastar templates.

src/models/Settings.php

@@ -18,7 +18,7 @@ class Settings extends Model
      * Whether the session should be closed when the event stream begins.
      * This is useful to allow other requests to be processed while the event stream is being sent.
      */
-    public bool $shouldCloseSession = false;
+    public bool $shouldCloseSession = true;
 
     /**
      * The name of the signals variable that will be injected into Datastar templates.

src/services/SseService.php

@@ -79,16 +79,16 @@ public function getEventStream(?callable $callable = null): Response
                 session_write_close();
             }
 
+            if (is_callable($callable)) {
+                $callable();
+            }
+
             echo $this->getEventOutput();
             if (ob_get_contents()) {
                 ob_flush();
             }
             flush();
 
-            if (is_callable($callable)) {
-                $callable();
-            }
-
             // Return an array to prevent Yii from throwing an exception.
             return [];
         };