Sep 21, 2018 · Sep 21, 2018 · Sep 21, 2018 · Sep 21, 2018
diff --git a/controller.rst b/controller.rst
        // redirects to a route with parameters
        return $this->redirectToRoute('blog_show', array('slug' => 'my-page'));

        // redirects to a route and mantains the original query string parameters
        return $this->redirectToRoute('blog_show', $request->query->all());

        // redirects externally
        return $this->redirect('http://symfony.com/doc');
    }
Original file line number	Diff line number	Diff line change
Expand Up		@@ -176,6 +176,9 @@ and ``redirect()`` methods::
		// redirects to a route with parameters
		return $this->redirectToRoute('blog_show', array('slug' => 'my-page'));

		// redirects to a route and mantains the original query string parameters
		return $this->redirectToRoute('blog_show', $request->query->all());
Copy link Contributor linaoriSep 21, 2018 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. This change looks good! I'm just thinking, would it be worth it having the base controller support the following syntax? return$this->redirectToRoute('blog_show',$request->query); Copy link MemberAuthor javiereguiluzSep 21, 2018 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. In my opinion it's better to not add a new way to get the query string params. Let's keep things simple and minimize the number of methods/features to learn. Thanks! linaori reacted with thumbs up emoji

		// redirects externally
		return $this->redirect('http://symfony.com/doc');
		}
Expand Down