Bugfix: use the correct assertion to test the value of an attribute-name of type basic

Author: tvdijen

src/XML/saml/Attribute.php

@@ -79,7 +79,7 @@ public function __construct(
                 );
                 break;
             case C::NAMEFORMAT_BASIC:
-                Assert::validNCName(
+                Assert::validName(
                     strval($name),
                     sprintf("Attribute name `%s` does not match its declared format `%s`", $name, $nameFormat),
                 );

src/XML/samlp/NameIDPolicy.php

@@ -47,7 +47,7 @@ public function __construct(
         protected ?SAMLStringValue $SPNameQualifier = null,
         protected ?BooleanValue $AllowCreate = null,
     ) {
-        if ($AllowCreate->equals(BooleanValue::fromBoolean(true)) {
+        if ($AllowCreate->equals(BooleanValue::fromBoolean(true))) {
             // Per Errata E14: AllowCreate
             Assert::notSame(
                 $Format->getValue(),