simplesamlphp / simplesamlphp-module-casserver

lib/Cas/Protocol/Cas20.php

Spacing +2 added lines, -2 removed lines

@@ -174,7 +174,7 @@ 
     private function workAroundForBuggyJasigXmlParser($xmlString)
     {
         // when will people stop hand coding xml handling....?
-        return str_replace('><', '>' . PHP_EOL . '<', str_replace(PHP_EOL, '', $xmlString));
+        return str_replace('><', '>'.PHP_EOL.'<', str_replace(PHP_EOL, '', $xmlString));
     }
 
     private function generateCas20Attribute($xmlDocument, $attributeName, $attributeValue)
@@ -182,7 +182,7 @@ 
         $attributeValueNode = $xmlDocument->createTextNode($this->base64EncodeAttributes ?
             base64_encode($attributeValue) : $attributeValue);
 
-        $attributeElement = $xmlDocument->createElement('cas:' . $attributeName);
+        $attributeElement = $xmlDocument->createElement('cas:'.$attributeName);
 
         $attributeElement->appendChild($attributeValueNode);
 

lib/Cas/Ticket/SQLTicketStore.php

Spacing +3 added lines, -3 removed lines

@@ -39,7 +39,7 @@ 
         $dsn = $storeConfig->getString('dsn');
         $username = $storeConfig->getString('username');
         $password = $storeConfig->getString('password');
-        $options =  $storeConfig->getArray('options', []);
+        $options = $storeConfig->getArray('options', []);
         $this->prefix = $storeConfig->getString('prefix', '');
 
         $this->pdo = new PDO($dsn, $username, $password, $options);
@@ -106,7 +106,7 @@ 
         }
 
         while (($row = $fetchTableVersion->fetch(PDO::FETCH_ASSOC)) !== false) {
-            $this->tableVersions[$row['_name']] = (int)$row['_version'];
+            $this->tableVersions[$row['_name']] = (int) $row['_version'];
         }
     }
 
@@ -195,7 +195,7 @@ 
             $insertQuery->execute($data);
             return;
         } catch (\PDOException $e) {
-            $ecode = (string)$e->getCode();
+            $ecode = (string) $e->getCode();
             switch ($ecode) {
                 case '23505': /* PostgreSQL */
                     break;

config-templates/module_casserver.php

Indentation +3 added lines, -3 removed lines

@@ -20,13 +20,13 @@ 
         ]
     ],
     'legal_service_urls' => [
-         //Any service url string matching any of the following prefixes is accepted
+            //Any service url string matching any of the following prefixes is accepted
         'http://host1.domain:1234/path1',
         'https://host2.domain:5678/path2/path3',
     ],
 
     'legal_target_service_urls' => [
-         //Any target service url string matching any of the following prefixes is accepted
+            //Any target service url string matching any of the following prefixes is accepted
         'http://host3.domain:4321/path4',
         'https://host4.domain:8765/path5/path6',
         // So is regex
@@ -34,7 +34,7 @@ 
     ],
 
     'ticketstore' => [
-         //defaults to filesystem ticket store using the directory 'ticketcache'
+            //defaults to filesystem ticket store using the directory 'ticketcache'
         'class' => 'casserver:FileSystemTicketStore', //Not intended for production
         'directory' => 'ticketcache',
 

www/logout.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
 if (!$casconfig->getValue('enable_logout', false)) {
     $message = 'Logout not allowed';
 
-    \SimpleSAML\Logger::debug('casserver:' . $message);
+    \SimpleSAML\Logger::debug('casserver:'.$message);
 
     throw new \Exception($message);
 }
@@ -39,7 +39,7 @@ 
 if ($skipLogoutPage && !array_key_exists('url', $_GET)) {
     $message = 'Required URL query parameter [url] not provided. (CAS Server)';
 
-    \SimpleSAML\Logger::debug('casserver:' . $message);
+    \SimpleSAML\Logger::debug('casserver:'.$message);
 
     throw new \Exception($message);
 }

www/login.php

Indentation +1 added lines, -1 removed lines

@@ -186,7 +186,7 @@
 
     \SimpleSAML\Utils\HTTP::redirectTrustedURL(\SimpleSAML\Utils\HTTP::addURLParameters($_GET['service'], $parameters));
 } else {
-   \ SimpleSAML\Utils\HTTP::redirectTrustedURL(
+    \ SimpleSAML\Utils\HTTP::redirectTrustedURL(
         \SimpleSAML\Utils\HTTP::addURLParameters(SimpleSAML\Module::getModuleURL('casserver/loggedIn.php'), $parameters)
     );
 }

www/utility/validateTicket.php

Spacing +16 added lines, -16 removed lines

@@ -79,8 +79,8 @@ 
                             'sessionId' => $serviceTicket['sessionId']
                         ]);
                         try {
-                            \SimpleSAML\Utils\HTTP::fetch($pgtUrl . '?pgtIou=' . $proxyGrantingTicket['iou']
-                                . '&pgtId=' . $proxyGrantingTicket['id']);
+                            \SimpleSAML\Utils\HTTP::fetch($pgtUrl.'?pgtIou='.$proxyGrantingTicket['iou']
+                                . '&pgtId='.$proxyGrantingTicket['id']);
 
                             $protocol->setProxyGrantingTicketIOU($proxyGrantingTicket['iou']);
 
@@ -93,29 +93,29 @@ 
                 echo $protocol->getValidateSuccessResponse($serviceTicket['userName']);
             } else {
                 if ($ticketFactory->isExpired($serviceTicket)) {
-                    $message = 'Ticket ' . var_export($_GET['ticket'], true) . ' has expired';
+                    $message = 'Ticket '.var_export($_GET['ticket'], true).' has expired';
 
-                    \SimpleSAML\Logger::debug('casserver:' . $message);
+                    \SimpleSAML\Logger::debug('casserver:'.$message);
 
                     echo $protocol->getValidateFailureResponse('INVALID_TICKET', $message);
                 } else {
                     if (sanitize($serviceTicket['service']) != sanitize($_GET['service'])) {
                         $message = 'Mismatching service parameters: expected '
                             . var_export($serviceTicket['service'], true)
-                            . ' but was: ' . var_export($_GET['service'], true);
+                            . ' but was: '.var_export($_GET['service'], true);
 
-                        \SimpleSAML\Logger::debug('casserver:' . $message);
+                        \SimpleSAML\Logger::debug('casserver:'.$message);
 
                         echo $protocol->getValidateFailureResponse('INVALID_SERVICE', $message);
                     } else {
                         if ($serviceTicket['forceAuthn'] != $forceAuthn) {
                             $message = 'Ticket was issue from single sign on session';
 
-                            \SimpleSAML\Logger::debug('casserver:' . $message);
+                            \SimpleSAML\Logger::debug('casserver:'.$message);
 
                             echo $protocol->getValidateFailureResponse('INVALID_TICKET', $message);
                         } else {
-                            \SimpleSAML\Logger::error('casserver:' . $method . ': internal server error.');
+                            \SimpleSAML\Logger::error('casserver:'.$method.': internal server error.');
 
                             echo $protocol->getValidateFailureResponse('INTERNAL_ERROR', 'Unknown internal error');
                         }
@@ -124,23 +124,23 @@ 
             }
         } else {
             if (is_null($serviceTicket)) {
-                $message = 'Ticket ' . var_export($_GET['ticket'], true) . ' not recognized';
+                $message = 'Ticket '.var_export($_GET['ticket'], true).' not recognized';
 
-                \SimpleSAML\Logger::debug('casserver:' . $message);
+                \SimpleSAML\Logger::debug('casserver:'.$message);
 
                 echo $protocol->getValidateFailureResponse('INVALID_TICKET', $message);
             } else {
                 if ($ticketFactory->isProxyTicket($serviceTicket) && $method == 'serviceValidate') {
-                    $message = 'Ticket ' . var_export($_GET['ticket'], true)
+                    $message = 'Ticket '.var_export($_GET['ticket'], true)
                         . ' is a proxy ticket. Use proxyValidate instead.';
 
-                    \SimpleSAML\Logger::debug('casserver:' . $message);
+                    \SimpleSAML\Logger::debug('casserver:'.$message);
 
                     echo $protocol->getValidateFailureResponse('INVALID_TICKET', $message);
                 } else {
-                    $message = 'Ticket ' . var_export($_GET['ticket'], true) . ' is not a service ticket';
+                    $message = 'Ticket '.var_export($_GET['ticket'], true).' is not a service ticket';
 
-                    \SimpleSAML\Logger::debug('casserver:' . $message);
+                    \SimpleSAML\Logger::debug('casserver:'.$message);
 
                     echo $protocol->getValidateFailureResponse('INVALID_TICKET', $message);
                 }
@@ -157,13 +157,13 @@ 
     if (!array_key_exists('service', $_GET)) {
         $message = 'Missing service parameter: [service]';
 
-        \SimpleSAML\Logger::debug('casserver:' . $message);
+        \SimpleSAML\Logger::debug('casserver:'.$message);
 
         echo $protocol->getValidateFailureResponse('INVALID_REQUEST', $message);
     } else {
         $message = 'Missing ticket parameter: [ticket]';
 
-        \SimpleSAML\Logger::debug('casserver:' . $message);
+        \SimpleSAML\Logger::debug('casserver:'.$message);
 
         echo $protocol->getValidateFailureResponse('INVALID_REQUEST', $message);
     }

www/utility/urlUtils.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
         }
         if (!ctype_alnum($legalUrl[0])) {
             // Probably a regex. Suppress errors incase the format is invalid
-            $result =  @preg_match($legalUrl, $service);
+            $result = @preg_match($legalUrl, $service);
             if ($result === 1) {
                 return true;
             } elseif ($result === false) {