The debug bar panel class render() method does too much.
It:
gets the logs.
adds wrapping markup, and loops through the logs.
loops through each item in the log, echoing with markup.
echos out CSS.
That is far too much responsibility for one method and couldn't easily be unit tested. It should be split into smaller methods, with render() pretty much making calls to those other methods.
GaryJones
commented
11 years ago
The debug bar panel class
render()method does too much.It:
That is far too much responsibility for one method and couldn't easily be unit tested. It should be split into smaller methods, with
render()pretty much making calls to those other methods.