summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFabien Potencier <fabien.potencier@gmail.com>2016-01-27 05:57:57 +0100
committerFabien Potencier <fabien.potencier@gmail.com>2016-01-27 05:57:57 +0100
commitca564737b36474a1e566d06585c3a01e849eb0b9 (patch)
treed2e002099f843d8ea48644da4039f426754d6b8c
parenta9c0831d7e4f6bd2a72edc235bb31e59ec05f995 (diff)
parentb27826b0ed6f72b12a66608bfb4074d36e1d4176 (diff)
downloadsymfony-security-ca564737b36474a1e566d06585c3a01e849eb0b9.zip
symfony-security-ca564737b36474a1e566d06585c3a01e849eb0b9.tar.gz
symfony-security-ca564737b36474a1e566d06585c3a01e849eb0b9.tar.bz2
minor #17480 Static code analysis (Koc)
This PR was merged into the 2.3 branch. Discussion ---------- Static code analysis | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - Things that done: * fix case in method calls * removed unused imports * use shorter concat where it possible * optimize some css * removed duplicated array keys * removed redurant return statements * removed one-time variables * do not pass arguments that not used in functions Commits ------- 8db691a Static code analysis
-rw-r--r--Acl/Dbal/MutableAclProvider.php2
-rw-r--r--Tests/Http/Firewall/RememberMeListenerTest.php2
-rw-r--r--Tests/Http/RememberMe/TokenBasedRememberMeServicesTest.php3
3 files changed, 3 insertions, 4 deletions
diff --git a/Acl/Dbal/MutableAclProvider.php b/Acl/Dbal/MutableAclProvider.php
index 074c4a0..3baaa99 100644
--- a/Acl/Dbal/MutableAclProvider.php
+++ b/Acl/Dbal/MutableAclProvider.php
@@ -856,7 +856,7 @@ QUERY;
*/
private function updateNewAceProperty($name, array $changes)
{
- list($old, $new) = $changes;
+ list(, $new) = $changes;
$sids = new \SplObjectStorage();
$classIds = new \SplObjectStorage();
diff --git a/Tests/Http/Firewall/RememberMeListenerTest.php b/Tests/Http/Firewall/RememberMeListenerTest.php
index ad96243..8316a8c 100644
--- a/Tests/Http/Firewall/RememberMeListenerTest.php
+++ b/Tests/Http/Firewall/RememberMeListenerTest.php
@@ -140,7 +140,7 @@ class RememberMeListenerTest extends \PHPUnit_Framework_TestCase
public function testSessionStrategy()
{
- list($listener, $tokenStorage, $service, $manager) = $this->getListener(false, true, true);
+ list($listener, $tokenStorage, $service, $manager) = $this->getListener();
$tokenStorage
->expects($this->once())
diff --git a/Tests/Http/RememberMe/TokenBasedRememberMeServicesTest.php b/Tests/Http/RememberMe/TokenBasedRememberMeServicesTest.php
index b988c7d..d1ec9b2 100644
--- a/Tests/Http/RememberMe/TokenBasedRememberMeServicesTest.php
+++ b/Tests/Http/RememberMe/TokenBasedRememberMeServicesTest.php
@@ -172,9 +172,8 @@ class TokenBasedRememberMeServicesTest extends \PHPUnit_Framework_TestCase
{
$service = $this->getService(null, array('name' => 'foo', 'path' => '/foo', 'domain' => 'foodomain.foo'));
$request = new Request();
- $response = new Response();
- $service->loginFail($request, $response);
+ $service->loginFail($request);
$cookie = $request->attributes->get(RememberMeServicesInterface::COOKIE_ATTR_NAME);
$this->assertTrue($cookie->isCleared());