Missing return statement in example

snebes · weaverryan · commit 7b5c7d089e64 · 2019-10-17T21:40:12.000-04:00
diff --git a/src/Security/SecurityControllerBuilder.php b/src/Security/SecurityControllerBuilder.php
@@ -36,7 +36,7 @@ public function addLoginMethod(ClassSourceManipulator $manipulator)
         $manipulator->addMethodBody($loginMethodBuilder, <<<'CODE'
 <?php
 // if ($this->getUser()) {
-//    $this->redirectToRoute('target_path');
+//     return $this->redirectToRoute('target_path');
 // }
 CODE
         );
diff --git a/tests/Security/fixtures/expected/SecurityController_login.php b/tests/Security/fixtures/expected/SecurityController_login.php
@@ -15,7 +15,7 @@ class SecurityController extends AbstractController
     public function login(AuthenticationUtils $authenticationUtils): Response
     {
         // if ($this->getUser()) {
-        //    $this->redirectToRoute('target_path');
+        //     return $this->redirectToRoute('target_path');
         // }
 
         // get the login error if there is one
diff --git a/tests/Security/fixtures/expected/SecurityController_login_logout.php b/tests/Security/fixtures/expected/SecurityController_login_logout.php
@@ -15,7 +15,7 @@ class SecurityController extends AbstractController
     public function login(AuthenticationUtils $authenticationUtils): Response
     {
         // if ($this->getUser()) {
-        //    $this->redirectToRoute('target_path');
+        //     return $this->redirectToRoute('target_path');
         // }
 
         // get the login error if there is one

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ class SecurityController extends AbstractController`
`15`	`15`	`public function login(AuthenticationUtils $authenticationUtils): Response`
`16`	`16`	`{`
`17`	`17`	`// if ($this->getUser()) {`
`18`		`- // $this->redirectToRoute('target_path');`
	`18`	`+ // return $this->redirectToRoute('target_path');`
`19`	`19`	`// }`
`20`	`20`
`21`	`21`	`// get the login error if there is one`