Fix CS

Author: core23

src/Command/ActivateUserCommand.php

@@ -56,7 +56,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
         $this->userManipulator->activate($username);
 
-        $output->writeln(sprintf('User "%s" has been activated.', $username));
+        $output->writeln(\sprintf('User "%s" has been activated.', $username));
 
         return 0;
     }

src/Command/ChangePasswordCommand.php

@@ -65,7 +65,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
         $this->userManipulator->changePassword($username, $password);
 
-        $output->writeln(sprintf('Changed password for user <comment>%s</comment>', $username));
+        $output->writeln(\sprintf('Changed password for user <comment>%s</comment>', $username));
 
         return 0;
     }

src/Command/CreateUserCommand.php

@@ -80,7 +80,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
         $this->userManipulator->create($username, $password, $email, !$inactive, $superadmin);
 
-        $output->writeln(sprintf('Created user <comment>%s</comment>', $username));
+        $output->writeln(\sprintf('Created user <comment>%s</comment>', $username));
 
         return 0;
     }

src/Command/DeactivateUserCommand.php

@@ -56,7 +56,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
         $this->userManipulator->deactivate($username);
 
-        $output->writeln(sprintf('User "%s" has been deactivated.', $username));
+        $output->writeln(\sprintf('User "%s" has been deactivated.', $username));
 
         return 0;
     }

src/Command/DemoteUserCommand.php

@@ -40,11 +40,11 @@ protected function executeRoleCommand(UserManipulator $manipulator, OutputInterf
     {
         if ($super) {
             $manipulator->demote($username);
-            $output->writeln(sprintf('User "%s" has been demoted as a simple user. This change will not apply until the user logs out and back in again.', $username));
+            $output->writeln(\sprintf('User "%s" has been demoted as a simple user. This change will not apply until the user logs out and back in again.', $username));
         } elseif ($manipulator->removeRole($username, $role)) {
-            $output->writeln(sprintf('Role "%s" has been removed from user "%s". This change will not apply until the user logs out and back in again.', $role, $username));
+            $output->writeln(\sprintf('Role "%s" has been removed from user "%s". This change will not apply until the user logs out and back in again.', $role, $username));
         } else {
-            $output->writeln(sprintf('User "%s" didn\'t have "%s" role.', $username, $role));
+            $output->writeln(\sprintf('User "%s" didn\'t have "%s" role.', $username, $role));
         }
     }
 }

src/Command/PromoteUserCommand.php

@@ -40,11 +40,11 @@ protected function executeRoleCommand(UserManipulator $manipulator, OutputInterf
     {
         if ($super) {
             $manipulator->promote($username);
-            $output->writeln(sprintf('User "%s" has been promoted as a super administrator. This change will not apply until the user logs out and back in again.', $username));
+            $output->writeln(\sprintf('User "%s" has been promoted as a super administrator. This change will not apply until the user logs out and back in again.', $username));
         } elseif ($manipulator->addRole($username, $role)) {
-            $output->writeln(sprintf('Role "%s" has been added to user "%s". This change will not apply until the user logs out and back in again.', $role, $username));
+            $output->writeln(\sprintf('Role "%s" has been added to user "%s". This change will not apply until the user logs out and back in again.', $role, $username));
         } else {
-            $output->writeln(sprintf('User "%s" did already have "%s" role.', $username, $role));
+            $output->writeln(\sprintf('User "%s" did already have "%s" role.', $username, $role));
         }
     }
 }

src/DependencyInjection/NucleosUserExtension.php

@@ -73,7 +73,7 @@ public function load(array $configs, ContainerBuilder $container): void
                 $loader->load('doctrine.php');
                 $container->setAlias('nucleos_user.doctrine_registry', new Alias(self::$doctrineDrivers[$config['db_driver']]['registry'], false));
             } else {
-                $loader->load(sprintf('%s.php', $config['db_driver']));
+                $loader->load(\sprintf('%s.php', $config['db_driver']));
             }
             $container->setParameter($this->getAlias().'.backend_type_'.$config['db_driver'], true);
         }
@@ -84,7 +84,7 @@ public function load(array $configs, ContainerBuilder $container): void
         }
 
         foreach (['validator', 'security', 'util', 'mailer', 'listeners', 'commands'] as $basename) {
-            $loader->load(sprintf('%s.php', $basename));
+            $loader->load(\sprintf('%s.php', $basename));
         }
 
         if (!$config['use_authentication_listener']) {
@@ -186,7 +186,7 @@ private function remapParametersNamespaces(array $config, ContainerBuilder $cont
                 $this->remapParameters($namespaceConfig, $container, $map);
             } else {
                 foreach ($namespaceConfig as $name => $value) {
-                    $container->setParameter(sprintf($map, $name), $value);
+                    $container->setParameter(\sprintf($map, $name), $value);
                 }
             }
         }
@@ -234,7 +234,7 @@ private function loadGroups(array $config, ContainerBuilder $container, FileLoad
             if (isset(self::$doctrineDrivers[$dbDriver])) {
                 $loader->load('doctrine_group.php');
             } else {
-                $loader->load(sprintf('%s_group.php', $dbDriver));
+                $loader->load(\sprintf('%s_group.php', $dbDriver));
             }
         }
 

src/Event/GetResponseLoginEvent.php

@@ -21,7 +21,7 @@ final class GetResponseLoginEvent extends Event
 
     private ?Response $response = null;
 
-    public function __construct(Request $request = null)
+    public function __construct(?Request $request = null)
     {
         $this->request = $request;
     }

src/Event/UserEvent.php

@@ -23,7 +23,7 @@ class UserEvent extends Event
 
     protected readonly UserInterface $user;
 
-    public function __construct(UserInterface $user, Request $request = null)
+    public function __construct(UserInterface $user, ?Request $request = null)
     {
         $this->user    = $user;
         $this->request = $request;

src/Model/User.php

@@ -208,7 +208,7 @@ public function setPlainPassword(?string $password): void
         $this->passwordRequestedAt = null;
     }
 
-    public function setLastLogin(DateTime $time = null): void
+    public function setLastLogin(?DateTime $time = null): void
     {
         $this->lastLogin = $time;
     }
@@ -218,7 +218,7 @@ public function setConfirmationToken(?string $confirmationToken): void
         $this->confirmationToken = $confirmationToken;
     }
 
-    public function setPasswordRequestedAt(DateTime $date = null): void
+    public function setPasswordRequestedAt(?DateTime $date = null): void
     {
         $this->passwordRequestedAt = $date;
     }