Feeling Lucky
?
?
[#23335] - [plugin][system][actionlog] use namespaced class
- Fixed in Code Base
- 5 Jan 2019
- Medium
- Build: staging
- # 23335
- Diff
- alikon:patch-112
User tests: Successful: Unsuccessful:
Summary of Changes
use namespaced class
Testing Instructions
code review
or action log work as before
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
viocassel
- comment
- 27 Dec 2018
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23335.
| Status | Pending | ⇒ | Ready to Commit |
Quy
- comment
- 27 Dec 2018
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23335.
mbabker
- comment
- 27 Dec 2018
Merge conflicts...
| Labels |
Added:
?
?
|
||
sandewt
- comment
- 29 Dec 2018
Hi @alikon,
I looked at the code, and have a suggestion.
class PlgSystemActionLogs extends JPlugin
{
Could be:
use Joomla\CMS\Plugin\CMSPlugin;
...
class PlgSystemActionLogs extends CMSPlugin
{
Also apply to issue #23349 and #23350
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23335.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-01-05 16:16:24 |
| Closed_By | ⇒ | mbabker |
I have tested this item✅ successfully on ab67f90
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23335.