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;

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/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));
     }
 

Service/ShibbolethServiceProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -205,7 +205,7 @@
 
     /**
      * @param string $name
-     * @param null   $fallback
+     * @param null|double   $fallback
      * @return null|string
      */
     public function getAttribute($name, $fallback = null)

Spacing +1 added lines, -1 removed lines

@@ -232,7 +232,7 @@
     {
         $result = true;
         $empty = microtime(true);
-        set_error_handler(function ($errno, $errstr) {
+        set_error_handler(function($errno, $errstr) {
             throw new \Exception($errstr, $errno);
         });
         foreach ($keyValues as $checkName => $checkValue) {

Compiler/KuleuvenShibbolethAttributeDefinitionsXmlParserPass.php

Spacing +1 added lines, -1 removed lines

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

Service/HeaderAttributesProvider.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     public function getAttributes()
     {
         $attributes = $this->requestStack->getCurrentRequest()->headers->all();
-        array_walk($attributes, function (&$value) {
+        array_walk($attributes, function(&$value) {
             $value = (is_array($value) ? implode(';', $value) : $value);
         });
         return $attributes;

Service/AttributeDefinitionsProvider.php

Spacing +1 added lines, -1 removed lines

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