From 1932af0f03e9dce27242e79761b264cddf130a61 Mon Sep 17 00:00:00 2001 From: David Maicher Date: Tue, 27 Apr 2021 20:45:04 +0200 Subject: [PATCH] add "setFormTheme" and "setFilterTheme" calls + change "initialize" priority --- .../AddDependencyCallsCompilerPass.php | 3 +- .../AdminAddInitializeCallCompilerPass.php | 32 ++++++++++++++++++ src/SonataAdminBundle.php | 2 ++ .../AddDependencyCallsCompilerPassTest.php | 26 +++++++++++++++ ...AdminAddInitializeCallCompilerPassTest.php | 33 +++++++++++++++++++ tests/SonataAdminBundleTest.php | 4 ++- 6 files changed, 98 insertions(+), 2 deletions(-) create mode 100644 src/DependencyInjection/Compiler/AdminAddInitializeCallCompilerPass.php create mode 100644 tests/DependencyInjection/Compiler/AdminAddInitializeCallCompilerPassTest.php diff --git a/src/DependencyInjection/Compiler/AddDependencyCallsCompilerPass.php b/src/DependencyInjection/Compiler/AddDependencyCallsCompilerPass.php index df5541da822..8477f6f64b0 100644 --- a/src/DependencyInjection/Compiler/AddDependencyCallsCompilerPass.php +++ b/src/DependencyInjection/Compiler/AddDependencyCallsCompilerPass.php @@ -386,7 +386,8 @@ public function applyDefaults(ContainerBuilder $container, $serviceId, array $at $definition->addMethodCall('setSecurityInformation', ['%sonata.admin.configuration.security.information%']); } - $definition->addMethodCall('initialize'); + $definition->addMethodCall('setFormTheme', [$overwriteAdminConfiguration['templates']['form'] ?? []]); + $definition->addMethodCall('setFilterTheme', [$overwriteAdminConfiguration['templates']['filter'] ?? []]); return $definition; } diff --git a/src/DependencyInjection/Compiler/AdminAddInitializeCallCompilerPass.php b/src/DependencyInjection/Compiler/AdminAddInitializeCallCompilerPass.php new file mode 100644 index 00000000000..b12f3f1c622 --- /dev/null +++ b/src/DependencyInjection/Compiler/AdminAddInitializeCallCompilerPass.php @@ -0,0 +1,32 @@ + + * + * For the full copyright and license information, please view the LICENSE + * file that was distributed with this source code. + */ + +namespace Sonata\AdminBundle\DependencyInjection\Compiler; + +use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface; +use Symfony\Component\DependencyInjection\ContainerBuilder; + +/** + * this compiler pass is registered with low priority to make sure it runs after all the other passes + * as we want the "initialize()" calls to come after all the other calls. + */ +final class AdminAddInitializeCallCompilerPass implements CompilerPassInterface +{ + public function process(ContainerBuilder $container): void + { + $admins = $container->findTaggedServiceIds('sonata.admin'); + foreach (array_keys($admins) as $id) { + $container->getDefinition($id)->addMethodCall('initialize'); + } + } +} diff --git a/src/SonataAdminBundle.php b/src/SonataAdminBundle.php index 8b7095a86e1..d6cf2eea29a 100644 --- a/src/SonataAdminBundle.php +++ b/src/SonataAdminBundle.php @@ -17,6 +17,7 @@ use Sonata\AdminBundle\DependencyInjection\Compiler\AddAuditReadersCompilerPass; use Sonata\AdminBundle\DependencyInjection\Compiler\AddDependencyCallsCompilerPass; use Sonata\AdminBundle\DependencyInjection\Compiler\AddFilterTypeCompilerPass; +use Sonata\AdminBundle\DependencyInjection\Compiler\AdminAddInitializeCallCompilerPass; use Sonata\AdminBundle\DependencyInjection\Compiler\AdminMakerCompilerPass; use Sonata\AdminBundle\DependencyInjection\Compiler\AdminSearchCompilerPass; use Sonata\AdminBundle\DependencyInjection\Compiler\AliasDeprecatedPublicServicesCompilerPass; @@ -63,6 +64,7 @@ public function build(ContainerBuilder $container) $container->addCompilerPass(new AdminMakerCompilerPass()); $container->addCompilerPass(new AddAuditReadersCompilerPass()); $container->addCompilerPass(new AliasDeprecatedPublicServicesCompilerPass(), PassConfig::TYPE_AFTER_REMOVING); + $container->addCompilerPass(new AdminAddInitializeCallCompilerPass(), PassConfig::TYPE_BEFORE_REMOVING, -100); $this->registerFormMapping(); } diff --git a/tests/DependencyInjection/Compiler/AddDependencyCallsCompilerPassTest.php b/tests/DependencyInjection/Compiler/AddDependencyCallsCompilerPassTest.php index 8faba6151e1..3c2009f6f56 100644 --- a/tests/DependencyInjection/Compiler/AddDependencyCallsCompilerPassTest.php +++ b/tests/DependencyInjection/Compiler/AddDependencyCallsCompilerPassTest.php @@ -200,6 +200,30 @@ public function testProcessResultingConfig(): void 'setRouteBuilder', ['sonata.admin.route.path_info_slashes'] ); + + $this->assertContainerBuilderHasServiceDefinitionWithMethodCall( + 'sonata_article_admin', + 'setFormTheme', + [[]] + ); + + $this->assertContainerBuilderHasServiceDefinitionWithMethodCall( + 'sonata_article_admin', + 'setFilterTheme', + [[]] + ); + + $this->assertContainerBuilderHasServiceDefinitionWithMethodCall( + 'sonata_post_admin', + 'setFormTheme', + [['some_form_template.twig']] + ); + + $this->assertContainerBuilderHasServiceDefinitionWithMethodCall( + 'sonata_post_admin', + 'setFilterTheme', + [['some_filter_template.twig']] + ); } public function testProcessSortAdmins(): void @@ -591,6 +615,8 @@ protected function getConfig() 'sonata_post_admin' => [ 'templates' => [ 'view' => ['user_block' => 'foobar.twig.html'], + 'form' => ['some_form_template.twig'], + 'filter' => ['some_filter_template.twig'], ], ], 'sonata_news_admin' => [ diff --git a/tests/DependencyInjection/Compiler/AdminAddInitializeCallCompilerPassTest.php b/tests/DependencyInjection/Compiler/AdminAddInitializeCallCompilerPassTest.php new file mode 100644 index 00000000000..4f21c4a5c02 --- /dev/null +++ b/tests/DependencyInjection/Compiler/AdminAddInitializeCallCompilerPassTest.php @@ -0,0 +1,33 @@ + + * + * For the full copyright and license information, please view the LICENSE + * file that was distributed with this source code. + */ + +namespace Sonata\AdminBundle\Tests\DependencyInjection\Compiler; + +use PHPUnit\Framework\TestCase; +use Sonata\AdminBundle\DependencyInjection\Compiler\AdminAddInitializeCallCompilerPass; +use Sonata\AdminBundle\Tests\App\Admin\FooAdmin; +use Symfony\Component\DependencyInjection\ContainerBuilder; + +final class AdminAddInitializeCallCompilerPassTest extends TestCase +{ + public function testProcess(): void + { + $builder = new ContainerBuilder(); + $builder->register('foo', FooAdmin::class) + ->addTag('sonata.admin'); + + (new AdminAddInitializeCallCompilerPass())->process($builder); + + $this->assertSame([['initialize', []]], $builder->getDefinition('foo')->getMethodCalls()); + } +} diff --git a/tests/SonataAdminBundleTest.php b/tests/SonataAdminBundleTest.php index 48b4ec4ab4c..560a7f79901 100644 --- a/tests/SonataAdminBundleTest.php +++ b/tests/SonataAdminBundleTest.php @@ -17,6 +17,7 @@ use Sonata\AdminBundle\DependencyInjection\Compiler\AddAuditReadersCompilerPass; use Sonata\AdminBundle\DependencyInjection\Compiler\AddDependencyCallsCompilerPass; use Sonata\AdminBundle\DependencyInjection\Compiler\AddFilterTypeCompilerPass; +use Sonata\AdminBundle\DependencyInjection\Compiler\AdminAddInitializeCallCompilerPass; use Sonata\AdminBundle\DependencyInjection\Compiler\AdminMakerCompilerPass; use Sonata\AdminBundle\DependencyInjection\Compiler\AdminSearchCompilerPass; use Sonata\AdminBundle\DependencyInjection\Compiler\AliasDeprecatedPublicServicesCompilerPass; @@ -37,7 +38,7 @@ public function testBuild(): void { $containerBuilder = $this->createMock(ContainerBuilder::class); - $containerBuilder->expects($this->exactly(11)) + $containerBuilder->expects($this->exactly(12)) ->method('addCompilerPass') ->withConsecutive( [new AddDependencyCallsCompilerPass()], @@ -51,6 +52,7 @@ public function testBuild(): void [new AdminMakerCompilerPass()], [new AddAuditReadersCompilerPass()], [new AliasDeprecatedPublicServicesCompilerPass()], + [new AdminAddInitializeCallCompilerPass()], ); $bundle = new SonataAdminBundle();