Fix codesniffer issue

simplesamlphp · Mar 16, 2023 · 31a2455 · 31a2455
1 parent 633c68d
commit 31a2455
Show file tree

Hide file tree

Showing 14 changed files with 27 additions and 41 deletions.
diff --git a/src/SAML2/Binding.php b/src/SAML2/Binding.php
@@ -111,10 +111,8 @@ public static function getCurrentBinding(ServerRequestInterface $request): Bindi
                     if (
                         ($contentType === 'text/xml' || $contentType === 'application/xml')
                         // See paragraph 3.2.3 of Binding for SAML2 (OASIS)
-                        || (
-                            $request->hasHeader('SOAPAction')
-                            && $request->getHeader('SOAPAction')[0] === 'http://www.oasis-open.org/committees/security'
-                        )
+                        || ($request->hasHeader('SOAPAction')
+                            && $request->getHeader('SOAPAction')[0] === 'http://www.oasis-open.org/committees/security')
                     ) {
                         return new SOAP();
                     }

diff --git a/src/SAML2/Utils/XPath.php b/src/SAML2/Utils/XPath.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace SimpleSAML\SAML2\Utils;
 
 use DOMNode;

diff --git a/src/SAML2/XML/md/AbstractMetadataDocument.php b/src/SAML2/XML/md/AbstractMetadataDocument.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace SimpleSAML\SAML2\XML\md;
 
 use DOMElement;

diff --git a/src/SAML2/XML/mdui/DiscoHints.php b/src/SAML2/XML/mdui/DiscoHints.php
@@ -99,12 +99,10 @@ public function addChild(Chunk $child): void
      */
     public function isEmptyElement(): bool
     {
-        return (
-            empty($this->elements)
+        return empty($this->elements)
             && empty($this->ipHint)
             && empty($this->domainHint)
-            && empty($this->geolocationHint)
-        );
+            && empty($this->geolocationHint);
     }
 
 

diff --git a/src/SAML2/XML/mdui/UIInfo.php b/src/SAML2/XML/mdui/UIInfo.php
@@ -202,15 +202,13 @@ public function addChild(Chunk $child): void
      */
     public function isEmptyElement(): bool
     {
-        return (
-            empty($this->displayName)
+        return empty($this->displayName)
             && empty($this->description)
             && empty($this->informationURL)
             && empty($this->privacyStatementURL)
             && empty($this->keywords)
             && empty($this->logo)
-            && empty($this->elements)
-        );
+            && empty($this->elements);
     }
 
 

diff --git a/src/SAML2/XML/saml/AbstractBaseID.php b/src/SAML2/XML/saml/AbstractBaseID.php
@@ -95,7 +95,7 @@ public static function fromXML(DOMElement $xml): static
             list($element) = $qname;
         }
         $ns = $xml->lookupNamespaceUri($prefix);
-        $type = ($ns === null ) ? $element : implode(':', [$ns, $element]);
+        $type = ($ns === null) ? $element : implode(':', [$ns, $element]);
 
         // now check if we have a handler registered for it
         $handler = Utils::getContainer()->getExtensionHandler($type);

diff --git a/src/SAML2/XML/saml/Advice.php b/src/SAML2/XML/saml/Advice.php
@@ -55,13 +55,11 @@ public function __construct(
      */
     public function isEmptyElement(): bool
     {
-        return (
-            empty($this->assertionIDRef)
+        return empty($this->assertionIDRef)
             && empty($this->assertionURIRef)
             && empty($this->assertion)
             && empty($this->encryptedAssertion)
-            && empty($this->getElements())
-        );
+            && empty($this->getElements());
     }
 
 

diff --git a/src/SAML2/XML/saml/Conditions.php b/src/SAML2/XML/saml/Conditions.php
@@ -117,14 +117,12 @@ public function getProxyRestriction(): ?ProxyRestriction
      */
     public function isEmptyElement(): bool
     {
-        return (
-            empty($this->notBefore)
+        return empty($this->notBefore)
             && empty($this->notOnOrAfter)
             && empty($this->condition)
             && empty($this->audienceRestriction)
             && $this->oneTimeUse === false
-            && empty($this->proxyRestriction)
-        );
+            && empty($this->proxyRestriction);
     }
 
 

diff --git a/src/SAML2/XML/saml/Evidence.php b/src/SAML2/XML/saml/Evidence.php
@@ -42,12 +42,10 @@ public function __construct(
      */
     public function isEmptyElement(): bool
     {
-        return (
-            empty($this->assertionIDRef)
+        return empty($this->assertionIDRef)
             && empty($this->assertionURIRef)
             && empty($this->assertion)
-            && empty($this->encryptedAssertion)
-        );
+            && empty($this->encryptedAssertion);
     }
 
 

diff --git a/src/SAML2/XML/saml/ProxyRestriction.php b/src/SAML2/XML/saml/ProxyRestriction.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace SimpleSAML\SAML2\XML\saml;
 
 use DOMElement;

diff --git a/src/SAML2/XML/saml/SubjectConfirmationData.php b/src/SAML2/XML/saml/SubjectConfirmationData.php
@@ -137,15 +137,13 @@ public function getInfo(): array
      */
     public function isEmptyElement(): bool
     {
-        return (
-            empty($this->notBefore)
+        return empty($this->notBefore)
             && empty($this->notOnOrAfter)
             && empty($this->recipient)
             && empty($this->inResponseTo)
             && empty($this->address)
             && empty($this->info)
-            && empty($this->namespacedAttributes)
-        );
+            && empty($this->namespacedAttributes);
     }
 
 

diff --git a/src/SAML2/XML/saml/SubjectLocality.php b/src/SAML2/XML/saml/SubjectLocality.php
@@ -60,10 +60,8 @@ public function getDnsName(): ?string
      */
     public function isEmptyElement(): bool
     {
-        return (
-            empty($this->address)
-            && empty($this->dnsName)
-        );
+        return empty($this->address)
+            && empty($this->dnsName);
     }
 
 

diff --git a/src/SAML2/XML/samlp/NameIDPolicy.php b/src/SAML2/XML/samlp/NameIDPolicy.php
@@ -68,11 +68,9 @@ public function getAllowCreate(): ?bool
      */
     public function isEmptyElement(): bool
     {
-        return (
-            empty($this->Format)
+        return empty($this->Format)
             && empty($this->SPNameQualifier)
-            && empty($this->AllowCreate)
-        );
+            && empty($this->AllowCreate);
     }
 
 

diff --git a/src/SAML2/XML/samlp/Scoping.php b/src/SAML2/XML/samlp/Scoping.php
@@ -71,11 +71,9 @@ public function getProxyCount(): ?int
      */
     public function isEmptyElement(): bool
     {
-        return (
-            empty($this->proxyCount)
+        return empty($this->proxyCount)
             && empty($this->IDPList)
-            && empty($this->requesterId)
-        );
+            && empty($this->requesterId);
     }