[DependencyInjection] Remove refs that point to container.excluded services when allowed

Author: nicolas-grekas

Compiler/ResolveInvalidReferencesPass.php

@@ -95,7 +95,7 @@ private function processValue(mixed $value, int $rootLevel = 0, int $level = 0):
                 $value = array_values($value);
             }
         } elseif ($value instanceof Reference) {
-            if ($this->container->has($id = (string) $value)) {
+            if ($this->container->hasDefinition($id = (string) $value) ? !$this->container->getDefinition($id)->hasTag('container.excluded') : $this->container->hasAlias($id)) {
                 return $value;
             }
 

Tests/Compiler/ResolveInvalidReferencesPassTest.php

@@ -165,6 +165,17 @@ public function testProcessSetOnlyDecoratedAsNullOnInvalid()
         $this->assertEquals($unknownArgument, $decoratorDefinition->getArguments()[1]);
     }
 
+    public function testProcessExcludedServiceAndNullOnInvalid()
+    {
+        $container = new ContainerBuilder();
+        $container->register('foo', \stdClass::class)->addTag('container.excluded');
+        $container->register('bar', \stdClass::class)->addArgument(new Reference('foo', $container::NULL_ON_INVALID_REFERENCE));
+
+        $this->process($container);
+
+        $this->assertSame([null], $container->getDefinition('bar')->getArguments());
+    }
+
     protected function process(ContainerBuilder $container)
     {
         $pass = new ResolveInvalidReferencesPass();