View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0019446||phpList 3 application||Sub-admins||public||05-10-18 14:01||12-10-18 17:31|
|Product Version||3.3.5 - RC1|
|Target Version||Fixed in Version||3.3.5|
|Summary||0019446: Inconsistencies with the new admin password function|
|Description||The form to enter a new admin looks a bit confusing|
1) "Send email" is by default selected for "choose how to set password" but fields are displayed for create password and confirm password. Those should be hidden until the appropriate radio button is selected.
2) The captions "Create password" and "confirm password" are in bold, which is inconsistent with all the other captions. The word "Create" is probably unnecessary because none of the other captions have it (e.g "Email" not "Create email").
3) The list of enhancements explains
"New option to specify the password of a new sub-account during the account creation process"
but entering a password directly seems to apply to a super admin as well.
4) There appears not to be any validation that the password has actually been entered. If I select the "create password" radio button and then submit then the new admin is added. That should really be rejected.
5) There is some sort of validation that the two passwords are the same but that appears not to be working properly. It displays "Not matching" after I entered the same value in the two fields.
|Tags||No tags attached.|
I suppose you haven't been testing it from sourceforge (https://sourceforge.net/projects/phplist/files/phplist-development/3.3.5-RC1/ ) but from phpList3 repo on GitHub and you haven't pulled the latest changes on Dressprow (https://github.com/phpList/phplist-ui-dressprow/ ). There are some js changes that are made in order for that feature to work as you are describing it should.
Regarding 2 is a just aesthetic issue, maybe @suela @samtuke have some thoughts.
I installed the release file from the link to SourceForge.
But I found that I had to refresh (F5 or ctrl-F5) to make the browser reload all files. Now, as you say, the password fields are hidden unless "create password" is selected.
I suggest that some guidance is added to the release information about needing to do the refresh.
||We should pull the CSS and JS assets using something like ?v=VERSION to avoid them being cached between versions. That should solve the issue|
Dressprow does that:
But bootlist doesn't:
This should resolve that https://github.com/phpList/phplist-ui-bootlist/commit/6a7cea175f3a876c1b393703da37eaf4eb9313b7
As for the actual admin creation function, I think it works ok to allow switching between entering the password, or having it sent by email.
||This can be marked resolved in 3.3.5 - RC2|