update psr/container >=2.0 (#132)

.gitignore

@@ -1 +1,2 @@
-composer.lock
+composer.lock
+vendor

Makefile

@@ -1,8 +1,6 @@
 install:
 	composer install
-test: phpcs phpstan phpunit infection clean
-clean:
-	rm -f tests/ExampleCompiled.php
+test: phpcs phpstan phpunit infection
 update:
 	composer update
 phpcs:

composer.json

@@ -10,7 +10,7 @@
     "container"
   ],
   "require": {
-    "psr/container": "^1.0",
+    "psr/container": ">=2.0",
     "php": ">=8.0"
   },
   "require-dev": {
@@ -25,7 +25,7 @@
     }
   },
   "provide": {
-    "psr/container-implementation": "^1.0"
+    "psr/container-implementation": "^2.0"
   },
   "autoload-dev": {
     "psr-4": {

src/ArrayCache.php

@@ -16,15 +16,15 @@ public function __construct(ContainerInterface $container)
         $this->container = $container;
     }
 
-    public function get($id)
+    public function get(string $id)
     {
         if (!array_key_exists($id, $this->values)) {
             $this->values[$id] = $this->container->get($id);
         }
         return $this->values[$id];
     }
 
-    public function has($id): bool
+    public function has(string $id): bool
     {
         return array_key_exists($id, $this->values) || $this->container->has($id);
     }

src/Container.php

@@ -27,12 +27,12 @@ public function __construct(array $params = [], array $alias = [], array $defini
         )));
     }
 
-    public function get($id)
+    public function get(string $id)
     {
         return $this->container->get($id);
     }
 
-    public function has($id): bool
+    public function has(string $id): bool
     {
         return $this->container->has($id);
     }

src/InfiniteRecursionDetector.php

@@ -20,7 +20,7 @@ public function __construct(ContainerInterface $container)
         $this->container = $container;
     }
 
-    public function get($id)
+    public function get(string $id)
     {
         if (in_array($id, $this->calls)) {
             throw new InfiniteRecursion($id, $this->calls);
@@ -31,7 +31,7 @@ public function get($id)
         return $result;
     }
 
-    public function has($id): bool
+    public function has(string $id): bool
     {
         return $this->container->has($id);
     }

src/Strategy.php

@@ -28,7 +28,7 @@ public function get($id)
         return $provider->get($id);
     }
 
-    public function has($id): bool
+    public function has(string $id): bool
     {
         return $this->findProvider($id) !== null;
     }

src/Strategy/Alias.php

@@ -16,7 +16,7 @@ public function __construct(array $values, ContainerInterface $container)
         $this->container = $container;
     }
 
-    public function get($id)
+    public function get(string $id)
     {
         $result = parent::get($id);
         if (is_string($result)) {

src/Strategy/Autowiring.php

@@ -28,7 +28,7 @@ public function __construct(Reflection $reflection, ContainerInterface $containe
         $this->alias = $alias;
     }
 
-    public function get($id)
+    public function get(string $id)
     {
         $args = [];
         foreach ($this->reflection->getDependencies($id) as $dependency) {
@@ -49,7 +49,7 @@ public function get($id)
         return new $id(...$args);
     }
 
-    public function has($id): bool
+    public function has(string $id): bool
     {
         return $this->reflection->isInstantiable($id);
     }

src/Strategy/Definition.php

@@ -16,7 +16,7 @@ public function __construct(array $values, ContainerInterface $container)
         $this->container = $container;
     }
 
-    public function get($id)
+    public function get(string $id)
     {
         $result = parent::get($id);
         if (is_callable($result)) {

src/Strategy/KeyValue.php

@@ -21,12 +21,12 @@ public function __construct(array $values)
         $this->values = $values;
     }
 
-    public function get($id)
+    public function get(string $id)
     {
         return $this->values[$id];
     }
 
-    public function has($id): bool
+    public function has(string $id): bool
     {
         return array_key_exists($id, $this->values);
     }

template/container.compiler.php

@@ -41,7 +41,7 @@ public function __construct(array $params, array $alias, array $definitions)
         $this->containers = <?= var_export(array_unique(array_merge(array_keys($containers), array_keys($alias)))) ?>;
     }
 
-    public function get($name)
+    public function get(string $name)
     {
         if (!array_key_exists($name, $this->params)) {
             $this->params[$name] = match($name) {
@@ -66,7 +66,7 @@ public function get($name)
         return $this->params[$name];
     }
 
-    public function has($name)
+    public function has(string $name): bool
     {
         return array_key_exists($name, $this->params)
 <?php if ($alias_overridable) { ?>

tests/ContainerCompiledTest.php

@@ -34,6 +34,9 @@ public static function tearDownAfterClass(): void
         if (file_exists(__DIR__ . '/ExampleCompiled.php')) {
             unlink(__DIR__ . '/ExampleCompiled.php');
         }
+        if (file_exists(__DIR__ . '/TestReflectionEnabled.php')) {
+            unlink(__DIR__ . '/TestReflectionEnabled.php');
+        }
     }
 
     /**