Remove deprecations

composer.json

@@ -37,9 +37,8 @@
     "require": {
         "php": "^8.1",
         "ext-pcre": "*",
-        "sonata-project/datagrid-bundle": "^2.3 || ^3.0",
         "symfony/routing": "^5.4 || ^6.2",
-        "symfony/translation-contracts": "^1.1 || ^2.0 || ^3.0",
+        "symfony/translation-contracts": "^2.0 || ^3.0",
         "twig/twig": "^2.4 || ^3.0"
     },
     "require-dev": {

src/Bridge/Symfony/DependencyInjection/NucleosTwigExtension.php

@@ -23,14 +23,7 @@ final class NucleosTwigExtension extends Extension
      */
     public function load(array $configs, ContainerBuilder $container): void
     {
-        $configuration = new Configuration();
-        $config        = $this->processConfiguration($configuration, $configs);
-
         $loader = new Loader\PhpFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('services.php');
-
-        $container->getDefinition('nucleos_twig.router.extension')
-            ->replaceArgument(2, $config['pagination'])
-        ;
     }
 }

src/Bridge/Symfony/Resources/config/services.php

@@ -26,9 +26,7 @@
         ->set('nucleos_twig.router.extension', RouterTwigExtension::class)
             ->tag('twig.extension')
             ->args([
-                new Reference('twig'),
                 new Reference('router'),
-                [],
             ])
     ;
 };

src/Extension/RouterTwigExtension.php

@@ -11,12 +11,7 @@
 
 namespace Nucleos\Twig\Extension;
 
-use Sonata\DatagridBundle\Pager\PagerInterface;
 use Symfony\Component\Routing\RouterInterface;
-use Twig\Environment;
-use Twig\Error\LoaderError;
-use Twig\Error\RuntimeError;
-use Twig\Error\SyntaxError;
 use Twig\Extension\AbstractExtension;
 use Twig\TwigFilter;
 use Twig\TwigFunction;
@@ -25,43 +20,15 @@
 {
     private RouterInterface $router;
 
-    /**
-     * @var array<string, int|string>
-     */
-    private array $options;
-
-    private Environment $environment;
-
-    /**
-     * @param array<string, int|string> $options
-     *
-     * @throws LoaderError
-     */
-    public function __construct(Environment $environment, RouterInterface $router, array $options = [])
+    public function __construct(RouterInterface $router)
     {
-        $this->environment = $environment;
-        $this->router      = $router;
-        $this->options     = $options;
-
-        if (!isset($this->options['template'])) {
-            throw new LoaderError('Pager template is not set.');
-        }
-        if (!isset($this->options['extremeLimit'])) {
-            throw new LoaderError('Pager extreme limit is not set.');
-        }
-        if (!isset($this->options['nearbyLimit'])) {
-            throw new LoaderError('Pager nearby limit is not set.');
-        }
+        $this->router = $router;
     }
 
     public function getFunctions(): array
     {
         return [
             new TwigFunction('routeExists', [$this, 'routeExists']),
-            new TwigFunction('page_pager', [$this, 'generatePager'], [
-                'is_safe'    => ['html'],
-                'deprecated' => true,
-            ]),
         ];
     }
 
@@ -82,44 +49,16 @@
      */
     public function splitTag(string $text, string $tag): array
     {
         if ('' === trim($tag)) {
             return [$text];
         }
 
         $split = preg_split('/(?=<'.$tag.'([^>])*>)/', $text, -1, PREG_SPLIT_NO_EMPTY | PREG_SPLIT_DELIM_CAPTURE);
 
         if (false !== $split) {
             return $split;
         }
 
         return [$text];
     }
-
-    /**
-     * @param PagerInterface<object>    $pager
-     * @param array<string, int|string> $options
-     *
-     * @throws LoaderError
-     * @throws RuntimeError
-     * @throws SyntaxError
-     *
-     * @deprecated
-     */
-    public function generatePager(PagerInterface $pager, array $options = []): string
-    {
-        $data = array_merge(array_merge($this->options, $options), [
-            'itemsCount'  => $pager->count(),
-            'limit'       => max(1, $pager->getMaxPerPage()),
-            'currentPage' => $pager->getPage(),
-        ]);
-
-        $data['lastPage'] = self::getNumPages((int) $data['limit'], (int) $data['itemsCount']);
-
-        return $this->environment->render((string) $data['template'], $data);
-    }
-
-    private static function getNumPages(int $limit, int $count): int
-    {
-        return (int) ceil($count / $limit);
-    }
 }

tests/Bridge/Symfony/Bundle/NucleosTwigBundleTest.php

@@ -21,13 +21,13 @@ public function testGetPath(): void
     {
         $bundle = new NucleosTwigBundle();
 
-        static::assertStringEndsWith('Bridge/Symfony/Bundle', \dirname($bundle->getPath()));
+        self::assertStringEndsWith('Bridge/Symfony/Bundle', \dirname($bundle->getPath()));
     }
 
     public function testGetContainerExtension(): void
     {
         $bundle = new NucleosTwigBundle();
 
-        static::assertInstanceOf(NucleosTwigExtension::class, $bundle->getContainerExtension());
+        self::assertInstanceOf(NucleosTwigExtension::class, $bundle->getContainerExtension());
     }
 }

tests/Bridge/Symfony/DependencyInjection/NucleosTwigExtensionTest.php

@@ -29,39 +29,6 @@ public function testLoadDefault(): void
         );
         $this->assertContainerBuilderHasService('nucleos_twig.string.extension', StringTwigExtension::class);
         $this->assertContainerBuilderHasService('nucleos_twig.router.extension', RouterTwigExtension::class);
-
-        $this->assertContainerBuilderHasServiceDefinitionWithArgument(
-            'nucleos_twig.router.extension',
-            2,
-            [
-                'template'     => '@NucleosTwig/Pager/pagination.html.twig',
-                'extremeLimit' => 3,
-                'nearbyLimit'  => 2,
-            ]
-        );
-    }
-
-    public function testLoadCustom(): void
-    {
-        $this->load(
-            [
-                'pagination' => [
-                    'template'     => '@Acme/Pager/pagination.html.twig',
-                    'extremeLimit' => 10,
-                    'nearbyLimit'  => 5,
-                ],
-            ]
-        );
-
-        $this->assertContainerBuilderHasServiceDefinitionWithArgument(
-            'nucleos_twig.router.extension',
-            2,
-            [
-                'template'     => '@Acme/Pager/pagination.html.twig',
-                'extremeLimit' => 10,
-                'nearbyLimit'  => 5,
-            ]
-        );
     }
 
     protected function getContainerExtensions(): array