Fix several TypeErrors when accessing unset properties

Author: Jonathon Hill

src/Config.php

@@ -23,7 +23,7 @@ public function setSavePath(string $save_path): self
 
     public function getSavePath(): ?string
     {
-        return $this->save_path;
+        return $this->save_path ?? null;
     }
 
     protected SessionHandlerInterface $save_handler;
@@ -36,7 +36,7 @@ public function setSaveHandler(SessionHandlerInterface $save_handler): self
 
     public function getSaveHandler(): ?SessionHandlerInterface
     {
-        return $this->save_handler;
+        return $this->save_handler ?? null;
     }
 
     protected SerializerInterface $serialize_handler;

src/Handlers/RedisHandler.php

@@ -31,13 +31,10 @@ public function __construct(Config $config)
 
     public function open($path, $name): bool
     {
-        if ($this->redis) {
+        if (isset($this->redis)) {
             return false;
         }
 
-        $this->redis = null;
-        $this->store = null;
-
         // Parse redis connection settings from save path
         $query = [];
         $config = parse_url($path);
@@ -69,12 +66,12 @@ public function close(): bool
     {
         $this->store = null;
 
-        if (!$this->redis) {
+        if (!isset($this->redis)) {
             return false;
         }
 
         $success = $this->redis->close();
-        $this->redis = null;
+        unset($this->redis);
 
         return $success;
     }

src/Manager.php

@@ -38,15 +38,15 @@ public function getConfig(): Config
      */
     public function getCurrentSession(): ?Session
     {
-        return $this->currentSession;
+        return $this->currentSession ?? null;
     }
 
     /**
      * Discard session array changes and finish session
      */
     public function abort(): bool
     {
-        if (!$this->currentSession) {
+        if (!isset($this->currentSession)) {
             return false;
         }
 
@@ -59,7 +59,7 @@ public function abort(): bool
      */
     public function commit(): bool
     {
-        if (!$this->currentSession) {
+        if (!isset($this->currentSession)) {
             return false;
         }
 
@@ -160,7 +160,7 @@ public function gc()
      */
     public function id(string $id = null): string
     {
-        $returnId = $this->currentSession
+        $returnId = isset($this->currentSession)
             ? $this->currentSession->getId()
             : "";
 
@@ -238,7 +238,7 @@ public function register_shutdown(): void
      */
     public function reset(): bool
     {
-        if (!$this->currentSession) {
+        if (!isset($this->currentSession)) {
             return false;
         }
 
@@ -324,7 +324,7 @@ public function start(): bool
         }
 
         if (
-            !$this->currentSession
+            !isset($this->currentSession)
             || (
                 $handler instanceof SessionUpdateTimestampHandlerInterface
                 && !$handler->validateId($this->currentSession->getId())
@@ -391,7 +391,7 @@ public function status(): int
             return \PHP_SESSION_DISABLED;
         }
 
-        if (!$this->currentSession || !$this->currentSession->isInitialized()) {
+        if (!isset($this->currentSession) || !$this->currentSession->isInitialized()) {
             return \PHP_SESSION_NONE;
         }
 
@@ -403,7 +403,7 @@ public function status(): int
      */
     public function unset(): bool
     {
-        if (!$this->currentSession) {
+        if (!isset($this->currentSession)) {
             return false;
         }
 
@@ -420,7 +420,7 @@ public function unset(): bool
      */
     public function write_close(): bool
     {
-        if (!$this->currentSession) {
+        if (!isset($this->currentSession)) {
             return false;
         }