Handle Symfony 7.1 containerxml (#344)

fixes #336
psalm · Jun 5, 2024 · 0fe09bf · 0fe09bf
1 parent 5716321
commit 0fe09bf
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 5 deletions.
diff --git a/src/Symfony/ContainerMeta.php b/src/Symfony/ContainerMeta.php
@@ -97,14 +97,20 @@ private function init(array $containerXmlPaths): void
         $xml->load($containerXmlPath);
 
         foreach ($this->container->getDefinitions() as $definition) {
+            if ($definition->hasTag('container.service_locator')) {
+                continue;
+            }
+
             $definitionFactory = $definition->getFactory();
             if ($definition->hasTag('container.service_locator_context') && is_array($definitionFactory)) {
                 /** @var Reference $reference */
                 $reference = $definitionFactory[0];
                 $id = $definition->getTag('container.service_locator_context')[0]['id'];
-                $this->classLocators[$this->container->getDefinition($id)->getClass() ?? $id] = (string) $reference;
-            } elseif ($definition->hasTag('container.service_locator')) {
-                continue;
+                try {
+                    $this->classLocators[$this->container->getDefinition($id)->getClass() ?? $id] = (string) $reference;
+                } catch (ServiceNotFoundException) {
+                    continue;
+                }
             } elseif (null !== $className = $definition->getClass()) {
                 $this->classNames[] = $className;
             }

diff --git a/tests/acceptance/container.xml b/tests/acceptance/container.xml
@@ -46,7 +46,7 @@
     <service id="dummy_private_service" class="Psalm\SymfonyPsalmPlugin\Tests\Fixture\DummyPrivateService" public="false"/>
     <service id="public_service_wo_public_attr" class="Foo\Bar"/>
     <service id="wronglyNamedService" class="Foo\Bar" public="true" />
-    <!-- service subscriber 1 example -->
+    <!-- service locator example 1 -->
     <service id="App\Controller\DummyController" class="App\Controller\DummyController">
       <tag name="container.service_subscriber"/>
       <argument type="service" id=".service_locator.123xyz.App\Controller\DummyController"/>
@@ -66,7 +66,7 @@
       <argument type="service" id="service_container"/>
       <factory service=".service_locator.123xyz" method="withContext"/>
     </service>
-    <!-- service subscriber 2 example -->
+    <!-- service locator example 2 -->
     <service id="service_subscriber_2" class="App\SomeClass" autowire="true" autoconfigure="true">
       <tag name="container.service_subscriber"/>
       <argument type="service" id=".service_locator.456xyz.service_subscriber_2"/>
@@ -87,5 +87,12 @@
         <argument key="1234" type="service" id="Psalm\SymfonyPsalmPlugin\Tests\Fixture\DummyPrivateService"/>
       </argument>
     </service>
+    <!-- service locator example 3 -->
+    <service id=".service_locator.890xyz.kernel::registerContainerConfiguration()" class="Symfony\Component\DependencyInjection\ServiceLocator">
+      <tag name="container.service_locator_context" id="kernel::registerContainerConfiguration()"/>
+      <argument>kernel::registerContainerConfiguration()</argument>
+      <argument type="service" id="service_container"/>
+      <factory service=".service_locator.890xyz" method="withContext"/>
+    </service>
   </services>
 </container>