generating template names moved out from controller to another class
This commit is contained in:
parent
6138e8029e
commit
abd0eb7faa
@ -41,6 +41,7 @@ class ProfilerController extends ContainerAware
|
||||
$panel = $this->container->get('request')->query->get('panel', 'request');
|
||||
$page = $this->container->get('request')->query->get('page', 'home');
|
||||
|
||||
|
||||
if (!$profile = $profiler->loadProfile($token)) {
|
||||
return $this->container->get('templating')->renderResponse('WebProfilerBundle:Profiler:info.html.twig', array('about' => 'no_token', 'token' => $token));
|
||||
}
|
||||
@ -49,13 +50,17 @@ class ProfilerController extends ContainerAware
|
||||
throw new NotFoundHttpException(sprintf('Panel "%s" is not available for token "%s".', $panel, $token));
|
||||
}
|
||||
|
||||
return $this->container->get('templating')->renderResponse($this->getTemplateName($profile, $panel), array(
|
||||
/** @var $templateManager \Symfony\Bundle\WebProfilerBundle\Profiler\Template */
|
||||
$templateManager = $this->container->get('web_profiler.profiler_template');
|
||||
$templateManager->setProfiler($profiler);
|
||||
|
||||
return $this->container->get('templating')->renderResponse($templateManager->getName($profile, $panel), array(
|
||||
'token' => $token,
|
||||
'profile' => $profile,
|
||||
'collector' => $profile->getCollector($panel),
|
||||
'panel' => $panel,
|
||||
'page' => $page,
|
||||
'templates' => $this->getTemplates($profile),
|
||||
'templates' => $templateManager->getTemplates($profile),
|
||||
'is_ajax' => $request->isXmlHttpRequest(),
|
||||
));
|
||||
}
|
||||
@ -178,10 +183,14 @@ class ProfilerController extends ContainerAware
|
||||
// the profiler is not enabled
|
||||
}
|
||||
|
||||
/** @var $templateManager \Symfony\Bundle\WebProfilerBundle\Profiler\Template */
|
||||
$templateManager = $this->container->get('web_profiler.profiler_template');
|
||||
$templateManager->setProfiler($profiler);
|
||||
|
||||
return $this->container->get('templating')->renderResponse('WebProfilerBundle:Profiler:toolbar.html.twig', array(
|
||||
'position' => $position,
|
||||
'profile' => $profile,
|
||||
'templates' => $this->getTemplates($profile),
|
||||
'templates' => $templateManager->getTemplates($profile),
|
||||
'profiler_url' => $url,
|
||||
'verbose' => $this->container->get('web_profiler.debug_toolbar')->isVerbose()
|
||||
));
|
||||
@ -292,60 +301,4 @@ class ProfilerController extends ContainerAware
|
||||
)));
|
||||
}
|
||||
|
||||
/**
|
||||
* Gets template names of templates that are
|
||||
* present in the viewed profile
|
||||
*
|
||||
* @param $profile
|
||||
* @return array
|
||||
* @throws \UnexpectedValueException
|
||||
*/
|
||||
protected function getTemplateNames($profile)
|
||||
{
|
||||
$templates = array();
|
||||
|
||||
foreach ($this->container->getParameter('data_collector.templates') as $arguments) {
|
||||
if (null === $arguments) {
|
||||
continue;
|
||||
}
|
||||
|
||||
list($name, $template) = $arguments;
|
||||
if (!$profile->hasCollector($name)) {
|
||||
continue;
|
||||
}
|
||||
|
||||
if ('.html.twig' === substr($template, -10)) {
|
||||
$template = substr($template, 0, -10);
|
||||
}
|
||||
|
||||
if (!$this->container->get('templating')->exists($template.'.html.twig')) {
|
||||
throw new \UnexpectedValueException(sprintf('The profiler template "%s.html.twig" for data collector "%s" does not exist.', $template, $name));
|
||||
}
|
||||
|
||||
$templates[$name] = $template.'.html.twig';
|
||||
}
|
||||
|
||||
return $templates;
|
||||
}
|
||||
|
||||
protected function getTemplateName($profile, $panel)
|
||||
{
|
||||
$templates = $this->getTemplateNames($profile);
|
||||
|
||||
if (!isset($templates[$panel])) {
|
||||
throw new NotFoundHttpException(sprintf('Panel "%s" is not registered.', $panel));
|
||||
}
|
||||
|
||||
return $templates[$panel];
|
||||
}
|
||||
|
||||
protected function getTemplates($profile)
|
||||
{
|
||||
$templates = $this->getTemplateNames($profile);
|
||||
foreach ($templates as $name => $template) {
|
||||
$templates[$name] = $this->container->get('twig')->loadTemplate($template);
|
||||
}
|
||||
|
||||
return $templates;
|
||||
}
|
||||
}
|
||||
|
133
src/Symfony/Bundle/WebProfilerBundle/Profiler/Template.php
Normal file
133
src/Symfony/Bundle/WebProfilerBundle/Profiler/Template.php
Normal file
@ -0,0 +1,133 @@
|
||||
<?php
|
||||
|
||||
/*
|
||||
* This file is part of the Symfony package.
|
||||
*
|
||||
* (c) Fabien Potencier <fabien@symfony.com>
|
||||
*
|
||||
* For the full copyright and license information, please view the LICENSE
|
||||
* file that was distributed with this source code.
|
||||
*/
|
||||
|
||||
namespace Symfony\Bundle\WebProfilerBundle\Profiler;
|
||||
|
||||
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
|
||||
use Symfony\Component\HttpKernel\Profiler\Profiler;
|
||||
use Symfony\Component\HttpKernel\Profiler\Profile;
|
||||
use Symfony\Bundle\TwigBundle\TwigEngine;
|
||||
|
||||
/**
|
||||
* ProfilerController.
|
||||
*
|
||||
* @author Fabien Potencier <fabien@symfony.com>
|
||||
* @author Artur Wielogórski <wodor@wodor.net>
|
||||
*/
|
||||
class Template {
|
||||
|
||||
/**
|
||||
* @var \Symfony\Bundle\TwigBundle\TwigEngine
|
||||
*/
|
||||
public $templating;
|
||||
|
||||
/**
|
||||
* @var \Twig_Environment
|
||||
*/
|
||||
public $twig;
|
||||
|
||||
/**
|
||||
* @var array
|
||||
*/
|
||||
public $templates;
|
||||
|
||||
/**
|
||||
* @var \Symfony\Component\HttpKernel\Profiler\Profiler
|
||||
*/
|
||||
protected $profiler;
|
||||
|
||||
/**
|
||||
* @param \Symfony\Bundle\TwigBundle\TwigEngine $templating
|
||||
* @param \Twig_Environment $twig
|
||||
* @param array $templates
|
||||
*/
|
||||
public function __construct(TwigEngine $templating, \Twig_Environment $twig, array $templates)
|
||||
{
|
||||
$this->templating = $templating;
|
||||
$this->twig = $twig;
|
||||
$this->templates = $templates;
|
||||
}
|
||||
|
||||
/**
|
||||
* @param \Symfony\Component\HttpKernel\Profiler\Profile $profile
|
||||
* @param $panel
|
||||
* @return mixed
|
||||
* @throws \Symfony\Component\HttpKernel\Exception\NotFoundHttpException
|
||||
*/
|
||||
public function getName(Profile $profile, $panel)
|
||||
{
|
||||
$templates = $this->getNames($profile);
|
||||
|
||||
if (!isset($templates[$panel])) {
|
||||
throw new NotFoundHttpException(sprintf('Panel "%s" is not registered in profiler or is not present in viewed profile.', $panel));
|
||||
}
|
||||
|
||||
return $templates[$panel];
|
||||
}
|
||||
|
||||
/**
|
||||
* @param \Symfony\Component\HttpKernel\Profiler\Profile $profile
|
||||
* @return array
|
||||
*/
|
||||
public function getTemplates(Profile $profile)
|
||||
{
|
||||
$templates = $this->getNames($profile);
|
||||
foreach ($templates as $name => $template) {
|
||||
$templates[$name] = $this->twig->loadTemplate($template);
|
||||
}
|
||||
|
||||
return $templates;
|
||||
}
|
||||
|
||||
/**
|
||||
* @param \Symfony\Component\HttpKernel\Profiler\Profiler $profiler
|
||||
*/
|
||||
public function setProfiler(Profiler $profiler)
|
||||
{
|
||||
$this->profiler = $profiler;
|
||||
}
|
||||
|
||||
/**
|
||||
* Gets template names of templates that are
|
||||
* present in the viewed profile
|
||||
* @param \Symfony\Component\HttpKernel\Profiler\Profile $profile
|
||||
* @return array
|
||||
* @throws \UnexpectedValueException
|
||||
*/
|
||||
protected function getNames(Profile $profile)
|
||||
{
|
||||
$templates = array();
|
||||
|
||||
foreach ($this->templates as $arguments) {
|
||||
if (null === $arguments) {
|
||||
continue;
|
||||
}
|
||||
|
||||
list($name, $template) = $arguments;
|
||||
|
||||
if (!$this->profiler->has($name) || !$profile->hasCollector($name)) {
|
||||
continue;
|
||||
}
|
||||
|
||||
if ('.html.twig' === substr($template, -10)) {
|
||||
$template = substr($template, 0, -10);
|
||||
}
|
||||
|
||||
if (!$this->templating->exists($template.'.html.twig')) {
|
||||
throw new \UnexpectedValueException(sprintf('The profiler template "%s.html.twig" for data collector "%s" does not exist.', $template, $name));
|
||||
}
|
||||
|
||||
$templates[$name] = $template.'.html.twig';
|
||||
}
|
||||
|
||||
return $templates;
|
||||
}
|
||||
}
|
@ -6,6 +6,7 @@
|
||||
|
||||
<parameters>
|
||||
<parameter key="web_profiler.debug_toolbar.class">Symfony\Bundle\WebProfilerBundle\EventListener\WebDebugToolbarListener</parameter>
|
||||
<parameter key="web_profiler.profiler_template">Symfony\Bundle\WebProfilerBundle\Profiler\Template</parameter>
|
||||
</parameters>
|
||||
|
||||
<services>
|
||||
@ -16,5 +17,11 @@
|
||||
<argument>%web_profiler.debug_toolbar.mode%</argument>
|
||||
<argument>%web_profiler.debug_toolbar.position%</argument>
|
||||
</service>
|
||||
|
||||
<service id="web_profiler.profiler_template" class="%web_profiler.profiler_template%">
|
||||
<argument type="service" id="templating.engine.twig" />
|
||||
<argument type="service" id="twig" />
|
||||
<argument>%data_collector.templates%</argument>
|
||||
</service>
|
||||
</services>
|
||||
</container>
|
||||
|
@ -65,6 +65,7 @@ class WebProfilerExtensionTest extends TestCase
|
||||
$this->container->setParameter('kernel.cache_dir', __DIR__);
|
||||
$this->container->setParameter('kernel.debug', false);
|
||||
$this->container->setParameter('kernel.root_dir', __DIR__);
|
||||
$this->container->setParameter('data_collector.templates', array());
|
||||
$this->container->set('kernel', $this->kernel);
|
||||
}
|
||||
|
||||
|
@ -0,0 +1,175 @@
|
||||
<?php
|
||||
|
||||
/*
|
||||
* This file is part of the Symfony package.
|
||||
*
|
||||
* (c) Fabien Potencier <fabien@symfony.com>
|
||||
*
|
||||
* For the full copyright and license information, please view the LICENSE
|
||||
* file that was distributed with this source code.
|
||||
*/
|
||||
|
||||
namespace Symfony\Bundle\WebProfilerBundle\Tests\Profiler;
|
||||
|
||||
use Symfony\Bundle\WebProfilerBundle\Tests\TestCase;
|
||||
use Symfony\Bundle\WebProfilerBundle\Profiler\Template;
|
||||
|
||||
/**
|
||||
* Test for Template class.
|
||||
*
|
||||
* @author Artur Wielogórski <wodor@wodor.net>
|
||||
*/
|
||||
class TemplateTest extends TestCase
|
||||
{
|
||||
/**
|
||||
* @var \Symfony\Bundle\TwigBundle\TwigEngine
|
||||
*/
|
||||
protected $twigEngine;
|
||||
|
||||
/**
|
||||
* @var \Twig_Environment
|
||||
*/
|
||||
protected $twigEnvironment;
|
||||
|
||||
/**
|
||||
* @var \Symfony\Component\HttpKernel\Profiler\Profiler
|
||||
*/
|
||||
protected $profiler;
|
||||
|
||||
/**
|
||||
* @var \PHPUnit_Framework_MockObject_MockObject
|
||||
*/
|
||||
protected $profile;
|
||||
|
||||
/**
|
||||
* @var \Symfony\Bundle\WebProfilerBundle\Profiler\Template
|
||||
*/
|
||||
protected $templateManager;
|
||||
|
||||
public function setUp()
|
||||
{
|
||||
parent::setUp();
|
||||
|
||||
$twigEngine = $this->mockTwigEngine();
|
||||
$twigEnvironment = $this->mockTwigEnvironment();
|
||||
$templates = array(
|
||||
'data_collector.foo'=>array('foo','FooBundle:Collector:foo'),
|
||||
'data_collector.bar'=>array('bar','FooBundle:Collector:bar'),
|
||||
'data_collector.baz'=>array('baz','FooBundle:Collector:baz')
|
||||
);
|
||||
|
||||
$this->templateManager = new Template($twigEngine, $twigEnvironment, $templates);
|
||||
$this->templateManager->setProfiler($this->mockProfiler());
|
||||
}
|
||||
|
||||
/**
|
||||
* @expectedException Symfony\Component\HttpKernel\Exception\NotFoundHttpException
|
||||
*/
|
||||
public function testGetNameOfInvalidTemplate()
|
||||
{
|
||||
$profile = $this->mockProfile();
|
||||
$this->templateManager->getName($profile, 'notexisitingpanel');
|
||||
}
|
||||
|
||||
/**
|
||||
* if template exists in both profile and profiler then it's name should be returned
|
||||
*/
|
||||
public function testGetNameValidTemplate()
|
||||
{
|
||||
$this->profiler->expects($this->any())
|
||||
->method('has')
|
||||
->withAnyParameters()
|
||||
->will($this->returnCallback(array($this, 'profilerHasCallback')));
|
||||
|
||||
$profile = $this->mockProfile();
|
||||
$profile->expects($this->any())
|
||||
->method('hasCollector')
|
||||
->will($this->returnCallback(array($this, 'profileHasCollectorCallback')));
|
||||
|
||||
$this->assertEquals('FooBundle:Collector:foo.html.twig', $this->templateManager->getName($profile, 'foo'));
|
||||
}
|
||||
|
||||
/**
|
||||
* template should be loaded for 'foo' because other collectors are
|
||||
* missing in profile or in profiler
|
||||
*/
|
||||
public function testGetTemplates() {
|
||||
|
||||
$profile = $this->mockProfile();
|
||||
$profile->expects($this->any())
|
||||
->method('hasCollector')
|
||||
->will($this->returnCallback(array($this, 'profilerHasCallback')));
|
||||
|
||||
$this->profiler->expects($this->any())
|
||||
->method('has')
|
||||
->withAnyParameters()
|
||||
->will($this->returnCallback(array($this, 'profileHasCollectorCallback')));
|
||||
|
||||
$result = $this->templateManager->getTemplates($profile);
|
||||
$this->assertArrayHasKey('foo',$result);
|
||||
$this->assertArrayNotHasKey('bar',$result);
|
||||
$this->assertArrayNotHasKey('baz',$result);
|
||||
}
|
||||
|
||||
public function profilerHasCallback($panel) {
|
||||
switch ($panel) {
|
||||
case 'foo':
|
||||
case 'bar':
|
||||
return true;
|
||||
default:
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
public function profileHasCollectorCallback($panel) {
|
||||
switch ($panel) {
|
||||
case 'foo':
|
||||
case 'baz':
|
||||
return true;
|
||||
default:
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
protected function mockProfile() {
|
||||
$this->profile = $this->getMockBuilder('Symfony\Component\HttpKernel\Profiler\Profile')
|
||||
->disableOriginalConstructor()
|
||||
->getMock();
|
||||
|
||||
return $this->profile;
|
||||
}
|
||||
|
||||
protected function mockTwigEnvironment()
|
||||
{
|
||||
$this->twigEnvironment = $this->getMockBuilder('Twig_Environment')->getMock();
|
||||
|
||||
$this->twigEnvironment->expects($this->any())
|
||||
->method('loadTemplate')
|
||||
->will($this->returnValue('loadedTemplate'));
|
||||
|
||||
return $this->twigEnvironment;
|
||||
|
||||
}
|
||||
|
||||
protected function mockTwigEngine()
|
||||
{
|
||||
$this->twigEngine = $this->getMockBuilder('Symfony\Bundle\TwigBundle\TwigEngine')
|
||||
->disableOriginalConstructor()
|
||||
->getMock();
|
||||
|
||||
$this->twigEngine->expects($this->any())
|
||||
->method('exists')
|
||||
->will($this->returnValue(true));
|
||||
|
||||
return $this->twigEngine;
|
||||
|
||||
}
|
||||
|
||||
protected function mockProfiler() {
|
||||
$this->profiler = $this->getMockBuilder('Symfony\Component\HttpKernel\Profiler\Profiler')
|
||||
->disableOriginalConstructor()
|
||||
->getMock();
|
||||
|
||||
return $this->profiler;
|
||||
}
|
||||
}
|
Reference in New Issue
Block a user