[FrameworkBundle] Deprecate setting the collect_serializer_data to false

mtarld · mtarld · commit f8424254de31 · 2025-03-28T09:42:19.000+01:00
diff --git a/UPGRADE-7.3.md b/UPGRADE-7.3.md
@@ -56,6 +56,7 @@ 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`
 
 Ldap
 ----
diff --git a/src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md b/src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md
@@ -19,6 +19,7 @@ CHANGELOG
  * Add DI alias from `ServicesResetterInterface` to `services_resetter`
  * Add `methods` argument in `#[IsCsrfTokenValid]` attribute
  * Allow configuring the logging channel per type of exceptions
+ * Deprecate setting the `framework.profiler.collect_serializer_data` config option to `false`
 
 7.2
 ---
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php
@@ -429,7 +429,7 @@ public function load(array $configs, ContainerBuilder $container): void
             }
             $loggers[$exception['log_channel']] = new Reference('monolog.logger.'.$exception['log_channel'], ContainerInterface::NULL_ON_INVALID_REFERENCE);
         }
-        
+
         $exceptionListener
             ->replaceArgument(3, $config['exceptions'])
             ->setArgument(4, $loggers)
@@ -984,6 +984,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');
         }

Original file line number	Diff line number	Diff line change
`@@ -429,7 +429,7 @@ public function load(array $configs, ContainerBuilder $container): void`
`429`	`429`	`}`
`430`	`430`	`$loggers[$exception['log_channel']] = new Reference('monolog.logger.'.$exception['log_channel'], ContainerInterface::NULL_ON_INVALID_REFERENCE);`
`431`	`431`	`}`
`432`		`-`
	`432`	`+`
`433`	`433`	`$exceptionListener`
`434`	`434`	`->replaceArgument(3, $config['exceptions'])`
`435`	`435`	`->setArgument(4, $loggers)`
`@@ -984,6 +984,10 @@ private function registerProfilerConfiguration(array $config, ContainerBuilder $`
`984`	`984`	`$loader->load('notifier_debug.php');`
`985`	`985`	`}`
`986`	`986`
	`987`	`+ if (false === $config['collect_serializer_data']) {`
	`988`	`+ trigger_deprecation('symfony/framework-bundle', '7.3', 'Setting the "framework.profiler.collect_serializer_data" config option to "false" is deprecated.');`
	`989`	`+ }`
	`990`	`+`
`987`	`991`	`if ($this->isInitializedConfigEnabled('serializer') && $config['collect_serializer_data']) {`
`988`	`992`	`$loader->load('serializer_debug.php');`
`989`	`993`	`}`