Fix the slug variable in Twig templates with extra compiler pass

Signed-off-by: Tim Goudriaan <tim@codedmonkey.com>

Author: codedmonkey

config/packages/twig.yaml

@@ -2,10 +2,6 @@ twig:
     file_name_pattern: '*.twig'
     form_themes: ['bootstrap_5_layout.html.twig']
 
-    globals:
-        dirigent:
-            slug: '%dirigent.slug%'
-
 when@test:
     twig:
         strict_variables: true

config/services.yaml

@@ -3,8 +3,6 @@
 
 # Put parameters here that don't need to change on each machine where the app is deployed
 # https://symfony.com/doc/current/best_practices.html#use-parameters-for-application-configuration
-parameters:
-    dirigent.slug: ~
 
 services:
     # default configuration for services in *this* file

src/DependencyInjection/Compiler/ParametersPass.php

@@ -0,0 +1,25 @@
+<?php
+
+namespace CodedMonkey\Dirigent\DependencyInjection\Compiler;
+
+use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
+use Symfony\Component\DependencyInjection\ContainerBuilder;
+
+class ParametersPass implements CompilerPassInterface
+{
+    public function process(ContainerBuilder $container): void
+    {
+        $this->setTwigGlobal($container);
+    }
+
+    private function setTwigGlobal(ContainerBuilder $container): void
+    {
+        $parameterBag = $container->getParameterBag();
+
+        $variables = [
+            'slug' => $parameterBag->get('dirigent.slug'),
+        ];
+
+        $container->getDefinition('twig')->addMethodCall('addGlobal', ['dirigent', $variables]);
+    }
+}

src/DependencyInjection/DirigentExtension.php

@@ -11,10 +11,8 @@ class DirigentExtension extends ConfigurableExtension
 {
     protected function loadInternal(array $mergedConfig, ContainerBuilder $container): void
     {
-        if (null === $slug = $mergedConfig['slug']) {
-            $slug = (new AsciiSlugger())->slug($mergedConfig['title']);
-            $slug = strtolower($slug);
-        }
+        $slug = $mergedConfig['slug'];
+        $slug ??= (new AsciiSlugger())->slug($mergedConfig['title'])->lower()->toString();
 
         $container->setParameter('dirigent.title', $mergedConfig['title']);
         $container->setParameter('dirigent.slug', $slug);

src/Kernel.php

@@ -3,6 +3,7 @@
 namespace CodedMonkey\Dirigent;
 
 use CodedMonkey\Dirigent\DependencyInjection\Compiler\EncryptionPass;
+use CodedMonkey\Dirigent\DependencyInjection\Compiler\ParametersPass;
 use CodedMonkey\Dirigent\DependencyInjection\DirigentExtension;
 use Symfony\Bundle\FrameworkBundle\Kernel\MicroKernelTrait;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
@@ -32,8 +33,8 @@ protected function build(ContainerBuilder $container): void
     {
         $container->registerExtension(new DirigentExtension());
 
-        // The encryption pass has to be the first pass to run as it removes sensitive data from the container
-        $container->addCompilerPass(new EncryptionPass(), priority: 2048);
+        $container->addCompilerPass(new EncryptionPass(), priority: 2048); // The encryption pass has to be the first pass to run as it removes sensitive data from the container
+        $container->addCompilerPass(new ParametersPass());
     }
 
     public function boot(): void