simplesamlphp / xml-security

src/XML/xenc11/PRF.php

<?php

declare(strict_types=1);

namespace SimpleSAML\XMLSecurity\XML\xenc11;

use DOMElement;
use SimpleSAML\Assert\Assert;
use SimpleSAML\XML\Exception\InvalidDOMElementException;

/**
 * A class implementing the xenc11:PRF element.
 *
 * @package simplesamlphp/xml-security
 */
final class PRF extends AbstractPRFAlgorithmIdentifierType
{
    /**
     * @inheritDoc
     *
     * @throws \SimpleSAML\XML\Exception\InvalidDOMElementException
     *   If the qualified name of the supplied element is wrong
     */
    public static function fromXML(DOMElement $xml): static
    {
        Assert::same($xml->localName, static::getLocalName(), InvalidDOMElementException::class);
        Assert::same($xml->namespaceURI, static::getNamespaceURI(), InvalidDOMElementException::class);

        return new static(
            self::getOptionalAttribute($xml, 'Algorithm', null),

It seems like self::getOptionalAttribute($xml, 'Algorithm', null) can also be of type null; however, parameter $Algorithm of SimpleSAML\XMLSecurity\XML\xenc11\PRF::__construct() does only seem to accept string, maybe add an additional type check?

        );
    }
}