Commit23d917f

authored and

committed

[HttpFoundation] Fix forward-compat of NativeSessionStorage with PHP 7.2

1 parentefb4891 commit23d917fCopy full SHA for 23d917f

File tree

+26

-1

lines changed

src/Symfony/Component/HttpFoundation
- Session/Storage
  - NativeSessionStorage.php
- Tests/Session/Storage
  - NativeSessionStorageTest.php

+26

-1

lines changed

Lines changed: 6 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -102,6 +102,12 @@ class NativeSessionStorage implements SessionStorageInterface`
`102`	`102`	`*/`
`103`	`103`	`publicfunction__construct(array$options =array(),$handler =null,MetadataBag$metaBag =null)`
`104`	`104`	`{`
	`105`	`+$this->setMetadataBag($metaBag);`
	`106`	`+`
	`107`	`+if (\PHP_SESSION_ACTIVE ===session_status()) {`
	`108`	`+return;`
	`109`	`+ }`
	`110`	`+`
`105`	`111`	`$options +=array(`
`106`	`112`	`// disable by default because it's managed by HeaderBag (if used)`
`107`	`113`	`'cache_limiter' =>'',`
`@@ -114,7 +120,6 @@ public function __construct(array $options = array(), $handler = null, MetadataB`
`114`	`120`	`register_shutdown_function('session_write_close');`
`115`	`121`	`}`
`116`	`122`
`117`		`-$this->setMetadataBag($metaBag);`
`118`	`123`	`$this->setOptions($options);`
`119`	`124`	`$this->setSaveHandler($handler);`
`120`	`125`	`}`

Lines changed: 20 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -270,4 +270,24 @@ public function testRestart()`
`270`	`270`	`$this->assertSame($id,$storage->getId(),'Same session ID after restarting');`
`271`	`271`	`$this->assertSame(7,$storage->getBag('attributes')->get('lucky'),'Data still available');`
`272`	`272`	`}`
	`273`	`+`
	`274`	`+publicfunctiontestCanCreateNativeSessionStorageWhenSessionAlreadyStarted()`
	`275`	`+ {`
	`276`	`+session_start();`
	`277`	`+$this->getStorage();`
	`278`	`+`
	`279`	+// Assert no exception has been thrown by `getStorage()`
	`280`	`+$this->assertTrue(true);`
	`281`	`+ }`
	`282`	`+`
	`283`	`+publicfunctiontestSetSessionOptionsOnceSessionStartedIsIgnored()`
	`284`	`+ {`
	`285`	`+session_start();`
	`286`	`+$this->getStorage(array(`
	`287`	`+'name' =>'something-else',`
	`288`	`+ ));`
	`289`	`+`
	`290`	+// Assert no exception has been thrown by `getStorage()`
	`291`	`+$this->assertTrue(true);`
	`292`	`+ }`
`273`	`293`	`}`

Comments

(0)