merged branch vicb/profiler (PR #4727)
Commits -------eda439f
[EventDataCollector] Display a better message when no events have been recorded6b87981
[TimeDataCollector] Do not throw an exception when no events are recorded Discussion ---------- [Profiler] Better support for collector in a production env relates to #3706. With this PR it is possible to: - enable only the profiler in a production environment - the wdt being disabled you have to switch to the development environment to inspect the collected data, - enable both the profiler and the wdt in a production environment (the use case form #3706). @jmikola would this solve your use case ?
This commit is contained in:
commit
391ee67d68
@ -10,6 +10,17 @@
|
||||
{% endblock %}
|
||||
|
||||
{% block panel %}
|
||||
{% if collector.calledlisteners|length %}
|
||||
{{ block('panelContent') }}
|
||||
{% else %}
|
||||
<h2>Events</h2>
|
||||
<p>
|
||||
<em>No events have been recorded. Are you sure that debugging is enabled in the kernel ?</em>
|
||||
</p>
|
||||
{% endif %}
|
||||
{% endblock %}
|
||||
|
||||
{% block panelContent %}
|
||||
<h2>Called Listeners</h2>
|
||||
|
||||
<table>
|
||||
|
@ -14,14 +14,15 @@
|
||||
{% endif %}
|
||||
|
||||
{% block toolbar %}
|
||||
{% set total_time = collector.events|length ? '%.0f ms'|format(collector.totaltime) : 'n/a' %}
|
||||
{% set icon %}
|
||||
<img width="16" height="28" alt="Time" src="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAcCAYAAABoMT8aAAABqUlEQVR42t2Vv0sCYRyHX9OmEhsMx/YKGlwLQ69DTEUSBJEQEy5J3FRc/BsuiFqEIIcQIRo6ysUhoaBBWhoaGoJwiMJLglRKrs8bXgienmkQdPDAwX2f57j3fhFJkkbiPwTK5bIiFoul3kmPud8MqKMewDXpwuGww+12n9hsNhFnlijYf/Z4PDmO45Yxo+10ZFGTyWRMEItU6AdCx7lczkgd6n7J2Wx2xm63P6jJMk6n80YQBBN1aUDv9XqvlAbbm2LE7/cLODRB0un0VveAeoDC8/waCQQC18MGQqHQOcEKvw8bcLlcL6TfYnVtCrGRAlartUUYhmn1jKg/E3USjUYfhw3E4/F7ks/nz4YNFIvFQ/ogbUYikdefyqlU6gnuOg2YK5XKvs/n+xhUDgaDTVEUt+HO04ABOBA5isViDTU5kUi81Wq1AzhWMEkDGmAEq2C3UCjcYXGauDvfEsuyUjKZbJRKpVvM8IABU9SVX+cxYABmwIE9cFqtVi9xtgvsC2AHbIAFoKey0gdlHEyDObAEWLACFsEsMALdIJ80+dK0bTS95v7+v/AJnis0eO906QwAAAAASUVORK5CYII="/>
|
||||
<span>{{ '%.0f'|format(collector.totaltime) }} ms</span>
|
||||
<span>{{ total_time }}</span>
|
||||
{% endset %}
|
||||
{% set text %}
|
||||
<div class="sf-toolbar-info-piece">
|
||||
<b>Total time</b>
|
||||
<span>{{ '%.0f'|format(collector.totaltime) }} ms</span>
|
||||
<span>{{ total_time }}</span>
|
||||
</div>
|
||||
{% endset %}
|
||||
{% include 'WebProfilerBundle:Profiler:toolbar_item.html.twig' with { 'link': profiler_url } %}
|
||||
@ -36,7 +37,16 @@
|
||||
|
||||
{% block panel %}
|
||||
<h2>Timeline</h2>
|
||||
{% if collector.events|length %}
|
||||
{{ block('panelContent') }}
|
||||
{% else %}
|
||||
<p>
|
||||
<em>No timing events have been recorded. Are you sure that debugging is enabled in the kernel ?</em>
|
||||
</p>
|
||||
{% endif %}
|
||||
{% endblock %}
|
||||
|
||||
{% block panelContent %}
|
||||
<form id="timeline-control" action="" method="get">
|
||||
<input type="hidden" name="panel" value="time" />
|
||||
<table>
|
||||
@ -413,7 +423,6 @@
|
||||
elementThresholdControl.onclick = canvasAutoUpdateOnThresholdChange;
|
||||
elementThresholdControl.onchange = canvasAutoUpdateOnThresholdChange;
|
||||
elementThresholdControl.onkeyup = canvasAutoUpdateOnThresholdChange;
|
||||
|
||||
//]]></script>
|
||||
{% endblock %}
|
||||
|
||||
|
@ -74,7 +74,7 @@ class TimeDataCollector extends DataCollector
|
||||
{
|
||||
$lastEvent = $this->data['events']['__section__'];
|
||||
|
||||
return $lastEvent->getOrigin() + $lastEvent->getTotalTime() - $this->data['start_time'];
|
||||
return $lastEvent->getOrigin() + $lastEvent->getTotalTime() - $this->getStartTime();
|
||||
}
|
||||
|
||||
/**
|
||||
|
Reference in New Issue
Block a user