FrontEndUsers bugs and fixes (when email is username)

[Nick Smart]

I've been setting up FrontEndUsers 1.5.2 on CMSMS 1.4.1 and selected the option to use the email address as the username. I've found and, I think, solved a couple of bugs with the lost password facility.

Using the lost password facility and entering a valid and registered email address gives the error:

Warning: Parameter mesage is not known... dropped in /home/xxx/public_html/lib/misc.functions.php on line 1301

This is caused by a spelling mistake in modules/FrontEndUsers/action.do_forgotpw.php on line 150. It should read:

Code: Select all

$params['message'] = $this->Lang('error_couldnotfindemail');

i.e. "message" not "mesage".

Trying the lost password again now returns the error:

Could not find an email address

This error is returned because the wrong column name is specified in modules/FrontEndUsers/FrontEndUsers.api.php on lines 943/944. They should read:

Code: Select all

$q = 'SELECT username FROM '.cms_db_prefix().
'module_feusers_users WHERE id = ?';

i.e. column "id", not "userid".

The lost password form now sends an email that provides a link to allow a new password to be selected.

Nick

[doodle]

Thanks, great help...

[sn3p]

Thanks a lot Nick.

After these fixes I get the "We could not find an email address for this account" error when trying to recover a password. This is because I don't have an "E-mail Address" type user property, but I don't understand the logic of creating an extra e-mail field when I've set "email is username".

Am I missing something or is there no other way then to create an e-mail field?

I'm still using CMSMS 1.4.1 and FEU 1.4.4 because I can't upgrade after modifying some core files.

[Jeff]

Sorry but you are using old versions and theses bugs have like already been fixed.

Please read the Forum rules we don't support old versions. If you are modifying core files then it is assume that you can handle your own support and can find the bugs and fix them.

[sn3p]

ajprog
I'm aware of that. I'm merely asking the community in helping me understand the module's logic.
Sorry if that means I'm breaking forum rules..

Anyway, I found my solution so I marked my post as [SOLVED].