simplesamlphp / saml2

src/SAML2/HTTPRedirect.php

Spacing +11 added lines, -11 removed lines

@@ -49,21 +49,21 @@ 
         $msg .= urlencode($msgStr);
 
         if ($relayState !== null) {
-            $msg .= '&RelayState=' . urlencode($relayState);
+            $msg .= '&RelayState='.urlencode($relayState);
         }
 
         if ($key !== null) {
             /* Add the signature. */
-            $msg .= '&SigAlg=' . urlencode($key->type);
+            $msg .= '&SigAlg='.urlencode($key->type);
 
             $signature = $key->signData($msg);
-            $msg .= '&Signature=' . urlencode(base64_encode($signature));
+            $msg .= '&Signature='.urlencode(base64_encode($signature));
         }
 
         if (strpos($destination, '?') === false) {
-            $destination .= '?' . $msg;
+            $destination .= '?'.$msg;
         } else {
-            $destination .= '&' . $msg;
+            $destination .= '&'.$msg;
         }
 
         return $destination;
@@ -79,7 +79,7 @@ 
     public function send(Message $message)
     {
         $destination = $this->getRedirectURL($message);
-        Utils::getContainer()->getLogger()->debug('Redirect to ' . strlen($destination) . ' byte URL: ' . $destination);
+        Utils::getContainer()->getLogger()->debug('Redirect to '.strlen($destination).' byte URL: '.$destination);
         Utils::getContainer()->redirect($destination);
     }
 
@@ -106,7 +106,7 @@ 
         }
 
         if (isset($data['SAMLEncoding']) && $data['SAMLEncoding'] !== self::DEFLATE) {
-            throw new \Exception('Unknown SAMLEncoding: ' . var_export($data['SAMLEncoding'], true));
+            throw new \Exception('Unknown SAMLEncoding: '.var_export($data['SAMLEncoding'], true));
         }
 
         $message = base64_decode($message);
@@ -182,18 +182,18 @@ 
             switch ($name) {
                 case 'SAMLRequest':
                 case 'SAMLResponse':
-                    $sigQuery = $name . '=' . $value;
+                    $sigQuery = $name.'='.$value;
                     break;
                 case 'RelayState':
-                    $relayState = '&RelayState=' . $value;
+                    $relayState = '&RelayState='.$value;
                     break;
                 case 'SigAlg':
-                    $sigAlg = '&SigAlg=' . $value;
+                    $sigAlg = '&SigAlg='.$value;
                     break;
             }
         }
 
-        $data['SignedQuery'] = $sigQuery . $relayState . $sigAlg;
+        $data['SignedQuery'] = $sigQuery.$relayState.$sigAlg;
 
         return $data;
     }

src/SAML2/Compat/Ssp/Logger.php

Spacing +8 added lines, -8 removed lines

@@ -15,7 +15,7 @@ 
      */
     public function emergency($message, array $context = array())
     {
-        \SimpleSAML\Logger::emergency($message . var_export($context, true));
+        \SimpleSAML\Logger::emergency($message.var_export($context, true));
     }
 
     /**
@@ -30,7 +30,7 @@ 
      */
     public function alert($message, array $context = array())
     {
-        \SimpleSAML\Logger::alert($message . var_export($context, true));
+        \SimpleSAML\Logger::alert($message.var_export($context, true));
     }
 
     /**
@@ -44,7 +44,7 @@ 
      */
     public function critical($message, array $context = array())
     {
-        \SimpleSAML\Logger::critical($message . var_export($context, true));
+        \SimpleSAML\Logger::critical($message.var_export($context, true));
     }
 
     /**
@@ -57,7 +57,7 @@ 
      */
     public function error($message, array $context = array())
     {
-        \SimpleSAML\Logger::error($message . var_export($context, true));
+        \SimpleSAML\Logger::error($message.var_export($context, true));
     }
 
     /**
@@ -72,7 +72,7 @@ 
      */
     public function warning($message, array $context = array())
     {
-        \SimpleSAML\Logger::warning($message . var_export($context, true));
+        \SimpleSAML\Logger::warning($message.var_export($context, true));
     }
 
     /**
@@ -84,7 +84,7 @@ 
      */
     public function notice($message, array $context = array())
     {
-        \SimpleSAML\Logger::notice($message . var_export($context, true));
+        \SimpleSAML\Logger::notice($message.var_export($context, true));
     }
 
     /**
@@ -98,7 +98,7 @@ 
      */
     public function info($message, array $context = array())
     {
-        \SimpleSAML\Logger::info($message . var_export($context, true));
+        \SimpleSAML\Logger::info($message.var_export($context, true));
     }
 
     /**
@@ -110,7 +110,7 @@ 
      */
     public function debug($message, array $context = array())
     {
-        \SimpleSAML\Logger::debug($message . var_export($context, true));
+        \SimpleSAML\Logger::debug($message.var_export($context, true));
     }
 
     /**

src/SAML2/Message.php

Doc Comments +1 added lines, -1 removed lines

@@ -355,7 +355,7 @@
     /**
      * Retrieve the issuer if this message.
      *
-     * @return string|\SAML2\XML\saml\Issuer|null The issuer of this message, or NULL if no issuer is given
+     * @return string The issuer of this message, or NULL if no issuer is given
      */
     public function getIssuer()
     {

src/SAML2/XML/saml/BaseIDType.php

Doc Comments -1 removed lines

@@ -70,7 +70,6 @@
     /**
      * Convert this BaseID to XML.
      *
-     * @param \DOMElement $element The element we are converting to XML.
      * @return \DOMElement The XML element after adding the data corresponding to this BaseID.
      */
     public function toXML(\DOMElement $parent = null)

src/SAML2/XML/saml/NameIDType.php

Doc Comments -1 removed lines

@@ -77,7 +77,6 @@
     /**
      * Convert this NameIDType to XML.
      *
-     * @param \DOMElement $element The element we are converting to XML.
      *
      * @return \DOMElement The XML element after adding the data corresponding to this NameIDType.
      */

src/SAML2/XML/mdui/DiscoHints.php

Spacing +2 added lines, -2 removed lines

@@ -54,8 +54,8 @@
             return;
         }
 
