simplesamlphp / simplesamlphp-module-cas

src/Controller/CAS.php

Spacing +1 added lines, -1 removed lines

@@ -106,7 +106,7 @@
         /** @var \SimpleSAML\Module\cas\Auth\Source\CAS|null $source */
         $source = $this->authSource::getById($sourceId);
         if ($source === null) {
-            throw new Exception('Could not find authentication source with id ' . $sourceId);
+            throw new Exception('Could not find authentication source with id '.$sourceId);
         }
 
         $source->finalStep($state);

src/Auth/Source/CAS.php

Indentation +4 added lines, -4 removed lines

@@ -183,13 +183,13 @@
             // array is empty or not set then an empty array will be returned.
             $attributesFromQueryConfiguration = $this->parseQueryAttributes($dom);
             if (!empty($attributesFromQueryConfiguration)) {
-              // Overwrite attributes from parseAuthenticationSuccess with configured
-              // XPath-based attributes, instead of combining them.
+                // Overwrite attributes from parseAuthenticationSuccess with configured
+                // XPath-based attributes, instead of combining them.
                 foreach ($attributesFromQueryConfiguration as $name => $values) {
-                  // Ensure a clean, unique list of string values
+                    // Ensure a clean, unique list of string values
                     $values = array_values(array_unique(array_map('strval', $values)));
 
-                  // Configuration wins: replace any existing attribute with the same name
+                    // Configuration wins: replace any existing attribute with the same name
                     $attributes[$name] = $values;
                 }
             }

Spacing +5 added lines, -5 removed lines

@@ -236,7 +236,7 @@ 
 
         $config = Configuration::loadFromArray(
             $this->ldapConfig,
-            'Authentication source ' . var_export($this->authId, true),
+            'Authentication source '.var_export($this->authId, true),
         );
         if (!empty($this->ldapConfig['servers'])) {
             $ldap = new Ldap(
@@ -320,7 +320,7 @@ 
      *   1: array<string, list<string>>
      * }
      */
-    private function parseAuthenticationSuccess(CasAuthnSuccess|SlateAuthnSuccess $message): array
+    private function parseAuthenticationSuccess(CasAuthnSuccess | SlateAuthnSuccess $message): array
     {
         /** @var array<string, list<string>> $result */
         $result = [];
@@ -347,7 +347,7 @@ 
             // Key selection rule
             $key = ($prefix === '' || $prefix === 'cas')
                 ? $localName
-                : ($prefix . ':' . $localName);
+                : ($prefix.':'.$localName);
 
             $value = trim($xmlElement->textContent ?? '');
 
@@ -372,7 +372,7 @@ 
      * @return void
      */
     private function parseAuthenticationSuccessMetadata(
-        CasAuthnSuccess|SlateAuthnSuccess $message,
+        CasAuthnSuccess | SlateAuthnSuccess $message,
         array &$attributes,
     ): void {
         if (!method_exists($message, 'getElements')) {
@@ -399,7 +399,7 @@ 
             // and just localName when there is none.
             $key = ($prefix === '')
                 ? $localName
-                : ($prefix . ':' . $localName);
+                : ($prefix.':'.$localName);
 
             $value = trim($element->getXML()->textContent ?? '');