Merge pull request #1 from hugoramosandrade/1.0.1

bugfix - solve properties with uniontypes
hugoramosandrade · Aug 1, 2024 · 07ad477 · 07ad477
2 parents 225926d + 7742dd7
commit 07ad477
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/app/Container/DI.php b/app/Container/DI.php
@@ -60,7 +60,11 @@ protected function resolve(string $class): object
             $constructorParams = [];
             foreach ($params as $param) {
                 $dependencyClass = $param->getType();
-                if ($dependencyClass instanceof \ReflectionType) {
+                if ($param->isDefaultValueAvailable()) {
+                    $constructorParams[] = $param->getDefaultValue();
+                } else if ($dependencyClass instanceof \ReflectionUnionType) {
+                    throw new ContainerException("Can't resolve entry.");
+                } else if ($dependencyClass instanceof \ReflectionType) {
                     /** @var ReflectionType $dependencyClass */
                     $type = $dependencyClass->getName();
                     if (class_exists($type)) {

diff --git a/composer.json b/composer.json
@@ -1,7 +1,7 @@
 {
     "name": "hugoandrade/container-di",
     "type": "library",
-    "version": "1.0.0",
+    "version": "1.0.1",
     "license": "MIT",
     "autoload": {
         "psr-4": {