Merge branch 'feature/lookup-check-empty' into feature/multiple-parameters

Author: clemzarch

src/Builder/AlternativeLookup.php

@@ -127,7 +127,14 @@ public function getNode(): Node
                 ...array_filter(
                     [
                         $this->getAlternativeLookupNode(),
-                        $this->merge?->getNode(),
+                        new Node\Stmt\If_(
+                            cond: new Node\Expr\Variable('lookup'),
+                            subNodes: [
+                                'stmts' => [
+                                    $this->merge?->getNode(),
+                                ],
+                            ]
+                        ),
                         new Node\Stmt\Return_(
                             new Node\Expr\Variable('output')
                         ),

src/Builder/ConditionalLookup.php

@@ -7,7 +7,6 @@
 use Kiboko\Contract\Configurator\StepBuilderInterface;
 use Kiboko\Contract\Mapping\CompiledMapperInterface;
 use PhpParser\Node;
-use Psr\Log\LoggerInterface;
 use Psr\Log\NullLogger;
 
 final class ConditionalLookup implements StepBuilderInterface
@@ -188,7 +187,8 @@ class: new Node\Name\FullyQualified(NullLogger::class)
                                                     default: new Node\Expr\ConstFetch(
                                                         name: new Node\Name(name: 'null'),
                                                     ),
-                                                    type: new Node\Name\FullyQualified(LoggerInterface::class)
+                                                    type: new Node\NullableType(\Psr\Log\LoggerInterface::class),
+                                                    flags: Node\Stmt\Class_::MODIFIER_PRIVATE
                                                 ),
                                             ],
                                         ],