Merge branch '6.4' into 7.0

* 6.4:
  skip tests that do not work with ICU 71.1
  [DependencyInjection] Fix dumping containers with null-referenced services
  [Routing] Fix removing aliases pointing to removed route in RouteCollection::remove()
  [VarExporter] Fix lazy ghost trait when using nullsafe operator
  [Routing] Fix conflicting FQCN aliases with route name
  Fix legacy class palceholder definitions for static analysis

Author: xabbuh

Loader/AttributeClassLoader.php

@@ -101,7 +101,9 @@ public function load(mixed $class, string $type = null): RouteCollection
 
             if (1 === $collection->count() - \count($routeNamesBefore)) {
                 $newRouteName = current(array_diff(array_keys($collection->all()), $routeNamesBefore));
-                $collection->addAlias(sprintf('%s::%s', $class->name, $method->name), $newRouteName);
+                if ($newRouteName !== $aliasName = sprintf('%s::%s', $class->name, $method->name)) {
+                    $collection->addAlias($aliasName, $newRouteName);
+                }
             }
         }
         if (0 === $collection->count() && $class->hasMethod('__invoke')) {
@@ -112,8 +114,14 @@ public function load(mixed $class, string $type = null): RouteCollection
             }
         }
         if ($fqcnAlias && 1 === $collection->count()) {
-            $collection->addAlias($class->name, $invokeRouteName = key($collection->all()));
-            $collection->addAlias(sprintf('%s::__invoke', $class->name), $invokeRouteName);
+            $invokeRouteName = key($collection->all());
+            if ($invokeRouteName !== $class->name) {
+                $collection->addAlias($class->name, $invokeRouteName);
+            }
+
+            if ($invokeRouteName !== $aliasName = sprintf('%s::__invoke', $class->name)) {
+                $collection->addAlias($aliasName, $invokeRouteName);
+            }
         }
 
         return $collection;

RouteCollection.php

@@ -142,8 +142,15 @@ public function get(string $name): ?Route
      */
     public function remove(string|array $name): void
     {
-        foreach ((array) $name as $n) {
-            unset($this->routes[$n], $this->priorities[$n], $this->aliases[$n]);
+        $names = (array) $name;
+        foreach ($names as $n) {
+            unset($this->routes[$n], $this->priorities[$n]);
+        }
+
+        foreach ($this->aliases as $k => $alias) {
+            if (\in_array($alias->getId(), $names, true)) {
+                unset($this->aliases[$k]);
+            }
         }
     }
 

Tests/Fixtures/AnnotationFixtures/InvokableFQCNAliasConflictController.php

@@ -0,0 +1,15 @@
+<?php
+
+namespace Symfony\Component\Routing\Tests\Fixtures\AnnotationFixtures;
+
+use Symfony\Component\Routing\Attribute\Route;
+
+/**
+ * @Route("/foobarccc", name="Symfony\Component\Routing\Tests\Fixtures\AnnotationFixtures\InvokableFQCNAliasConflictController")
+ */
+class InvokableFQCNAliasConflictController
+{
+    public function __invoke()
+    {
+    }
+}

Tests/Fixtures/AttributeFixtures/InvokableFQCNAliasConflictController.php

@@ -0,0 +1,13 @@
+<?php
+
+namespace Symfony\Component\Routing\Tests\Fixtures\AttributeFixtures;
+
+use Symfony\Component\Routing\Attribute\Route;
+
+#[Route(path: '/foobarccc', name: self::class)]
+class InvokableFQCNAliasConflictController
+{
+    public function __invoke()
+    {
+    }
+}

Tests/Loader/AttributeClassLoaderTestCase.php

@@ -99,6 +99,15 @@ public function testInvokableControllerLoader()
         $this->assertEquals(new Alias('lol'), $routes->getAlias('Symfony\Component\Routing\Tests\Fixtures\AttributeFixtures\InvokableController::__invoke'));
     }
 
+    public function testInvokableFQCNAliasConflictController()
+    {
+        $routes = $this->loader->load($this->getNamespace().'\InvokableFQCNAliasConflictController');
+        $this->assertCount(1, $routes);
+        $this->assertEquals('/foobarccc', $routes->get($this->getNamespace().'\InvokableFQCNAliasConflictController')->getPath());
+        $this->assertNull($routes->getAlias($this->getNamespace().'\InvokableFQCNAliasConflictController'));
+        $this->assertEquals(new Alias($this->getNamespace().'\InvokableFQCNAliasConflictController'), $routes->getAlias($this->getNamespace().'\InvokableFQCNAliasConflictController::__invoke'));
+    }
+
     public function testInvokableMethodControllerLoader()
     {
         $routes = $this->loader->load(InvokableMethodController::class);

Tests/RouteCollectionTest.php

@@ -225,11 +225,13 @@ public function testRemove()
         $collection1->add('bar', $bar = new Route('/bar'));
         $collection->addCollection($collection1);
         $collection->add('last', $last = new Route('/last'));
+        $collection->addAlias('ccc_my_custom_alias', 'foo');
 
         $collection->remove('foo');
         $this->assertSame(['bar' => $bar, 'last' => $last], $collection->all(), '->remove() can remove a single route');
         $collection->remove(['bar', 'last']);
         $this->assertSame([], $collection->all(), '->remove() accepts an array and can remove multiple routes at once');
+        $this->assertNull($collection->getAlias('ccc_my_custom_alias'));
     }
 
     public function testSetHost()