Skip to content

Commit 3513e77

Browse files
Merge branch '3.4'
* 3.4: [HttpKernel] Disable container inlining when legacy inlining has been used Let getFlashes starts the session Update default translations path [FrameworkBundle] Ignore failures when removing the old cache dir
2 parents 5695b11 + bd27a85 commit 3513e77

File tree

2 files changed

+6
-11
lines changed

2 files changed

+6
-11
lines changed

AppVariable.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -156,10 +156,9 @@ public function getDebug()
156156
*/
157157
public function getFlashes($types = null)
158158
{
159-
// needed to avoid starting the session automatically when looking for flash messages
160159
try {
161160
$session = $this->getSession();
162-
if (null === $session || !$session->isStarted()) {
161+
if (null === $session) {
163162
return array();
164163
}
165164
} catch (\RuntimeException $e) {

Tests/AppVariableTest.php

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -173,12 +173,8 @@ public function testGetFlashesWithNoRequest()
173173
*/
174174
public function testGetFlashesWithNoSessionStarted()
175175
{
176-
$request = $this->getMockBuilder('Symfony\Component\HttpFoundation\Request')->getMock();
177-
$request->method('getSession')->willReturn(new Session());
178-
179-
$this->setRequestStack($request);
180-
181-
$this->assertEquals(array(), $this->appVariable->getFlashes());
176+
$flashMessages = $this->setFlashMessages(false);
177+
$this->assertEquals($flashMessages, $this->appVariable->getFlashes());
182178
}
183179

184180
/**
@@ -256,7 +252,7 @@ protected function setTokenStorage($user)
256252
$token->method('getUser')->willReturn($user);
257253
}
258254

259-
private function setFlashMessages()
255+
private function setFlashMessages($sessionHasStarted = true)
260256
{
261257
$flashMessages = array(
262258
'notice' => array('Notice #1 message'),
@@ -266,8 +262,8 @@ private function setFlashMessages()
266262
$flashBag = new FlashBag();
267263
$flashBag->initialize($flashMessages);
268264

269-
$session = $this->getMockBuilder('Symfony\Component\HttpFoundation\Session\Session')->getMock();
270-
$session->method('isStarted')->willReturn(true);
265+
$session = $this->getMockBuilder('Symfony\Component\HttpFoundation\Session\Session')->disableOriginalConstructor()->getMock();
266+
$session->method('isStarted')->willReturn($sessionHasStarted);
271267
$session->method('getFlashBag')->willReturn($flashBag);
272268

273269
$request = $this->getMockBuilder('Symfony\Component\HttpFoundation\Request')->getMock();

0 commit comments

Comments
 (0)