New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix session restart on PHP 5.3 #12961
Conversation
this also removes some useless code
@@ -89,7 +89,7 @@ public function setSessionData(array $array) | |||
*/ | |||
public function start() | |||
{ | |||
if ($this->started && !$this->closed) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was useless because closed is always false if started is true
Thank you @Tobion. |
This PR was merged into the 2.3 branch. Discussion ---------- fix session restart on PHP 5.3 | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | no | Fixed tickets | #12562 | License | MIT | Doc PR | - this also removes some useless code Commits ------- b9d3c92 fix session restart on PHP 5.3
@@ -209,18 +211,20 @@ public function testSetSaveHandler54() | |||
/** | |||
* @expectedException \RuntimeException | |||
*/ | |||
public function testStartedOutside53() | |||
public function testStartedOutside() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This method should be named testStarted
. I'll send a pull...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no it should not
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Opps, oh yeh. Sorry.
this also removes some useless code