Replace array_pop with array_last (#416)

* Bump the all-actions group across 1 directory with 2 updates (#412)

Bumps the all-actions group with 2 updates in the / directory: [actions/checkout](https://github.com/actions/checkout) and [codecov/codecov-action](https://github.com/codecov/codecov-action).


Updates `actions/checkout` from 6 to 7
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v6...v7)

Updates `codecov/codecov-action` from 6 to 7
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](codecov/codecov-action@v6...v7)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: all-actions
- dependency-name: codecov/codecov-action
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: all-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Fix stray whitespace

* Bump actions/cache from 5 to 6 in the all-actions group (#413)

Bumps the all-actions group with 1 update: [actions/cache](https://github.com/actions/cache).


Updates `actions/cache` from 5 to 6
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](actions/cache@v5...v6)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: all-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Remove last case of XMLSeclibs

* Replace array_pop with array_last

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Author: tvdijen

src/Configuration/IdentityProvider.php

@@ -8,7 +8,7 @@
 use SimpleSAML\XMLSecurity\Constants as C;
 
 use function array_filter;
-use function array_pop;
+use function array_last;
 use function count;
 use function sprintf;
 
@@ -94,7 +94,7 @@ public function getPrivateKey(string $name, ?bool $required = null)
             return null;
         }
 
-        return array_pop($key);
+        return array_last($key);
     }
 
 

src/Configuration/ServiceProvider.php

@@ -8,7 +8,7 @@
 use SimpleSAML\XMLSecurity\Constants as C;
 
 use function array_filter;
-use function array_pop;
+use function array_last;
 use function count;
 use function sprintf;
 
@@ -102,7 +102,7 @@ public function getPrivateKey(string $name, ?bool $required = null)
             return null;
         }
 
-        return array_pop($key);
+        return array_last($key);
     }
 
 

src/XML/IdentifierTrait.php

@@ -12,7 +12,7 @@
 use SimpleSAML\SAML2\XML\saml\NameID;
 use SimpleSAML\XMLSchema\Exception\TooManyElementsException;
 
-use function array_pop;
+use function array_last;
 
 /**
  * Trait grouping common functionality for elements that can hold identifiers.
@@ -94,7 +94,7 @@ protected static function getIdentifierFromXML(DOMElement $xml): ?IdentifierInte
             TooManyElementsException::class,
         );
 
-        $identifier = array_pop($identifiers);
+        $identifier = array_last($identifiers);
 
         return $identifier;
     }

src/XML/SignedElementTrait.php

@@ -11,6 +11,8 @@
 use SimpleSAML\XMLSecurity\XML\ds\Signature;
 use SimpleSAML\XMLSecurity\XML\SignedElementTrait as BaseSignedElementTrait;
 
+use function array_last;
+
 /**
  * Helper trait for processing signed elements.
  *
@@ -37,7 +39,7 @@ protected function setSignature(Signature $signature): void
         $references = $signature->getSignedInfo()->getReferences();
         Assert::count($references, 1, "A signature needs to have exactly one Reference, %d found.");
 
-        $reference = array_pop($references);
+        $reference = array_last($references);
         Assert::notNull($reference->getURI(), "URI attribute not found.", ReferenceValidationFailedException::class);
         Assert::startsWith(
             $reference->getURI()->getValue(),

src/XML/ecp/Request.php

@@ -19,6 +19,7 @@
 use SimpleSAML\XMLSchema\Exception\TooManyElementsException;
 use SimpleSAML\XMLSchema\Type\BooleanValue;
 
+use function array_last;
 use function intval;
 use function strval;
 
@@ -140,8 +141,8 @@ public static function fromXML(DOMElement $xml): static
         $idpList = IDPList::getChildrenOfClass($xml);
 
         return new static(
-            array_pop($issuer),
-            array_pop($idpList),
+            array_last($issuer),
+            array_last($idpList),
             self::getOptionalAttribute($xml, 'ProviderName', SAMLStringValue::class, null),
             self::getOptionalAttribute($xml, 'IsPassive', BooleanValue::class, null),
         );

src/XML/ecp/SubjectConfirmation.php

@@ -17,6 +17,8 @@
 use SimpleSAML\XMLSchema\Exception\MissingAttributeException;
 use SimpleSAML\XMLSchema\Exception\TooManyElementsException;
 
+use function array_last;
+
 /**
  * Class representing the ECP SubjectConfirmation element.
  *
@@ -113,7 +115,7 @@ public static function fromXML(DOMElement $xml): static
 
         return new static(
             self::getAttribute($xml, 'Method', SAMLAnyURIValue::class),
-            array_pop($subjectConfirmationData),
+            array_last($subjectConfirmationData),
         );
     }
 

src/XML/emd/RepublishRequest.php

@@ -14,7 +14,7 @@
 use SimpleSAML\XMLSchema\Exception\InvalidDOMElementException;
 use SimpleSAML\XMLSchema\Exception\SchemaViolationException;
 
-use function array_pop;
+use function array_last;
 
 /**
  * Class implementing RepublishRequest.
@@ -71,7 +71,7 @@ public static function fromXML(DOMElement $xml): static
             SchemaViolationException::class,
         );
 
-        return new static(array_pop($republishTarget));
+        return new static(array_last($republishTarget));
     }
 
 

src/XML/md/AbstractRoleDescriptor.php

@@ -24,7 +24,7 @@
 use SimpleSAML\XMLSchema\Type\IDValue;
 use SimpleSAML\XMLSchema\Type\QNameValue;
 
-use function array_pop;
+use function array_last;
 
 /**
  * Class representing a SAML2 RoleDescriptor element.
@@ -148,10 +148,10 @@ public static function fromXML(DOMElement $xml): static
                 self::getOptionalAttribute($xml, 'ID', IDValue::class, null),
                 self::getOptionalAttribute($xml, 'validUntil', SAMLDateTimeValue::class, null),
                 self::getOptionalAttribute($xml, 'cacheDuration', DurationValue::class, null),
-                array_pop($extensions),
+                array_last($extensions),
                 self::getOptionalAttribute($xml, 'errorURL', SAMLAnyURIValue::class, null),
                 KeyDescriptor::getChildrenOfClass($xml),
-                array_pop($orgs),
+                array_last($orgs),
                 ContactPerson::getChildrenOfClass($xml),
                 self::getAttributesNSFromXML($xml),
             );

src/XML/md/ContactPerson.php

@@ -23,8 +23,8 @@
 use function array_filter;
 use function array_key_exists;
 use function array_keys;
+use function array_last;
 use function array_map;
-use function array_pop;
 use function count;
 
 /**
@@ -191,9 +191,9 @@ public static function fromXML(DOMElement $xml): static
 
         return new static(
             $contactType,
-            array_pop($company),
-            array_pop($givenName),
-            array_pop($surName),
+            array_last($company),
+            array_last($givenName),
+            array_last($surName),
             (count($extensions) === 1) ? $extensions[0] : null,
             $email,
             $telephone,

src/XML/md/KeyDescriptor.php

@@ -15,7 +15,7 @@
 use SimpleSAML\XMLSchema\Exception\TooManyElementsException;
 use SimpleSAML\XMLSecurity\XML\ds\KeyInfo;
 
-use function array_pop;
+use function array_last;
 
 /**
  * Class representing a KeyDescriptor element.
@@ -97,7 +97,7 @@ public static function fromXML(DOMElement $xml): static
         Assert::maxCount($keyInfo, 1, 'Too many ds:KeyInfo in the KeyDescriptor.', TooManyElementsException::class);
 
         return new static(
-            array_pop($keyInfo),
+            array_last($keyInfo),
             self::getOptionalAttribute($xml, 'use', KeyTypesValue::class, null),
             EncryptionMethod::getChildrenOfClass($xml),
         );