When getting the session's id, check if the session is not closed

Taluu · Taluu · commit 5b9a727705c9 · 2013-11-27T10:30:56.000+01:00
This introduced a regression from #9246, with an incomplete fix ; As the `started` flag on the NativeSessionStorage was not `true` anymore when saving the session, the session id was always empty when saving it, and thus when sending the `PHPSESSID` cookie
diff --git a/src/Symfony/Component/HttpFoundation/Session/Storage/NativeSessionStorage.php b/src/Symfony/Component/HttpFoundation/Session/Storage/NativeSessionStorage.php
@@ -161,7 +161,7 @@ public function start()
      */
     public function getId()
     {
-        if (!$this->started) {
+        if (!$this->started && !$this->closed) {
             return ''; // returning empty is consistent with session_id() behaviour
         }
 
diff --git a/src/Symfony/Component/HttpFoundation/Tests/Session/Storage/NativeSessionStorageTest.php b/src/Symfony/Component/HttpFoundation/Tests/Session/Storage/NativeSessionStorageTest.php
@@ -62,8 +62,12 @@ public function testGetId()
     {
         $storage = $this->getStorage();
         $this->assertEquals('', $storage->getId());
+
         $storage->start();
         $this->assertNotEquals('', $storage->getId());
+
+        $storage->save();
+        $this->assertNotEquals('', $storage->getId());
     }
 
     public function testRegenerate()

Original file line number	Diff line number	Diff line change
`@@ -161,7 +161,7 @@ public function start()`
`161`	`161`	`*/`
`162`	`162`	`public function getId()`
`163`	`163`	`{`
`164`		`- if (!$this->started) {`
	`164`	`+ if (!$this->started && !$this->closed) {`
`165`	`165`	`return ''; // returning empty is consistent with session_id() behaviour`
`166`	`166`	`}`
`167`	`167`
Original file line number	Diff line number	Diff line change
`@@ -62,8 +62,12 @@ public function testGetId()`
`62`	`62`	`{`
`63`	`63`	`$storage = $this->getStorage();`
`64`	`64`	`$this->assertEquals('', $storage->getId());`
	`65`	`+`
`65`	`66`	`$storage->start();`
`66`	`67`	`$this->assertNotEquals('', $storage->getId());`
	`68`	`+`
	`69`	`+ $storage->save();`
	`70`	`+ $this->assertNotEquals('', $storage->getId());`
`67`	`71`	`}`
`68`	`72`
`69`	`73`	`public function testRegenerate()`