simplesamlphp / simplesamlphp-module-consent

www/getconsent.php

Spacing +21 added lines, -21 removed lines

@@ -72,12 +72,12 @@ 
         $attributeSet = $state['consent:store.attributeSet'];
 
         \SimpleSAML\Logger::debug(
-            'Consent - saveConsent() : [' . $userId . '|' . $targetedId . '|' . $attributeSet . ']'
+            'Consent - saveConsent() : ['.$userId.'|'.$targetedId.'|'.$attributeSet.']'
         );
         try {
             $store->saveConsent($userId, $targetedId, $attributeSet);
         } catch (\Exception $e) {
-            \SimpleSAML\Logger::error('Consent: Error writing to storage: ' . $e->getMessage());
+            \SimpleSAML\Logger::error('Consent: Error writing to storage: '.$e->getMessage());
         }
     }
 
@@ -228,38 +228,38 @@ 
 
     $alternate = ['odd', 'even'];
     $i = 0;
-    $summary = 'summary="' . $translator->t('{consent:consent:table_summary}') . '"';
+    $summary = 'summary="'.$translator->t('{consent:consent:table_summary}').'"';
 
     if (strlen($nameParent) > 0) {
-        $parentStr = strtolower($nameParent) . '_';
-        $str = '<table class="attributes" ' . $summary . '>';
+        $parentStr = strtolower($nameParent).'_';
+        $str = '<table class="attributes" '.$summary.'>';
     } else {
         $parentStr = '';
-        $str = '<table id="table_with_attributes" class="attributes" ' . $summary . '>';
-        $str .= "\n" . '<caption>' . $translator->t('{consent:consent:table_caption}') . '</caption>';
+        $str = '<table id="table_with_attributes" class="attributes" '.$summary.'>';
+        $str .= "\n".'<caption>'.$translator->t('{consent:consent:table_caption}').'</caption>';
     }
 
     foreach ($attributes as $name => $value) {
         $nameraw = $name;
-        $name = $translator->getAttributeTranslation($parentStr . $nameraw);
+        $name = $translator->getAttributeTranslation($parentStr.$nameraw);
 
         if (preg_match('/^child_/', $nameraw)) {
             // insert child table
             $parentName = preg_replace('/^child_/', '', $nameraw);
             foreach ($value as $child) {
-                $str .= "\n" . '<tr class="odd"><td class="td_odd">' .
-                    present_attributes($t, $child, $parentName) . '</td></tr>';
+                $str .= "\n".'<tr class="odd"><td class="td_odd">'.
+                    present_attributes($t, $child, $parentName).'</td></tr>';
             }
         } else {
             // insert values directly
 
-            $str .= "\n" . '<tr class="' . $alternate[($i++ % 2)] .
-                '"><td><span class="attrname">' . htmlspecialchars($name) . '</span></td>';
+            $str .= "\n".'<tr class="'.$alternate[($i++ % 2)].
+                '"><td><span class="attrname">'.htmlspecialchars($name).'</span></td>';
 
             $isHidden = in_array($nameraw, $t->data['hiddenAttributes'], true);
             if ($isHidden) {
                 $hiddenId = \SimpleSAML\Utils\Random::generateID();
-                $str .= '<td><span class="attrvalue hidden" id="hidden_' . $hiddenId . '">';
+                $str .= '<td><span class="attrvalue hidden" id="hidden_'.$hiddenId.'">';
             } else {
                 $hiddenId = '';
                 $str .= '<td><span class="attrvalue">';
@@ -270,18 +270,18 @@ 
                 $str .= '<ul>';
                 foreach ($value as $listitem) {
                     if ($nameraw === 'jpegPhoto') {
-                        $str .= '<li><img src="data:image/jpeg;base64,' .
-                            htmlspecialchars($listitem) . '" alt="User photo" /></li>';
+                        $str .= '<li><img src="data:image/jpeg;base64,'.
+                            htmlspecialchars($listitem).'" alt="User photo" /></li>';
                     } else {
-                        $str .= '<li>' . htmlspecialchars($listitem) . '</li>';
+                        $str .= '<li>'.htmlspecialchars($listitem).'</li>';
                     }
                 }
                 $str .= '</ul>';
             } elseif (isset($value[0])) {
                 // we hawe only one value
                 if ($nameraw === 'jpegPhoto') {
-                    $str .= '<img src="data:image/jpeg;base64,' .
-                        htmlspecialchars($value[0]) . '" alt="User photo" />';
+                    $str .= '<img src="data:image/jpeg;base64,'.
+                        htmlspecialchars($value[0]).'" alt="User photo" />';
                 } else {
                     $str .= htmlspecialchars($value[0]);
                 }
@@ -289,10 +289,10 @@ 
             $str .= '</span>';
 
             if ($isHidden) {
-                $str .= '<div class="attrvalue consent_showattribute" id="visible_' . $hiddenId . '">';
+                $str .= '<div class="attrvalue consent_showattribute" id="visible_'.$hiddenId.'">';
                 $str .= '... ';
-                $str .= '<a class="consent_showattributelink" href="javascript:SimpleSAML_show(\'hidden_' . $hiddenId;
-                $str .= '\'); SimpleSAML_hide(\'visible_' . $hiddenId . '\');">';
+                $str .= '<a class="consent_showattributelink" href="javascript:SimpleSAML_show(\'hidden_'.$hiddenId;
+                $str .= '\'); SimpleSAML_hide(\'visible_'.$hiddenId.'\');">';
                 $str .= $translator->t('{consent:consent:show_attribute}');
                 $str .= '</a>';
                 $str .= '</div>';

lib/Consent/Store/Cookie.php

Spacing +9 added lines, -9 removed lines

@@ -42,19 +42,19 @@ 
     {
         $cookieName = self::getCookieName($userId, $destinationId);
 
-        $data = $userId . ':' . $attributeSet . ':' . $destinationId;
+        $data = $userId.':'.$attributeSet.':'.$destinationId;
 
-        \SimpleSAML\Logger::debug('Consent cookie - Get [' . $data . ']');
+        \SimpleSAML\Logger::debug('Consent cookie - Get ['.$data.']');
 
         if (!array_key_exists($cookieName, $_COOKIE)) {
             \SimpleSAML\Logger::debug(
-                'Consent cookie - no cookie with name \'' . $cookieName . '\'.'
+                'Consent cookie - no cookie with name \''.$cookieName.'\'.'
             );
             return false;
         }
         if (!is_string($_COOKIE[$cookieName])) {
             \SimpleSAML\Logger::warning(
-                'Value of consent cookie wasn\'t a string. Was: ' .
+                'Value of consent cookie wasn\'t a string. Was: '.
                 var_export($_COOKIE[$cookieName], true)
             );
             return false;
@@ -92,9 +92,9 @@ 
     public function saveConsent(string $userId, string $destinationId, string $attributeSet)
     {
         $name = self::getCookieName($userId, $destinationId);
-        $value = $userId . ':' . $attributeSet . ':' . $destinationId;
+        $value = $userId.':'.$attributeSet.':'.$destinationId;
 
-        \SimpleSAML\Logger::debug('Consent cookie - Set [' . $value . ']');
+        \SimpleSAML\Logger::debug('Consent cookie - Set ['.$value.']');
 
         $value = self::sign($value);
         return $this->setConsentCookie($name, $value);
@@ -164,7 +164,7 @@ 
             $tmp = explode(':', $value, 3);
             if (count($tmp) !== 3) {
                 \SimpleSAML\Logger::warning(
-                    'Consent cookie with invalid value: ' . $value
+                    'Consent cookie with invalid value: '.$value
                 );
                 continue;
             }
@@ -195,7 +195,7 @@ 
     {
         $secretSalt = \SimpleSAML\Utils\Config::getSecretSalt();
 
-        return sha1($secretSalt . $data . $secretSalt) . ':' . $data;
+        return sha1($secretSalt.$data.$secretSalt).':'.$data;
     }
 
 
@@ -239,7 +239,7 @@ 
      */
     private static function getCookieName(string $userId, string $destinationId)
     {
-        return '\SimpleSAML\Module\consent:' . sha1($userId . ':' . $destinationId);
+        return '\SimpleSAML\Module\consent:'.sha1($userId.':'.$destinationId);
     }
 
 

lib/Consent/Store/Database.php

Spacing +20 added lines, -20 removed lines

@@ -169,8 +169,8 @@ 
     public function hasConsent(string $userId, string $destinationId, string $attributeSet)
     {
         $st = $this->execute(
-            'UPDATE ' . $this->table . ' ' .
-            'SET usage_date = ' . $this->dateTime . ' ' .
+            'UPDATE '.$this->table.' '.
+            'SET usage_date = '.$this->dateTime.' '.
             'WHERE hashed_user_id = ? AND service_id = ? AND attribute = ?',
             [$userId, $destinationId, $attributeSet]
         );
@@ -206,8 +206,8 @@ 
     {
         // Check for old consent (with different attribute set)
         $st = $this->execute(
-            'UPDATE ' . $this->table . ' ' .
-            'SET consent_date = ' . $this->dateTime . ', usage_date = ' . $this->dateTime . ', attribute = ? ' .
+            'UPDATE '.$this->table.' '.
+            'SET consent_date = '.$this->dateTime.', usage_date = '.$this->dateTime.', attribute = ? '.
             'WHERE hashed_user_id = ? AND service_id = ?',
             [$attributeSet, $userId, $destinationId]
         );
@@ -224,8 +224,8 @@ 
 
         // Add new consent
         $st = $this->execute(
-            'INSERT INTO ' . $this->table . ' (' . 'consent_date, usage_date, hashed_user_id, service_id, attribute' .
-            ') ' . 'VALUES (' . $this->dateTime . ', ' . $this->dateTime . ', ?, ?, ?)',
+            'INSERT INTO '.$this->table.' ('.'consent_date, usage_date, hashed_user_id, service_id, attribute'.
+            ') '.'VALUES ('.$this->dateTime.', '.$this->dateTime.', ?, ?, ?)',
             [$userId, $destinationId, $attributeSet]
         );
 
@@ -249,7 +249,7 @@ 
     public function deleteConsent(string $userId, string $destinationId)
     {
         $st = $this->execute(
-            'DELETE FROM ' . $this->table . ' WHERE hashed_user_id = ? AND service_id = ?;',
+            'DELETE FROM '.$this->table.' WHERE hashed_user_id = ? AND service_id = ?;',
             [$userId, $destinationId]
         );
 
@@ -277,7 +277,7 @@ 
     public function deleteAllConsents(string $userId)
     {
         $st = $this->execute(
-            'DELETE FROM ' . $this->table . ' WHERE hashed_user_id = ?',
+            'DELETE FROM '.$this->table.' WHERE hashed_user_id = ?',
             [$userId]
         );
 
@@ -286,7 +286,7 @@ 
         }
 
         if ($st->rowCount() > 0) {
-            \SimpleSAML\Logger::debug('consent:Database - Deleted (' . $st->rowCount() . ') consent(s) . ');
+            \SimpleSAML\Logger::debug('consent:Database - Deleted ('.$st->rowCount().') consent(s) . ');
             return $st->rowCount();
         }
 
@@ -309,7 +309,7 @@ 
         $ret = [];
 
         $st = $this->execute(
-            'SELECT service_id, attribute, consent_date, usage_date FROM ' . $this->table .
+            'SELECT service_id, attribute, consent_date, usage_date FROM '.$this->table.
             ' WHERE hashed_user_id = ?',
             [$userId]
         );
@@ -348,16 +348,16 @@ 
         $st = $db->prepare($statement);
         if ($st === false) {
             \SimpleSAML\Logger::error(
-                'consent:Database - Error preparing statement \'' .
-                $statement . '\': ' . self::formatError($db->errorInfo())
+                'consent:Database - Error preparing statement \''.
+                $statement.'\': '.self::formatError($db->errorInfo())
             );
             return false;
         }
 
         if ($st->execute($parameters) !== true) {
             \SimpleSAML\Logger::error(
-                'consent:Database - Error executing statement \'' .
-                $statement . '\': ' . self::formatError($st->errorInfo())
+                'consent:Database - Error executing statement \''.
+                $statement.'\': '.self::formatError($st->errorInfo())
             );
             return false;
         }
@@ -381,7 +381,7 @@ 
         $ret = [];
 
         // Get total number of consents
-        $st = $this->execute('SELECT COUNT(*) AS no FROM ' . $this->table, []);
+        $st = $this->execute('SELECT COUNT(*) AS no FROM '.$this->table, []);
 
         if ($st === false) {
             return [];
@@ -393,8 +393,8 @@ 
 
         // Get total number of users that has given consent
         $st = $this->execute(
-            'SELECT COUNT(*) AS no ' .
-            'FROM (SELECT DISTINCT hashed_user_id FROM ' . $this->table . ' ) AS foo',
+            'SELECT COUNT(*) AS no '.
+            'FROM (SELECT DISTINCT hashed_user_id FROM '.$this->table.' ) AS foo',
             []
         );
 
@@ -408,7 +408,7 @@ 
 
         // Get total number of services that has been given consent to
         $st = $this->execute(
-            'SELECT COUNT(*) AS no FROM (SELECT DISTINCT service_id FROM ' . $this->table . ') AS foo',
+            'SELECT COUNT(*) AS no FROM (SELECT DISTINCT service_id FROM '.$this->table.') AS foo',
             []
         );
 
@@ -464,7 +464,7 @@ 
     {
         Assert::greaterThanEq(count($error), 3);
 
-        return $error[0] . ' - ' . $error[2] . ' (' . $error[1] . ')';
+        return $error[0].' - '.$error[2].' ('.$error[1].')';
     }
 
 
@@ -476,7 +476,7 @@ 
     public function selftest()
     {
         $st = $this->execute(
-            'SELECT * FROM ' . $this->table . ' WHERE hashed_user_id = ? AND service_id = ? AND attribute = ?',
+            'SELECT * FROM '.$this->table.' WHERE hashed_user_id = ? AND service_id = ? AND attribute = ?',
             ['test', 'test', 'test']
         );