Merge pull request overblog#723 from overblog/murtukov-patch-1

Fix bug

Author: murtukov

src/Config/Parser/AnnotationParser.php

@@ -734,11 +734,11 @@ private static function getDescriptionConfiguration(array $annotations, bool $wi
     private static function getArgs(array $args = null, \ReflectionMethod $method = null): array
     {
         $config = [];
-        if ($args && !empty($args)) {
+        if (!empty($args)) {
             foreach ($args as $arg) {
                 $config[$arg->name] = ['type' => $arg->type]
-                    + ($arg->description ? ['description' => $arg->description] : [])
-                    + ($arg->default ? ['defaultValue' => $arg->default] : []);
+                    + (null !== $arg->description ? ['description' => $arg->description] : [])
+                    + (null !== $arg->default ? ['defaultValue' => $arg->default] : []);
             }
         } elseif ($method) {
             $config = self::guessArgs($method);

tests/Config/Parser/AnnotationParserTest.php

@@ -117,7 +117,7 @@ public function testTypes(): void
                 'currentMaster' => ['type' => 'Sith', 'resolve' => "@=service('master_resolver').getMaster(value)"],
                 'victims' => [
                     'type' => '[Character]',
-                    'args' => ['jediOnly' => ['type' => 'Boolean', 'description' => 'Only Jedi victims']],
+                    'args' => ['jediOnly' => ['type' => 'Boolean', 'description' => 'Only Jedi victims', 'defaultValue' => false]],
                     'resolve' => '@=call(value.getVictims, arguments({jediOnly: "Boolean"}, args))',
                 ],
             ],