Address CodeStyle issues

OpenConext · Aug 28, 2024 · bbc84ff · bbc84ff
1 parent edbe853
commit bbc84ff
Show file tree

Hide file tree

Showing 8 changed files with 13 additions and 22 deletions.
diff --git a/src/OpenConext/Kernel.php b/src/OpenConext/Kernel.php
@@ -38,5 +38,4 @@ public function getLogDir(): string
     {
         return $this->getCacheDir();
     }
-
 }
diff --git a/src/OpenConext/UserLifecycle/Application/QueryHandler/FindUserInformationQueryHandler.php b/src/OpenConext/UserLifecycle/Application/QueryHandler/FindUserInformationQueryHandler.php
@@ -33,8 +33,7 @@ public function __construct(
 
     public function handle(
         FindUserInformationQuery $query,
-    ): InformationResponseInterface
-    {
+    ): InformationResponseInterface {
         return $this->informationService->readInformationFor($query->getCollabPersonId());
     }
 }
diff --git a/src/OpenConext/UserLifecycle/Application/QueryHandler/InactiveUsersQueryHandler.php b/src/OpenConext/UserLifecycle/Application/QueryHandler/InactiveUsersQueryHandler.php
@@ -33,8 +33,7 @@ public function __construct(
 
     public function handle(
         InactiveUsersQuery $query,
-    ): LastLoginCollectionInterface
-    {
+    ): LastLoginCollectionInterface {
         return $this->lastLoginRepository->findDeprovisionCandidates($query->getInactivityPeriod());
     }
 }
diff --git a/src/OpenConext/UserLifecycle/Application/Service/DeprovisionService.php b/src/OpenConext/UserLifecycle/Application/Service/DeprovisionService.php
@@ -55,8 +55,7 @@ public function deprovision(
         ProgressReporterInterface $progressReporter,
         string $personId,
         bool $dryRun = false,
-    ): InformationResponseCollectionInterface
-    {
+    ): InformationResponseCollectionInterface {
         $this->logger->debug('Received a request to deprovision a user.');
 
         $collabPersonId = $this->buildCollabPersonId($personId);

diff --git a/src/OpenConext/UserLifecycle/Domain/ValueObject/Client/Name.php b/src/OpenConext/UserLifecycle/Domain/ValueObject/Client/Name.php
@@ -29,7 +29,6 @@ public function __construct(
         private readonly string $name,
     ) {
         Assert::notEmpty(trim($name), 'Name can not be empty.');
-
     }
 
     public function getName(): string

diff --git a/src/OpenConext/UserLifecycle/Domain/ValueObject/CollabPersonId.php b/src/OpenConext/UserLifecycle/Domain/ValueObject/CollabPersonId.php
@@ -31,9 +31,7 @@ class CollabPersonId implements \Stringable
     public function __construct(
         string $collabUserId,
     ) {
-
-       $collabUserId = trim($collabUserId);
-
+        $collabUserId = trim($collabUserId);
         if (empty($collabUserId)) {
             throw new InvalidCollabPersonIdException('The collabPersonId must be a non empty string');
         }

diff --git a/src/OpenConext/UserLifecycle/Domain/ValueObject/InactivityPeriod.php b/src/OpenConext/UserLifecycle/Domain/ValueObject/InactivityPeriod.php
@@ -25,15 +25,13 @@
 
 class InactivityPeriod
 {
-
     public function __construct(
         #[Autowire(param: 'inactivity_period')]
         private readonly int $inactivityPeriodInMonths,
     ) {
         if ($inactivityPeriodInMonths <= 0) {
             throw new InvalidInactivityPeriodException('The inactivity period must be an positive integer value');
         }
-
     }
 
     public function getInactivityPeriodInMonths(): int

diff --git a/...penConext/UserLifecycle/Infrastructure/UserLifecycleBundle/Command/DeprovisionCommand.php b/...penConext/UserLifecycle/Infrastructure/UserLifecycleBundle/Command/DeprovisionCommand.php
@@ -44,7 +44,7 @@ public function __construct(
         private readonly ProgressReporterInterface $progressReporter,
         private readonly LoggerInterface $logger,
     ) {
-        parent::__construct(null);
+        parent::__construct();
     }
 
     protected function configure(): void
@@ -108,14 +108,14 @@ protected function execute(
             );
         }
         return $this->executeSingleUser(
-                $input,
-                $output,
-                $userIdInput,
-                $dryRun,
-                $noInteraction,
-                $outputOnlyJson,
-                $prettyJson,
-            );
+            $input,
+            $output,
+            $userIdInput,
+            $dryRun,
+            $noInteraction,
+            $outputOnlyJson,
+            $prettyJson,
+        );
     }
 
     private function executeBatch(