flipboxfactory / saml-core

src/helpers/SecurityHelper.php

Spacing +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
     {
         $algorithm = $reader->getAlgorithm();
 
-        if (! in_array($algorithm, [
+        if (!in_array($algorithm, [
             XMLSecurityKey::RSA_SHA1,
             XMLSecurityKey::RSA_SHA256,
             XMLSecurityKey::RSA_SHA384,

src/migrations/AbstractInstall.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 
     protected function installKeyChain()
     {
-        if (! \Craft::$app->plugins->getPlugin('keychain')) {
+        if (!\Craft::$app->plugins->getPlugin('keychain')) {
             \Craft::$app->plugins->installPlugin('keychain');
         }
     }

src/AbstractPlugin.php

Spacing +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
         /**
          * Register Core Module on Craft
          */
-        if (! \Craft::$app->getModule(Saml::MODULE_ID)) {
+        if (!\Craft::$app->getModule(Saml::MODULE_ID)) {
             \Craft::$app->setModule(Saml::MODULE_ID, [
                 'class' => Saml::class
             ]);
@@ -77,7 +77,7 @@ 
         Event::on(
             View::class,
             View::EVENT_REGISTER_CP_TEMPLATE_ROOTS,
-            function (RegisterTemplateRootsEvent $e) {
+            function(RegisterTemplateRootsEvent $e) {
                 if (is_dir($baseDir = Saml::getTemplateRoot())) {
                     $e->roots[Saml::getTemplateRootKey($this)] = $baseDir;
                 }

src/models/AbstractSettings.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
      */
     public function getEntityId()
     {
-        if (! $this->entityId) {
+        if (!$this->entityId) {
             $this->entityId = UrlHelper::baseUrl();
         }
 

src/cli/AbstractMetadata.php

Spacing +5 added lines, -5 removed lines

@@ -112,7 +112,7 @@ 
             );
         }
 
-        if (! $this->keyPairId) {
+        if (!$this->keyPairId) {
             $this->keyPairId = null;
             $this->createKeyPair = true;
         }
@@ -128,7 +128,7 @@ 
     {
 
         $this->newProviderRecord([]);
-        if (! $this->keyPairId) {
+        if (!$this->keyPairId) {
             $this->chooseFromKeyChain();
         }
 
@@ -136,14 +136,14 @@ 
         if ($this->keyPairId === null && $this->createKeyPair) {
             $openSslConfig = $this->promptKeyPair();
             $keyPairRecord = $openSslConfig->create();
-            if (! KeyChain::getInstance()->getService()->save($keyPairRecord)) {
+            if (!KeyChain::getInstance()->getService()->save($keyPairRecord)) {
                 $this->stderr(
                     sprintf('Failed to save new key pair to the database') . PHP_EOL
                     , Console::FG_RED);
                 return ExitCode::DATAERR;
             }
         } else if ($this->keyPairId) {
-            if (! ($keyPairRecord = KeyChainRecord::findOne([
+            if (!($keyPairRecord = KeyChainRecord::findOne([
                 'id' => $this->keyPairId,
             ]))) {
                 $this->stderr(
@@ -185,7 +185,7 @@ 
     public function actionImport($file, $type)
     {
 
-        if (! in_array($type, ['idp', 'sp'])) {
+        if (!in_array($type, ['idp', 'sp'])) {
             throw new \InvalidArgumentException('Type must be idp or sp');
         }
 

src/services/AbstractProviderService.php

Spacing +6 added lines, -6 removed lines

@@ -50,7 +50,7 @@ 
             return $this->cache[$condition['entityId']];
         }
 
-        if (! $provider = $class::find()->where($condition)->one()) {
+        if (!$provider = $class::find()->where($condition)->one()) {
             return null;
         }
 
@@ -80,7 +80,7 @@ 
      */
     protected function findByType($type)
     {
-        if (! in_array($type, ['sp', 'idp'])) {
+        if (!in_array($type, ['sp', 'idp'])) {
             throw new \InvalidArgumentException("Type must be idp or sp.");
         }
         return $this->find([
@@ -135,7 +135,7 @@ 
         }
 
         //save record
-        if (! $record->save($runValidation, $attributeNames)) {
+        if (!$record->save($runValidation, $attributeNames)) {
             throw new \Exception(Json::encode($record->getErrors()));
         }
 
@@ -159,7 +159,7 @@ 
         $attributeNames = null
     )
     {
-        if (! $provider->id && ! $keyChain->id) {
+        if (!$provider->id && !$keyChain->id) {
             throw new \Exception('Provider id and keychain id must exist before linking.');
         }
         $linkAttributes = [
@@ -167,7 +167,7 @@ 
         ];
 
         /** @var LinkRecord $link */
-        if (! $link = LinkRecord::find()->where($linkAttributes)->one()) {
+        if (!$link = LinkRecord::find()->where($linkAttributes)->one()) {
             $link = new LinkRecord($linkAttributes);
         }
 
@@ -176,7 +176,7 @@ 
             $link,
             $linkAttributes
         );
-        if (! $link->save($runValidation, $attributeNames)) {
+        if (!$link->save($runValidation, $attributeNames)) {
             throw new \Exception(Json::encode($record->getErrors()));
         }
     }

Braces +3 added lines, -2 removed lines

@@ -119,8 +119,9 @@
             'metadata' => SerializeHelper::toXml($entityDescriptor),
         ]);
 
-        if ($keyChainRecord)
-            $provider->setKeychain($keyChainRecord);
+        if ($keyChainRecord) {
+                    $provider->setKeychain($keyChainRecord);
+        }
 
         return $provider;
     }

src/services/bindings/AbstractHttpPost.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
         $provider = $this->getSamlPlugin()->getProvider()->findByEntityId(
             $issuer->getValue()
         );
-        if (! $provider) {
+        if (!$provider) {
             throw new InvalidIssuer(
                 sprintf("Invalid issuer: %s", $issuer->getValue())
             );
@@ -156,7 +156,7 @@ 
         /**
          * Validate Signature
          */
-        if ($message->getSignature() && ! SecurityHelper::validSignature($message, $key)) {
+        if ($message->getSignature() && !SecurityHelper::validSignature($message, $key)) {
             throw new InvalidSignature("Invalid request", 400);
         }
 

src/services/bindings/AbstractFactory.php

Spacing +5 added lines, -5 removed lines

@@ -79,7 +79,7 @@ 
             $binding = static::determineBindingToSp($message, $provider);
         }
 
-        return static::getService($binding)->send($message,  $provider);
+        return static::getService($binding)->send($message, $provider);
     }
 
     /**
@@ -94,7 +94,7 @@ 
 
         switch (true) {
             case MessageHelper::isRequest($message):
-                if (! $provider->getMetadataModel()->getFirstSpSsoDescriptor()->getFirstAssertionConsumerService(
+                if (!$provider->getMetadataModel()->getFirstSpSsoDescriptor()->getFirstAssertionConsumerService(
                     SamlConstants::BINDING_SAML2_HTTP_POST
                 )) {
                     if ($provider->getMetadataModel()->getFirstSpSsoDescriptor()->getFirstAssertionConsumerService(
@@ -108,7 +108,7 @@ 
 
                 break;
             case MessageHelper::isResponse($message):
-                if (! $provider->getMetadataModel()->getFirstSpSsoDescriptor()->getFirstSingleLogoutService(
+                if (!$provider->getMetadataModel()->getFirstSpSsoDescriptor()->getFirstSingleLogoutService(
                     SamlConstants::BINDING_SAML2_HTTP_POST
                 )) {
                     if ($provider->getMetadataModel()->getFirstSpSsoDescriptor()->getFirstSingleLogoutService(
@@ -139,7 +139,7 @@ 
 
         switch (true) {
             case MessageHelper::isRequest($message):
-                if (! $provider->getMetadataModel()->getFirstIdpSsoDescriptor()->getFirstSingleSignOnService(
+                if (!$provider->getMetadataModel()->getFirstIdpSsoDescriptor()->getFirstSingleSignOnService(
                     SamlConstants::BINDING_SAML2_HTTP_POST
                 )) {
                     if ($provider->getMetadataModel()->getFirstIdpSsoDescriptor()->getFirstSingleSignOnService(
@@ -152,7 +152,7 @@ 
                 }
                 break;
             case MessageHelper::isResponse($message):
-                if (! $provider->getMetadataModel()->getFirstIdpSsoDescriptor()->getFirstSingleLogoutService(
+                if (!$provider->getMetadataModel()->getFirstIdpSsoDescriptor()->getFirstSingleLogoutService(
                     SamlConstants::BINDING_SAML2_HTTP_POST
                 )) {
                     if ($provider->getMetadataModel()->getFirstIdpSsoDescriptor()->getFirstSingleLogoutService(

src/services/AbstractProviderIdentityService.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
     public function save(ProviderIdentityInterface $record, $runValidation = true, $attributeNames = null): ProviderIdentityInterface
     {
 
-        if (! $record->save($runValidation, $attributeNames)) {
+        if (!$record->save($runValidation, $attributeNames)) {
             throw new \Exception(Json::encode($record->getErrors()));
         }