Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit 6776a43

Browse filesBrowse files
fancywebfancyweb
committed
[WebProfilerBundle] Add special "_best" panel
1 parent 2b71c6f commit 6776a43
Copy full SHA for 6776a43

File tree

3 files changed

+115
-5
lines changed
Filter options

3 files changed

+115
-5
lines changed

‎src/Symfony/Bundle/WebProfilerBundle/Controller/ProfilerController.php

Copy file name to clipboardExpand all lines: src/Symfony/Bundle/WebProfilerBundle/Controller/ProfilerController.php
+19-1Lines changed: 19 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@
1717
use Symfony\Component\HttpFoundation\Request;
1818
use Symfony\Component\HttpFoundation\Response;
1919
use Symfony\Component\HttpFoundation\Session\Flash\AutoExpireFlashBag;
20+
use Symfony\Component\HttpKernel\DataCollector\DumpDataCollector;
21+
use Symfony\Component\HttpKernel\DataCollector\ExceptionDataCollector;
2022
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
2123
use Symfony\Component\HttpKernel\Profiler\Profiler;
2224
use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
@@ -78,7 +80,7 @@ public function panelAction(Request $request, $token)
7880
$this->cspHandler->disableCsp();
7981
}
8082

81-
$panel = $request->query->get('panel', 'request');
83+
$panel = $request->query->get('panel');
8284
$page = $request->query->get('page', 'home');
8385

8486
if ('latest' === $token && $latest = current($this->profiler->find(null, null, 1, null, null, null))) {
@@ -89,6 +91,22 @@ public function panelAction(Request $request, $token)
8991
return new Response($this->twig->render('@WebProfiler/Profiler/info.html.twig', ['about' => 'no_token', 'token' => $token, 'request' => $request]), 200, ['Content-Type' => 'text/html']);
9092
}
9193

94+
if (null === $panel) {
95+
$panel = 'request';
96+
97+
foreach ($profile->getCollectors() as $collector) {
98+
if ($collector instanceof ExceptionDataCollector && $collector->hasException()) {
99+
$panel = $collector->getName();
100+
101+
break;
102+
}
103+
104+
if ($collector instanceof DumpDataCollector && $collector->getDumpsCount() > 0) {
105+
$panel = $collector->getName();
106+
}
107+
}
108+
}
109+
92110
if (!$profile->hasCollector($panel)) {
93111
throw new NotFoundHttpException(sprintf('Panel "%s" is not available for token "%s".', $panel, $token));
94112
}

‎src/Symfony/Bundle/WebProfilerBundle/Resources/views/Profiler/base_js.html.twig

Copy file name to clipboardExpand all lines: src/Symfony/Bundle/WebProfilerBundle/Resources/views/Profiler/base_js.html.twig
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -267,7 +267,7 @@
267267
if (request.profilerUrl) {
268268
profilerCell.textContent = '';
269269
var profilerLink = document.createElement('a');
270-
profilerLink.setAttribute('href', request.statusCode < 400 ? request.profilerUrl : request.profilerUrl + '?panel=exception');
270+
profilerLink.setAttribute('href', request.profilerUrl);
271271
profilerLink.textContent = request.profile;
272272
profilerCell.appendChild(profilerLink);
273273
}

‎src/Symfony/Bundle/WebProfilerBundle/Tests/Controller/ProfilerControllerTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Bundle/WebProfilerBundle/Tests/Controller/ProfilerControllerTest.php
+95-3Lines changed: 95 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,16 @@
1515
use Symfony\Bundle\WebProfilerBundle\Controller\ProfilerController;
1616
use Symfony\Bundle\WebProfilerBundle\Csp\ContentSecurityPolicyHandler;
1717
use Symfony\Component\HttpFoundation\Request;
18+
use Symfony\Component\HttpFoundation\Response;
19+
use Symfony\Component\HttpKernel\DataCollector\DumpDataCollector;
20+
use Symfony\Component\HttpKernel\DataCollector\ExceptionDataCollector;
21+
use Symfony\Component\HttpKernel\DataCollector\RequestDataCollector;
1822
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
1923
use Symfony\Component\HttpKernel\Profiler\Profile;
24+
use Symfony\Component\HttpKernel\Profiler\Profiler;
25+
use Twig\Environment;
26+
use Twig\Loader\SourceContextLoaderInterface;
27+
use Twig\Source;
2028

2129
class ProfilerControllerTest extends TestCase
2230
{
@@ -185,17 +193,101 @@ public function provideCspVariants()
185193
];
186194
}
187195

