[WebProfilerBundle] Display trace and context in the logger profiler

lyrixx · lyrixx · commit ef1e50802ed4 · 2017-07-21T10:28:01.000+02:00
diff --git a/src/Symfony/Bundle/WebProfilerBundle/Resources/views/Collector/logger.html.twig b/src/Symfony/Bundle/WebProfilerBundle/Resources/views/Collector/logger.html.twig
@@ -224,31 +224,33 @@
 {% endmacro %}
 
 {% macro render_log_message(category, log_index, log) %}
-    {% if log.context.exception.trace is defined %}
-        {{ profiler_dump_log(log.message, log.context) }}
-
-        {% set context_id = 'context-' ~ category ~ '-' ~ log_index %}
+    {% set has_context = log.context is defined and log.context is not empty %}
+    {% set has_trace = log.context.exception.trace is defined %}
 
-        <span class="metadata">
-             <a class="btn btn-link text-small sf-toggle" data-toggle-selector="#{{ context_id }}" data-toggle-alt-content="Hide trace">Show trace</a>
-
-            <div id="{{ context_id }}" class="context sf-toggle-content sf-toggle-hidden">
-                {{ profiler_dump(log.context.exception.trace, maxDepth=1) }}
-            </div>
-        </span>
-    {% elseif log.context is defined and log.context is not empty %}
+    {% if not has_context %}
+        {{ profiler_dump_log(log.message) }}
+    {% else %}
         {{ profiler_dump_log(log.message, log.context) }}
 
-        {% set context_id = 'context-' ~ category ~ '-' ~ log_index %}
+        <div class="text-small font-normal">
+            {% set context_id = 'context-' ~ category ~ '-' ~ log_index %}
+            <a class="btn btn-link text-small sf-toggle" data-toggle-selector="#{{ context_id }}" data-toggle-alt-content="Hide context">Show context</a>
 
-        <span class="metadata">
-             <a class="btn btn-link text-small sf-toggle" data-toggle-selector="#{{ context_id }}" data-toggle-alt-content="Hide context">Show context</a>
+            {% if has_trace %}
+                &nbsp;&nbsp;
+                {% set trace_id = 'trace-' ~ category ~ '-' ~ log_index %}
+                <a class="btn btn-link text-small sf-toggle" data-toggle-selector="#{{ trace_id }}" data-toggle-alt-content="Hide trace">Show trace</a>
+            {% endif %}
+        </div>
+
+        <div id="{{ context_id }}" class="context sf-toggle-content sf-toggle-hidden">
+            {{ profiler_dump(log.context, maxDepth=1) }}
+        </div>
 
-             <div id="{{ context_id }}" class="context sf-toggle-content sf-toggle-hidden">
-                {{ profiler_dump(log.context, maxDepth=1) }}
+        {% if has_trace %}
+            <div id="{{ trace_id }}" class="context sf-toggle-content sf-toggle-hidden">
+                {{ profiler_dump(log.context.exception.trace, maxDepth=1) }}
             </div>
-        </span>
-    {% else %}
-        {{ profiler_dump_log(log.message) }}
+        {% endif %}
     {% endif %}
 {% endmacro %}
