[#8899] - Joomla Debug Console Improvements
- Closed
- 26 Jan 2016
- Medium
- Build: staging
- # 8899
- Diff
- andrepereiradasilva:improve-debugger
- Success Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Description
This PR adds some visual and informative improvements to the Joomla Debug Console.
Before PR (Database Queries -> "Call stack" slider)
After PR (Database Queries -> "Call stack" slider)
Before PR (Log Messages)
After PR (Log Messages)
How to test
- Apply this patch.
- Turn on the debug in global config and enable the debug system plugin.
- Test any page in frontend or administrator and check the "Joomla Debug Console" in the bottom of the page.
- Verify the info and visual of "Database Queries" -> "Call stack" slider
- Verify the info and visual of "Log Messages"
Observations
Suggestions and improvements are welcome.
Votes
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
?
|
||
I'll see if I can test this soon.
Would be nice if the "visual and informative improvements" also applied to the debug session info
I have tested this item 
successfully on 7fcc15a
Definitely a valuable improvement.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8899.
Would be nice if the "visual and informative improvements" also applied to the debug session info
I can change it to pretty-print the session json values (in PHP 5.4+)? ok for you? or are you referring to something else?
@test success, applied the patch and indeed better layout now
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8899.
@watchfulli-dev please mark as tested https://issues.joomla.org/tracker/joomla-cms/8899 (after login the Test button appears). See https://docs.joomla.org/Testing_Joomla!_patches#Recording_test_results
I have tested this item successfully on 7fcc15a
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8899.
Don't bother. The Framework's Log library is deprecated and all but abandoned at this point. We typehint the PSR-3 logging interface in the Framework where packages support logging and the consensus at the time was Monolog is exponentially better so we weren't rewriting our code to support the interface.
ok thanks.
@andrepereiradasilva Yes, something like pretty-print the session json values. Anything would likely look better than the raw JSON values.
This PR has received new commits.
CC: @photodude, @watchfulli-dev
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8899.
This PR has received new commits.
CC: @photodude, @watchfulli-dev
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8899.
Done, can you test the session part now?
This PR has received new commits.
CC: @photodude, @watchfulli-dev
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8899.
This PR has received new commits.
CC: @photodude, @watchfulli-dev
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8899.
This PR has received new commits.
CC: @photodude, @watchfulli-dev
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8899.
ok session
ok call stack
ko Log messages
gives different result items
This PR has received new commits.
CC: @photodude, @watchfulli-dev
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8899.
if we can have a subtotal for each logged category messages (not only for deprecated) should be OK
Joomla core has a LOT of deprecated messages .... 
What we can do is making tabs there for priority and with the totals
ex: CRITICAL xx || WARNING xx || INFO xx || DEBUG xx
or by category
ex: Deprecated xx || Databasequery xx || etc xx
I think the deprecated warning should stay alone.
I have tested this item successfully on f224435
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8899.
I have tested this item successfully on f224435
I do agree that we should have a summary subtotal for each logged category in the log section.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8899.
Ok. i will try do that in other PR.
This one has already lot of changes and is RTC.
I will also make some other improvements in the new PR.
| Status | Pending | ⇒ | Ready to Commit |
RTC then it is!
| Labels |
Added:
?
|
||
| Milestone |
Added: |
||
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-01-26 23:34:41 |
| Closed_By | ⇒ | wilsonge |
Merged - thanks @andrepereiradasilva
| Labels |
Removed:
?
|
||