Merge pull request #66 from lucatume/php-84-compat

solve PHP 8.4 deprecation issues

Author: lucatume

.github/workflows/qa.yaml

@@ -7,38 +7,38 @@ jobs:
     steps:
       - name: Checkout code
         uses: actions/checkout@v3
-      - name: Setup PHP 5.6
+      - name: Setup PHP 7.1
         uses: shivammathur/setup-php@v2
         with:
-          php-version: 5.6
+          php-version: 7.1
       - name: Install dependencies
         uses: "ramsey/composer-install@v2"
       - name: Run phpcs
         run: make phpcs
   phpstan:
-    name: phpstan on PHP 5.6
+    name: phpstan on PHP 7.1
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
         uses: actions/checkout@v3
-      - name: Setup PHP 5.6
+      - name: Setup PHP 7.1
         uses: shivammathur/setup-php@v2
         with:
-          php-version: 5.6
+          php-version: 7.1
       - name: Install dependencies
         uses: "ramsey/composer-install@v2"
       - name: Run phpstan
         run: make phpstan
   phan:
-    name: phan on PHP 5.6
+    name: phan on PHP 7.1
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
         uses: actions/checkout@v3
-      - name: Setup PHP 5.6
+      - name: Setup PHP 7.1
         uses: shivammathur/setup-php@v2
         with:
-          php-version: 5.6
+          php-version: 7.1
       - name: Install dependencies
         uses: "ramsey/composer-install@v2"
       - name: Run phan

.github/workflows/test.yml

@@ -6,7 +6,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        php: [ '5.6', '7.0' , '7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2', '8.3' ]
+        php: [ '7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2', '8.3', '8.4' ]
     steps:
       - name: Checkout code
         uses: actions/checkout@v3

CHANGELOG.md

@@ -5,6 +5,11 @@ to [Semantic Versioning](http://semver.org/).
 
 ## [unreleased] Unreleased
 
+### Changed
+
+- Bumped minimum supported version to PHP 7.1.
+- Fixed PHP 8.4 deprecation warnings.
+
 ## [3.3.7] 2024-04-26;
 
 ### Fixed

composer.json

@@ -14,7 +14,7 @@
     }
   },
   "require": {
-    "php": ">=5.6",
+    "php": ">=7.1",
     "psr/container": "^1.0",
     "ext-json": "*"
   },

config/phan-config.php

@@ -43,7 +43,7 @@
     // Note that the **only** effect of choosing `'5.6'` is to infer that functions removed in php 7.0 exist.
     // (See `backward_compatibility_checks` for additional options)
     // Automatically inferred from composer.json requirement for "php" of ">=5.6"
-    'target_php_version' => '5.6',
+    'target_php_version' => '7.1',
 
     // If enabled, missing properties will be created when
     // they are first seen. If false, we'll report an

src/App.php

@@ -38,7 +38,7 @@ public static function container()
 
         return static::$container;
     }
-    
+
     /**
      * Sets the container instance the Application should use as a Service Locator.
      *
@@ -54,7 +54,7 @@ public static function setContainer(Container $container)
     {
         static::$container = $container;
     }
-    
+
         /**
      * Sets a variable on the container.
      *
@@ -99,7 +99,7 @@ public static function offsetSet($offset, $value)
      *
      * @throws ContainerException If there's any issue reflecting on the class, interface or the implementation.
      */
-    public static function singleton($id, $implementation = null, array $afterBuildMethods = null)
+    public static function singleton($id, $implementation = null, ?array $afterBuildMethods = null)
     {
         static::container()->singleton($id, $implementation, $afterBuildMethods);
     }
@@ -311,7 +311,7 @@ public static function register($serviceProviderClass, ...$alias)
      *
      * @throws ContainerException      If there's an issue while trying to bind the implementation.
      */
-    public static function bind($id, $implementation = null, array $afterBuildMethods = null)
+    public static function bind($id, $implementation = null, ?array $afterBuildMethods = null)
     {
         static::container()->bind($id, $implementation, $afterBuildMethods);
     }
@@ -346,7 +346,7 @@ public static function boot()
      * @return void This method does not return any value.
      * @throws ContainerException
      */
-    public static function singletonDecorators($id, $decorators, array $afterBuildMethods = null)
+    public static function singletonDecorators($id, $decorators, ?array $afterBuildMethods = null)
     {
         static::container()->singletonDecorators($id, $decorators, $afterBuildMethods);
     }
