OpenConext / Stepup-Middleware

src/Surfnet/StepupMiddleware/ApiBundle/Identity/Value/AuthorizedInstitutionCollection.php

Spacing +5 added lines, -5 removed lines

@@ -45,18 +45,18 @@
 
         /** @var Institution $institution */
         foreach ($raInstitutions as $institution) {
-            $collection->authorizations[(string)$institution][] = AuthorityRole::ROLE_RA;
+            $collection->authorizations[(string) $institution][] = AuthorityRole::ROLE_RA;
         }
         if ($raaInstitutions instanceof InstitutionCollection) {
             /** @var Institution $institution */
             foreach ($raaInstitutions as $institution) {
                 // Override existing lower role
-                if (isset($collection->authorizations[(string)$institution])
-                    && in_array(AuthorityRole::ROLE_RA, $collection->authorizations[(string)$institution])
+                if (isset($collection->authorizations[(string) $institution])
+                    && in_array(AuthorityRole::ROLE_RA, $collection->authorizations[(string) $institution])
                 ) {
-                    $collection->authorizations[(string)$institution] = [];
+                    $collection->authorizations[(string) $institution] = [];
                 }
-                $collection->authorizations[(string)$institution][] = AuthorityRole::ROLE_RAA;
+                $collection->authorizations[(string) $institution][] = AuthorityRole::ROLE_RAA;
             }
         }
         return $collection;

src/Surfnet/StepupMiddleware/ManagementBundle/Controller/InstitutionConfigurationController.php

Spacing +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
         foreach ($configuration as $institution => $options) {
             assert(is_array($options));
             $command = new ReconfigureInstitutionConfigurationOptionsCommand();
-            $command->UUID = (string)Uuid::uuid4();
+            $command->UUID = (string) Uuid::uuid4();
             $command->institution = $institution;
             $command->useRaLocationsOption = $options['use_ra_locations'];
             $command->showRaaContactInformationOption = $options['show_raa_contact_information'];

src/Surfnet/StepupMiddleware/ApiBundle/Identity/Repository/VettedSecondFactorRepository.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
         if ($query->identityId instanceof IdentityId) {
             $queryBuilder
                 ->andWhere('sf.identityId = :identityId')
-                ->setParameter('identityId', (string)$query->identityId);
+                ->setParameter('identityId', (string) $query->identityId);
         }
 
         return $queryBuilder->getQuery();

src/Surfnet/StepupMiddleware/ApiBundle/Identity/Projector/AuditLogProjector.php

Spacing +8 added lines, -8 removed lines

@@ -82,7 +82,7 @@ 
 
         $metadata = $domainMessage->getMetadata()->serialize();
         $entry = new AuditLogEntry();
-        $entry->id = (string)Uuid::uuid4();
+        $entry->id = (string) Uuid::uuid4();
 
         if (isset($metadata['actorId'])) {
             $actor = $this->identityRepository->find($metadata['actorId']);
@@ -106,36 +106,36 @@ 
             $entry->actorInstitution = new Institution($metadata['actorInstitution']);
         }
 
-        $entry->identityId = (string)$auditLogMetadata->identityId;
+        $entry->identityId = (string) $auditLogMetadata->identityId;
         $entry->identityInstitution = $auditLogMetadata->identityInstitution;
         $entry->event = $event::class;
         $entry->recordedOn = new DateTime(new CoreDateTime($domainMessage->getRecordedOn()->toString()));
 
         if ($auditLogMetadata->secondFactorId instanceof SecondFactorId) {
-            $entry->secondFactorId = (string)$auditLogMetadata->secondFactorId;
+            $entry->secondFactorId = (string) $auditLogMetadata->secondFactorId;
         }
 
         if ($auditLogMetadata->secondFactorType instanceof SecondFactorType) {
-            $entry->secondFactorType = (string)$auditLogMetadata->secondFactorType;
+            $entry->secondFactorType = (string) $auditLogMetadata->secondFactorType;
         }
 
         if (!$event instanceof RecoveryTokenRevokedEvent
             && !$event instanceof CompliedWithRecoveryCodeRevocationEvent
             && $auditLogMetadata->recoveryTokenId
         ) {
-            $entry->recoveryTokenIdentifier = (string)$auditLogMetadata->recoveryTokenId;
+            $entry->recoveryTokenIdentifier = (string) $auditLogMetadata->recoveryTokenId;
         }
 
         if ($auditLogMetadata->recoveryTokenType instanceof RecoveryTokenType) {
-            $entry->recoveryTokenType = (string)$auditLogMetadata->recoveryTokenType;
+            $entry->recoveryTokenType = (string) $auditLogMetadata->recoveryTokenType;
         }
 
         if ($auditLogMetadata->secondFactorIdentifier instanceof SecondFactorIdentifier) {
-            $entry->secondFactorIdentifier = (string)$auditLogMetadata->secondFactorIdentifier;
+            $entry->secondFactorIdentifier = (string) $auditLogMetadata->secondFactorIdentifier;
         }
 
         if ($auditLogMetadata->raInstitution instanceof Institution) {
-            $entry->raInstitution = (string)$auditLogMetadata->raInstitution;
+            $entry->raInstitution = (string) $auditLogMetadata->raInstitution;
         }
 
         $this->auditLogRepository->save($entry);

src/Surfnet/StepupMiddleware/ApiBundle/Identity/Repository/RaListingRepository.php

Spacing +8 added lines, -8 removed lines

@@ -45,14 +45,14 @@ 
 
     public function findByIdentityId(IdentityId $identityId): ?array
     {
-        return parent::findBy(['identityId' => (string)$identityId]);
+        return parent::findBy(['identityId' => (string) $identityId]);
     }
 
     public function findByIdentityIdAndRaInstitution(IdentityId $identityId, Institution $raInstitution): ?RaListing
     {
         return parent::findOneBy([
-            'identityId' => (string)$identityId,
-            'raInstitution' => (string)$raInstitution,
+            'identityId' => (string) $identityId,
+            'raInstitution' => (string) $raInstitution,
         ]);
     }
 
@@ -65,7 +65,7 @@ 
             ->where('r.identityId = :identityId')
             ->andWhere('r.raInstitution = :raInstitution')
             ->setParameter('identityId', $identityId)
-            ->setParameter('raInstitution', (string)$raInstitution)
+            ->setParameter('raInstitution', (string) $raInstitution)
             ->orderBy('r.raInstitution');
 
         // Modify query to filter on authorization:
@@ -88,8 +88,8 @@ 
     public function findByIdentityIdAndInstitution(IdentityId $identityId, Institution $institution): array
     {
         return parent::findBy([
-            'identityId' => (string)$identityId,
-            'institution' => (string)$institution,
+            'identityId' => (string) $identityId,
+            'institution' => (string) $institution,
         ]);
     }
 
@@ -116,7 +116,7 @@ 
         if ($query->identityId instanceof IdentityId) {
             $queryBuilder
                 ->andWhere('r.identityId = :identityId')
-                ->setParameter('identityId', (string)$query->identityId);
+                ->setParameter('identityId', (string) $query->identityId);
         }
 
         if ($query->name) {
@@ -240,6 +240,6 @@ 
 
     public function contains(IdentityId $identityId): bool
     {
-        return count(parent::findBy(['identityId' => (string)$identityId])) > 0;
+        return count(parent::findBy(['identityId' => (string) $identityId])) > 0;
     }
 }

src/Surfnet/StepupMiddleware/ApiBundle/Identity/Repository/UnverifiedSecondFactorRepository.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
         if ($query->identityId instanceof IdentityId) {
             $queryBuilder
                 ->andWhere('sf.identityId = :identityId')
-                ->setParameter('identityId', (string)$query->identityId);
+                ->setParameter('identityId', (string) $query->identityId);
         }
 
         if ($query->verificationNonce) {

src/Surfnet/StepupMiddleware/ApiBundle/Identity/Repository/VerifiedSecondFactorRepository.php

Spacing +3 added lines, -3 removed lines

@@ -76,13 +76,13 @@ 
         if ($query->identityId instanceof IdentityId) {
             $queryBuilder
                 ->andWhere('sf.identityId = :identityId')
-                ->setParameter('identityId', (string)$query->identityId);
+                ->setParameter('identityId', (string) $query->identityId);
         }
 
         if ($query->secondFactorId instanceof SecondFactorId) {
             $queryBuilder
                 ->andWhere('sf.id = :secondFactorId')
-                ->setParameter('secondFactorId', (string)$query->secondFactorId);
+                ->setParameter('secondFactorId', (string) $query->secondFactorId);
         }
 
         if (is_string($query->registrationCode)) {
@@ -109,7 +109,7 @@ 
 
         $queryBuilder
             ->andWhere('sf.identityId = :identityId')
-            ->setParameter('identityId', (string)$query->identityId);
+            ->setParameter('identityId', (string) $query->identityId);
 
         return $queryBuilder->getQuery();
     }

src/Surfnet/StepupMiddleware/ApiBundle/Identity/Entity/AuditLogEntry.php

Spacing +5 added lines, -5 removed lines

@@ -168,18 +168,18 @@
     {
         return [
             'actor_id' => $this->actorId,
-            'actor_institution' => $this->actorInstitution instanceof Institution ? (string)$this->actorInstitution : null,
-            'actor_common_name' => (string)$this->actorCommonName,
+            'actor_institution' => $this->actorInstitution instanceof Institution ? (string) $this->actorInstitution : null,
+            'actor_common_name' => (string) $this->actorCommonName,
             'identity_id' => $this->identityId,
-            'identity_institution' => (string)$this->identityInstitution,
-            'ra_institution' => (string)$this->raInstitution,
+            'identity_institution' => (string) $this->identityInstitution,
+            'ra_institution' => (string) $this->raInstitution,
             'second_factor_id' => $this->secondFactorId,
             'second_factor_type' => $this->secondFactorType ?: null,
             'second_factor_identifier' => $this->secondFactorIdentifier,
             'recovery_token_type' => $this->recoveryTokenType,
             'recovery_token_identifier' => $this->recoveryTokenIdentifier,
             'action' => $this->mapEventToAction($this->event),
-            'recorded_on' => (string)$this->recordedOn,
+            'recorded_on' => (string) $this->recordedOn,
         ];
     }
 

src/Surfnet/StepupMiddleware/ApiBundle/Doctrine/Type/SecondFactorStatusType.php

Spacing +3 added lines, -3 removed lines

@@ -51,7 +51,7 @@ 
                 sprintf(
                     "Encountered illegal second factor status of type %s '%s', expected a SecondFactorStatus instance",
                     get_debug_type($value),
-                    is_scalar($value) ? (string)$value : '',
+                    is_scalar($value) ? (string) $value : '',
                 ),
             );
         }
@@ -68,7 +68,7 @@ 
             return 40;
         }
 
-        throw new ConversionException(sprintf("Encountered inconvertible second factor status '%s'", (string)$value));
+        throw new ConversionException(sprintf("Encountered inconvertible second factor status '%s'", (string) $value));
     }
 
     /**
@@ -77,7 +77,7 @@ 
     public function convertToPHPValue(mixed $value, AbstractPlatform $platform): SecondFactorStatus
     {
         if (is_scalar($value)) {
-            $value = (string)$value;
+            $value = (string) $value;
         }
         if ($value === '0') {
             return SecondFactorStatus::unverified();