From 522c2cc48f7a6188b14c125b04371d20b672ada8 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Rapha=C3=ABl=20Gertz?= Date: Thu, 7 Mar 2024 18:48:13 +0100 Subject: [PATCH 1/1] Cleanup --- Controller/AbstractController.php | 8 ++++---- DependencyInjection/Configuration.php | 4 ++-- Exception/UnactivatedException.php | 4 ++-- Factory.php | 1 + Handler/AuthenticationSuccessHandler.php | 20 ++++++++++---------- Repository.php | 4 ++-- 6 files changed, 21 insertions(+), 20 deletions(-) diff --git a/Controller/AbstractController.php b/Controller/AbstractController.php index 8a0a2c8..9f771e6 100644 --- a/Controller/AbstractController.php +++ b/Controller/AbstractController.php @@ -41,9 +41,9 @@ use Rapsys\PackBundle\Util\SluggerUtil; use Rapsys\UserBundle\RapsysUserBundle; /** - * Provides common features needed in controllers. - * * {@inheritdoc} + * + * Provides common features needed in controllers. */ abstract class AbstractController extends BaseAbstractController implements ServiceSubscriberInterface { /** @@ -268,9 +268,9 @@ abstract class AbstractController extends BaseAbstractController implements Serv } /** - * Renders a view - * * {@inheritdoc} + * + * Renders a view */ protected function render(string $view, array $parameters = [], Response $response = null): Response { //Create response when null diff --git a/DependencyInjection/Configuration.php b/DependencyInjection/Configuration.php index 7c053c9..80a3552 100644 --- a/DependencyInjection/Configuration.php +++ b/DependencyInjection/Configuration.php @@ -17,11 +17,11 @@ use Symfony\Component\Config\Definition\ConfigurationInterface; use Rapsys\UserBundle\RapsysUserBundle; /** + * {@inheritdoc} + * * This is the class that validates and merges configuration from your app/config files. * * @link http://symfony.com/doc/current/cookbook/bundles/configuration.html - * - * {@inheritdoc} */ class Configuration implements ConfigurationInterface { /** diff --git a/Exception/UnactivatedException.php b/Exception/UnactivatedException.php index cbf367d..b157149 100644 --- a/Exception/UnactivatedException.php +++ b/Exception/UnactivatedException.php @@ -14,9 +14,9 @@ namespace Rapsys\UserBundle\Exception; use Symfony\Component\Security\Core\Exception\AccountStatusException; /** - * UnactivatedException is thrown when the user account is unactivated. - * * {@inheritdoc} + * + * UnactivatedException is thrown when the user account is unactivated. */ class UnactivatedException extends AccountStatusException { /** diff --git a/Factory.php b/Factory.php index e2d05d5..abfbc5f 100644 --- a/Factory.php +++ b/Factory.php @@ -14,6 +14,7 @@ namespace Rapsys\UserBundle; use Doctrine\ORM\EntityManagerInterface; use Doctrine\ORM\Repository\RepositoryFactory as RepositoryFactoryInterface; use Doctrine\Persistence\ObjectRepository; + use Symfony\Component\HttpFoundation\RequestStack; use Symfony\Component\Routing\RouterInterface; use Symfony\Contracts\Translation\TranslatorInterface; diff --git a/Handler/AuthenticationSuccessHandler.php b/Handler/AuthenticationSuccessHandler.php index 11e8325..c57c68a 100644 --- a/Handler/AuthenticationSuccessHandler.php +++ b/Handler/AuthenticationSuccessHandler.php @@ -37,13 +37,13 @@ class AuthenticationSuccessHandler extends DefaultAuthenticationSuccessHandler { /** * Default options */ - protected array $defaultOptions = [ - 'always_use_default_target_path' => false, - 'default_target_path' => '/', - 'login_path' => '/login', - 'target_path_parameter' => '_target_path', - 'use_referer' => false, - ]; + protected array $defaultOptions = [ + 'always_use_default_target_path' => false, + 'default_target_path' => '/', + 'login_path' => '/login', + 'target_path_parameter' => '_target_path', + 'use_referer' => false, + ]; /** * {@inheritdoc} @@ -53,12 +53,12 @@ class AuthenticationSuccessHandler extends DefaultAuthenticationSuccessHandler { $this->setOptions($options); } - /** + /** + * {@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 diff --git a/Repository.php b/Repository.php index b21b0ca..c7a654b 100644 --- a/Repository.php +++ b/Repository.php @@ -20,9 +20,9 @@ use Symfony\Contracts\Translation\TranslatorInterface; use Rapsys\PackBundle\Util\SluggerUtil; /** - * Repository - * * {@inheritdoc} + * + * Repository */ class Repository extends EntityRepository { /** -- 2.41.1