[SecurityBundle] avoid unneeded work

Author: fabpot

DependencyInjection/SecurityExtension.php

@@ -567,17 +567,17 @@ private function createSwitchUserListener($container, $id, $config, $defaultProv
 
     private function createRequestMatcher($container, $path = null, $host = null, $methods = array(), $ip = null, array $attributes = array())
     {
+        if ($methods) {
+            $methods = array_map('strtoupper', (array) $methods);
+        }
+
         $serialized = serialize(array($path, $host, $methods, $ip, $attributes));
         $id = 'security.request_matcher.'.md5($serialized).sha1($serialized);
 
         if (isset($this->requestMatchers[$id])) {
             return $this->requestMatchers[$id];
         }
 
-        if ($methods) {
-            $methods = array_map('strtoupper', (array) $methods);
-        }
-
         // only add arguments that are necessary
         $arguments = array($path, $host, $methods, $ip, $attributes);
         while (count($arguments) > 0 && !end($arguments)) {