[Serializer] TranslatableNormalizer before BackedEnumNormalizer

Author: Seb33300

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

@@ -119,7 +119,7 @@
 
         ->set('serializer.normalizer.translatable', TranslatableNormalizer::class)
             ->args(['$translator' => service('translator')])
-            ->tag('serializer.normalizer', ['built_in' => true, 'priority' => -920])
+            ->tag('serializer.normalizer', ['built_in' => true, 'priority' => -910])
 
         ->set('serializer.normalizer.form_error', FormErrorNormalizer::class)
             ->tag('serializer.normalizer', ['built_in' => true, 'priority' => -915])

src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/FrameworkExtensionTestCase.php

@@ -1863,7 +1863,7 @@ public function testTranslatableNormalizerRegistered()
         $tag = $definition->getTag('serializer.normalizer');
 
         $this->assertSame(TranslatableNormalizer::class, $definition->getClass());
-        $this->assertSame(-920, $tag[0]['priority']);
+        $this->assertSame(-910, $tag[0]['priority']);
         $this->assertEquals(new Reference('translator'), $definition->getArgument('$translator'));
     }
 

src/Symfony/Bundle/FrameworkBundle/Tests/Functional/SerializerTest.php

@@ -15,6 +15,7 @@
 use Symfony\Bundle\FrameworkBundle\Tests\Functional\app\AppKernel;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
+use Symfony\Component\Serializer\Normalizer\TranslatableNormalizer;
 
 /**
  * @author Kévin Dunglas <dunglas@gmail.com>
@@ -60,6 +61,18 @@ public function testNormalizersAndEncodersUseDefaultContextConfigOption()
         }
     }
 
+    public function testSerializeTranslatableBackedEnum()
+    {
+        static::bootKernel(['test_case' => 'Serializer']);
+
+        $serializer = static::getContainer()->get('serializer.alias');
+
+        $this->assertEquals('custom_get_string', $serializer->serialize(TranslatableBackedEnum::Get, 'yaml'));
+        $this->assertEquals('GET', $serializer->serialize(TranslatableBackedEnum::Get, 'yaml', [
+            TranslatableNormalizer::NORMALIZATION_LOCALE_KEY => false,
+        ]));
+    }
+
     protected static function getKernelClass(): string
     {
         return SerializerKernel::class;
@@ -91,15 +104,6 @@ public function process(ContainerBuilder $container): void
             }
         }
     }
-
-    public function testSerializeTranslatableBackedEnum()
-    {
-        static::bootKernel(['test_case' => 'Serializer']);
-
-        $serializer = static::getContainer()->get('serializer.alias');
-
-        $this->assertEquals('GET', $serializer->serialize(TranslatableBackedEnum::Get, 'yaml'));
-    }
 }
 
 class Foo

src/Symfony/Component/Serializer/Normalizer/TranslatableNormalizer.php

@@ -45,11 +45,12 @@ public function normalize(mixed $data, ?string $format = null, array $context =
 
     public function supportsNormalization(mixed $data, ?string $format = null, array $context = []): bool
     {
-        return $data instanceof TranslatableInterface;
+        return $data instanceof TranslatableInterface
+            && false !== ($context[self::NORMALIZATION_LOCALE_KEY] ?? $this->defaultContext[self::NORMALIZATION_LOCALE_KEY]);
     }
 
     public function getSupportedTypes(?string $format): array
     {
-        return [TranslatableInterface::class => true];
+        return [TranslatableInterface::class => false];
     }
 }