[#26688] - [4.0] Blocked user logout action should not be Logged when user is not logged in Frontend
- Closed
- 3 Jan 2020
- Medium
- Build: 4.0-dev
- # 26688
- Diff
- sudhir-j-sapkal:userBlockLog
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Summary of Changes
If user is blocked , Logout activity should not be logged.
Testing Instructions
Login to Joomla Backend with Superuser access
Go to Users => Manage
Disable any user
Go to Users => Users Action Log
See the latest activity. "User vjtest logged out from admin"
Expected result
The logout entry for blocked user should not be logged when user is not logged in Front end, It should only log user is updated by Super user.
Actual result
There were two activities
- For update of user by super user.
- User is logged out from system.
Documentation Changes Required
No
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
| Title |
|
||||||
| Labels |
Added:
?
|
||
Why should this not be logged. It seems sensible to me that this is logged
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26688.
It's only happens when you block him from the table but not when you do it in the edit view.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26688.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26688.
| Title |
|
||||||
@brianteeman - It should log the activity of User Logged out only when user is log-ed in front end and Admin is blocking him from back-end.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26688.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26688.
@sudhir-j-sapkal sorry but I stilll dont understand why this is needed
It seems like there should be a more appropriate "Blocked user X tried to log in" log message, not no log message at all. I'm with Brian here, I don't get what cognitive benefit this change is adding.
alikon
- | Status | Ready to Commit | ⇒ | Needs Review |
| Labels |
Added:
Information Required
|
||
Closing for reason stated.
| Status | Needs Review | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-01-03 14:10:36 |
| Closed_By | ⇒ | Quy | |
| Labels |
Added:
?
Removed: Information Required |
||
I have tested this item✅ successfully on e6655ea
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26688.