bug #45193 [FrameworkBundle] Fix missing arguments when a serialization default context is bound (ArnoudThibaut)

nicolas-grekas · nicolas-grekas · commit 111459ef1597 · 2022-01-26T19:42:13.000+01:00
This PR was squashed before being merged into the 5.4 branch. Discussion ---------- [FrameworkBundle] Fix missing arguments when a serialization default context is bound | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | Fix #44960 | License | MIT I used `null` as first argument for `JsonSerializableNormalizer` definition even if a service exist for the `ClassMetadataFactoryInterface` to keep the actual behavior Commits ------- 1855798 [FrameworkBundle] Fix missing arguments when a serialization default context is bound
diff --git a/src/Symfony/Bundle/FrameworkBundle/Resources/config/serializer.php b/src/Symfony/Bundle/FrameworkBundle/Resources/config/serializer.php
@@ -99,6 +99,7 @@
             ->tag('serializer.normalizer', ['priority' => -910])
 
         ->set('serializer.normalizer.json_serializable', JsonSerializableNormalizer::class)
+            ->args([null, null])
             ->tag('serializer.normalizer', ['priority' => -900])
 
         ->set('serializer.normalizer.problem', ProblemNormalizer::class)
@@ -178,6 +179,7 @@
             ->tag('serializer.encoder')
 
         ->set('serializer.encoder.yaml', YamlEncoder::class)
+            ->args([null, null])
             ->tag('serializer.encoder')
 
         ->set('serializer.encoder.csv', CsvEncoder::class)
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/Functional/app/Serializer/config.yml b/src/Symfony/Bundle/FrameworkBundle/Tests/Functional/app/Serializer/config.yml
@@ -2,7 +2,10 @@ imports:
     - { resource: ../config/default.yml }
 
 framework:
-    serializer: { enabled: true }
+    serializer:
+        enabled: true
+        default_context:
+            enable_max_depth: true
     property_info: { enabled: true }
 
 services:
