[TwigBridge] Use CPP

symfony · nicolas-grekas · Mar 12, 2024 · Mar 12, 2024 · Mar 12, 2024 · d4f8f3940a169b38be3a06d3e85c1af11312c90b
commit d4f8f3940a169b38be3a06d3e85c1af11312c90b
diff --git a/src/Symfony/Bridge/Twig/Command/DebugCommand.php b/src/Symfony/Bridge/Twig/Command/DebugCommand.php
@@ -36,27 +36,19 @@
 #[AsCommand(name: 'debug:twig', description: 'Show a list of twig functions, filters, globals and tests')]
 class DebugCommand extends Command
 {
-    private Environment $twig;
-    private ?string $projectDir;
-    private array $bundlesMetadata;
-    private ?string $twigDefaultPath;
-
    /**
     * @var FilesystemLoader[]
     */
    private array $filesystemLoaders;

-    private ?FileLinkFormatter $fileLinkFormatter;
-
-    public function __construct(Environment $twig, ?string $projectDir = null, array $bundlesMetadata = [], ?string $twigDefaultPath = null, ?FileLinkFormatter $fileLinkFormatter = null)
-    {
+    public function __construct(
+        private Environment $twig,
+        private ?string $projectDir = null,
+        private array $bundlesMetadata = [],
+        private ?string $twigDefaultPath = null,
+        private ?FileLinkFormatter $fileLinkFormatter = null,
+    ) {
        parent::__construct();
-
-        $this->twig = $twig;
-        $this->projectDir = $projectDir;
-        $this->bundlesMetadata = $bundlesMetadata;
-        $this->twigDefaultPath = $twigDefaultPath;
-        $this->fileLinkFormatter = $fileLinkFormatter;
    }

    protected function configure(): void

diff --git a/src/Symfony/Bridge/Twig/DataCollector/TwigDataCollector.php b/src/Symfony/Bridge/Twig/DataCollector/TwigDataCollector.php
@@ -28,14 +28,12 @@
 */
 class TwigDataCollector extends DataCollector implements LateDataCollectorInterface
 {
-    private Profile $profile;
-    private ?Environment $twig;
    private array $computed;

-    public function __construct(Profile $profile, ?Environment $twig = null)
-    {
-        $this->profile = $profile;
-        $this->twig = $twig;
+    public function __construct(
+        private Profile $profile,
+        private ?Environment $twig = null,
+    ) {
    }

    public function collect(Request $request, Response $response, ?\Throwable $exception = null): void

diff --git a/src/Symfony/Bridge/Twig/ErrorRenderer/TwigErrorRenderer.php b/src/Symfony/Bridge/Twig/ErrorRenderer/TwigErrorRenderer.php
@@ -25,16 +25,17 @@
 */
 class TwigErrorRenderer implements ErrorRendererInterface
 {
-    private Environment $twig;
    private HtmlErrorRenderer $fallbackErrorRenderer;
    private \Closure|bool $debug;

    /**
     * @param bool|callable $debug The debugging mode as a boolean or a callable that should return it
     */
-    public function __construct(Environment $twig, ?HtmlErrorRenderer $fallbackErrorRenderer = null, bool|callable $debug = false)
-    {
-        $this->twig = $twig;
+    public function __construct(
+        private Environment $twig,
+        ?HtmlErrorRenderer $fallbackErrorRenderer = null,
+        bool|callable $debug = false,
+    ) {
        $this->fallbackErrorRenderer = $fallbackErrorRenderer ?? new HtmlErrorRenderer();
        $this->debug = \is_bool($debug) ? $debug : $debug(...);
    }

diff --git a/src/Symfony/Bridge/Twig/Extension/AssetExtension.php b/src/Symfony/Bridge/Twig/Extension/AssetExtension.php
@@ -22,11 +22,9 @@
 */
 final class AssetExtension extends AbstractExtension
 {
-    private Packages $packages;
-
-    public function __construct(Packages $packages)
-    {
-        $this->packages = $packages;
+    public function __construct(
+        private Packages $packages,
+    ) {
    }

    public function getFunctions(): array

diff --git a/src/Symfony/Bridge/Twig/Extension/CsrfRuntime.php b/src/Symfony/Bridge/Twig/Extension/CsrfRuntime.php
@@ -19,11 +19,9 @@
 */
 final class CsrfRuntime
 {
-    private CsrfTokenManagerInterface $csrfTokenManager;
-
-    public function __construct(CsrfTokenManagerInterface $csrfTokenManager)
-    {
-        $this->csrfTokenManager = $csrfTokenManager;
+    public function __construct(
+        private CsrfTokenManagerInterface $csrfTokenManager,
+    ) {
    }

    public function getCsrfToken(string $tokenId): string

diff --git a/src/Symfony/Bridge/Twig/Extension/DumpExtension.php b/src/Symfony/Bridge/Twig/Extension/DumpExtension.php
@@ -26,13 +26,10 @@
 */
 final class DumpExtension extends AbstractExtension
 {
-    private ClonerInterface $cloner;
-    private ?HtmlDumper $dumper;
-
-    public function __construct(ClonerInterface $cloner, ?HtmlDumper $dumper = null)
-    {
-        $this->cloner = $cloner;
-        $this->dumper = $dumper;
+    public function __construct(
+        private ClonerInterface $cloner,
+        private ?HtmlDumper $dumper = null,
+    ) {
    }

    public function getFunctions(): array

@@ -33,11 +33,9 @@
 */
 final class FormExtension extends AbstractExtension
 {
-    private ?TranslatorInterface $translator;
-
-    public function __construct(?TranslatorInterface $translator = null)
-    {
-        $this->translator = $translator;
+    public function __construct(
+        private ?TranslatorInterface $translator = null,
+    ) {
    }

    public function getTokenParsers(): array

diff --git a/src/Symfony/Bridge/Twig/Extension/HttpFoundationExtension.php b/src/Symfony/Bridge/Twig/Extension/HttpFoundationExtension.php
@@ -23,11 +23,9 @@
 */
 final class HttpFoundationExtension extends AbstractExtension
 {
-    private UrlHelper $urlHelper;
-
-    public function __construct(UrlHelper $urlHelper)
-    {
-        $this->urlHelper = $urlHelper;
+    public function __construct(
+        private UrlHelper $urlHelper,
+    ) {
    }

    public function getFunctions(): array

diff --git a/src/Symfony/Bridge/Twig/Extension/HttpKernelRuntime.php b/src/Symfony/Bridge/Twig/Extension/HttpKernelRuntime.php
@@ -22,13 +22,10 @@
 */
 final class HttpKernelRuntime
 {
-    private FragmentHandler $handler;
-    private ?FragmentUriGeneratorInterface $fragmentUriGenerator;
-
-    public function __construct(FragmentHandler $handler, ?FragmentUriGeneratorInterface $fragmentUriGenerator = null)
-    {
-        $this->handler = $handler;
-        $this->fragmentUriGenerator = $fragmentUriGenerator;
+    public function __construct(
+        private FragmentHandler $handler,
+        private ?FragmentUriGeneratorInterface $fragmentUriGenerator = null,
+    ) {
    }

    /**

diff --git a/src/Symfony/Bridge/Twig/Extension/ImportMapRuntime.php b/src/Symfony/Bridge/Twig/Extension/ImportMapRuntime.php
@@ -18,8 +18,9 @@
 */
 class ImportMapRuntime
 {
-    public function __construct(private readonly ImportMapRenderer $importMapRenderer)
-    {
+    public function __construct(
+        private readonly ImportMapRenderer $importMapRenderer,
+    ) {
    }

    public function importmap(string|array $entryPoint = 'app', array $attributes = []): string

diff --git a/src/Symfony/Bridge/Twig/Extension/LogoutUrlExtension.php b/src/Symfony/Bridge/Twig/Extension/LogoutUrlExtension.php
@@ -22,11 +22,9 @@
 */
 final class LogoutUrlExtension extends AbstractExtension
 {
-    private LogoutUrlGenerator $generator;
-
-    public function __construct(LogoutUrlGenerator $generator)
-    {
-        $this->generator = $generator;
+    public function __construct(
+        private LogoutUrlGenerator $generator,
+    ) {
    }

    public function getFunctions(): array

diff --git a/src/Symfony/Bridge/Twig/Extension/ProfilerExtension.php b/src/Symfony/Bridge/Twig/Extension/ProfilerExtension.php
@@ -21,18 +21,17 @@
 */
 final class ProfilerExtension extends BaseProfilerExtension
 {
-    private ?Stopwatch $stopwatch;
-
    /**
     * @var \SplObjectStorage<Profile, StopwatchEvent>
     */
    private \SplObjectStorage $events;

-    public function __construct(Profile $profile, ?Stopwatch $stopwatch = null)
-    {
+    public function __construct(
+        Profile $profile,
+        private ?Stopwatch $stopwatch = null,
+    ) {
        parent::__construct($profile);

-        $this->stopwatch = $stopwatch;
        $this->events = new \SplObjectStorage();
    }


diff --git a/src/Symfony/Bridge/Twig/Extension/RoutingExtension.php b/src/Symfony/Bridge/Twig/Extension/RoutingExtension.php
@@ -25,11 +25,9 @@
 */
 final class RoutingExtension extends AbstractExtension
 {
-    private UrlGeneratorInterface $generator;
-
-    public function __construct(UrlGeneratorInterface $generator)
-    {
-        $this->generator = $generator;
+    public function __construct(
+        private UrlGeneratorInterface $generator,
+    ) {
    }

    public function getFunctions(): array

diff --git a/src/Symfony/Bridge/Twig/Extension/SecurityExtension.php b/src/Symfony/Bridge/Twig/Extension/SecurityExtension.php
@@ -25,13 +25,10 @@
 */
 final class SecurityExtension extends AbstractExtension
 {
-    private ?AuthorizationCheckerInterface $securityChecker;
-    private ?ImpersonateUrlGenerator $impersonateUrlGenerator;
-
-    public function __construct(?AuthorizationCheckerInterface $securityChecker = null, ?ImpersonateUrlGenerator $impersonateUrlGenerator = null)
-    {
-        $this->securityChecker = $securityChecker;
-        $this->impersonateUrlGenerator = $impersonateUrlGenerator;
+    public function __construct(
+        private ?AuthorizationCheckerInterface $securityChecker = null,
+        private ?ImpersonateUrlGenerator $impersonateUrlGenerator = null,
+    ) {
    }

    public function isGranted(mixed $role, mixed $object = null, ?string $field = null): bool

diff --git a/src/Symfony/Bridge/Twig/Extension/SerializerRuntime.php b/src/Symfony/Bridge/Twig/Extension/SerializerRuntime.php
@@ -19,11 +19,9 @@
 */
 final class SerializerRuntime implements RuntimeExtensionInterface
 {
-    private SerializerInterface $serializer;
-
-    public function __construct(SerializerInterface $serializer)
-    {
-        $this->serializer = $serializer;
+    public function __construct(
+        private SerializerInterface $serializer,
+    ) {
    }

    public function serialize(mixed $data, string $format = 'json', array $context = []): string

diff --git a/src/Symfony/Bridge/Twig/Extension/StopwatchExtension.php b/src/Symfony/Bridge/Twig/Extension/StopwatchExtension.php
@@ -23,13 +23,10 @@
 */
 final class StopwatchExtension extends AbstractExtension
 {
-    private ?Stopwatch $stopwatch;
-    private bool $enabled;
-
-    public function __construct(?Stopwatch $stopwatch = null, bool $enabled = true)
-    {
-        $this->stopwatch = $stopwatch;
-        $this->enabled = $enabled;
+    public function __construct(
+        private ?Stopwatch $stopwatch = null,
+        private bool $enabled = true,
+    ) {
    }

    public function getStopwatch(): Stopwatch

diff --git a/src/Symfony/Bridge/Twig/Extension/TranslationExtension.php b/src/Symfony/Bridge/Twig/Extension/TranslationExtension.php
@@ -34,13 +34,10 @@ class_exists(TranslatorTrait::class);
 */
 final class TranslationExtension extends AbstractExtension
 {
-    private ?TranslatorInterface $translator;
-    private ?TranslationNodeVisitor $translationNodeVisitor;
-
-    public function __construct(?TranslatorInterface $translator = null, ?TranslationNodeVisitor $translationNodeVisitor = null)
-    {
-        $this->translator = $translator;
-        $this->translationNodeVisitor = $translationNodeVisitor;
+    public function __construct(
+        private ?TranslatorInterface $translator = null,
+        private ?TranslationNodeVisitor $translationNodeVisitor = null,
+    ) {
    }

    public function getTranslator(): TranslatorInterface

diff --git a/src/Symfony/Bridge/Twig/Extension/WebLinkExtension.php b/src/Symfony/Bridge/Twig/Extension/WebLinkExtension.php
@@ -24,11 +24,9 @@
 */
 final class WebLinkExtension extends AbstractExtension
 {
-    private RequestStack $requestStack;
-
-    public function __construct(RequestStack $requestStack)
-    {
-        $this->requestStack = $requestStack;
+    public function __construct(
+        private RequestStack $requestStack,
+    ) {
    }

    public function getFunctions(): array

@@ -25,11 +25,9 @@
 */
 final class WorkflowExtension extends AbstractExtension
 {
-    private Registry $workflowRegistry;
-
-    public function __construct(Registry $workflowRegistry)
-    {
-        $this->workflowRegistry = $workflowRegistry;
+    public function __construct(
+        private Registry $workflowRegistry,
+    ) {
    }

    public function getFunctions(): array

@@ -21,13 +21,13 @@
 */
 class TwigRendererEngine extends AbstractRendererEngine
 {
-    private Environment $environment;
    private Template $template;

-    public function __construct(array $defaultThemes, Environment $environment)
-    {
+    public function __construct(
+        array $defaultThemes,
+        private Environment $environment,
+    ) {
        parent::__construct($defaultThemes);
-        $this->environment = $environment;
    }

    public function renderBlock(FormView $view, mixed $resource, string $blockName, array $variables = []): string

diff --git a/src/Symfony/Bridge/Twig/Mime/BodyRenderer.php b/src/Symfony/Bridge/Twig/Mime/BodyRenderer.php
@@ -26,17 +26,15 @@
 */
 final class BodyRenderer implements BodyRendererInterface
 {
-    private Environment $twig;
-    private array $context;
    private HtmlToTextConverterInterface $converter;
-    private ?LocaleSwitcher $localeSwitcher = null;

-    public function __construct(Environment $twig, array $context = [], ?HtmlToTextConverterInterface $converter = null, ?LocaleSwitcher $localeSwitcher = null)
-    {
-        $this->twig = $twig;
-        $this->context = $context;
+    public function __construct(
+        private Environment $twig,
+        private array $context = [],
+        ?HtmlToTextConverterInterface $converter = null,
+        private ?LocaleSwitcher $localeSwitcher = null,
+    ) {
        $this->converter = $converter ?: (interface_exists(HtmlConverterInterface::class) ? new LeagueHtmlToMarkdownConverter() : new DefaultHtmlToTextConverter());
-        $this->localeSwitcher = $localeSwitcher;
    }

    public function render(Message $message): void

diff --git a/src/Symfony/Bridge/Twig/Mime/WrappedTemplatedEmail.php b/src/Symfony/Bridge/Twig/Mime/WrappedTemplatedEmail.php
@@ -23,13 +23,10 @@
 */
 final class WrappedTemplatedEmail
 {
-    private Environment $twig;
-    private TemplatedEmail $message;
-
-    public function __construct(Environment $twig, TemplatedEmail $message)
-    {
-        $this->twig = $twig;
-        $this->message = $message;
+    public function __construct(
+        private Environment $twig,
+        private TemplatedEmail $message,
+    ) {
    }

    public function toName(): string