[JsonEncoder] Fix retrieving encodable classes

Author: mtarld

DependencyInjection/Configuration.php

@@ -2580,16 +2580,6 @@ private function addJsonEncoderSection(ArrayNodeDefinition $rootNode, callable $
                 ->arrayNode('json_encoder')
                     ->info('JSON encoder configuration')
                     ->{$enableIfStandalone('symfony/json-encoder', EncoderInterface::class)}()
-                    ->fixXmlConfig('path')
-                    ->children()
-                        ->arrayNode('paths')
-                            ->info('Namespaces and paths of encodable/decodable classes.')
-                            ->normalizeKeys(false)
-                            ->useAttributeAsKey('namespace')
-                            ->scalarPrototype()->end()
-                            ->defaultValue([])
-                        ->end()
-                    ->end()
                 ->end()
             ->end()
         ;

DependencyInjection/FrameworkExtension.php

@@ -2026,15 +2026,6 @@ private function registerJsonEncoderConfiguration(array $config, ContainerBuilde
         $container->setParameter('.json_encoder.decoders_dir', '%kernel.cache_dir%/json_encoder/decoder');
         $container->setParameter('.json_encoder.lazy_ghosts_dir', '%kernel.cache_dir%/json_encoder/lazy_ghost');
 
-        $encodableDefinition = (new Definition())
-            ->setAbstract(true)
-            ->addTag('container.excluded')
-            ->addTag('json_encoder.encodable');
-
-        foreach ($config['paths'] as $namespace => $path) {
-            $loader->registerClasses($encodableDefinition, $namespace, $path);
-        }
-
         if (\PHP_VERSION_ID >= 80400) {
             $container->removeDefinition('.json_encoder.cache_warmer.lazy_ghost');
         }

FrameworkBundle.php

@@ -54,6 +54,7 @@
 use Symfony\Component\HttpKernel\DependencyInjection\RemoveEmptyControllerArgumentLocatorsPass;
 use Symfony\Component\HttpKernel\DependencyInjection\ResettableServicePass;
 use Symfony\Component\HttpKernel\KernelEvents;
+use Symfony\Component\JsonEncoder\DependencyInjection\EncodablePass;
 use Symfony\Component\Messenger\DependencyInjection\MessengerPass;
 use Symfony\Component\Mime\DependencyInjection\AddMimeTypeGuesserPass;
 use Symfony\Component\PropertyInfo\DependencyInjection\PropertyInfoPass;
@@ -186,6 +187,7 @@ public function build(ContainerBuilder $container): void
         $container->addCompilerPass(new ErrorLoggerCompilerPass(), PassConfig::TYPE_BEFORE_OPTIMIZATION, -32);
         $container->addCompilerPass(new VirtualRequestStackPass());
         $container->addCompilerPass(new TranslationUpdateCommandPass(), PassConfig::TYPE_BEFORE_REMOVING);
+        $container->addCompilerPass(new EncodablePass());
 
         if ($container->getParameter('kernel.debug')) {
             $container->addCompilerPass(new AddDebugLogProcessorPass(), PassConfig::TYPE_BEFORE_OPTIMIZATION, 2);

Resources/config/json_encoder.php

@@ -107,7 +107,7 @@
         // cache
         ->set('.json_encoder.cache_warmer.encoder_decoder', EncoderDecoderCacheWarmer::class)
             ->args([
-                tagged_iterator('json_encoder.encodable'),
+                abstract_arg('encodable class names'),
                 service('json_encoder.encode.property_metadata_loader'),
                 service('json_encoder.decode.property_metadata_loader'),
                 param('.json_encoder.encoders_dir'),
@@ -118,7 +118,7 @@
 
         ->set('.json_encoder.cache_warmer.lazy_ghost', LazyGhostCacheWarmer::class)
             ->args([
-                tagged_iterator('json_encoder.encodable'),
+                abstract_arg('encodable class names'),
                 param('.json_encoder.lazy_ghosts_dir'),
             ])
             ->tag('kernel.cache_warmer')

Resources/config/schema/symfony-1.0.xsd

@@ -1006,11 +1006,7 @@
     </xsd:complexType>
 
     <xsd:complexType name="json-encoder">
-        <xsd:choice minOccurs="0" maxOccurs="unbounded">
-            <xsd:sequence>
-                <xsd:element name="default-context" type="metadata" minOccurs="0" maxOccurs="1" />
-            </xsd:sequence>
-        </xsd:choice>
         <xsd:attribute name="enabled" type="xsd:boolean" />
     </xsd:complexType>
+
 </xsd:schema>

Tests/DependencyInjection/ConfigurationTest.php

@@ -973,7 +973,6 @@ class_exists(SemaphoreStore::class) && SemaphoreStore::isSupported() ? 'semaphor
             ],
             'json_encoder' => [
                 'enabled' => !class_exists(FullStack::class) && class_exists(JsonEncoder::class),
-                'paths' => [],
             ],
         ];
     }

Tests/Functional/app/JsonEncoder/config.yml

@@ -4,10 +4,7 @@ imports:
 framework:
     http_method_override: false
     type_info: ~
-    json_encoder:
-        enabled: true
-        paths:
-            Symfony\Bundle\FrameworkBundle\Tests\Functional\app\JsonEncoder\Dto\: '../../Tests/Functional/app/JsonEncoder/Dto/*'
+    json_encoder: ~
 
 services:
     _defaults: