Merge pull request #343 from FlorentTorregrosa/342-php81

Issue #342 by Grimreaper: PHP 8.1 compatibility. First try.
nuvoleweb · Aug 22, 2022 · 6a20799 · 6a20799
2 parents fd1657b + 5a6c050
commit 6a20799
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -5,7 +5,7 @@ jobs:
  runs-on: ubuntu-latest
  strategy:
  matrix:
- PHP_VERSION: ["7.4", "8.0"]
+ PHP_VERSION: ["7.4", "8.0", "8.1"]
  env:
  PHP_VERSION: ${{ matrix.php_version }}
  steps:

diff --git a/src/Definition/ArrayAccessDefinitionTrait.php b/src/Definition/ArrayAccessDefinitionTrait.php
@@ -14,27 +14,31 @@ trait ArrayAccessDefinitionTrait {
  /**
  * {@inheritdoc}
  */
+ #[\ReturnTypeWillChange]
  public function offsetExists($offset) {
  return array_key_exists($offset, $this->definition);
  }
 
  /**
  * {@inheritdoc}
  */
+ #[\ReturnTypeWillChange]
  public function offsetGet($offset) {
  return $this->definition[$offset] ?? NULL;
  }
 
  /**
  * {@inheritdoc}
  */
+ #[\ReturnTypeWillChange]
  public function offsetSet($offset, $value) {
  $this->definition[$offset] = $value;
  }
 
  /**
  * {@inheritdoc}
  */
+ #[\ReturnTypeWillChange]
  public function offsetUnset($offset) {
  unset($this->definition[$offset]);
  }