Conversation
|
Downside of own method: Using debug() should be preferred, but my tests revealed that core one is loaded first, so not sure how to fix that up. |
|
I have no strong opinions about this, since I wouldn't really use this feature. But if people like to have more options to do that, I am fine with that as well. |
|
File and line is still visible, same as logging it always is |
markstory
left a comment
There was a problem hiding this comment.
Since you want to defer output, why would someone use this over Log?
| <div class="c-dump-panel"> | ||
| <?php | ||
| foreach ($statements as $statement) { | ||
| echo '<pre class="cake-debug-string">' . h($statement['var']) . '</pre>'; |
There was a problem hiding this comment.
Don't we need to handle more complex values as well? If we're only going to support simple values, one could use logging instead.
Normal debug() breaks templates and JS, so with debug kit enabled this should go into dump panel IMO
Anywhere:
Proof of concept.
Before

After

and
