bug #43914 [5.4][WebProfiler] Fixing missing full_stack variable that's needed by toolbar.html.twig (weaverryan)

fabpot · fabpot · commit d254e8d2a279 · 2021-11-03T16:16:52.000+01:00
This PR was merged into the 5.4 branch. Discussion ---------- [5.4][WebProfiler] Fixing missing full_stack variable that's needed by toolbar.html.twig | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | None | License | MIT | Doc PR | Not needed The combination of #43526 and then #41257 created an undefined variable. By adding the variable here, it will flow from `toolbar_js.html.twig` into `toolbar.html.twig`. Tested locally after reproducing the issue. Thanks to symfony/ux test suite for catching this :) Commits ------- 0e8b864 Fixing missing full_stack variable that's needed by toolbar.html.twig
diff --git a/src/Symfony/Bundle/WebProfilerBundle/EventListener/WebDebugToolbarListener.php b/src/Symfony/Bundle/WebProfilerBundle/EventListener/WebDebugToolbarListener.php
@@ -11,6 +11,7 @@
 
 namespace Symfony\Bundle\WebProfilerBundle\EventListener;
 
+use Symfony\Bundle\FullStack;
 use Symfony\Bundle\WebProfilerBundle\Csp\ContentSecurityPolicyHandler;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 use Symfony\Component\HttpFoundation\Request;
@@ -142,6 +143,7 @@ protected function injectToolbar(Response $response, Request $request, array $no
             $toolbar = "\n".str_replace("\n", '', $this->twig->render(
                 '@WebProfiler/Profiler/toolbar_js.html.twig',
                 [
+                    'full_stack' => class_exists(FullStack::class),
                     'excluded_ajax_paths' => $this->excludedAjaxPaths,
                     'token' => $response->headers->get('X-Debug-Token'),
                     'request' => $request,
