Merge pull request #10 from tflori/feature-destroyEmpty

add option not to destroy empty sessions

Author: tflori

src/SessionInstance.php

@@ -14,6 +14,10 @@ class SessionInstance implements SessionInterface
      * @var bool */
     protected static $useCookies;
 
+    /** Weather or not an empty session should be destroyed
+     * @var bool */
+    public $destroyEmptySession = true;
+
     /** Whether the session has been started or not.
      * @var bool $init */
     protected $init = false;
@@ -152,7 +156,7 @@ protected function updateSession(array $data = [])
             if ($val === null) {
                 unset($_SESSION[$key]);
                 // destroy the session when empty
-                if (empty($_SESSION)) {
+                if (empty($_SESSION) && $this->destroyEmptySession) {
                     $this->destroy();
                     return;
                 }

tests/DeleteTest.php

@@ -67,7 +67,7 @@ public function doesNotWriteWhenNothingDeleted()
     }
 
     /** @test */
-    public function destorysTheSessionWhenLastKeyGotDeleted()
+    public function destroysTheSessionWhenLastKeyGotDeleted()
     {
         $session = new SessionInstance('session');
         $session->set('foo', 'bar');
@@ -76,4 +76,16 @@ public function destorysTheSessionWhenLastKeyGotDeleted()
 
         $session->delete('foo');
     }
+
+    /** @test */
+    public function doesNotDestroyWhenConfigured()
+    {
+        $session = new SessionInstance('session');
+        $session->set('foo', 'bar');
+        $session->destroyEmptySession = false;
+
+        $this->sessionHandler->shouldNotReceive('destroy');
+
+        $session->delete('foo');
+    }
 }