8000 [Serializer] Fix Traceable decoration priorities · symfony/symfony@96fc2dc · GitHub
[go: up one dir, main page]

Skip to content

Commit 96fc2dc

Browse files
committed
[Serializer] Fix Traceable decoration priorities
1 parent e92323d commit 96fc2dc

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

src/Symfony/Bundle/FrameworkBundle/Resources/config/serializer_debug.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
return static function (ContainerConfigurator $container) {
1818
$container->services()
1919
->set('debug.serializer', TraceableSerializer::class)
20-
->decorate('serializer', null, 255)
20+
->decorate('serializer')
2121
->args([
2222
service('debug.serializer.inner'),
2323
service('serializer.data_collector'),

src/Symfony/Component/Serializer/DependencyInjection/SerializerPass.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ public function process(ContainerBuilder $container)
4444
if ($container->getParameter('kernel.debug') && $container->hasDefinition('serializer.data_collector')) {
4545
foreach (array_keys($normalizers) as $normalizer) {
4646
$container->register('debug.'.$normalizer, TraceableNormalizer::class)
47-
->setDecoratedService($normalizer, null, 255)
47+
->setDecoratedService($normalizer)
4848
->setArguments([new Reference('debug.'.$normalizer.'.inner'), new Reference('serializer.data_collector')]);
4949
}
5050
}
@@ -59,7 +59,7 @@ public function process(ContainerBuilder $container)
5959
if ($container->getParameter('kernel.debug') && $container->hasDefinition('serializer.data_collector')) {
6060
foreach (array_keys($encoders) as $encoder) {
6161
$container->register('debug.'.$encoder, TraceableEncoder::class)
62-
->setDecoratedService($encoder, null, 255)
62+
->setDecoratedService($encoder)
6363
->setArguments([new Reference('debug.'.$encoder.'.inner'), new Reference('serializer.data_collector')]);
6464
}
6565
}

src/Symfony/Component/Serializer/Tests/DependencyInjection/SerializerPassTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,12 +108,12 @@ public function testNormalizersAndEncodersAreDecoredAndOrderedWhenCollectingData
108108
$traceableEncoderDefinition = $container->getDefinition('debug.e');
109109

110110
$this->assertEquals(TraceableNormalizer::class, $traceableNormalizerDefinition->getClass());
111-
$this->assertEquals(['n', null, 255], $traceableNormalizerDefinition->getDecoratedService());
111+
$this->assertEquals(['n', null, 0], $traceableNormalizerDefinition->getDecoratedService());
112112
$this->assertEquals(new Reference('debug.n.inner'), $traceableNormalizerDefinition->getArgument(0));
113113
$this->assertEquals(new Reference('serializer.data_collector'), $traceableNormalizerDefinition->getArgument(1));
114114

115115
$this->assertEquals(TraceableEncoder::class, $traceableEncoderDefinition->getClass());
116-
$this->assertEquals(['e', null, 255], $traceableEncoderDefinition->getDecoratedService());
116+
$this->assertEquals(['e', null, 0], $traceableEncoderDefinition->getDecoratedService());
117117
$this->assertEquals(new Reference('debug.e.inner'), $traceableEncoderDefinition->getArgument(0));
118118
$this->assertEquals(new Reference('serializer.data_collector'), $traceableEncoderDefinition->getArgument(1));
119119
}

0 commit comments

Comments
 (0)
0