feature symfony#59401 [JsonEncoder] Add JsonEncodable attribute (mtarld)

This PR was merged into the 7.3 branch.

Discussion
----------

[JsonEncoder] Add `JsonEncodable` attribute

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

Add `JsonEncodable` attribute that autoconfigures `json_encoder.encodable` tag.

Commits
-------

a12ad34 [JsonEncoder] Add `JsonEncodable` attribute

Author: chalasr

src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/UnusedTagsPass.php

@@ -54,7 +54,6 @@ class UnusedTagsPass implements CompilerPassInterface
         'html_sanitizer',
         'http_client.client',
         'json_encoder.denormalizer',
-        'json_encoder.encodable',
         'json_encoder.normalizer',
         'kernel.cache_clearer',
         'kernel.cache_warmer',

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

@@ -100,6 +100,7 @@
 use Symfony\Component\HttpKernel\DataCollector\DataCollectorInterface;
 use Symfony\Component\HttpKernel\DependencyInjection\Extension;
 use Symfony\Component\HttpKernel\Log\DebugLoggerConfigurator;
+use Symfony\Component\JsonEncoder\Attribute\JsonEncodable;
 use Symfony\Component\JsonEncoder\Decode\Denormalizer\DenormalizerInterface as JsonEncoderDenormalizerInterface;
 use Symfony\Component\JsonEncoder\DecoderInterface as JsonEncoderDecoderInterface;
 use Symfony\Component\JsonEncoder\Encode\Normalizer\NormalizerInterface as JsonEncoderNormalizerInterface;
@@ -745,6 +746,10 @@ static function (ChildDefinition $definition, AsPeriodicTask|AsCronTask $attribu
                 }
             );
         }
+        $container->registerAttributeForAutoconfiguration(JsonEncodable::class, static function (ChildDefinition $definition): void {
+            $definition->addTag('json_encoder.encodable');
+            $definition->addTag('container.excluded');
+        });
 
         if (!$container->getParameter('kernel.debug')) {
             // remove tagged iterator argument for resource checkers

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

@@ -12,6 +12,7 @@
 namespace Symfony\Bundle\FrameworkBundle\Tests\Functional;
 
 use Symfony\Bundle\FrameworkBundle\Tests\Functional\app\JsonEncoder\Dto\Dummy;
+use Symfony\Component\Filesystem\Filesystem;
 use Symfony\Component\JsonEncoder\DecoderInterface;
 use Symfony\Component\JsonEncoder\EncoderInterface;
 use Symfony\Component\TypeInfo\Type;
@@ -21,10 +22,13 @@
  */
 class JsonEncoderTest extends AbstractWebTestCase
 {
-    public function testEncode()
+    protected function setUp(): void
     {
         static::bootKernel(['test_case' => 'JsonEncoder']);
+    }
 
+    public function testEncode()
+    {
         /** @var EncoderInterface $encoder */
         $encoder = static::getContainer()->get('json_encoder.encoder.alias');
 
@@ -33,8 +37,6 @@ public function testEncode()
 
     public function testDecode()
     {
-        static::bootKernel(['test_case' => 'JsonEncoder']);
-
         /** @var DecoderInterface $decoder */
         $decoder = static::getContainer()->get('json_encoder.decoder.alias');
 
@@ -44,4 +46,22 @@ public function testDecode()
 
         $this->assertEquals($expected, $decoder->decode('{"@name": "DUMMY", "range": "0..1"}', Type::object(Dummy::class)));
     }
+
+    public function testWarmupEncodableClasses()
+    {
+        /** @var Filesystem $fs */
+        $fs = static::getContainer()->get('filesystem');
+
+        $encodersDir = \sprintf('%s/json_encoder/encoder/', static::getContainer()->getParameter('kernel.cache_dir'));
+
+        // clear already created encoders
+        if ($fs->exists($encodersDir)) {
+            $fs->remove($encodersDir);
+        }
+
+        static::getContainer()->get('json_encoder.cache_warmer.encoder_decoder.alias')->warmUp(static::getContainer()->getParameter('kernel.cache_dir'));
+
+        $this->assertFileExists($encodersDir);
+        $this->assertCount(1, glob($encodersDir.'/*'));
+    }
 }

src/Symfony/Bundle/FrameworkBundle/Tests/Functional/app/JsonEncoder/Dto/Dummy.php

@@ -14,11 +14,13 @@
 use Symfony\Bundle\FrameworkBundle\Tests\Functional\app\JsonEncoder\RangeNormalizer;
 use Symfony\Component\JsonEncoder\Attribute\Denormalizer;
 use Symfony\Component\JsonEncoder\Attribute\EncodedName;
+use Symfony\Component\JsonEncoder\Attribute\JsonEncodable;
 use Symfony\Component\JsonEncoder\Attribute\Normalizer;
 
 /**
  * @author Mathias Arlaud <[email protected]>
  */
+#[JsonEncodable]
 class Dummy
 {
     #[EncodedName('@name')]

src/Symfony/Bundle/FrameworkBundle/Tests/Functional/app/JsonEncoder/config.yml

@@ -18,4 +18,9 @@ services:
         alias: json_encoder.decoder
         public: true
 
+    json_encoder.cache_warmer.encoder_decoder.alias:
+        alias: .json_encoder.cache_warmer.encoder_decoder
+        public: true
+
+    Symfony\Bundle\FrameworkBundle\Tests\Functional\app\JsonEncoder\Dto\Dummy: ~
     Symfony\Bundle\FrameworkBundle\Tests\Functional\app\JsonEncoder\RangeNormalizer: ~

src/Symfony/Component/JsonEncoder/Attribute/JsonEncodable.php

@@ -0,0 +1,22 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\JsonEncoder\Attribute;
+
+/**
+ * @author Mathias Arlaud <[email protected]>
+ *
+ * @experimental
+ */
+#[\Attribute(\Attribute::TARGET_CLASS)]
+final class JsonEncodable
+{
+}

src/Symfony/Component/JsonEncoder/DependencyInjection/EncodablePass.php

@@ -30,7 +30,11 @@ public function process(ContainerBuilder $container): void
         $encodableClassNames = [];
 
         // retrieve concrete services tagged with "json_encoder.encodable" tag
-        foreach ($container->findTaggedServiceIds('json_encoder.encodable') as $id => $tags) {
+        foreach ($container->getDefinitions() as $id => $definition) {
+            if (!$definition->hasTag('json_encoder.encodable')) {
+                continue;
+            }
+
             if (($className = $container->getDefinition($id)->getClass()) && !$container->getDefinition($id)->isAbstract()) {
                 $encodableClassNames[] = $className;
             }