diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[SecurityBundle] fix constructor argument index

Browse Source 
As @excelwebzone pointed out in fb9dc6a, `catchExceptions` is the 6th
argument of the RememberMeListener constructor.
This commit is contained in:
Christian Flothmann 2014-07-26 20:45:30 +02:00
parent e4dd60ae0d
commit a32646e35b
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Symfony/Bundle/SecurityBundle/DependencyInjection/Security/Factory/RememberMeFactory.php
View File

@ -102,7 +102,7 @@ class RememberMeFactory implements SecurityFactoryInterface
$listenerId = 'security.authentication.listener.rememberme.'.$id;
$listener = $container->setDefinition($listenerId, new DefinitionDecorator('security.authentication.listener.rememberme'));
$listener->replaceArgument(1, new Reference($rememberMeServicesId));
$listener->replaceArgument(4, $config['catch_exceptions']);
$listener->replaceArgument(5, $config['catch_exceptions']);
return array($authProviderId, $listenerId, $defaultEntryPoint);
}

4
src/Symfony/Bundle/SecurityBundle/Tests/DependencyInjection/CompleteConfigurationTest.php
View File

@ -223,7 +223,7 @@ abstract class CompleteConfigurationTest extends \PHPUnit_Framework_TestCase
public function testRememberMeThrowExceptionsDefault()
{
$container = $this->getContainer('container1');
$this->assertTrue($container->getDefinition('security.authentication.listener.rememberme.secure')->getArgument(4));
$this->assertTrue($container->getDefinition('security.authentication.listener.rememberme.secure')->getArgument(5));
}
public function testRememberMeThrowExceptions()
@ -231,7 +231,7 @@ abstract class CompleteConfigurationTest extends \PHPUnit_Framework_TestCase
$container = $this->getContainer('remember_me_options');
$service = $container->getDefinition('security.authentication.listener.rememberme.main');
$this->assertEquals('security.authentication.rememberme.services.persistent.main', $service->getArgument(1));
$this->assertFalse($service->getArgument(4));
$this->assertFalse($service->getArgument(5));
}
protected function getContainer($file)