exception when registering bags for started sessions

xabbuh · xabbuh · commit c4a5b67a5af0 · 2016-03-05T08:51:23.000+01:00
diff --git a/src/Symfony/Component/HttpFoundation/Session/Storage/NativeSessionStorage.php b/src/Symfony/Component/HttpFoundation/Session/Storage/NativeSessionStorage.php
@@ -260,6 +260,10 @@ public function clear()
      */
     public function registerBag(SessionBagInterface $bag)
     {
+        if ($this->started) {
+            throw new \LogicException('Cannot register a bag when the session is already started.');
+        }
+
         $this->bags[$bag->getName()] = $bag;
     }
 
diff --git a/src/Symfony/Component/HttpFoundation/Tests/Session/Storage/NativeSessionStorageTest.php b/src/Symfony/Component/HttpFoundation/Tests/Session/Storage/NativeSessionStorageTest.php
@@ -83,6 +83,16 @@ public function testRegisterBagException()
         $storage->getBag('non_existing');
     }
 
+    /**
+     * @expectedException \LogicException
+     */
+    public function testRegisterBagForAStartedSessionThrowsException()
+    {
+        $storage = $this->getStorage();
+        $storage->start();
+        $storage->registerBag(new AttributeBag());
+    }
+
     public function testGetId()
     {
         $storage = $this->getStorage();

Original file line number	Diff line number	Diff line change
`@@ -260,6 +260,10 @@ public function clear()`
`260`	`260`	`*/`
`261`	`261`	`public function registerBag(SessionBagInterface $bag)`
`262`	`262`	`{`
	`263`	`+ if ($this->started) {`
	`264`	`+ throw new \LogicException('Cannot register a bag when the session is already started.');`
	`265`	`+ }`
	`266`	`+`
`263`	`267`	`$this->bags[$bag->getName()] = $bag;`
`264`	`268`	`}`
`265`	`269`