[#8112] - Remove empty category from addLogEntry
- Closed
- 24 Oct 2015
- Medium
- Build: staging
- Easy Test
- # 8112
- Diff
- digitalgarage:jlogchanges
- Success Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
This prevents logging of messages where no category is specified as part of JLog::add.
Test Instructions
Add JLog::add("This shouldn't be here", JLog::INFO); to your template's index.php file.
Before this patched the logging statement This shouldn't be here will be included in the messages block. According to the docs this should ONLY be displayed when the jerror category is set. (https://docs.joomla.org/Using_JLog)
After this patch the message will not be displayed. Logging entries will only be logged when category is included. For example:
JLog::add("This should be here", JLog::INFO,"jerror");
if using the MessageQueue logger.
Eric.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | Libraries |
| Easy | No | ⇒ | Yes |
| Rel_Number | 0 | ⇒ | 8094 |
| Relation Type | ⇒ | Pull Request for |
I have tested this item 
successfully on fd3428f
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8112.
I have tested this item successfully on fd3428f
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8112.
Pizzaaaaaa :-) It works as expected
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8112.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
?
|
||
| Milestone |
Added: |
||
rdeutz
- | Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-10-24 14:12:33 |
| Closed_By | ⇒ | rdeutz |
| Labels |
Removed:
?
|
||
| Milestone | |||
| Milestone |
Added: |
||
| Milestone |
Added: |
||
| Milestone | |||
Sorry folks forgot the issue reference. For history and discussion see: Issue #8094.