[StimulusBundle] Remove deprecations for 3.0

Author: Kocal

UPGRADE-3.0.md

@@ -62,6 +62,11 @@ class MyLiveComponent {
         * Options `authReferrerPolicy`, `mapIds`, `channel`, `solutionChannel` have been added
         * Options `ìd`, `nonce`, `retries`, `url` have been removed
 
+## StimulusBundle
+
+* The Twig function `ux_controller_link_tags()` has been removed, which requires Symfony AssetMapper >=6.4,
+  run `composer require symfony/asset-mapper:>=6.4` if you don't have it installed yet.
+
 ## Swup
 
 * The package has been removed, see the [previous README](https://raw.githubusercontent.com/symfony/ux/refs/heads/2.x/src/Turbo/README.md)

src/StimulusBundle/CHANGELOG.md

@@ -4,6 +4,7 @@
 
 -   Minimum required Symfony version is now 6.4
 -   Minimum required PHP version is now 8.2
+-   Remove Twig function `ux_controller_link_tags()`, which requires Symfony AssetMapper >=6.4
 
 ## 2.30
 

src/StimulusBundle/composer.json

@@ -21,6 +21,9 @@
         "twig/twig": "^2.15.3|^3.8",
         "symfony/deprecation-contracts": "^2.0|^3.0"
     },
+    "conflict": {
+        "symfony/asset-mapper": "<6.4"
+    },
     "require-dev": {
         "symfony/asset-mapper": "^6.4|^7.0|^8.0",
         "symfony/framework-bundle": "^6.4|^7.0|^8.0",

src/StimulusBundle/doc/index.rst

@@ -416,10 +416,6 @@ will import all your custom controllers as well as those from ``controllers.json
 It will also dynamically enable "debug" mode in Stimulus when your application
 is running in debug mode.
 
-.. tip::
-
-    For AssetMapper 6.3 only, you also need a ``{{ ux_controller_link_tags() }}``
-    in ``base.html.twig``. This is not needed in AssetMapper 6.4+.
 
 With WebpackEncoreBundle
 ~~~~~~~~~~~~~~~~~~~~~~~~

src/StimulusBundle/src/DependencyInjection/Compiler/RemoveAssetMapperServicesCompiler.php

@@ -24,7 +24,6 @@ class RemoveAssetMapperServicesCompiler implements CompilerPassInterface
     public function process(ContainerBuilder $container): void
     {
         if (!$container->hasDefinition('asset_mapper')) {
-            $container->removeDefinition('stimulus.ux_controllers_twig_runtime');
             $container->removeDefinition('stimulus.asset_mapper.controllers_map_generator');
             $container->removeDefinition('stimulus.asset_mapper.loader_javascript_compiler');
         }

src/StimulusBundle/src/Twig/UxControllersTwigExtension.php

@@ -22,13 +22,6 @@
 
 class AutoImportLocatorTest extends TestCase
 {
-    protected function setUp(): void
-    {
-        if (!class_exists(ImportMapConfigReader::class)) {
-            $this->markTestSkipped('Test requires AssetMapper >= 6.4.');
-        }
-    }
-
     public function testLocateAutoImportCanHandleAssetMapperPath()
     {
         $assetMapper = $this->createMock(AssetMapperInterface::class);

src/StimulusBundle/src/Twig/UxControllersTwigRuntime.php

@@ -54,17 +54,11 @@ public function testGetControllersMap()
         $packageReader = new UxPackageReader(__DIR__.'/../fixtures');
 
         $autoImportLocator = $this->createMock(AutoImportLocator::class);
-        if (class_exists(ImportMapConfigReader::class)) {
-            $autoImportLocator->expects($this->any())
-                ->method('locateAutoImport')
-                ->willReturnCallback(function ($path) {
-                    return new MappedControllerAutoImport('/path/to'.$path, false);
-                });
-        } else {
-            // @legacy for AssetMapper 6.3
-            $autoImportLocator->expects($this->never())
-                ->method('locateAutoImport');
-        }
+        $autoImportLocator->expects($this->any())
+            ->method('locateAutoImport')
+            ->willReturnCallback(function ($path) {
+                return new MappedControllerAutoImport('/path/to'.$path, false);
+            });
 
         $generator = new ControllersMapGenerator(
             $mapper,
@@ -106,11 +100,7 @@ public function testGetControllersMap()
         $this->assertSame('fake-vendor/ux-package1/package-controller-second.js', $controllerSecond->asset->logicalPath);
         // lazy from user's controller.json
         $this->assertTrue($controllerSecond->isLazy);
-        // @legacy: assert can be without the conditional for AssetMapper 6.4+
-        if (class_exists(ImportMapConfigReader::class)) {
-            // 4 auto imports from package.json
-            $this->assertCount(4, $controllerSecond->autoImports);
-        }
+        $this->assertCount(4, $controllerSecond->autoImports);
 
         $helloControllerFromPackage = $map['fake-vendor--ux-package2--hello-controller'];
         $this->assertSame('fake-vendor/ux-package2/package-hello-controller.js', $helloControllerFromPackage->asset->logicalPath);