From d05cffb40916426f0252f5a635ea3c8fa1ed1819 Mon Sep 17 00:00:00 2001 From: Vladimir Reznichenko Date: Wed, 11 Mar 2015 21:36:41 +0100 Subject: [PATCH] [2.3] Static Code Analysis for Components --- src/Symfony/Component/BrowserKit/Response.php | 3 ++- src/Symfony/Component/Security/Acl/Dbal/MutableAclProvider.php | 3 --- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/src/Symfony/Component/BrowserKit/Response.php b/src/Symfony/Component/BrowserKit/Response.php index d00836fa83..7a7130b95e 100644 --- a/src/Symfony/Component/BrowserKit/Response.php +++ b/src/Symfony/Component/BrowserKit/Response.php @@ -123,8 +123,9 @@ class Response */ public function getHeader($header, $first = true) { + $normalizedHeader = str_replace('-', '_', strtolower($header)); foreach ($this->headers as $key => $value) { - if (str_replace('-', '_', strtolower($key)) == str_replace('-', '_', strtolower($header))) { + if (str_replace('-', '_', strtolower($key)) === $normalizedHeader) { if ($first) { return is_array($value) ? (count($value) ? $value[0] : '') : $value; } diff --git a/src/Symfony/Component/Security/Acl/Dbal/MutableAclProvider.php b/src/Symfony/Component/Security/Acl/Dbal/MutableAclProvider.php index a710bf75e7..c34d752942 100644 --- a/src/Symfony/Component/Security/Acl/Dbal/MutableAclProvider.php +++ b/src/Symfony/Component/Security/Acl/Dbal/MutableAclProvider.php @@ -785,7 +785,6 @@ QUERY; { $sids = new \SplObjectStorage(); $classIds = new \SplObjectStorage(); - $currentIds = array(); foreach ($changes[1] as $field => $new) { for ($i = 0, $c = count($new); $i<$c; $i++) { $ace = $new[$i]; @@ -813,8 +812,6 @@ QUERY; $aceIdProperty = new \ReflectionProperty('Symfony\Component\Security\Acl\Domain\Entry', 'id'); $aceIdProperty->setAccessible(true); $aceIdProperty->setValue($ace, (int) $aceId); - } else { - $currentIds[$ace->getId()] = true; } } }