Fix bcit-ci#4179

narfbg · narfbg · commit de8b82ca8c4e · 2015-10-18T20:58:38.000+03:00
diff --git a/system/libraries/Session/drivers/Session_database_driver.php b/system/libraries/Session/drivers/Session_database_driver.php
@@ -159,6 +159,10 @@ public function read($session_id)
 
 			if (($result = $this->_db->get()->row()) === NULL)
 			{
+				// PHP7 will reuse the same SessionHandler object after
+				// ID regeneration, so we need to explicitly set this to
+				// FALSE instead of relying on the default ...
+				$this->_row_exists = FALSE;
 				$this->_fingerprint = md5('');
 				return '';
 			}
diff --git a/user_guide_src/source/changelog.rst b/user_guide_src/source/changelog.rst
@@ -15,6 +15,7 @@ Bug fixes for 3.0.3
 -------------------
 
 -  Fixed a bug (#4170) - :doc:`Database <database/index>` method ``insert_id()`` could return an identity from the wrong scope with the 'sqlsrv' driver.
+-  Fixed a bug (#4179) - :doc:`Session Library <libraries/sessions>` doesn't properly maintain its state after ID regeneration with the 'database' driver on PHP7.
 
 Version 3.0.2
 =============

Original file line number	Diff line number	Diff line change
`@@ -159,6 +159,10 @@ public function read($session_id)`
`159`	`159`
`160`	`160`	`if (($result = $this->_db->get()->row()) === NULL)`
`161`	`161`	`{`
	`162`	`+ // PHP7 will reuse the same SessionHandler object after`
	`163`	`+ // ID regeneration, so we need to explicitly set this to`
	`164`	`+ // FALSE instead of relying on the default ...`
	`165`	`+ $this->_row_exists = FALSE;`
`162`	`166`	`$this->_fingerprint = md5('');`
`163`	`167`	`return '';`
`164`	`168`	`}`