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

Make Email Adresse check case-insensitiv #6

Closed
Spegeli opened this issue Dec 19, 2016 · 2 comments
Closed

Make Email Adresse check case-insensitiv #6

Spegeli opened this issue Dec 19, 2016 · 2 comments
Assignees

Comments

@Spegeli
Copy link

Spegeli commented Dec 19, 2016

As the title say, currently its case-sensitiv.
So when i send Emails to maxmuster@mydomain.de and i check via the Website for Maxmuster@mydomain.de it not found the Emails.

Would be better to make it case-insensitiv.
So it no matters in i search for maxmuster, MaxMuster or MaXmUsTeR.

@Spegeli Spegeli changed the title Check Email Adresse is case-sensitiv Make Email Adresse check case-insensitiv Dec 19, 2016
@Spegeli
Copy link
Author

Spegeli commented Dec 19, 2016

I found out that email adresses are always lowercase and here is the Solution for the Problem:
In the index.html change:

                    <input ng-model="$ctrl.inputFieldUsername"
                           placeholder="username"
                           type="text" class="form-control"/>

to

                    <input ng-model="$ctrl.inputFieldUsername"
                           placeholder="username"
                           type="text" class="form-control"
                           ng-change="$ctrl.inputFieldUsername=$ctrl.inputFieldUsername.toLowerCase();" />

this will auto convert the input in the Username Field to lowercase ;-)
This is normaly enough but i would change in the backend.php
$name = clean_name($username);
to
$name = strtolower(clean_name($username));
in case your doing a check via /backend.php?action=get&username=MaxMuster
so it will auto convert it to lowercase too

@synox synox self-assigned this Dec 19, 2016
synox added a commit that referenced this issue Dec 19, 2016
@synox
Copy link
Owner

synox commented Dec 19, 2016

fixed.

@synox synox closed this as completed Dec 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants