Password reset form is case sensitive for email address

Hello Danny.Wouters,

Nice catch ! Thanks a lot for providing a patch to.

I need time to test this but I guess this patch is not the right one : the

variable_get('dbee_loggin_email', 0)

expression should not be checked here.
And I need to check if the patch should handle specific stuff about compatibility with Email registration and login tobogan modules.

Ok Here is the right patch.
I have added new simpleTests for sending email in case of lost password.

Fixed into the dbee module v7.x-2.3.
Thank you Danny.Wouters for reporting this issue.

Note : the v6x version meets this bug to. A patch for the D6 version is needed.
The v6x version does not meet this issue.
This issue only concerned users with sensitive case email, trying to reset their password. As a workaround for previous versions those users could succeed in reseting their password providing their username ou their exact sensitive case email.

Extra note to the above comment (#7): the notice might be caused by the fact that dbee_extract unsets the dbee_*-fields, which are checked when doing the user_load_multiple-call.

ps. when using randomName, you can't be certain the name/email contains upper and lower case characters :)

Oups, I have released this commit to early.
Not so big deal, it's only an Notice error. I change the priority of this issue to "Minor".
According to your advises, I'm going to update the tests, making absolutly sure the tested random name will contain both uppercase and lowercase characters.

For the record : In a perfect world, the form for reseting the password should not need be altered from the dbee module. The dbee_query_alter() function should take care of matching datas regarding the encryption. Unfornatelly, the User core functions may need some fixes. The User_load_multiple function does not handle natively sensitive case emails but thanks to MySQL collation (where is it set ??), it succeed to load a user from a sensitive case email (see this post and its answer). This behavior corresponds to an opened issue for postgreSQL.
In the future, depending on changes on the user load core function, the changes on the reseting password form in the dbee module may be deleted.

I am not able to reproduce the notice error : Notice: Undefined property: stdClass::$dbee_mail in DrupalDefaultEntityController->cacheGet() (line 414 of /includes/entity.inc).
It would help me if you provide the whole backtrace ("called fom etc..., called from etc...").

I provide here a new patch, that handle the sensitive case email issue, replicating the MySQL collation (I am not altering the user_pass_form anymore).
Tell me if the new patch fixes the notice error please.

Fixed into the dbee module v7.x-2.4.
Thank you Danny.Wouters and SpadXIII for your help.