-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[5.3] Fixed manual Unblock sets user as Activated (Verified) issue #45118
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 5.3-dev
Are you sure you want to change the base?
[5.3] Fixed manual Unblock sets user as Activated (Verified) issue #45118
Conversation
Co-authored-by: Richard Fath <[email protected]>
5b43327
to
1edeac1
Compare
1edeac1
to
7330d11
Compare
Your assumptions are wrong. If the site is set to no verfication then the user is marked as activated on registration As stated before - until all the possible registration scenarios are documented it is not a good idea to change anything here |
Hi @brianteeman sir, |
@richard67 |
The Summary of Changes looks very structured and clear. Compliments and many thanks for this. Not all PRs are so well documented, which makes it often difficult for testers to get the perspective. I am the person who originally reported this issue. But I am afraid, my request was somewhat misunderstood. My concern is NOT about the manual addition of users by the administrator. I wanted to discuss whether the user registration process should be improved in case the option ‘New User Account Activation’ is set to ’Administrator’. IS
MY PROPOSAL
Summarising I would think, that my propsal would make the whole user registration and approval process more transparent and clear then it was until now. BY NO MEANS is it necessary, to completely rewrite this function from the scratch. It basically works, but could be clarified a little bit. |
Thank you for the clarification and the detailed explanation of your proposal sir . I now better understand that your concern is specifically about improving transparency in the ‘New User Account Activation: Administrator’ scenario rather than manual user addition. I also acknowledge the importance of fully documenting all possible registration scenarios before making any changes, as suggested by @brianteeman . To move forward productively, would it be helpful if I first work on a draft documentation of the different registration scenarios, including the current behavior and potential improvements? Once we have a well-structured document covering all cases, we could then reassess whether the proposed change aligns with Joomla’s existing logic and best practices. Please let me know if this approach makes sense or if there are any specific guidelines I should follow for documenting these scenarios. Looking forward to your thoughts sir |
@Amitesh007z I appreciate very much your suggestion «draft documentation of the different registration scenarios, including the current behavior and potential improvements». Maybe there already exists a documentation? Or am I the one who should work on this? |
In mattermost chat ,Mr.Emmanuel L. asked me to check once in https://manual.joomla.org/ for any documentation, Looking forward to your thoughts sir |
@Amitesh007z Very good :-) I would be happy to review your documentation at the appropriate time. Please let me know when you are ready. |
@Amitesh007z Any progress with the documentation yet? I look forward to reviewing your documentation at the appropriate time. Please let me know when you are ready. |
@crimle , yes sir its already done , just formatting is to be done , will share the draft now sir , Sorry for the delay |
@crimle This document outlines the possible user registration processes in Joomla and includes a summary table at the end to compare different registration types sir . I would appreciate your feedback on any improvements or corrections needed. Looking forward to your thoughts sir , Thanks in advance , Also sir ,On making this , I noticed a discrepancy in the Admin-User Activation process is that email activation is not utilized, as the Enabled and Activated statuses are solely controlled by the admin. I think there is an inconsistency on this activation logic compared to others . Would be happy to hear your thoughts sir , Sorry for the delay sir |
Pull Request for Issue #45063 .
Summary of Changes:
This PR fixes an issue where manually unblocking a user does not set them as Activated (Verified). Previously, when an admin unblocked a user, the activation state remained unchanged, which could cause login issues. This fix ensures that when an admin unblocks a user, their activation status is also updated to Verified.
Testing Instructions
Testing Instructions
Log in to the Joomla administrator panel.
Navigate to Users → Manage Users.
Now new create account ,
traverse back to manage users panel and check unblocking (it must eventually activate respective user which happens when the same user verifies mail and admin verifies mail)
Actual result BEFORE applying this Pull Request
above image is result of manual unblock by admin where activation is red
above image is result of other method i.e through mail verification of user and admin .
Expected result AFTER applying this Pull Request
on manual click on unblock (i.e manually unblocking), gives green to activation too, which results in consistency.
Link to documentations
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed