Add legacy class to fix old Form compilation pass

DependencyInjection/Compiler/FormFactoryCompilerPass.php

@@ -11,8 +11,8 @@
 
 namespace Sonata\CoreBundle\DependencyInjection\Compiler;
 
-use Symfony\Bundle\FrameworkBundle\DependencyInjection\Compiler\FormPass;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
+use Symfony\Component\Form\DependencyInjection\FormPass;
 
 class FormFactoryCompilerPass extends FormPass
 {
@@ -42,7 +42,6 @@ public function process(ContainerBuilder $container)
         // get factories
         $original = $container->getDefinition('form.extension');
         parent::process($container);
-
         $factory = $container->getDefinition('sonata.core.form.extension.dependency');
         $factory->replaceArgument(1, $original->getArgument(1));
         $factory->replaceArgument(2, $original->getArgument(2));

DependencyInjection/Compiler/LegacyFormFactoryCompilerPass.php

@@ -0,0 +1,55 @@
+<?php
+
+/*
+ * This file is part of the Sonata Project package.
+ *
+ * (c) Thomas Rabaix <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Sonata\CoreBundle\DependencyInjection\Compiler;
+
+use Symfony\Bundle\FrameworkBundle\DependencyInjection\Compiler\FormPass;
+use Symfony\Component\DependencyInjection\ContainerBuilder;
+
+class LegacyFormFactoryCompilerPass extends FormPass
+{
+    /**
+     * {@inheritdoc}
+     */
+    public function process(ContainerBuilder $container)
+    {
+        $typeIdx = [];
+        foreach ($container->findTaggedServiceIds('form.type') as $id => $tags) {
+            $typeIdx[] = $id;
+        }
+
+        $typeExtensionIdx = [];
+        foreach ($container->findTaggedServiceIds('form.type_extension') as $id => $tag) {
+            $typeExtensionIdx[] = $id;
+        }
+
+        $container->setParameter('sonata.core.form.types', $typeIdx);
+        $container->setParameter('sonata.core.form.type_extensions', $typeExtensionIdx);
+
+        // nothing to do
+        if (!$container->hasDefinition('sonata.core.form.extension.dependency')) {
+            return;
+        }
+
+        // get factories
+        $original = $container->getDefinition('form.extension');
+        parent::process($container);
+        $factory = $container->getDefinition('sonata.core.form.extension.dependency');
+        $factory->replaceArgument(1, $original->getArgument(1));
+        $factory->replaceArgument(2, $original->getArgument(2));
+        $factory->replaceArgument(3, $original->getArgument(3));
+
+        $container->removeDefinition('form.extension');
+        $container->removeDefinition('sonata.core.form.extension.dependency');
+
+        $container->setDefinition('form.extension', $factory);
+    }
+}

SonataCoreBundle.php

@@ -11,11 +11,12 @@
 
 namespace Sonata\CoreBundle;
 
+use Sonata\CoreBundle\DependencyInjection\Compiler;
 use Sonata\CoreBundle\DependencyInjection\Compiler\AdapterCompilerPass;
-use Sonata\CoreBundle\DependencyInjection\Compiler\FormFactoryCompilerPass;
 use Sonata\CoreBundle\DependencyInjection\Compiler\StatusRendererCompilerPass;
 use Sonata\CoreBundle\Form\FormHelper;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
+use Symfony\Component\Form\DependencyInjection\FormPass;
 use Symfony\Component\HttpKernel\Bundle\Bundle;
 
 class SonataCoreBundle extends Bundle
@@ -27,7 +28,11 @@ public function build(ContainerBuilder $container)
     {
         $container->addCompilerPass(new StatusRendererCompilerPass());
         $container->addCompilerPass(new AdapterCompilerPass());
-        $container->addCompilerPass(new FormFactoryCompilerPass());
+        if (class_exists(FormPass::class)) {
+            $container->addCompilerPass(new Compiler\FormFactoryCompilerPass());
+        } else {
+            $container->addCompilerPass(new Compiler\LegacyFormFactoryCompilerPass());
+        }
 
         $this->registerFormMapping();
     }