simplesamlphp / simplesamlphp-module-consentadmin

hooks/hook_configpage.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
  * @param \SimpleSAML\XHTML\Template $template The template that we should alter in this hook.
  * @return void
  */
-function consentAdmin_hook_configpage(Template &$template): void
+function consentAdmin_hook_configpage(Template&$template): void
 {
     $template->data['links'][] = [
         'href' => Module::getModuleURL('consentAdmin/consentAdmin.php'),

public/consentAdmin.php

Spacing +12 added lines, -12 removed lines

@@ -77,14 +77,14 @@ 
     /*
      * Generate identifiers and hashes
      */
-    $destination = $sp_metadata['metadata-set'] . '|' . $sp_entityid;
+    $destination = $sp_metadata['metadata-set'].'|'.$sp_entityid;
 
     $targeted_id = \SimpleSAML\Module\consent\Auth\Process\Consent::getTargetedID($userid, $source, $destination);
     $attribute_hash = \SimpleSAML\Module\consent\Auth\Process\Consent::getAttributeHash($attributes, $hashAttributes);
 
-    \SimpleSAML\Logger::info('consentAdmin: user: ' . $userid);
-    \SimpleSAML\Logger::info('consentAdmin: target: ' . $targeted_id);
-    \SimpleSAML\Logger::info('consentAdmin: attribute: ' . $attribute_hash);
+    \SimpleSAML\Logger::info('consentAdmin: user: '.$userid);
+    \SimpleSAML\Logger::info('consentAdmin: target: '.$targeted_id);
+    \SimpleSAML\Logger::info('consentAdmin: attribute: '.$attribute_hash);
 
     // Return values
     return [$targeted_id, $attribute_hash, $attributes];
@@ -126,10 +126,10 @@ 
 // Calc correct source
 if ($as->getAuthData('saml:sp:IdP') !== null) {
     // from a remote idp (as bridge)
-    $source = 'saml20-idp-remote|' . $as->getAuthData('saml:sp:IdP');
+    $source = 'saml20-idp-remote|'.$as->getAuthData('saml:sp:IdP');
 } else {
     // from the local idp
-    $source = $idp_metadata['metadata-set'] . '|' . $idp_entityid;
+    $source = $idp_metadata['metadata-set'].'|'.$idp_entityid;
 }
 
 // Get user ID
@@ -142,8 +142,8 @@ 
 $userids = $attributes[$userid_attributename];
 
 if (empty($userids)) {
-    throw new \Exception('Could not generate useridentifier for storing consent. Attribute [' .
-        $userid_attributename . '] was not available.');
+    throw new \Exception('Could not generate useridentifier for storing consent. Attribute ['.
+        $userid_attributename.'] was not available.');
 }
 
 $userid = $userids[0];
@@ -161,7 +161,7 @@ 
     $action = $_GET["action"];
 }
 
-\SimpleSAML\Logger::critical('consentAdmin: sp: ' . $sp_entityid . ' action: ' . $action);
+\SimpleSAML\Logger::critical('consentAdmin: sp: '.$sp_entityid.' action: '.$action);
 
 // Remove services, which have consent disabled
 if (isset($idp_metadata['consent.disable'])) {
@@ -172,7 +172,7 @@ 
     }
 }
 
-\SimpleSAML\Logger::info('consentAdmin: ' . $idp_entityid);
+\SimpleSAML\Logger::info('consentAdmin: '.$idp_entityid);
 
 // Parse consent config
 $consent_storage = \SimpleSAML\Module\consent\Store::parseStoreConfig($cA_config->getValue('consentadmin'));
@@ -264,8 +264,8 @@ 
 
     // Translate attribute-names
     foreach ($attributes_new as $orig_name => $value) {
-        if (isset($template->data['attribute_' . htmlspecialchars(strtolower($orig_name))])) {
-            $old_name = $template->data['attribute_' . htmlspecialchars(strtolower($orig_name))];
+        if (isset($template->data['attribute_'.htmlspecialchars(strtolower($orig_name))])) {
+            $old_name = $template->data['attribute_'.htmlspecialchars(strtolower($orig_name))];
         }
         $name = $translator->getAttributeTranslation(strtolower($orig_name)); // translate