QUESTION: Why remove "Request Password" link?
ANSWER: So internal users logging in with their LDAP account don't get the option to change the password.
But what about the rest, i still want external users to be able to create accounts and request new passwords.
Cant we allow users to request a new password, but if there account is associated with an LDAP account then just display a user configurable) message?
This is a problem for me because i want to add the ability for LDAP users to login to our external website, but i don't want to loose any functionality (creating accounts, requesting new passwords) for external non LDAP users.
Comments
Comment #1
peter panes commentedActually i think this is a bug because, i only have two choices as follows:
Allows LDAP users to change their password, except it doesn't work.
Hide change password fields for LDAP Users, but then non LDAP users cant create new accounts or request new passwords.
Is there something I'm missing or is this how its supposed to work?
Comment #2
shadybar commentedWe're having the same issue with accommodating public users and employees. Has there been any movement with this issue?
Comment #3
verta commentedsubscribing
Comment #4
bwinett commentedsubscribing
Comment #5
Anonymous (not verified) commentedI made this patch to solve the problem, hope it is usefull
Comment #6
johnbarclay commentedin the ldap project for drupal 7, I tried to do this in the theming and form level and it became a pain. Some of it worked better in the menu layer. (Below is the code I used to hide/show the password reset link.) For example, by changing the access callback for the user/password link, I can show or hide the link and the form based on whether ldap exclusive or mixed mode is selected.
Here are the use cases and behavior I'm looking at for user forms and link alterations in ldap for drupal 7. I would welcome feedback in the project/ldap issue queue
Comment #7
did1979 commentedHello,
I am on a drupal 6.20 verion, with the uptodate ldapauth module.
how works your patch. I tried to install it but nothing changes.
Comment #8
johnbarclay commentedThis is committed (patch on #5).