Update SessionSavePath()

joanhey · Sep 10, 2023 · 2dae9e7 · 2dae9e7
1 parent dedfc6c
commit 2dae9e7
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/src/AdapterFunctions.php b/src/AdapterFunctions.php
@@ -150,7 +150,7 @@ function session_name(string $name = ''): string
  * @param string $path
  * @return string
  */
-function session_save_path(string $path = ''): string
+function session_save_path(?string $path = null): string|false
 {
     return Http::sessionSavePath($path);
 }

diff --git a/src/Session.php b/src/Session.php
@@ -147,14 +147,20 @@ public static function sessionName(string $name = null): string
 
     /**
      * Get and/or set the current session save path.
+     * 
+     * @see https://www.php.net/manual/en/function.session-save-path.php
      */
-    public static function sessionSavePath(string $path = null): string
+    public static function sessionSavePath(?string $path = null): string|false
     {
-        if ($path && \is_dir($path) && \is_writable($path) && ! static::sessionStarted()) {
-            static::$sessionSavePath = $path;
+        if ($path === null) {
+            return static::$sessionSavePath;
         }
 
-        return static::$sessionSavePath;
+        if (! static::sessionStarted() && \is_dir($path) && \is_writable($path)) {
+            return static::$sessionSavePath = $path;
+        }
+
+        return false;
     }
 
     /**