feature symfony#60069 [FrameworkBundle] Deprecate setting the collect_serializer_data to false (mtarld)

This PR was merged into the 7.3 branch.

Discussion
----------

[FrameworkBundle] Deprecate setting the `collect_serializer_data` to `false`

| Q             | A
| ------------- | ---
| Branch?       | 7.3
| Bug fix?      | no
| New feature?  | no
| Deprecations? | yes
| Issues        |
| License       | MIT

Then, in 8.1, we'll be able to deprecate this option to finally remove it in 9.0

Commits
-------

408d09a [FrameworkBundle] Deprecate setting the `collect_serializer_data` to `false`

Author: nicolas-grekas

UPGRADE-7.3.md

@@ -58,6 +58,21 @@ FrameworkBundle
    because its default value will change in version 8.0
  * Deprecate the `--show-arguments` option of the `container:debug` command, as arguments are now always shown
  * Deprecate the `framework.validation.cache` config option
+ * Deprecate setting the `framework.profiler.collect_serializer_data` config option to `false`
+
+   When set to `true`, normalizers must be injected using the `NormalizerInterface`, and not using any concrete implementation.
+
+   Before:
+
+   ```php
+   public function __construct(ObjectNormalizer $normalizer) {}
+   ```
+
+   After:
+
+   ```php
+   public function __construct(#[Autowire('@serializer.normalizer.object')] NormalizerInterface $normalizer) {}
+   ```
 
 Ldap
 ----

src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md

@@ -21,6 +21,7 @@ CHANGELOG
  * Allow configuring the logging channel per type of exceptions
  * Enable service argument resolution on classes that use the `#[Route]` attribute,
    the `#[AsController]` attribute is no longer required
+ * Deprecate setting the `framework.profiler.collect_serializer_data` config option to `false`
 
 7.2
 ---

src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php

@@ -994,6 +994,10 @@ private function registerProfilerConfiguration(array $config, ContainerBuilder $
             $loader->load('notifier_debug.php');
         }
 
+        if (false === $config['collect_serializer_data']) {
+            trigger_deprecation('symfony/framework-bundle', '7.3', 'Setting the "framework.profiler.collect_serializer_data" config option to "false" is deprecated.');
+        }
+
         if ($this->isInitializedConfigEnabled('serializer') && $config['collect_serializer_data']) {
             $loader->load('serializer_debug.php');
         }

src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/php/profiler.php

@@ -7,6 +7,7 @@
     'php_errors' => ['log' => true],
     'profiler' => [
         'enabled' => true,
+        'collect_serializer_data' => true,
     ],
     'serializer' => [
         'enabled' => true,

src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/php/profiler_collect_serializer_data.php

@@ -9,7 +9,7 @@
     <framework:config http-method-override="false" handle-all-throwables="true">
         <framework:annotations enabled="false" />
         <framework:php-errors log="true" />
-        <framework:profiler enabled="true" />
+        <framework:profiler enabled="true" collect-serializer-data="true" />
         <framework:serializer enabled="true" />
     </framework:config>
 </container>

src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/xml/profiler.xml

@@ -6,5 +6,6 @@ framework:
         log: true
     profiler:
         enabled: true
+        collect_serializer_data: true
     serializer:
         enabled: true

src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/xml/profiler_collect_serializer_data.xml

@@ -278,22 +278,13 @@ public function testDisabledProfiler()
 
     public function testProfilerCollectSerializerDataEnabled()
     {
-        $container = $this->createContainerFromFile('profiler_collect_serializer_data');
+        $container = $this->createContainerFromFile('profiler');
 
         $this->assertTrue($container->hasDefinition('profiler'));
         $this->assertTrue($container->hasDefinition('serializer.data_collector'));
         $this->assertTrue($container->hasDefinition('debug.serializer'));
     }
 
-    public function testProfilerCollectSerializerDataDefaultDisabled()
-    {
-        $container = $this->createContainerFromFile('profiler');
-
-        $this->assertTrue($container->hasDefinition('profiler'));
-        $this->assertFalse($container->hasDefinition('serializer.data_collector'));
-        $this->assertFalse($container->hasDefinition('debug.serializer'));
-    }
-
     public function testWorkflows()
     {
         $container = $this->createContainerFromFile('workflows');