-        $this->IPHint =          Utils::extractStrings($xml, Common::NS, 'IPHint');
-        $this->DomainHint =      Utils::extractStrings($xml, Common::NS, 'DomainHint');
+        $this->IPHint = Utils::extractStrings($xml, Common::NS, 'IPHint');
+        $this->DomainHint = Utils::extractStrings($xml, Common::NS, 'DomainHint');
         $this->GeolocationHint = Utils::extractStrings($xml, Common::NS, 'GeolocationHint');
 
         foreach (Utils::xpQuery($xml, "./*[namespace-uri()!='".Common::NS."']") as $node) {

src/SAML2/HTTPArtifact.php

Spacing +5 added lines, -5 removed lines

@@ -39,7 +39,7 @@ 
         }
 
         $generatedId = pack('H*', ((string) SimpleSAML_Utilities::stringToHex(SimpleSAML_Utilities::generateRandomBytes(20))));
-        $artifact = base64_encode("\x00\x04\x00\x00" . sha1($message->getIssuer(), true) . $generatedId) ;
+        $artifact = base64_encode("\x00\x04\x00\x00".sha1($message->getIssuer(), true).$generatedId);
         $artifactData = $message->toUnsignedXML();
         $artifactDataString = $artifactData->ownerDocument->saveXML($artifactData);
 
@@ -81,7 +81,7 @@ 
     {
         if (array_key_exists('SAMLart', $_REQUEST)) {
             $artifact = base64_decode($_REQUEST['SAMLart']);
-            $endpointIndex =  bin2hex(substr($artifact, 2, 2));
+            $endpointIndex = bin2hex(substr($artifact, 2, 2));
             $sourceId = bin2hex(substr($artifact, 4, 20));
         } else {
             throw new \Exception('Missing SAMLart parameter.');
@@ -92,12 +92,12 @@ 
         $idpMetadata = $metadataHandler->getMetaDataConfigForSha1($sourceId, 'saml20-idp-remote');
 
         if ($idpMetadata === null) {
-            throw new \Exception('No metadata found for remote provider with SHA1 ID: ' . var_export($sourceId, true));
+            throw new \Exception('No metadata found for remote provider with SHA1 ID: '.var_export($sourceId, true));
         }
 
         $endpoint = null;
         foreach ($idpMetadata->getEndpoints('ArtifactResolutionService') as $ep) {
-            if ($ep['index'] ===  hexdec($endpointIndex)) {
+            if ($ep['index'] === hexdec($endpointIndex)) {
                 $endpoint = $ep;
                 break;
             }
@@ -107,7 +107,7 @@ 
             throw new \Exception('No ArtifactResolutionService with the correct index.');
         }
 
-        Utils::getContainer()->getLogger()->debug("ArtifactResolutionService endpoint being used is := " . $endpoint['Location']);
+        Utils::getContainer()->getLogger()->debug("ArtifactResolutionService endpoint being used is := ".$endpoint['Location']);
 
         //Construct the ArtifactResolve Request
         $ar = new ArtifactResolve();

src/SAML2/XML/mdui/Keywords.php

Spacing +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
             if (strpos($keyword, "+") !== false) {
                 throw new \Exception('Keywords may not contain a "+" character.');
             }
-            $value .= str_replace(' ', '+', $keyword) . ' ';
+            $value .= str_replace(' ', '+', $keyword).' ';
         }
         $value = rtrim($value);
         $e->appendChild($doc->createTextNode($value));

src/SAML2/Signature/ValidatorChain.php

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
 
         throw new MissingConfigurationException(sprintf(
             'No certificates or fingerprints have been configured%s',
-            $configuration->has('entityid') ? ' for "' . $configuration->get('entityid') . '"' : ''
+            $configuration->has('entityid') ? ' for "'.$configuration->get('entityid').'"' : ''
         ));
     }
 }