MetalArend / authentication-bundle

Security/ShibbolethAuthenticationListenerFactory.php

Doc Comments +11 added lines

@@ -31,6 +31,10 @@ 
             ->end();
     }
 
+    /**
+     * @param string $id
+     * @param string $userProvider
+     */
     protected function createAuthenticationProvider(ContainerBuilder $container, $id, $userProvider)
     {
         $providerId = 'security.authentication.provider.' . $this->key . '.' . $id;
@@ -41,6 +45,10 @@ 
         return $providerId;
     }
 
+    /**
+     * @param string $id
+     * @param string $defaultEntryPoint
+     */
     protected function createEntryPoint(ContainerBuilder $container, $id, $defaultEntryPoint)
     {
         if (null !== $defaultEntryPoint) {
@@ -51,6 +59,9 @@ 
         return $entryPointId;
     }
 
+    /**
+     * @param string $id
+     */
     protected function createListener(ContainerBuilder $container, $id, $defaultRoles)
     {
         $listenerId = 'security.authentication.listener.' . $this->key . '.' . $id;

Collector/AuthenticationDataCollector.php

Spacing +4 added lines, -4 removed lines

@@ -159,10 +159,10 @@ 
                 'display_name'            => $token->getUser()->getDisplayName(),
                 'affiliation'             => $token->getUser()->getAffiliation(),
                 'attributes'              => $token->getUser()->getAttributes(),
-                'roles'                   => array_map(function (RoleInterface $role) {
+                'roles'                   => array_map(function(RoleInterface $role) {
                     return $role->getRole();
                 }, $token->getRoles()),
-                'inherited_roles'         => array_map(function (RoleInterface $role) {
+                'inherited_roles'         => array_map(function(RoleInterface $role) {
                     return $role->getRole();
                 }, $this->findInheritedRoles($token->getRoles())),
                 'supports_role_hierarchy' => null !== $this->roleHierarchy,
@@ -188,10 +188,10 @@ 
                 'source_affiliation'     => $source->getUser()->getAffiliation(),
                 'source_display_name'    => $source->getUser()->getDisplayName(),
                 'source_attributes'      => $source->getUser()->getAttributes(),
-                'source_roles'           => array_map(function (RoleInterface $role) {
+                'source_roles'           => array_map(function(RoleInterface $role) {
                     return $role->getRole();
                 }, $source->getRoles()),
-                'source_inherited_roles' => array_map(function (RoleInterface $role) {
+                'source_inherited_roles' => array_map(function(RoleInterface $role) {
                     return $role->getRole();
                 }, $this->findInheritedRoles($source->getRoles())),
             ]);

Service/PersonDataService.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
             throw new \Exception('Could not get person data for ' . $uid . ': ' . json_encode($decodedOutput));
         }
         $user = $decodedOutput->d;
-        $mainWorkAddresses = array_filter($user->WorkAddresses->results, function (\stdClass $value) {
+        $mainWorkAddresses = array_filter($user->WorkAddresses->results, function(\stdClass $value) {
             return $value->isMainWorkAddress;
         });
         $mainWorkAddress = reset($mainWorkAddresses);

Service/AttributeDefinitionsProvider.php

Spacing +2 added lines, -2 removed lines

@@ -122,7 +122,7 @@ 
                 }
             }
             foreach ($aliases as $alias) {
-                $attributeDefinitions[$alias] =& $attributeDefinitions[$id];
+                $attributeDefinitions[$alias] = & $attributeDefinitions[$id];
             }
         }
 
@@ -137,7 +137,7 @@ 
             }
             $attributeDefinitions[$id] = $attributeDefinition;
             foreach ($aliases as $alias) {
-                $attributeDefinitions[$alias] =& $attributeDefinitions[$id];
+                $attributeDefinitions[$alias] = & $attributeDefinitions[$id];
             }
         }
 

Service/LdapAttributesProvider.php

Spacing +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
             $ldapSingleResult = $ldapResults->toArray()['0'];
             // Extract attributes as strings
             $attributes = $ldapSingleResult->getAttributes();
-            array_walk($attributes, function (&$value) {
+            array_walk($attributes, function(&$value) {
                 $value = (is_array($value) ? implode(';', $value) : $value);
             });
             // Save to the session (cache)

Service/ShibbolethAttributesInjectionProviderManager.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
             }
             $server = $event->getRequest()->server;
             $providerPropertiesCollectionIterator = $this->providerPropertiesCollection->getIterator();
-            $providerPropertiesCollectionIterator->uasort(function ($first, $second) {
+            $providerPropertiesCollectionIterator->uasort(function($first, $second) {
                 // Place highest priority first
                 if ($first['priority'] === $second['priority']) {
                     return 0;

Traits/ShibbolethAttributesResolverTrait.php

Braces +3 added lines, -1 removed lines

@@ -104,7 +104,9 @@
      */
     public function hasAttributeValue($name, $value = null)
     {
-        if (!$this->hasAttribute($name)) return false;
+        if (!$this->hasAttribute($name)) {
+            return false;
+        }
         return (empty($value) ? true : (array_search($value, $this->getArrayAttribute($name)) !== false));
     }