I have theses options checked in CAS Settings and they are not removing the fields from user view.

CommentFileSizeAuthor
#3 cas-282917.patch866 bytesmetzlerd

Comments

fillo’s picture

fillo commented

For now I just installed the User Protect module and was able to hide these fields.

Hannu’s picture

Hannu commented

I confirm that this bug (password and email fields are editable even if hide checkboxes are checked) exists, and that the User Product module is a workaround.

metzlerd’s picture

metzlerd commented
Version: 6.x-1.0 » master
Status: Active » Needs review
StatusFileSize
new cas-282917.patch866 bytes

Isolated the bug.... Yep, broken in 6.x here's the fix. Review/tests welcome. Will likely commit to head today, but would love to see tests.

metzlerd’s picture

metzlerd commented
Assigned: Unassigned » metzlerd

Woops forgot to assign.

Hannu’s picture

Hannu commented

Would like to test, but I have the patch mentioned in http://drupal.org/node/176841#comment-946191 applied (as it fixes an earlier problem), and this new patch does not want to go together with the earlier patch.

Also, I made a typo in #2. Meant to say "User Protect" instead of "User Product". Dunno what I was thinking. :)

metzlerd’s picture

metzlerd commented
Status: Needs review » Fixed

Since this is such a minor change and an important bug fix, I'm going to go ahead and commit. Hannu, I'll turn my attention to that issue soon. And make additional comments on it.

Status: Fixed » Closed (fixed)

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

  • Commit c774c64 on 6.x-2.x, 6.x-3.x, 7.x-1.x, master, 8.x-1.x by metzlerd: 
    #282917 Fixed bug - password and email hide functionality broken.
  • Commit 77610cf on 6.x-2.x, 6.x-3.x, 7.x-1.x, master, 8.x-1.x by metzlerd:
    Again bad commits #384756 Add persistent login support by danep #282917