Fix qodana report

pimcore · Feb 8, 2024 · ab52399 · ab52399
1 parent 83ea88b
commit ab52399
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 8 deletions.
diff --git a/src/DependencyInjection/PimcoreStudioApiExtension.php b/src/DependencyInjection/PimcoreStudioApiExtension.php
@@ -51,7 +51,10 @@ public function load(array $configs, ContainerBuilder $container): void
         }
 
         // Pass the configuration to the custom normalizer
-        $container->setParameter('pimcore_studio_api.serializer.mapping.paths', $config['serializer']['mapping']['paths']);
+        $container->setParameter(
+            'pimcore_studio_api.serializer.mapping.paths',
+            $config['serializer']['mapping']['paths']
+        );
     }
 
     public function prepend(ContainerBuilder $container): void

diff --git a/src/State/ResetPasswordProcessor.php b/src/State/ResetPasswordProcessor.php
@@ -35,16 +35,20 @@
 final class ResetPasswordProcessor implements ProcessorInterface
 {
     public function __construct(
-        private UrlGeneratorInterface $router,
-        private RequestStack $requestStack,
-        private RateLimiterFactory $resetPasswordLimiter,
-        private EventDispatcherInterface $eventDispatcher
+        private readonly UrlGeneratorInterface $router,
+        private readonly RequestStack          $requestStack,
+        private readonly RateLimiterFactory    $resetPasswordLimiter,
+        private readonly EventDispatcherInterface $eventDispatcher
     ) {
     }
 
     public function process(mixed $data, Operation $operation, array $uriVariables = [], array $context = [])
     {
-        if (!$operation instanceof Post || $operation->getUriTemplate() !== '/users/reset-password' || !$data instanceof ResetPasswordRequest) {
+        if (
+            !$operation instanceof Post ||
+            !$data instanceof ResetPasswordRequest ||
+            $operation->getUriTemplate() !== '/users/reset-password'
+        ) {
             // wrong operation
             throw new OperationNotFoundException();
         }
@@ -57,7 +61,7 @@ public function process(mixed $data, Operation $operation, array $uriVariables =
         $currentRequest = $this->requestStack->getCurrentRequest();
         $limiter = $this->resetPasswordLimiter->create($currentRequest->getClientIp());
 
-        if (false === $limiter->consume(1)->isAccepted()) {
+        if (false === $limiter->consume()->isAccepted()) {
             throw new InvalidValueException('Rate limit exceeded');
         }
 
@@ -102,7 +106,10 @@ public function process(mixed $data, Operation $operation, array $uriVariables =
             if ($event->getSendMail()) {
                 $mail = Tool::getMail([$user->getEmail()], 'Pimcore lost password service');
                 $mail->setIgnoreDebugMode(true);
-                $mail->text("Login to pimcore and change your password using the following link. This temporary login link will expire in 24 hours: \r\n\r\n" . $loginUrl);
+                $mail->text(
+                    "Login to pimcore and change your password using the following link. ".
+                    "This temporary login link will expire in 24 hours: \r\n\r\n" . $loginUrl
+                );
                 $mail->send();
             }