Fix routing

nucleos · Aug 23, 2023 · 547d739 · 547d739
1 parent a21a12f
commit 547d739
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 3 deletions.
diff --git a/docs/installation.rst b/docs/installation.rst
@@ -199,7 +199,7 @@ in your application:
                 anonymous:    true
 
         access_control:
-            - { path: ^/change-password, role: IS_AUTHENTICATED_REMEMBERED }
+            - { path: ^/update-security, role: IS_AUTHENTICATED_REMEMBERED }
             # If you have an admin backend, uncomment the following line
             # - { path: ^/admin/, role: ROLE_ADMIN }
 

diff --git a/src/Resources/config/routing/update_security.php b/src/Resources/config/routing/update_security.php
@@ -14,10 +14,20 @@
 namespace Symfony\Component\Routing\Loader\Configurator;
 
 use Nucleos\UserBundle\Action\UpdateSecurityAction;
+use Symfony\Bundle\FrameworkBundle\Controller\RedirectController;
 
 return static function (RoutingConfigurator $routes): void {
-    $routes->add('nucleos_user_update_security', '/change-password')
+    $routes->add('nucleos_user_update_security', '/update-security')
         ->controller(UpdateSecurityAction::class)
         ->methods(['GET', 'POST'])
     ;
+
+    $routes->add('nucleos_user_update_security_legacy', '/change-password')
+        ->controller(RedirectController::class)
+        ->methods(['GET', 'POST'])
+        ->defaults([
+            'route'     => 'nucleos_user_update_security',
+            'permanent' => true,
+        ])
+    ;
 };
diff --git a/tests/Routing/RoutingTest.php b/tests/Routing/RoutingTest.php
@@ -50,7 +50,9 @@ public function testLoadRouting(string $routeName, string $path, array $methods)
      */
     public static function provideLoadRoutingCases(): iterable
     {
-        yield ['nucleos_user_update_security', '/change-password', ['GET', 'POST']];
+        yield ['nucleos_user_update_security', '/update-security', ['GET', 'POST']];
+
+        yield ['nucleos_user_update_security_legacy', '/change-password', ['GET', 'POST']];
 
         yield ['nucleos_user_resetting_request', '/resetting/request', ['GET', 'POST']];