@@ -367,7 +367,7 @@ public static function singletonDecorators($id, $decorators, array $afterBuildMe
      * @return void This method does not return any value.
      * @throws ContainerException If there's any issue binding the decorators.
      */
-    public static function bindDecorators($id, array $decorators, array $afterBuildMethods = null)
+    public static function bindDecorators($id, array $decorators, ?array $afterBuildMethods = null)
     {
         static::container()->bindDecorators($id, $decorators, $afterBuildMethods);
     }
@@ -471,7 +471,7 @@ public static function callback($id, $method)
      * The callable will be a closure on PHP 5.3+ or a lambda function on PHP 5.2.
      *
      * @param  string|class-string|mixed  $id                 The fully qualified name of a class or an interface.
-     * @param  array<mixed>               $buildArgs          An array of arguments that should be used to build the
+     * @param  array<mixed>|null               $buildArgs     An array of arguments that should be used to build the
      *                                                        instance; note that any argument will be resolved using
      *                                                        the container itself and bindings will apply.
      * @param  string[]|null              $afterBuildMethods  An array of methods that should be called on the built
@@ -480,9 +480,9 @@ public static function callback($id, $method)
      * @return callable  A callable function that will return an instance of the specified class when
      *                   called.
      */
-    public static function instance($id, array $buildArgs = [], array $afterBuildMethods = null)
+    public static function instance($id, ?array $buildArgs = [], ?array $afterBuildMethods = null)
     {
-        return static::container()->instance($id, $buildArgs, $afterBuildMethods);
+        return static::container()->instance($id, $buildArgs ?? [], $afterBuildMethods);
     }
 
     /**

src/Builders/CallableBuilder.php

@@ -53,7 +53,7 @@ class CallableBuilder implements BuilderInterface, ReinitializableBuilderInterfa
     public function __construct(
         Container $container,
         callable $callable,
-        array $afterBuildMethods = null,
+        ?array $afterBuildMethods = null,
         ...$buildArgs
     ) {
         $this->container = $container;
@@ -86,7 +86,7 @@ public function build()
      *
      * @return void This method does not return any value.
      */
-    public function reinit(array $afterBuildMethods = null, ...$buildArgs)
+    public function reinit(?array $afterBuildMethods = null, ...$buildArgs)
     {
         $this->afterBuildMethods = $afterBuildMethods ?: [];
         $this->buildArgs = $buildArgs;

src/Builders/ClassBuilder.php

@@ -75,7 +75,7 @@ class ClassBuilder implements BuilderInterface, ReinitializableBuilderInterface
      *
      * @throws NotFoundException If the class does not exist.
      */
-    public function __construct($id, Resolver $resolver, $className, array $afterBuildMethods = null, ...$buildArgs)
+    public function __construct($id, Resolver $resolver, $className, ?array $afterBuildMethods = null, ...$buildArgs)
     {
         if (!class_exists($className)) {
             throw new NotFoundException(
@@ -236,7 +236,7 @@ protected function resolveParameter(Parameter $parameter)
     /**
      * {@inheritdoc}
      */
-    public function reinit(array $afterBuildMethods = null, ...$buildArgs)
+    public function reinit(?array $afterBuildMethods = null, ...$buildArgs)
     {
         $this->afterBuildMethods = $afterBuildMethods;
         $this->buildArgs = $buildArgs;

src/Builders/Factory.php

@@ -56,7 +56,7 @@ public function __construct(Container $container, Resolver $resolver)
      *
      * @throws NotFoundException If a builder cannot find its implementation target.
      */
-    public function getBuilder($id, $implementation = null, array $afterBuildMethods = null, ...$buildArgs)
+    public function getBuilder($id, $implementation = null, ?array $afterBuildMethods = null, ...$buildArgs)
     {
         if ($implementation === null) {
             $implementation = $id;

src/Builders/ReinitializableBuilderInterface.php

@@ -22,5 +22,5 @@ interface ReinitializableBuilderInterface
      *
      * @return void This method does not return any value.
      */
-    public function reinit(array $afterBuildMethods = null, ...$buildArgs);
+    public function reinit(?array $afterBuildMethods = null, ...$buildArgs);
 }