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

Major Bug In Jquery tools validator #115

Closed
emadhegab opened this Issue Jul 12, 2010 · 3 comments

Comments

Projects
None yet
2 participants
@emadhegab

I found a major bug in the validator in the attribute check

if i give field min attribute (e.g <input type="text" min="0" value="" name="age /> )

and i submit it give client side validation error cause it need at least 0 number!!!! even if it's not required..

the check on it must be if the value length of the field is equal to zero it should return true

thanks alot
Best Regards,
Mohamed Hegab
emadhegab@hotmail.com

@tipiirai

This comment has been minimized.

Show comment
Hide comment
@tipiirai

tipiirai Jul 22, 2010

Contributor

this is now fixed on github. see patch:

http://github.com/jquerytools/jquerytools/commit/10c7156994fff9dd08c6b4b13998e878f3a36df8

will be out in 1.2.4

THANKS! this was important

Contributor

tipiirai commented Jul 22, 2010

this is now fixed on github. see patch:

http://github.com/jquerytools/jquerytools/commit/10c7156994fff9dd08c6b4b13998e878f3a36df8

will be out in 1.2.4

THANKS! this was important

@emadhegab

This comment has been minimized.

Show comment
Hide comment
@emadhegab

emadhegab Jul 22, 2010

i'm happy that i've made a deffrince :)
just put in mind that the same issue is in the max attr as will
waiting for the fix

i'm happy that i've made a deffrince :)
just put in mind that the same issue is in the max attr as will
waiting for the fix

@tipiirai

This comment has been minimized.

Show comment
Hide comment
@tipiirai

tipiirai Jul 22, 2010

Contributor

you can see it from the patch that the max is also fixed.

Contributor

tipiirai commented Jul 22, 2010

you can see it from the patch that the max is also fixed.

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment