yiisoft / yii2-debug

Debug Extension for Yii 2
http://www.yiiframework.com
BSD 3-Clause "New" or "Revised" License
202 stars 149 forks source link

Show DB "callers" in the DB panel #501

Closed rhertogh closed 1 year ago

rhertogh commented 1 year ago
Q A
Is bugfix?
New feature? ✔️
Breaks BC?
Fixed issues #499

Existing DB panel is unchanged and located under the "Queries" tab:

image

A new tab "Callers" is added:

image

what-the-diff[bot] commented 1 year ago

PR Summary

rhertogh commented 1 year ago

@samdark @bizley Originally this issue was intended to show "repeating callers" (as discussed in #499) and is currently implemented as such (minimum of 5 calls). Since it's now displayed on a separate tab do we want to show all callers on that tab (so including callers that, for example, only make 1 query call)? The "repeating calls" would that simply be "Total calls". Looking at it now it would actually make more sense that way.

samdark commented 1 year ago

@rhertogh agree.

rhertogh commented 1 year ago

@samdark @bizley The PR has been updated to list all "callers" (please see updated screenshot in opening post).

samdark commented 1 year ago

👍