Skip to content

Commit dc53845

Browse files
fabpotfabpot
committed
Merge branch '2.3' into 2.4
* 2.3: fixed CS [Process] fixed some volatile tests [HttpKernel] fixed a volatile test [HttpFoundation] fixed some volatile tests Use getPathname() instead of string casting to get BinaryFileReponse file path Conflicts: src/Symfony/Bundle/FrameworkBundle/Command/ConfigDumpReferenceCommand.php src/Symfony/Bundle/FrameworkBundle/Command/ContainerDebugCommand.php src/Symfony/Bundle/FrameworkBundle/EventListener/SessionListener.php src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/FrameworkExtensionTest.php src/Symfony/Component/ClassLoader/Tests/ApcUniversalClassLoaderTest.php src/Symfony/Component/Config/Definition/ReferenceDumper.php src/Symfony/Component/Config/Tests/Definition/Dumper/YamlReferenceDumperTest.php src/Symfony/Component/Console/Application.php src/Symfony/Component/Console/Tests/ApplicationTest.php src/Symfony/Component/Filesystem/Exception/IOException.php src/Symfony/Component/Form/Extension/Templating/TemplatingExtension.php src/Symfony/Component/Form/Tests/Extension/Core/DataTransformer/NumberToLocalizedStringTransformerTest.php src/Symfony/Component/HttpKernel/Debug/TraceableEventDispatcher.php src/Symfony/Component/HttpKernel/Tests/Bundle/BundleTest.php src/Symfony/Component/HttpKernel/Tests/Fragment/RoutableFragmentRendererTest.php src/Symfony/Component/HttpKernel/Tests/Profiler/Mock/RedisMock.php src/Symfony/Component/Intl/NumberFormatter/NumberFormatter.php src/Symfony/Component/PropertyAccess/PropertyAccessor.php src/Symfony/Component/PropertyAccess/Tests/PropertyAccessorCollectionTest.php src/Symfony/Component/PropertyAccess/Tests/PropertyAccessorTest.php src/Symfony/Component/Routing/Matcher/Dumper/ApacheMatcherDumper.php src/Symfony/Component/Routing/Tests/Annotation/RouteTest.php src/Symfony/Component/Routing/Tests/Loader/AnnotationClassLoaderTest.php src/Symfony/Component/Serializer/Encoder/XmlEncoder.php src/Symfony/Component/Validator/Constraints/CollectionValidator.php src/Symfony/Component/Validator/Tests/ExecutionContextTest.php
2 parents 70524d6 + 6d040c6 commit dc53845

File tree

8 files changed

+32
-32
lines changed

8 files changed

+32
-32
lines changed

‎DependencyInjection/MainConfiguration.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -337,11 +337,11 @@ private function addProvidersSection(ArrayNodeDefinition $rootNode)
337337
'memory' => array(
338338
'users' => array(
339339
'foo' => array('password' => 'foo', 'roles' => 'ROLE_USER'),
340-
'bar' => array('password' => 'bar', 'roles' => '[ROLE_USER, ROLE_ADMIN]')
340+
'bar' => array('password' => 'bar', 'roles' => '[ROLE_USER, ROLE_ADMIN]'),
341341
),
342-
)
342+
),
343343
),
344-
'my_entity_provider' => array('entity' => array('class' => 'SecurityBundle:User', 'property' => 'username'))
344+
'my_entity_provider' => array('entity' => array('class' => 'SecurityBundle:User', 'property' => 'username')),
345345
))
346346
->disallowNewKeysInSubsequentConfigs()
347347
->isRequired()
@@ -398,8 +398,8 @@ private function addEncodersSection(ArrayNodeDefinition $rootNode)
398398
'Acme\DemoBundle\Entity\User2' => array(
399399
'algorithm' => 'sha512',
400400
'encode_as_base64' => 'true',
401-
'iterations'=> 5000
402-
)
401+
'iterations' => 5000,
402+
),
403403
))
404404
->requiresAtLeastOneElement()
405405
->useAttributeAsKey('class')

