Fix CS

nucleos · Dec 3, 2023 · 2e2694e · 2e2694e
1 parent a7b9df4
commit 2e2694e
Show file tree

Hide file tree

Showing 9 changed files with 20 additions and 16 deletions.
diff --git a/phpstan-baseline.neon b/phpstan-baseline.neon
@@ -135,6 +135,11 @@ parameters:
 			count: 1
 			path: tests/DependencyInjection/NucleosUserExtensionTest.php
 
+		-
+			message: "#^Dynamic call to static method PHPUnit\\\\Framework\\\\TestCase\\:\\:createStub\\(\\)\\.$#"
+			count: 1
+			path: tests/Form/Type/ResettingFormTypeTest.php
+
 		-
 			message: "#^Method Nucleos\\\\UserBundle\\\\Tests\\\\Model\\\\BaseUserManagerTest\\:\\:getUserManager\\(\\) has parameter \\$args with no value type specified in iterable type array\\.$#"
 			count: 1

diff --git a/psalm-baseline.xml b/psalm-baseline.xml
@@ -1,14 +1,14 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<files psalm-version="5.7.7@e028ba46ba0d7f9a78bc3201c251e137383e145f">
-  <file src="src/Action/UpdateSecurityAction.php">
+<files psalm-version="5.16.0@2897ba636551a8cb61601cc26f6ccfbba6c36591">
+  <file src="src/Action/ResetAction.php">
     <InvalidNullableReturnType>
       <code>Response</code>
     </InvalidNullableReturnType>
     <NullableReturnStatement>
       <code><![CDATA[$event->getResponse()]]></code>
     </NullableReturnStatement>
   </file>
-  <file src="src/Action/ResetAction.php">
+  <file src="src/Action/UpdateSecurityAction.php">
     <InvalidNullableReturnType>
       <code>Response</code>
     </InvalidNullableReturnType>

diff --git a/src/Action/RequestResetAction.php b/src/Action/RequestResetAction.php
@@ -53,12 +53,17 @@ final class RequestResetAction
 
     private readonly int $retryTtl;
 
+    /**
+     * @var UserProviderInterface<UserInterface>
+     */
     private readonly UserProviderInterface $userProvider;
 
     private readonly TranslatorInterface $translator;
 
     /**
      * @SuppressWarnings(PHPMD.ExcessiveParameterList)
+     *
+     * @param UserProviderInterface<UserInterface> $userProvider
      */
     public function __construct(
         Environment $twig,

diff --git a/src/Event/GetResponseNullableUserEvent.php b/src/Event/GetResponseNullableUserEvent.php
@@ -13,6 +13,4 @@
 
 namespace Nucleos\UserBundle\Event;
 
-final class GetResponseNullableUserEvent extends GetResponseUserEvent
-{
-}
+final class GetResponseNullableUserEvent extends GetResponseUserEvent {}
diff --git a/src/Mailer/NoopResettingMailer.php b/src/Mailer/NoopResettingMailer.php
@@ -17,7 +17,5 @@
 
 final class NoopResettingMailer implements ResettingMailer
 {
-    public function sendResettingEmailMessage(UserInterface $user): void
-    {
-    }
+    public function sendResettingEmailMessage(UserInterface $user): void {}
 }
diff --git a/src/Security/UserProvider.php b/src/Security/UserProvider.php
@@ -21,6 +21,9 @@
 use Symfony\Component\Security\Core\User\UserInterface as SecurityUserInterface;
 use Symfony\Component\Security\Core\User\UserProviderInterface;
 
+/**
+ * @phpstan-implements UserProviderInterface<UserInterface>
+ */
 class UserProvider implements UserProviderInterface
 {
     protected UserManager $userManager;

diff --git a/src/Validator/Constraints/Pattern.php b/src/Validator/Constraints/Pattern.php
@@ -18,7 +18,6 @@
 
 /**
  * @Annotation
- *
  * @Target({"PROPERTY", "METHOD", "ANNOTATION"})
  */
 #[Attribute(Attribute::TARGET_PROPERTY | Attribute::TARGET_METHOD | Attribute::IS_REPEATABLE)]

diff --git a/tests/App/Entity/TestGroup.php b/tests/App/Entity/TestGroup.php
@@ -15,6 +15,4 @@
 
 use Nucleos\UserBundle\Model\Group;
 
-class TestGroup extends Group
-{
-}
+class TestGroup extends Group {}
diff --git a/tests/App/Entity/TestUser.php b/tests/App/Entity/TestUser.php
@@ -19,6 +19,4 @@
 /**
  * @phpstan-extends User<GroupInterface>
  */
-class TestUser extends User
-{
-}
+class TestUser extends User {}