simplesamlphp / simplesamlphp-module-consentsimpleadmin

www/consentAdmin.php

Spacing +8 added lines, -8 removed lines

@@ -23,8 +23,8 @@ 
 // Get user ID
 $userid_attributename = $consentconfig->getValue('userid', 'eduPersonPrincipalName');
 if (empty($attributes[$userid_attributename])) {
-    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 = $attributes[$userid_attributename][0];
@@ -43,9 +43,9 @@ 
     $idp_metadata = $metadata->getMetaData($idp_entityid, 'saml20-idp-hosted');
 }
 
-\SimpleSAML\Logger::debug('consentAdmin: IdP is [' . $idp_entityid . ']');
+\SimpleSAML\Logger::debug('consentAdmin: IdP is ['.$idp_entityid.']');
 
-$source = $idp_metadata['metadata-set'] . '|' . $idp_entityid;
+$source = $idp_metadata['metadata-set'].'|'.$idp_entityid;
 
 // Parse consent config
 $consent_storage = \SimpleSAML\Module\consent\Store::parseStoreConfig($consentconfig->getValue('store'));
@@ -57,7 +57,7 @@ 
 // Check if button with withdraw all consent was clicked
 if (array_key_exists('withdraw', $_REQUEST)) {
     \SimpleSAML\Logger::info(
-        'consentAdmin: UserID [' . $hashed_user_id . '] has requested to withdraw all consents given...'
+        'consentAdmin: UserID ['.$hashed_user_id.'] has requested to withdraw all consents given...'
     );
 
     $consent_storage->deleteAllConsents($hashed_user_id);
@@ -73,7 +73,7 @@ 
 }
 
 \SimpleSAML\Logger::debug(
-    'consentAdmin: no of consents [' . count($user_consent_list) . '] no of services [' . count($consentServices) . ']'
+    'consentAdmin: no of consents ['.count($user_consent_list).'] no of services ['.count($consentServices).']'
 );
 
 // Init template
@@ -82,8 +82,8 @@ 
 $t->data['consentServices'] = count($consentServices);
 $t->data['consents'] = count($user_consent_list);
 $t->data['granted'] = $t->getTranslator()->t('{consentSimpleAdmin:consentsimpleadmin:granted}', [
-        '%NO%' => (string)$this->data['consents'],
-        '%OF%' => (string)$this->data['consentServices'],
+        '%NO%' => (string) $this->data['consents'],
+        '%OF%' => (string) $this->data['consentServices'],
 ]);
 
 $t->send();

hooks/hook_sanitycheck.php

Spacing +1 added lines, -1 removed lines

@@ -25,6 +25,6 @@
             $hookinfo['errors'][] = '[consentSimpleAdmin] Consent Storage selftest failed.';
         }
     } catch (\Exception $e) {
-        $hookinfo['errors'][] = '[consentSimpleAdmin] Error connecting to storage: ' . $e->getMessage();
+        $hookinfo['errors'][] = '[consentSimpleAdmin] Error connecting to storage: '.$e->getMessage();
     }
 }