Skip to content
This repository has been archived by the owner on Jan 15, 2021. It is now read-only.

Update The-User-Behavior.md #28

Closed
wants to merge 1 commit into from
Closed

Conversation

djonatanb
Copy link
Contributor

I was a bit confused with this, I thought it would be possible to send the user a temporary password.

@@ -35,7 +35,7 @@ Configure::write('UserTools.Behavior', [/* Config goes here */]);
* **defaultRole**: Used for a role, default is null, enter a string if you want a default role
* **hashPassword**: Hash the password or not, default is true
* **userActive**: Use the `active` field or not. By default users are set to active.
* **generatePassword**: Generate and send the password instead of using token verification, default is false
* **generatePassword**: Generate and send the password instead of using token verification, default is false. Applies to password reset only.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not correct, it doesn't apply to the reset, in fact it's used to send the user an auto-generated password when you decide that you don't want to allow users to sign up with their own password.

        if ($options['generatePassword'] === true) {
            $password = $this->generatePassword();
            $entity->{$this->_field('password')} = $password;
            $entity->clear_password = $password;
        }

@burzum burzum closed this Jan 28, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants