simplesamlphp / simplesamlphp-module-adfs

lib/IdP/ADFS.php

Spacing +6 added lines, -6 removed lines

@@ -32,7 +32,7 @@ 
             $metadata = MetaDataStorageHandler::getMetadataHandler();
             $spMetadata = $metadata->getMetaDataConfig($issuer, 'adfs-sp-remote');
 
-            Logger::info('ADFS - IdP.prp: Incoming Authentication request: ' . $issuer . ' id ' . $requestid);
+            Logger::info('ADFS - IdP.prp: Incoming Authentication request: '.$issuer.' id '.$requestid);
         } catch (\Exception $exception) {
             throw new Error\Error('PROCESSAUTHNREQUEST', $exception);
         }
@@ -347,7 +347,7 @@ 
         $spEntityId = $spMetadata['entityid'];
         $spMetadata = Configuration::loadFromArray(
             $spMetadata,
-            '$metadata[' . var_export($spEntityId, true) . ']'
+            '$metadata['.var_export($spEntityId, true).']'
         );
 
         $attributes = $state['Attributes'];
@@ -367,7 +367,7 @@ 
         $idpEntityId = $idpMetadata->getString('entityid');
 
         $idp->addAssociation([
-            'id' => 'adfs:' . $spEntityId,
+            'id' => 'adfs:'.$spEntityId,
             'Handler' => ADFS::class,
             'adfs:entityID' => $spEntityId,
         ]);
@@ -390,7 +390,7 @@ 
         $wresult = ADFS::signResponse($response, $privateKeyFile, $certificateFile, $algo, $passphrase);
 
         $wctx = $state['adfs:wctx'];
-        $wreply = $state['adfs:wreply'] ? : $spMetadata->getValue('prp');
+        $wreply = $state['adfs:wreply'] ?: $spMetadata->getValue('prp');
         ADFS::postResponse($wreply, $wresult, $wctx);
     }
 
@@ -448,8 +448,8 @@ 
         $metadata = MetaDataStorageHandler::getMetadataHandler();
         $spMetadata = $metadata->getMetaDataConfig($association['adfs:entityID'], 'adfs-sp-remote');
         $returnTo = Module::getModuleURL(
-            'adfs/idp/prp.php?assocId=' . urlencode($association["id"]) . '&relayState=' . urlencode($relayState)
+            'adfs/idp/prp.php?assocId='.urlencode($association["id"]).'&relayState='.urlencode($relayState)
         );
-        return $spMetadata->getValue('prp') . '?wa=wsignoutcleanup1.0&wreply=' . urlencode($returnTo);
+        return $spMetadata->getValue('prp').'?wa=wsignoutcleanup1.0&wreply='.urlencode($returnTo);
     }
 }

lib/Controller/AdfsController.php

Spacing +9 added lines, -9 removed lines

@@ -111,7 +111,7 @@ 
                 ];
             }
 
-            $adfs_service_location = Module::getModuleURL('adfs') . '/idp/prp.php';
+            $adfs_service_location = Module::getModuleURL('adfs').'/idp/prp.php';
             $metaArray = [
                 'metadata-set'        => 'adfs-idp-remote',
                 'entityid'            => $idpentityid,
@@ -173,7 +173,7 @@ 
                 $metaArray['RegistrationInfo'] = $idpmeta->getArray('RegistrationInfo');
             }
 
-            $metaflat = '$metadata[' . var_export($idpentityid, true) . '] = ' . var_export($metaArray, true) . ';';
+            $metaflat = '$metadata['.var_export($idpentityid, true).'] = '.var_export($metaArray, true).';';
 
             $metaBuilder = new Metadata\SAMLBuilder($idpentityid);
             $metaBuilder->addSecurityTokenServiceType($metaArray);
@@ -203,12 +203,12 @@ 
                 $certdata = [];
                 foreach (array_keys($availableCerts) as $availableCert) {
                     $certdata[$availableCert]['name'] = $availableCert;
-                    $certdata[$availableCert]['url'] = Module::getModuleURL('saml/idp/certs.php') .
-                        '/' . $availableCert;
+                    $certdata[$availableCert]['url'] = Module::getModuleURL('saml/idp/certs.php').
+                        '/'.$availableCert;
 
                     $certdata[$availableCert]['comment'] = '';
                     if ($availableCerts[$availableCert]['certFingerprint'][0] === 'afe71c28ef740bc87425be13a2263d37971da1f9') {
-                        $certdata[$availableCert]['comment'] = 'This is the default certificate.' .
+                        $certdata[$availableCert]['comment'] = 'This is the default certificate.'.
                             ' Generate a new certificate if this is a production system.';
                     }
                 }
@@ -250,20 +250,20 @@ 
         Logger::info('ADFS - IdP.prp: Accessing ADFS IdP endpoint prp');
 
         $idpEntityId = $this->metadata->getMetaDataCurrentEntityID('adfs-idp-hosted');
-        $idp = IdP::getById('adfs:' . $idpEntityId);
+        $idp = IdP::getById('adfs:'.$idpEntityId);
 
         if (isset($_GET['wa'])) {
             if ($_GET['wa'] === 'wsignout1.0') {
                 return new StreamedResponse(
                     /** @return void */
-                    function () use ($idp) {
+                    function() use ($idp) {
                         ADFS::receiveLogoutMessage($idp);
                     }
                 );
             } elseif ($_GET['wa'] === 'wsignin1.0') {
                 return new StreamedResponse(
                     /** @return void */
-                    function () use ($idp) {
+                    function() use ($idp) {
                         ADFS::receiveAuthnRequest($idp);
                     }
                 );
@@ -277,7 +277,7 @@ 
 
             return new StreamedResponse(
                 /** @return void */
-                function () use ($idp, $assocId, $relayState, $logoutError) {
+                function() use ($idp, $assocId, $relayState, $logoutError) {
                     $idp->handleLogoutResponse($assocId, $relayState, $logoutError);
                 }
             );