OpenConext / Stepup-Middleware

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

Spacing +6 added lines, -6 removed lines

@@ -72,7 +72,7 @@ 
             ->where("r.identityId = :identityId AND r.role IN(:roles)")
             ->groupBy("a.institution");
 
-        $qb->setParameter('identityId', (string)$actorId);
+        $qb->setParameter('identityId', (string) $actorId);
         $qb->setParameter(
             'authorizationRoles',
             $this->getAllowedInstitutionRoles($role)
@@ -88,7 +88,7 @@ 
             $this->logger->notice(
                 sprintf('Adding %s to authorized institutions', $institution['institution'])
             );
-            $result->add(new Institution((string)$institution['institution']));
+            $result->add(new Institution((string) $institution['institution']));
         }
 
         // Also get the institutions that are linked to the user via the 'institution_relation' field.
@@ -105,13 +105,13 @@ 
             ->andWhere("ia.institutionRole = :role") // Only filter on use_ra and use_raa roles here.
             ->groupBy('ia.institution');
 
-        $qb->setParameter('identityId', (string)$actorId);
+        $qb->setParameter('identityId', (string) $actorId);
         $qb->setParameter('role', $this->getInstitutionRoleByRaRole($role));
         $qb->setParameter('identityRoles', $identityRoles);
 
         $institutions = $qb->getQuery()->getArrayResult();
         foreach ($institutions as $institution) {
-            $institutionVo = new Institution((string)$institution['institution']);
+            $institutionVo = new Institution((string) $institution['institution']);
             if (!$result->contains($institutionVo)) {
                 $result->add($institutionVo);
                 $this->logger->notice(
@@ -138,7 +138,7 @@ 
             ->where('ia.institutionRole = :institutionRole AND ra.role = :authorizationRole')
             ->groupBy("ci.institution");
 
-        $qb->setParameter('identityId', (string)$actorId);
+        $qb->setParameter('identityId', (string) $actorId);
         // The identity requires RAA role to perform this search
         $qb->setParameter(
             'authorizationRole',
@@ -154,7 +154,7 @@ 
 
         $result = new InstitutionCollection();
         foreach ($institutions as $institution) {
-            $result->add(new Institution((string)$institution['institution']));
+            $result->add(new Institution((string) $institution['institution']));
         }
 
         return $result;

src/Surfnet/StepupMiddleware/ApiBundle/Authorization/Service/AuthorizationContextService.php

Spacing +2 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
             $institutions = new InstitutionCollection();
             $configuredInstitutions = $this->institutionRepository->findAll();
             foreach ($configuredInstitutions as $institution) {
-                $institutions->add(new Institution((string)$institution->institution));
+                $institutions->add(new Institution((string) $institution->institution));
             }
         } else {
             // Get the institutions the identity is RA(A) for.
@@ -104,7 +104,7 @@ 
 
     private function isSraa(IdentityId $actorId)
     {
-        $identity = $this->identityService->find((string)$actorId);
+        $identity = $this->identityService->find((string) $actorId);
         if (!$identity) {
             throw new InvalidArgumentException('The provided id is not associated with any known identity');
         }

tests/database/AuthorizationRepositoryMatrixTest.php

Spacing +1 added lines, -1 removed lines

@@ -122,7 +122,7 @@
     {
         $institutions = [];
         /** @var Institution $institution */
-        foreach($collection->jsonSerialize()['institutions'] as $institution)
+        foreach ($collection->jsonSerialize()['institutions'] as $institution)
         {
             $institutions[] = $institution->getInstitution();
         }