Merge pull request #640 from tienvx/revert-auto-configuration

Revert auto configuration
tienvx · Jul 26, 2022 · e50699f · e50699f
2 parents ecefec2 + 71f565e
commit e50699f
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/src/Plugin/PluginInterface.php b/src/Plugin/PluginInterface.php
@@ -2,9 +2,6 @@
 
 namespace Tienvx\Bundle\MbtBundle\Plugin;
 
-use Symfony\Component\DependencyInjection\Attribute\Autoconfigure;
-
-#[Autoconfigure(tags: [PluginInterface::TAG], lazy: true)]
 interface PluginInterface
 {
     public const TAG = 'mbt_bundle.plugin';

diff --git a/src/TienvxMbtBundle.php b/src/TienvxMbtBundle.php
@@ -8,6 +8,7 @@
 use Symfony\Component\HttpKernel\Bundle\AbstractBundle;
 use Tienvx\Bundle\MbtBundle\Command\CommandManager;
 use Tienvx\Bundle\MbtBundle\DependencyInjection\Compiler\PluginPass;
+use Tienvx\Bundle\MbtBundle\Plugin\PluginInterface;
 use Tienvx\Bundle\MbtBundle\Service\SelenoidHelper;
 
 class TienvxMbtBundle extends AbstractBundle
@@ -37,5 +38,8 @@ public function loadExtension(array $config, ContainerConfigurator $container, C
             ->get(CommandManager::class)
             ->call('setUploadDir', [$config[static::UPLOAD_DIR]])
         ;
+        $builder->registerForAutoconfiguration(PluginInterface::class)
+            ->setLazy(true)
+            ->addTag(PluginInterface::TAG);
     }
 }
diff --git a/tests/TienvxMbtBundleTest.php b/tests/TienvxMbtBundleTest.php
@@ -11,6 +11,7 @@
 use Symfony\Component\DependencyInjection\Loader\PhpFileLoader;
 use Tienvx\Bundle\MbtBundle\Command\CommandManager;
 use Tienvx\Bundle\MbtBundle\DependencyInjection\Compiler\PluginPass;
+use Tienvx\Bundle\MbtBundle\Plugin\PluginInterface;
 use Tienvx\Bundle\MbtBundle\Service\SelenoidHelper;
 use Tienvx\Bundle\MbtBundle\TienvxMbtBundle;
 
@@ -73,5 +74,8 @@ public function testLoadExtension(): void
         $this->assertSame([
             ['setUploadDir', [static::CONFIG[TienvxMbtBundle::UPLOAD_DIR]]],
         ], $this->builder->findDefinition(CommandManager::class)->getMethodCalls());
+        $autoConfigured = $this->builder->getAutoconfiguredInstanceof()[PluginInterface::class];
+        $this->assertTrue($autoConfigured->hasTag(PluginInterface::TAG));
+        $this->assertTrue($autoConfigured->isLazy());
     }
 }