Hi,
I hide the password field fron non-admins. In D7 I still get a field "Current Password" visible.
I get Password and Confirm password for admin.

CommentFileSizeAuthor
#5 user_readonly-1356844-3.patch833 bytesfilsterjisah
#4 user_readonly-1356844-2.patch854 bytesfilsterjisah
#3 user_readonly-1356844.patch854 bytesfilsterjisah

Comments

sw3b’s picture

sw3b commented

same for me !

FNGR’s picture

FNGR commented

same here!

filsterjisah’s picture

filsterjisah commented
StatusFileSize
new user_readonly-1356844.patch854 bytes

This current password field is used for when a user changes some fields in his own account: I think e-mail and password. Because the form is loading as anonymous user, this field is never included in the admin settings.

Please try attached patch.

filsterjisah’s picture

filsterjisah commented
StatusFileSize
new user_readonly-1356844-2.patch854 bytes

Fixed typo user_Load()

filsterjisah’s picture

filsterjisah commented
StatusFileSize
new user_readonly-1356844-3.patch833 bytes

and removed dpm() .... third time ftw

deekayen’s picture

deekayen commented
Status: Active » Needs review

This wasn't marked to need review, so it got missed.

rdrh555’s picture

rdrh555 commented
Status: Needs review » Reviewed & tested by the community

Confirmed "Current Password" field still visible/enabled when configured to be hidden (using "pass" in admin settings). Only the "pass" field is available in admin settings prior to patch. After applying patch #5 current_pass_required_values & current_pass available in admin settings. Configuring current_pass allows disabling/hiding of "Current Password" field for non-admin users.

  • deekayen committed 591fe12 on 7.x-1.x authored by filsterjisah 
    Issue #1356844 by filsterjisah, SangersDrupalDude, sw3b, FNGR, deekayen...
deekayen’s picture

deekayen commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.