Knockout-Validation icon indicating copy to clipboard operation
Knockout-Validation copied to clipboard

validation may not work when relying only on parseInputAttributes

Open crissdev opened this issue 11 years ago • 0 comments

When parseInputAttributes: true and the observable is not validatable the input does not get a validation message inserted, nor the validation class validationElement.

The problem is in the validationCore binding as it asynchronously executes parseInputValidationAttributes. The later method will make the observable validatable if it matches some criteria (eg. input type is number/email, has a required attribute etc.)

Solutions:

  • execute parseInputValidationAttributes synchronously - this adds some overhead when bindings are applied.
  • duplicate code in the validationCore binding to ensure config.insertMessages and config.decorateInputElement are applied if the observable becomes validatable.

I've created a fiddle for this - it has both the solutions and some intructions on how to fix it. The interesting code starts at line 1016 in the JavaScript panel of JSFiddle.

http://jsfiddle.net/3594jm1o/

Any thoughts?

crissdev avatar Jan 24 '15 12:01 crissdev