Start using PHP 8.1 features

Author: tvdijen

src/SAML2/Constants.php

@@ -96,33 +96,6 @@ class Constants extends \SimpleSAML\XMLSecurity\Constants
      */
     public const CM_VOUCHES = 'urn:oasis:names:tc:SAML:2.0:cm:sender-vouches';
 
-    /**
-     * Request Authentication Context Comparison indicating that  the resulting authentication context in the
-     * authentication statement MUST be stronger (as deemed by the responder) than any one of the authentication
-     * contexts specified
-     */
-    public const COMPARISON_BETTER = 'better';
-
-    /**
-     * Request Authentication Context Comparison indicating that the resulting authentication context in the
-     * authentication statement MUST be the exact match of at least one of the authentication contexts specified
-     */
-    public const COMPARISON_EXACT = 'exact';
-
-    /**
-     * Request Authentication Context Comparison indicating that the resulting authentication context in the
-     * authentication statement MUST be as strong as possible (as deemed by the responder) without exceeding the
-     * strength of at least one of the authentication contexts specified.
-     */
-    public const COMPARISON_MAXIMUM = 'maximum';
-
-    /**
-     * Request Authentication Context Comparison indicating that he resulting authentication context in the
-     * authentication statement MUST be at least as strong (as deemed by the responder) as one of the authentication
-     * contexts specified.
-     */
-    public const COMPARISON_MINIMUM = 'minimum';
-
     /**
      * Indicates that a principal’s consent has been explicitly obtained by the issuer of the message during the
      * action that initiated the message.
@@ -498,17 +471,4 @@ class Constants extends \SimpleSAML\XMLSecurity\Constants
      * The format to express a timestamp in SAML2
      */
     public const DATETIME_FORMAT = 'Y-m-d\\TH:i:sp';
-
-    /**
-     * Valid values for saml:DecisionType
-     */
-    public const AUTHZ_DECISION_PERMIT = 'Permit';
-    public const AUTHZ_DECISION_DENY = 'Deny';
-    public const AUTHZ_DECISION_INDETERMINATE = 'Indeterminate';
-
-    public const AUTHZ_DECISIONS = [
-        self::AUTHZ_DECISION_PERMIT,
-        self::AUTHZ_DECISION_DENY,
-        self::AUTHZ_DECISION_INDETERMINATE,
-    ];
 }

src/SAML2/XML/Comparison.php

@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types=1);
+
+namespace SimpleSAML\SAML2\XML;
+
+enum Comparison: string
+{
+    /**
+     * Request Authentication Context Comparison indicating that  the resulting authentication context in the
+     * authentication statement MUST be stronger (as deemed by the responder) than any one of the authentication
+     * contexts specified
+     */
+    case BETTER = 'better';
+
+    /**
+     * Request Authentication Context Comparison indicating that the resulting authentication context in the
+     * authentication statement MUST be the exact match of at least one of the authentication contexts specified
+     */
+    case EXACT = 'exact';
+
+    /**
+     * Request Authentication Context Comparison indicating that the resulting authentication context in the
+     * authentication statement MUST be as strong as possible (as deemed by the responder) without exceeding the
+     * strength of at least one of the authentication contexts specified.
+     */
+    case MAXIMUM = 'maximum';
+
+    /**
+     * Request Authentication Context Comparison indicating that he resulting authentication context in the
+     * authentication statement MUST be at least as strong (as deemed by the responder) as one of the authentication
+     * contexts specified.
+     */
+    case MINIMUM = 'minimum';
+}

src/SAML2/XML/Decision.php

@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+namespace SimpleSAML\SAML2\XML;
+
+/**
+ * Valid values for saml:DecisionType
+ */
+enum Decision: string
+{
+    case PERMIT = 'Permit';
+    case DENY = 'Deny';
+    case INDETERMINATE = 'Indeterminate';
+}

src/SAML2/XML/alg/DigestMethod.php

@@ -7,11 +7,11 @@
 use DOMElement;
 use SimpleSAML\Assert\Assert;
 use SimpleSAML\XML\Chunk;
-use SimpleSAML\XML\Constants as C;
 use SimpleSAML\XML\DOMDocumentFactory;
 use SimpleSAML\XML\Exception\InvalidDOMElementException;
 use SimpleSAML\XML\Exception\SchemaViolationException;
 use SimpleSAML\XML\ExtendableElementTrait;
+use SimpleSAML\XML\XsNamespace as NS;
 
 /**
  * Class for handling the alg:DigestMethod element.
@@ -24,7 +24,7 @@ final class DigestMethod extends AbstractAlgElement
     use ExtendableElementTrait;
 
     /** The namespace-attribute for the xs:any element */
-    public const XS_ANY_ELT_NAMESPACE = C::XS_ANY_NS_ANY;
+    public const XS_ANY_ELT_NAMESPACE = NS::ANY;
 
 
     /**

src/SAML2/XML/alg/SigningMethod.php

@@ -7,12 +7,12 @@
 use DOMElement;
 use SimpleSAML\Assert\Assert;
 use SimpleSAML\XML\Chunk;
-use SimpleSAML\XML\Constants as C;
 use SimpleSAML\XML\DOMDocumentFactory;
 use SimpleSAML\XML\Exception\InvalidDOMElementException;
 use SimpleSAML\XML\Exception\MissingAttributeException;
 use SimpleSAML\XML\Exception\SchemaViolationException;
 use SimpleSAML\XML\ExtendableElementTrait;
+use SimpleSAML\XML\XsNamespace as NS;
 
 use function strval;
 
@@ -27,7 +27,7 @@ final class SigningMethod extends AbstractAlgElement
     use ExtendableElementTrait;
 
     /** The namespace-attribute for the xs:any element */
