Update src/Illuminate/Session/Store.php

Fixing the issue #58 If session's lifetime parameter equals to 0 session should expire after browser closes. But it doesn't, it expires immediately. If lifetime is something like 30 mins, session class works as expected. This patch fixes the problem and improves our math knowledge.
laravel · devmach · Jan 21, 2013 · Jan 21, 2013 · 08c3b75244aee6cbb1680552a75c3042b029fbe2
commit 08c3b75244aee6cbb1680552a75c3042b029fbe2
diff --git a/src/Illuminate/Session/Store.php b/src/Illuminate/Session/Store.php
@@ -159,7 +159,10 @@ protected function isInvalid($session)
 	{
 		if ( ! is_array($session)) return true;
 
-		return (time() - $session['last_activity']) > ($this->lifetime * 60);
+		return ( $this->lifetime === 0 
+		         ? false 
+		         : (time() - $session['last_activity']) > ($this->lifetime * 60) 
+		);
 	}
 
 	/**
@@ -623,4 +626,4 @@ public function offsetUnset($key)
 		$this->forget($key);
 	}
 
-}
+}