Content-Length: 292976 | pFad | http://github.com/jquery-validation/jquery-validation/issues/2217

DD Issue with v1.18.0, IE and Required · Issue #2217 · jquery-validation/jquery-validation · GitHub
Skip to content

Issue with v1.18.0, IE and Required #2217

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

Closed
phenry2 opened this issue Sep 26, 2018 · 6 comments
Closed

Issue with v1.18.0, IE and Required #2217

phenry2 opened this issue Sep 26, 2018 · 6 comments

Comments

@phenry2
Copy link

phenry2 commented Sep 26, 2018

Summary of Issue

Since upgrading to v1.18.0, text boxes trigger a required validation error even though there is text in the text box. It does not happen in chrome. Rolling back to v1.17.0 fixed the issue.

Your environment

  • version of jquery-validate: v1.18.0
  • which browser and its version: Internet Explorer v11

Steps to reproduce

Simple MVC Core 2.1 application with a text box created using Html.TextBoxFor. The input element is generated with data-val="true". Html.ValidationFor used to generate the validation error.

Expected behaviour

Would expect no validation error when submit clicked. Normally then when typing in the field, the validation error cleared.

Actual behaviour

Validation error persists and cannot submit form.

Extra debugging revealed that when jquery-validate checked if the element was editable (isContentEditable), it then either did val = $element.text() or val = $element.val(). In the case of my text box, the isContentEditable flag was true, get $element.text() returned "". Hence the required validation failed. $element.val() return the actual contents of the textbox, but this was not used.

@worthy7
Copy link
Contributor

worthy7 commented Sep 26, 2018

Could perhaps be in #2053

@worthy7
Copy link
Contributor

worthy7 commented Sep 26, 2018

Just a note to whoever fixes this - please add a proper failing Required test if possible on this fix.

@worthy7
Copy link
Contributor

worthy7 commented Sep 26, 2018

(This is actually a massive bug, it will nuke all forms with Required, quick patch for NPM?)

@vijay18-91
Copy link

More details - The 'Required' functions 3rd parameter - param is always undefined even if the parameters are added while adding new validation methods which then called 'Depend' function with param and elements and returns "dependency-mismatch" for IE11 browser.

@sytheveenje
Copy link

Got the same issue, went back to v1.17.0 and everything works again.

@Arkni
Copy link
Member

Arkni commented Sep 30, 2018

Duplicate of #2211

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








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/jquery-validation/jquery-validation/issues/2217

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy