Duration added to log about yii2-debug HOT 9 CLOSED

Duration of what?

from yii2-debug.

ahh, now I understand why it isnt in there. It can not be called "duration" because that would imply a duration of the specific logcategory in the message.

Lets call it something else: Time to Next Log (TTNL) or something else.

from yii2-debug.

we have Yii::beginProfile() and Yii::endProfile() for that so you can measure timings for some parts of the app.

from yii2-debug.

I am aware of that, but this is a totally different way of performance optimization. Let me put it this way:
A) active optimization: implement Yii::beginProfile() and Yii::endProfile()
B) passive optimization: compare timeframes in log to see performance issues

from yii2-debug.

@dynasource Does #54 somehow solve this?

from yii2-debug.

@Deele: you've create a JS solution? Why in Js ;)? If it computes the duration, it would solve this topic, but its rather unconventional to 'not' do this by PHP.

from yii2-debug.

@dynasource Because, that is easier, and it works already. It is not precise, but allows to search faster when you have hundreds of log entries, and you want to find that one that causes 2 second loading time. Total time between logs does not match total time mentioned in header, that gives me hint, that to create trustful solution, would require through knowledge in debug module and Yii internals.

from yii2-debug.

well, I tried your solution. The JS looks OK, but durations are connected to the wrong lines, it should have an offset of -1.

Although its quite useful in the current form, I don't know if its logical from an architectural perspective. I would not expect a JS solution. Secondary, if Gridviews are applied with pagination, this solution wouldnt do.

from yii2-debug.

Should be PHP-based.

from yii2-debug.