‎DependencyInjection/Security/Factory/RememberMeFactory.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ public function create(ContainerBuilder $container, $id, $config, $userProvider,
6161

6262
if (isset($config['token_provider'])) {
6363
$rememberMeServices->addMethodCall('setTokenProvider', array(
64-
new Reference($config['token_provider'])
64+
new Reference($config['token_provider']),
6565
));
6666
}
6767

‎DependencyInjection/SecurityExtension.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@ private function configureDbalAclProvider(array $config, ContainerBuilder $conta
144144
->addTag('doctrine.event_listener', array(
145145
'connection' => $config['connection'],
146146
'event' => 'postGenerateSchema',
147-
'lazy' => true
147+
'lazy' => true,
148148
))
149149
;
150150

‎Tests/DependencyInjection/Fixtures/php/custom_acl_provider.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,5 @@
55
$container->loadFromExtension('security', array(
66
'acl' => array(
77
'provider' => 'foo',
8-
)
8+
),
99
));

‎Tests/DependencyInjection/Fixtures/php/merge.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,5 +16,5 @@
1616

1717
'role_hierarchy' => array(
1818
'FOO' => array('MOO'),
19-
)
19+
),
2020
));

‎Tests/DependencyInjection/Fixtures/php/merge_import.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@
55
'main' => array(
66
'form_login' => array(
77
'login_path' => '/login',
8-
)
9-
)
8+
),
9+
),
1010
),
1111
'role_hierarchy' => array(
1212
'FOO' => 'BAR',

‎Tests/DependencyInjection/SecurityExtensionTest.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,9 @@ public function testInvalidCheckPath()
3535
'pattern' => '/secured_area/.*',
3636
'form_login' => array(
3737
'check_path' => '/some_area/login_check',
38-
)
39-
)
40-
)
38+
),
39+
),
40+
),
4141
));
4242

4343
$container->compile();
@@ -59,8 +59,8 @@ public function testFirewallWithoutAuthenticationListener()
5959
'firewalls' => array(
6060
'some_firewall' => array(
6161
'pattern' => '/.*',
62-
)
63-
)
62+
),
63+
),
6464
));
6565

6666
$container->compile();

‎Tests/Functional/SecurityRoutingIntegrationTest.php

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -96,22 +96,22 @@ public function testSecurityConfigurationForMultipleIPAddresses($config)
9696
*/
9797
public function testSecurityConfigurationForExpression($config)
9898
{
99-
$allowedClient = $this->createClient(array('test_case' => 'StandardFormLogin', 'root_config' => $config), array('HTTP_USER_AGENT' => 'Firefox 1.0'));
100-
$this->assertAllowed($allowedClient, '/protected-via-expression');
101-
102-
$barredClient = $this->createClient(array('test_case' => 'StandardFormLogin', 'root_config' => $config), array());
103-
$this->assertRestricted($barredClient, '/protected-via-expression');
104-
105-
$allowedClient = $this->createClient(array('test_case' => 'StandardFormLogin', 'root_config' => $config), array());
106-
107-
$allowedClient->request('GET', '/protected-via-expression');
108-
$form = $allowedClient->followRedirect()->selectButton('login')->form();
109-
$form['_username'] = 'johannes';
110-
$form['_password'] = 'test';
111-
$allowedClient->submit($form);
112-
$this->assertRedirect($allowedClient->getResponse(), '/protected-via-expression');
113-
$this->assertAllowed($allowedClient, '/protected-via-expression');
114-
}
99+
$allowedClient = $this->createClient(array('test_case' => 'StandardFormLogin', 'root_config' => $config), array('HTTP_USER_AGENT' => 'Firefox 1.0'));
100+
$this->assertAllowed($allowedClient, '/protected-via-expression');
101+
102+
$barredClient = $this->createClient(array('test_case' => 'StandardFormLogin', 'root_config' => $config), array());
103+
$this->assertRestricted($barredClient, '/protected-via-expression');
104+
105+
$allowedClient = $this->createClient(array('test_case' => 'StandardFormLogin', 'root_config' => $config), array());
106+
107+
$allowedClient->request('GET', '/protected-via-expression');
108+
$form = $allowedClient->followRedirect()->selectButton('login')->form();
109+
$form['_username'] = 'johannes';
110+
$form['_password'] = 'test';
111+
$allowedClient->submit($form);
112+
$this->assertRedirect($allowedClient->getResponse(), '/protected-via-expression');
113+
$this->assertAllowed($allowedClient, '/protected-via-expression');
114+
}
115115

116116
private function assertAllowed($client, $path)
117117
{

0 commit comments

Comments
 (0)