Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem With email validation. #52

Closed
toome123 opened this issue May 21, 2013 · 5 comments
Closed

Problem With email validation. #52

toome123 opened this issue May 21, 2013 · 5 comments

Comments

@toome123
Copy link

After install login system I wanted to try to do register ,when I type my own email adress i receive : "Your email adress is not in a valid email format". Any idea how to fix that???

@toome123
Copy link
Author

Repeatedly tried different emails, But still same .... example@gmail.com in following formats.

@panique
Copy link
Owner

panique commented May 21, 2013

What exactly do you mean by "i receive" ? Have you changed something ? What's your PHP version ? What version of the script are you using ? Please give some more information... The email checker used here is the internal email checking method of PHP, so i think it's working perfectly! Are you sure that you haven't changed the $_GET stuff ? Can you debug this and show what exactly is given to the filter_var($email, FILTER_VALIDATE_EMAIL) ? I just tried this with the 1-minimal and the 2-advanced version and everything runs perfectly.

@blak3r
Copy link

blak3r commented May 21, 2013

@toome123 I had to change the HTML5 email validation.

As I reported in Issue #43:

change in views/register.php the login_input_email line from:

<input id="login_input_email" class="login_input" type="email" pattern="{3,64}" name="user_email" required /> 

to:

<input id="login_input_email" class="login_input" type="email" pattern="(.*){3,64}" name="user_email" required />

or remove the pattern altogether and rely on just server side validation.

@panique
Copy link
Owner

panique commented May 21, 2013

Okay, this has been fixed (in 1-minimal and 2-advanced (which can be found in the development branch now))

@panique panique closed this as completed May 21, 2013
@toome123
Copy link
Author

@blak3r this work.... Thanks for help. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants