Merge pull request #14 from pana1990/bug/13-empty-trace

fix null access

Author: pana1990

src/DataCollectors/PdoDataCollector.php

@@ -51,9 +51,27 @@ public function count(): int
     public function trace(): \Closure
     {
         return function (array $trace): array {
+            $file = $trace['file'] ?? null;
+            $line = $trace['line'] ?? null;
+            $class = $trace['class'] ?? null;
+
+            if (true === isset($file, $line)) {
+                return [
+                    'url'  => sprintf($this->xdebugLinkTemplate, $file, $line),
+                    'path' => sprintf('%s:%s', $file, $line),
+                ];
+            }
+
+            if (true === isset($class)) {
+                return [
+                    'url' => null,
+                    'path' => sprintf('%s:%s', $class, $line),
+                ];
+            }
+
             return [
-                'url' => sprintf($this->xdebugLinkTemplate, $trace['file'], $trace['line']),
-                'path' => sprintf('%s:%s', $trace['file'], $trace['line']),
+                'url' => null,
+                'path' => 'Unkown',
             ];
         };
     }

src/Resources/pages/pdo/pdo.html.twig

@@ -8,9 +8,9 @@
 
         <table id="table-pdo" class="table table-bordered table-striped">
             <colgroup>
-                <col />
-                <col />
-                <col />
+                <col/>
+                <col/>
+                <col/>
             </colgroup>
             <thead>
             <tr>
@@ -34,7 +34,11 @@
                             </p>
                             <div class="collapse" id="trace-{{ index }}">
                                 {% for trace in value['trace'] %}
-                                    <a href="{{ trace['url'] }}">{{ trace['path'] }}</a><br>
+                                    {% if trace['url'] is not null %}
+                                        <a href="{{ trace['url'] }}">{{ trace['path'] }}</a><br>
+                                    {% else %}
+                                        {{ trace['path'] }}<br>
+                                    {% endif %}
                                 {% endfor %}
                             </div>
                         </div>
@@ -52,6 +56,6 @@
     <script src="//cdnjs.cloudflare.com/ajax/libs/highlight.js/11.6.0/highlight.min.js"></script>
 
     <script>
-      hljs.highlightAll();
+        hljs.highlightAll();
     </script>
 {% endblock %}