-    public const XS_ANY_ELT_NAMESPACE = C::XS_ANY_NS_ANY;
+    public const XS_ANY_ELT_NAMESPACE = NS::ANY;
 
 
     /**

src/SAML2/XML/md/AbstractEndpointType.php

@@ -4,9 +4,9 @@
 
 namespace SimpleSAML\SAML2\XML\md;
 
+use CASE_LOWER;
 use DOMElement;
 use SimpleSAML\Assert\Assert;
-use SimpleSAML\SAML2\Constants as C;
 use SimpleSAML\SAML2\Exception\ArrayValidationException;
 use SimpleSAML\XML\ArrayizableElementInterface;
 use SimpleSAML\XML\Attribute as XMLAttribute;
@@ -16,6 +16,7 @@
 use SimpleSAML\XML\ExtendableAttributesTrait;
 use SimpleSAML\XML\ExtendableElementTrait;
 use SimpleSAML\XML\SerializableElementInterface;
+use SimpleSAML\XML\XsNamespace as NS;
 
 use function array_change_key_case;
 use function array_key_exists;
@@ -43,10 +44,10 @@ abstract class AbstractEndpointType extends AbstractMdElement implements Arrayiz
     use ExtendableElementTrait;
 
     /** The namespace-attribute for the xs:any element */
-    public const XS_ANY_ELT_NAMESPACE = C::XS_ANY_NS_OTHER;
+    public const XS_ANY_ELT_NAMESPACE = NS::OTHER;
 
     /** The namespace-attribute for the xs:anyAttribute element */
-    public const XS_ANY_ATTR_NAMESPACE = C::XS_ANY_NS_OTHER;
+    public const XS_ANY_ATTR_NAMESPACE = NS::OTHER;
 
 
     /**
@@ -140,13 +141,11 @@ public static function fromXML(DOMElement $xml): static
 
         $children = [];
         foreach ($xml->childNodes as $child) {
-            if ($child->namespaceURI === C::NS_MD) {
+            if (!($child instanceof DOMElement)) {
                 continue;
-            } elseif (!($child instanceof DOMElement)) {
-                continue;
-            }
-
-            $children[] = new Chunk($child);
+            } elseif ($child->namespaceURI !== static::NS) {
+                $children[] = new Chunk($child);
+            } // else continue
         }
 
         return new static(

src/SAML2/XML/md/AbstractRoleDescriptorType.php

@@ -10,6 +10,7 @@
 use SimpleSAML\SAML2\Constants as C;
 use SimpleSAML\XML\Exception\SchemaViolationException;
 use SimpleSAML\XML\ExtendableAttributesTrait;
+use SimpleSAML\XML\XsNamespace as NS;
 
 use function implode;
 
@@ -23,7 +24,7 @@ abstract class AbstractRoleDescriptorType extends AbstractMetadataDocument
     use ExtendableAttributesTrait;
 
     /** The namespace-attribute for the xs:anyAttribute element */
-    public const XS_ANY_ATTR_NAMESPACE = C::XS_ANY_NS_OTHER;
+    public const XS_ANY_ATTR_NAMESPACE = NS::OTHER;
 
 
     /**

src/SAML2/XML/md/AffiliationDescriptor.php

@@ -14,6 +14,7 @@
 use SimpleSAML\XML\Exception\SchemaViolationException;
 use SimpleSAML\XML\Exception\TooManyElementsException;
 use SimpleSAML\XML\ExtendableAttributesTrait;
+use SimpleSAML\XML\XsNamespace as NS;
 use SimpleSAML\XMLSecurity\XML\ds\Signature;
 
 /**
@@ -26,7 +27,7 @@ final class AffiliationDescriptor extends AbstractMetadataDocument
     use ExtendableAttributesTrait;
 
     /** The namespace-attribute for the xs:anyAttribute element */
-    public const XS_ANY_ATTR_NAMESPACE = C::XS_ANY_NS_OTHER;
+    public const XS_ANY_ATTR_NAMESPACE = NS::OTHER;
 
 
     /**

src/SAML2/XML/md/ContactPerson.php

@@ -20,6 +20,7 @@
 use SimpleSAML\XML\ExtendableAttributesTrait;
 use SimpleSAML\XML\SerializableElementInterface;
 use SimpleSAML\XML\Utils as XMLUtils;
+use SimpleSAML\XML\XsNamespace as NS;
 
 use function array_filter;
 use function array_change_key_case;
@@ -43,7 +44,7 @@ final class ContactPerson extends AbstractMdElement implements ArrayizableElemen
     use ExtendableAttributesTrait;
 
     /** The namespace-attribute for the xs:anyAttribute element */
-    public const XS_ANY_ATTR_NAMESPACE = C::XS_ANY_NS_OTHER;
+    public const XS_ANY_ATTR_NAMESPACE = NS::OTHER;
 
 
     /**

src/SAML2/XML/md/EntityDescriptor.php

@@ -13,6 +13,7 @@
 use SimpleSAML\XML\Exception\SchemaViolationException;
 use SimpleSAML\XML\Exception\TooManyElementsException;
 use SimpleSAML\XML\ExtendableAttributesTrait;
+use SimpleSAML\XML\XsNamespace as NS;
 use SimpleSAML\XMLSecurity\XML\ds\Signature;
 
 use function is_null;
@@ -27,7 +28,7 @@ final class EntityDescriptor extends AbstractMetadataDocument
     use ExtendableAttributesTrait;
 
     /** The namespace-attribute for the xs:anyAttribute element */
-    public const XS_ANY_ATTR_NAMESPACE = C::XS_ANY_NS_OTHER;
+    public const XS_ANY_ATTR_NAMESPACE = NS::OTHER;
 
 
     /**