Skip to content

[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

Open
wants to merge 4 commits into
base: 5.3-dev
Choose a base branch
from

Conversation

Amitesh007z
Copy link

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

image
above image is result of manual unblock by admin where activation is red


image

above image is result of other method i.e through mail verification of user and admin .

Expected result AFTER applying this Pull Request

image
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

@Amitesh007z Amitesh007z force-pushed the fix-admin-unblock-activation branch from 5b43327 to 1edeac1 Compare March 12, 2025 20:29
@Amitesh007z Amitesh007z force-pushed the fix-admin-unblock-activation branch from 1edeac1 to 7330d11 Compare March 12, 2025 20:42
@brianteeman
Copy link
Contributor

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

@Amitesh007z
Copy link
Author

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,
I’ve made the changes to address the inconsistency as instructed by MacJoom while ensuring the registration process remains unaffected sir . I really appreciate your guidance on this sir . Please let me know if there’s anything I might have missed or if further adjustments are needed sir . Happy to make any necessary changes

@richard67 richard67 requested review from richard67 and brianteeman and removed request for brianteeman and richard67 March 13, 2025 15:17
@Amitesh007z
Copy link
Author

Amitesh007z commented Mar 13, 2025

@richard67
i would be happy to hear further review and feedback sir , also further steps to be done sir

@crimle
Copy link

crimle commented Mar 13, 2025

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

  • A user registers for a new account
  • He clicks the verification link in the email he receives
  • This action does not change the state of the two icons, neither «Enabled» nor «Activated». Literally speaking this is correct, since the user only has verified his email address, but not activated his account. Keep in mind that the option ‘New User Account Activation’ is set to ’Administrator’.

MY PROPOSAL

  • A user registers for a new account
  • He clicks the verification link in the email he receives
  • This action should change the state of the second of these icons. This allows the administrator to determine that the account has been verified. Otherwise he would have to guess the verification status from the fact, that he has received a notification email from the new user account.
  • My proposal would make it necessary that the column's heading is renamed from «Activated» to «Verified».

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.

@Amitesh007z
Copy link
Author

As stated before - until all the possible registration scenarios are documented it is not a good idea to change anything here

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

@crimle
Copy link

crimle commented Mar 14, 2025

@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?

@Amitesh007z
Copy link
Author

Amitesh007z commented Mar 14, 2025

@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,
I can start looking into Joomla’s official documentation and any related discussions to see if this has already been documented. If nothing comprehensive exists, I’d be happy to contribute by drafting a structured document outlining the current behavior and potential improvements, If you happen to know of any references or previous discussions that might be useful as a starting point? That would definitely help in making sure we don’t miss any important details sir

Looking forward to your thoughts sir

@crimle
Copy link

crimle commented Mar 14, 2025

@Amitesh007z Very good :-) I would be happy to review your documentation at the appropriate time. Please let me know when you are ready.

@crimle
Copy link

crimle commented Apr 2, 2025

@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.

@Amitesh007z
Copy link
Author

@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

@Amitesh007z
Copy link
Author

Amitesh007z commented Apr 2, 2025

@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
Joomla User Registration Scenarios Documentation.docx

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
Thanks

@HLeithner HLeithner changed the title [5.3]Fixed manual Unblock sets user as Activated (Verified) issue [5.3] Fixed manual Unblock sets user as Activated (Verified) issue Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants