[FrameworkBundle] object mapper service definition without form

soyuka · soyuka · commit 82d7ecdec3f8 · 2025-05-20T13:57:50.000+02:00
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php
@@ -642,6 +642,14 @@ public function load(array $configs, ContainerBuilder $container): void
             $loader->load('mime_type.php');
         }
 
+        if (ContainerBuilder::willBeAvailable('symfony/object-mapper', ObjectMapperInterface::class, ['symfony/framework-bundle'])) {
+            $loader->load('object_mapper.php');
+            $container->registerForAutoconfiguration(TransformCallableInterface::class)
+                ->addTag('object_mapper.transform_callable');
+            $container->registerForAutoconfiguration(ConditionCallableInterface::class)
+                ->addTag('object_mapper.condition_callable');
+        }
+
         $container->registerForAutoconfiguration(PackageInterface::class)
             ->addTag('assets.package');
         $container->registerForAutoconfiguration(AssetCompilerInterface::class)
@@ -880,14 +888,6 @@ private function registerFormConfiguration(array $config, ContainerBuilder $cont
         if (!ContainerBuilder::willBeAvailable('symfony/translation', Translator::class, ['symfony/framework-bundle', 'symfony/form'])) {
             $container->removeDefinition('form.type_extension.upload.validator');
         }
-
-        if (ContainerBuilder::willBeAvailable('symfony/object-mapper', ObjectMapperInterface::class, ['symfony/framework-bundle'])) {
-            $loader->load('object_mapper.php');
-            $container->registerForAutoconfiguration(TransformCallableInterface::class)
-                ->addTag('object_mapper.transform_callable');
-            $container->registerForAutoconfiguration(ConditionCallableInterface::class)
-                ->addTag('object_mapper.condition_callable');
-        }
     }
 
     private function registerHttpCacheConfiguration(array $config, ContainerBuilder $container, bool $httpMethodOverride): void
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/FrameworkExtensionTestCase.php b/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/FrameworkExtensionTestCase.php
@@ -2600,6 +2600,14 @@ public function testJsonStreamerEnabled()
         $this->assertTrue($container->has('json_streamer.stream_writer'));
     }
 
+    public function testObjectMapperEnabled()
+    {
+        $container = $this->createContainerFromClosure(function (ContainerBuilder $container) {
+            $container->loadFromExtension('framework', []);
+        });
+        $this->assertTrue($container->has('object_mapper'));
+    }
+
     protected function createContainer(array $data = [])
     {
         return new ContainerBuilder(new EnvPlaceholderParameterBag(array_merge([

Original file line number	Diff line number	Diff line change
`@@ -2600,6 +2600,14 @@ public function testJsonStreamerEnabled()`
`2600`	`2600`	`$this->assertTrue($container->has('json_streamer.stream_writer'));`
`2601`	`2601`	`}`
`2602`	`2602`
	`2603`	`+ public function testObjectMapperEnabled()`
	`2604`	`+ {`
	`2605`	`+ $container = $this->createContainerFromClosure(function (ContainerBuilder $container) {`
	`2606`	`+ $container->loadFromExtension('framework', []);`
	`2607`	`+ });`
	`2608`	`+ $this->assertTrue($container->has('object_mapper'));`
	`2609`	`+ }`
	`2610`	`+`
`2603`	`2611`	`protected function createContainer(array $data = [])`
`2604`	`2612`	`{`
`2605`	`2613`	`return new ContainerBuilder(new EnvPlaceholderParameterBag(array_merge([`