[#23848] - [com_actionlog] Fix username log checkin actions
- Closed
- 10 Feb 2019
- Medium
- Build: staging
- # 23848
- Diff
- tecpromotion:fix-username-log-checkin-actions
User tests: Successful: Unsuccessful:
Pull Request for PR #23526
Summary of Changes
Based on the function onUserAfterSave, the variable username was set with username and not user->name
Testing Instructions
Install 3.9.3-rc
Go to User -> User Action Log
Expected result
User username performed a check in to table #__content
username = Login Name
Actual result
User Super User performed a check in to table #__content
Super User here is the name (Name of the user) and not the username.
| Status | New | ⇒ | Pending |
| Category | ⇒ | Front End Plugins |
| Labels |
Added:
?
|
||
This introduces warnings because $user is now undefined. Reverting these changes and just replacing $user->name with $user->username should be fine.
@HLeithner sure
Quy
- | Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-02-10 08:14:21 |
| Closed_By | ⇒ | Quy |
| Closed_By | Quy | ⇒ | joomla-cms-bot |
Set to "closed" on behalf of @Quy by The JTracker Application at issues.joomla.org/joomla-cms/23848
@alikon can you check this fix for your PR?