deprecate templating component

.github/workflows/build.yaml

@@ -37,8 +37,6 @@ jobs:
                     - description: 'Lowest deps'
                       php: '8.0'
                       composer_option: '--prefer-lowest'
-                      env:
-                          SYMFONY_DEPRECATIONS_HELPER: max[self]=0
                     - description: 'Symfony 5.4'
                       php: '8.0'
                       symfony: 5.4.*
@@ -59,9 +57,11 @@ jobs:
               with:
                   php-version: ${{ matrix.php }}
             - run: |
-                  sed -ri 's/"symfony\/(.+)": "(.+)"/"symfony\/\1": "'${{ matrix.symfony }}'"/' composer.json;
+                  sed -ri 's/"symfony\/(expression-language|framework-bundle|templating)": "(.+)"/"symfony\/\1": "'${{ matrix.symfony }}'"/' composer.json;
               if: matrix.symfony
             - run: composer config minimum-stability dev
               if: matrix.dev
             - run: composer update --no-interaction --no-progress --ansi ${{ matrix.composer_option }}
             - run: vendor/bin/phpunit
+              env:
+                  SYMFONY_DEPRECATIONS_HELPER: max[self]=1

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 * Increased minimum PHP version to 8.0 (to be consistent with KnpMenu 3.4)
 * Dropped support for Symfony 3 and 4
+* Deprecated use of Symfony templating component
 
 ## 3.2.0 (2021-10-30)
 

composer.json

@@ -21,6 +21,7 @@
     "require": {
         "php": "^8.0",
         "knplabs/knp-menu": "^3.3",
+        "symfony/deprecation-contracts": "^2.5 | ^3.3",
         "symfony/framework-bundle": "^5.4 | ^6.0"
     },
     "require-dev": {

docs/index.rst

@@ -63,7 +63,8 @@ You can define these options if you need to change them:
             # use "twig: false" to disable the Twig extension and the TwigRenderer
             twig:
                 template: KnpMenuBundle::menu.html.twig
-            #  if true, enables the helper for PHP templates
+            # if true, enables the helper for PHP templates
+            # support for templating is deprecated, it will be removed in next major version
             templating: false
             # the renderer to use, list is also available by default
             default_renderer: twig
@@ -76,7 +77,7 @@ You can define these options if you need to change them:
             xmlns:knp-menu="http://knplabs.com/schema/dic/menu">
 
             <!--
-                templating:       if true, enabled the helper for PHP templates
+                templating:       if true, enable the helper for PHP templates (deprecated)
                 default-renderer: the renderer to use, list is also available by default
             -->
             <knp-menu:config
@@ -96,7 +97,7 @@ You can define these options if you need to change them:
             'twig' => [
                 'template' => 'KnpMenuBundle::menu.html.twig'
             ],
-            // if true, enabled the helper for PHP templates
+            // if true, enabled the helper for PHP templates (deprecated)
             'templating' => false,
             // the renderer to use, list is also available by default
             'default_renderer' => 'twig',

src/DependencyInjection/KnpMenuExtension.php

@@ -36,6 +36,7 @@ public function load(array $configs, ContainerBuilder $container): void
             $container->setParameter('knp_menu.renderer.twig.template', $config['twig']['template']);
         }
         if ($config['templating']) {
+            trigger_deprecation('knplabs/knp-menu-bundle', '3.3', 'Using the templating component is deprecated since version 3.3, this option will be removed in version 4.');
             $loader->load('templating.xml');
         }
 

src/Provider/BuilderAliasProvider.php

@@ -88,7 +88,7 @@ private function getBuilder(string $bundleName, string $className): object
             }
 
             $builder = new $try();
-            if ($builder instanceof ContainerAwareInterface) {
+            if (interface_exists(ContainerAwareInterface::class) && $builder instanceof ContainerAwareInterface) {
                 $builder->setContainer($this->container);
             }
 

tests/Provider/BuilderAliasProviderTest.php

@@ -5,10 +5,18 @@
 use Knp\Bundle\MenuBundle\Provider\BuilderAliasProvider;
 use Knp\Bundle\MenuBundle\Tests\Stubs\TestKernel;
 use PHPUnit\Framework\TestCase;
-use Symfony\Component\HttpKernel\Bundle\BundleInterface;
+use Symfony\Component\DependencyInjection\ContainerAwareInterface;
 
 class BuilderAliasProviderTest extends TestCase
 {
+    protected function setUp(): void
+    {
+        parent::setUp();
+        if (!interface_exists(ContainerAwareInterface::class)) {
+            self::markTestSkipped();
+        }
+    }
+
     public function testHas(): void
     {
         $provider = new BuilderAliasProvider(