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

login error msg is not accurate #220

Open
jmikerq opened this issue Mar 16, 2018 · 1 comment
Open

login error msg is not accurate #220

jmikerq opened this issue Mar 16, 2018 · 1 comment

Comments

@jmikerq
Copy link

jmikerq commented Mar 16, 2018

In the case where I use only email to login, and if the password doesn't match, the error msg return is:

aauth_error_login_failed_all: E-mail, Username or Password do not match.

you can find this at line 298 in the stable version

$this->error($this->CI->lang->line('aauth_error_login_failed_all'));

I think the login function should check whether user is using email only, user only, or allowing both, and then return the correct error msg.

In my case, it should return

$lang['aauth_error_login_failed_email'] = 'E-mail Address and Password do not match.';

@REJack REJack self-assigned this Mar 18, 2018
@REJack
Copy link
Collaborator

REJack commented Mar 18, 2018

I agree with this but i gone change this in v3.0.0

@REJack REJack added this to the Aauth v3.0.0 milestone Mar 18, 2018
@REJack REJack removed their assignment Mar 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants