Cleanup

[userbundle] / Handler / AuthenticationSuccessHandler.php

diff --git a/Handler/AuthenticationSuccessHandler.php b/Handler/AuthenticationSuccessHandler.php
index 961c1b15419b334e3eb2db30c2e30b06a71e0fb3..c57c68a03f9fa19c890504af6d199d7f8261422e 100644 (file)
--- a/Handler/AuthenticationSuccessHandler.php
+++ b/Handler/AuthenticationSuccessHandler.php
@@ -37,41 +37,28 @@ class AuthenticationSuccessHandler extends DefaultAuthenticationSuccessHandler {
        /**
         * Default options
         */
-    protected $defaultOptions = [
-        'always_use_default_target_path' => false,
-        'default_target_path' => '/',
-        'login_path' => '/login',
-        'target_path_parameter' => '_target_path',
-        'use_referer' => false,
-    ];
-
-       /**
-        * Options
-        */
-    protected $options;
-
-       /**
-        * Router instance
-        */
-       protected $router;
+       protected array $defaultOptions = [
+               'always_use_default_target_path' => false,
+               'default_target_path' => '/',
+               'login_path' => '/login',
+               'target_path_parameter' => '_target_path',
+               'use_referer' => false,
+       ];
 
        /**
         * {@inheritdoc}
         */
-       public function __construct(RouterInterface $router, array $options = []) {
+       public function __construct(protected RouterInterface $router, protected array $options = []) {
                //Set options
                $this->setOptions($options);
-
-               //Set router
-               $this->router = $router;
        }
 
-    /**
+       /**
+        * {@inheritdoc}
+        *
         * This is called when an interactive authentication attempt succeeds
         *
         * In use_referer case it will handle correctly when login_path is a route name or path
-        *
-        * {@inheritdoc}
         */
        public function onAuthenticationSuccess(Request $request, TokenInterface $token): Response {
                //Set login route