Feeling Lucky
?
[#16342] - Use the JLogEntry class
- Fixed in Code Base
- 29 May 2017
- Medium
- Build: 3.8-dev
- # 16342
- Diff
- Digital-Peak:j3/fix/jlogentry-error
User tests: Successful: Unsuccessful:
Pull Request for Issue #16315 and #16311.
Summary of Changes
class_alias is actually not loading the alias class. Additionally when you have typed function arguments and instanceof calls, the class is also not loaded. Because the logger is called very early it can lead to a situation that the class JLogEntryis not loaded but a logger which is not namespaced requires still JLogEntry and then we get an error like:
Argument 1 passed to PlgSystemDebug::logger() must be an instance of JLogEntry, instance of Joomla\CMS\Log\LogEntry given
This pr is a workaround to use JLogEntryinstead of the namespaced class.
Testing Instructions
- Enable debugging in Joomla configuration
- Enable in the debug plugin Logging
Expected result
No Error is thrown.
Actual result
Error is thrown.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-05-29 17:17:52 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
|
||
This is bullshit. But I dunno what's better