feature #266 Fix support for PHP 8.1 (glensc, come-nc, fabpot)

This PR was merged into the main branch.

Discussion
----------

Fix support for PHP 8.1

Avoid deprecated warnings about Iterator and ArrayAccess interfaces methods typing.

Commits
-------

7a4a44b Fix return types
dade71d Fix support for PHP 8.1
8c28b6e Add php 8.1 to CI testing

Author: fabpot

.github/workflows/tests.yml

@@ -16,6 +16,7 @@ jobs:
           - "7.3"
           - "7.4"
           - "8.0"
+          - "8.1"
         dependencies:
           - "psr/container:^1.1"
           - "psr/container:^2.0"

composer.json

@@ -16,7 +16,7 @@
         "psr/container": "^1.1 || ^2.0"
     },
     "require-dev": {
-        "symfony/phpunit-bridge": "^5.0"
+        "symfony/phpunit-bridge": "^5.4@dev"
     },
     "autoload": {
         "psr-0": { "Pimple": "src/" }

src/Pimple/Container.php

@@ -74,8 +74,11 @@ public function __construct(array $values = [])
      * @param string $id    The unique identifier for the parameter or object
      * @param mixed  $value The value of the parameter or a closure to define an object
      *
+     * @return void
+     *
      * @throws FrozenServiceException Prevent override of a frozen service
      */
+    #[\ReturnTypeWillChange]
     public function offsetSet($id, $value)
     {
         if (isset($this->frozen[$id])) {
@@ -95,6 +98,7 @@ public function offsetSet($id, $value)
      *
      * @throws UnknownIdentifierException If the identifier is not defined
      */
+    #[\ReturnTypeWillChange]
     public function offsetGet($id)
     {
         if (!isset($this->keys[$id])) {
@@ -130,6 +134,7 @@ public function offsetGet($id)
      *
      * @return bool
      */
+    #[\ReturnTypeWillChange]
     public function offsetExists($id)
     {
         return isset($this->keys[$id]);
@@ -139,7 +144,10 @@ public function offsetExists($id)
      * Unsets a parameter or an object.
      *
      * @param string $id The unique identifier for the parameter or object
+     *
+     * @return void
      */
+    #[\ReturnTypeWillChange]
     public function offsetUnset($id)
     {
         if (isset($this->keys[$id])) {

src/Pimple/ServiceIterator.php

@@ -42,26 +42,46 @@ public function __construct(Container $container, array $ids)
         $this->ids = $ids;
     }
 
+    /**
+     * @return void
+     */
+    #[\ReturnTypeWillChange]
     public function rewind()
     {
         \reset($this->ids);
     }
 
+    /**
+     * @return mixed
+     */
+    #[\ReturnTypeWillChange]
     public function current()
     {
         return $this->container[\current($this->ids)];
     }
 
+    /**
+     * @return mixed
+     */
+    #[\ReturnTypeWillChange]
     public function key()
     {
         return \current($this->ids);
     }
 
+    /**
+     * @return void
+     */
+    #[\ReturnTypeWillChange]
     public function next()
     {
         \next($this->ids);
     }
 
+    /**
+     * @return bool
+     */
+    #[\ReturnTypeWillChange]
     public function valid()
     {
         return null !== \key($this->ids);