Added a pretty_print option to the log class #1473
Conversation
It isn't useful to log arrays if the result is 'Array'. I noticed that I often use `Log::info(print_r($array, true));` and I'am probably not alone, so why not add this as a feature to the framework?
|
If you'd ask me we could get rid of manually specifying pretty print arguments and just use |
|
Yeah that would work for me too. But you need to have backwards compatibility in mind. Anyway, can someone accept this pull request? I think many people want this feature. |
|
I just found this pull. I did something similar, but I approached it from a different angle. I can now just use "Profiler::log('test', $array)" and get the array fully printed (including objects) - with one single line change: #1525 |
|
Smart! But one problem with your code is that the log file will still just contain "array". And sometimes you need to check to log file, in example when you do AJAX request. |
|
Yeah - true - but since Laravel 4 is just around the corner I didnt want to bother with massive changes for the current profiler. This way it allows for simple debugging on my dev site |
|
I made a pull to solve the 'Array' 4 months ago, #1208 . I used |
Added a pretty_print option to the log class
It isn't useful to log arrays if the result is 'Array'.
I noticed that I often use
Log::info(print_r($array, true));and I'am probably not alone, so why not add this as a feature to the framework?With this change you just need to write
Log::info($array, true);