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

Password validation failure #52

Closed
PenzK opened this issue Dec 15, 2014 · 13 comments
Closed

Password validation failure #52

PenzK opened this issue Dec 15, 2014 · 13 comments
Assignees

Comments

@PenzK
Copy link

PenzK commented Dec 15, 2014

I have such an EditText:
@password
@SiZe(min = 6, message = "Enter at least 8 characters.")
EditText editTxtPassword;

this rule works, except the case if my pass long enough. When my edittext has string with 6 chars It returns emty error

@ragunathjawahar
Copy link
Owner

@PenzK v2 doesn't have all the rules implemented yet. @password is one of them. I'll keep the snapshots coming. Stable release will have all rules implemented.

@PenzK
Copy link
Author

PenzK commented Dec 15, 2014

You are fast ) I was editing my issue )

@ragunathjawahar
Copy link
Owner

Hi, you'll receive two errors for the same password field.

  1. Password rule returns false all the time and does not have a message yet. The empty message that you are seeing is from the Password annotation.
  2. The second error message will be the one from your Size annotation.

For now, I suggest you to remove the @password annotation.

@PenzK
Copy link
Author

PenzK commented Dec 16, 2014

Yes, removing @password did the trick

@ragunathjawahar
Copy link
Owner

@PenzK Hang on, we are nearing completion :)

@mehrad-rafigh
Copy link

Has this issue been fixed by a snapshot yet?

@ragunathjawahar
Copy link
Owner

@mehrad-rafigh not yet. But will be today.

@samar-moursi
Copy link

Hi
@ragunathjawahar @ConfirmPassword is not working as well. Will it be fixed soon?

Thanks.

@ragunathjawahar
Copy link
Owner

@samar-moursi Yes, both will be fixed soon. Meanwhile, you can use the 1.0.3 version if you need it immediately.

@javierdac
Copy link

Great work! Waiting for this fix :-)

@samar-moursi
Copy link

@ragunathjawahar Thanks a lot. Waiting for the fix :-)

@ragunathjawahar
Copy link
Owner

Feature added in 2e241c7, please check the latest snapshot.

@Password(min = 6, scheme = Password.Scheme.ALPHA_NUMERIC_MIXED_CASE_SYMBOLS)
private EditText mPasswordEditText;

@samar-moursi I'll create another issue for @ConfirmPassword.

@ragunathjawahar
Copy link
Owner

@PenzK You don't have to use @Size along with @Password. @Password now contains a min attribute.

@ragunathjawahar ragunathjawahar self-assigned this Dec 22, 2014
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

5 participants