simplesamlphp / simplesamlphp-module-adfs

lib/IdP/ADFS.php

Spacing +6 added lines, -6 removed lines

@@ -35,7 +35,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);
 
         $state = [
             'Responder' => [ADFS::class, 'sendResponse'],
@@ -354,7 +354,7 @@ 
         $spEntityId = $spMetadata['entityid'];
         $spMetadata = Configuration::loadFromArray(
             $spMetadata,
-            '$metadata[' . var_export($spEntityId, true) . ']'
+            '$metadata['.var_export($spEntityId, true).']'
         );
 
         $attributes = $state['Attributes'];
@@ -375,7 +375,7 @@ 
         $idpEntityId = $idpMetadata->getString('entityid');
 
         $idp->addAssociation([
-            'id' => 'adfs:' . $spEntityId,
+            'id' => 'adfs:'.$spEntityId,
             'Handler' => ADFS::class,
             'adfs:entityID' => $spEntityId,
         ]);
@@ -399,7 +399,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);
     }
 
@@ -457,8 +457,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/Adfs.php

Spacing +7 added lines, -7 removed lines

@@ -121,7 +121,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,
@@ -183,7 +183,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);
@@ -213,8 +213,8 @@ 
                 $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'] = '';
                 }
@@ -256,13 +256,13 @@ 
         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 ($request->query->has('wa')) {
             $wa = $request->query->get('wa');
             if ($wa === 'wsignout1.0') {
                 return new StreamedResponse(
-                    function () use ($idp) {
+                    function() use ($idp) {
                         ADFS_IDP::receiveLogoutMessage($idp);
                     }
                 );
@@ -280,7 +280,7 @@ 
             $logoutError = null;
 
             return new StreamedResponse(
-                function () use ($idp, /** @scrutinizer ignore-type */ $assocId, $relayState, $logoutError) {
+                function() use ($idp, /** @scrutinizer ignore-type */ $assocId, $relayState, $logoutError) {
                     $idp->handleLogoutResponse($assocId, $relayState, $logoutError);
                 }
             );