188-
private function createController($profiler, $twig, $withCSP): ProfilerController
196+
/**
197+
* @dataProvider defaultPanelProvider
198+
*/
199+
public function testDefaultPanel(string $expectedPanel, Profile $profile)
200+
{
201+
$profiler = $this->createMock(Profiler::class);
202+
$profiler
203+
->expects($this->atLeastOnce())
204+
->method('loadProfile')
205+
->with($profile->getToken())
206+
->willReturn($profile);
207+
208+
$profiler
209+
->expects($this->atLeastOnce())
210+
->method('has')
211+
->with($this->logicalXor($collectorsNames = array_keys($profile->getCollectors())))
212+
->willReturn(true);
213+
214+
$loader = $this->createMock(SourceContextLoaderInterface::class);
215+
$loader
216+
->expects($this->atLeastOnce())
217+
->method('getSourceContext')
218+
->with($this->logicalXor($expectedTemplate = 'expected_template.html.twig', 'other_template.html.twig'))
219+
->willReturn(new Source('foo', 'bar'));
220+
221+
$twig = $this->createMock(Environment::class);
222+
$twig
223+
->expects($this->atLeastOnce())
224+
->method('getLoader')
225+
->willReturn($loader);
226+
$twig
227+
->expects($this->once())
228+
->method('render')
229+
->with($expectedTemplate);
230+
231+
$this
232+
->createController($profiler, $twig, false, array_map(function (string $collectorName) use ($expectedPanel, $expectedTemplate): array {
233+
if ($collectorName === $expectedPanel) {
234+
return [$expectedPanel, $expectedTemplate];
235+
}
236+
237+
return [$collectorName, 'other_template.html.twig'];
238+
}, $collectorsNames))
239+
->panelAction(new Request(), $profile->getToken());
240+
}
241+
242+
public function defaultPanelProvider(): \Generator
243+
{
244+
// Test default behavior
245+
$profile = new Profile('xxxxxx');
246+
$profile->addCollector($requestDataCollector = new RequestDataCollector());
247+
yield [$requestDataCollector->getName(), $profile];
248+
249+
// Test exception
250+
$profile = new Profile('xxxxxx');
251+
$profile->addCollector($exceptionDataCollector = new ExceptionDataCollector());
252+
$exceptionDataCollector->collect(new Request(), new Response(), new \DomainException());
253+
yield [$exceptionDataCollector->getName(), $profile];
254+
255+
// Test exception priority
256+
$dumpDataCollector = $this->createMock(DumpDataCollector::class);
257+
$dumpDataCollector
258+
->expects($this->atLeastOnce())
259+
->method('getName')
260+
->willReturn('dump');
261+
$dumpDataCollector
262+
->expects($this->atLeastOnce())
263+
->method('getDumpsCount')
264+
->willReturn(1);
265+
$profile = new Profile('xxxxxx');
266+
$profile->setCollectors([$exceptionDataCollector, $dumpDataCollector]);
267+
yield [$exceptionDataCollector->getName(), $profile];
268+
269+
// Test exception priority when defined afterwards
270+
$profile = new Profile('xxxxxx');
271+
$profile->setCollectors([$dumpDataCollector, $exceptionDataCollector]);
272+
yield [$exceptionDataCollector->getName(), $profile];
273+
274+
// Test dump
275+
$profile = new Profile('xxxxxx');
276+
$profile->addCollector($dumpDataCollector);
277+
yield [$dumpDataCollector->getName(), $profile];
278+
}
279+
280+
private function createController($profiler, $twig, $withCSP, array $templates = []): ProfilerController
189281
{
190282
$urlGenerator = $this->getMockBuilder('Symfony\Component\Routing\Generator\UrlGeneratorInterface')->getMock();
191283

192284
if ($withCSP) {
193285
$nonceGenerator = $this->getMockBuilder('Symfony\Bundle\WebProfilerBundle\Csp\NonceGenerator')->getMock();
194286
$nonceGenerator->method('generate')->willReturn('dummy_nonce');
195287

196-
return new ProfilerController($urlGenerator, $profiler, $twig, [], new ContentSecurityPolicyHandler($nonceGenerator));
288+
return new ProfilerController($urlGenerator, $profiler, $twig, $templates, new ContentSecurityPolicyHandler($nonceGenerator));
197289
}
198290

199-
return new ProfilerController($urlGenerator, $profiler, $twig, []);
291+
return new ProfilerController($urlGenerator, $profiler, $twig, $templates);
200292
}
201293
}

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.