The ignoreAttributes itself should be ignored, too.

symfony · Jul 24, 2013 · dc1fff0 · dc1fff0
1 parent b45f18b
commit dc1fff0
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/Symfony/Bundle/FrameworkBundle/Controller/RedirectController.php b/src/Symfony/Bundle/FrameworkBundle/Controller/RedirectController.php
@@ -49,7 +49,7 @@ public function redirectAction(Request $request, $route, $permanent = false, $ig
         $attributes = array();
         if (false === $ignoreAttributes || is_array($ignoreAttributes)) {
             $attributes = $request->attributes->get('_route_params');
-            unset($attributes['route'], $attributes['permanent']);
+            unset($attributes['route'], $attributes['permanent'], $attributes['ignoreAttributes']);
             if ($ignoreAttributes) {
                 $attributes = array_diff_key($attributes, array_flip($ignoreAttributes));
             }

diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/Controller/RedirectControllerTest.php b/src/Symfony/Bundle/FrameworkBundle/Tests/Controller/RedirectControllerTest.php
@@ -53,6 +53,7 @@ public function testRoute($permanent, $ignoreAttributes, $expectedCode, $expecte
                 'route' => $route,
                 'permanent' => $permanent,
                 'additional-parameter' => 'value',
+                'ignoreAttributes' => $ignoreAttributes
             ),
         );