Merge pull request christianalfoni#242 from easyrider/master

Semigradsky · Semigradsky · commit cfebf17aea99 · 2015-11-03T14:34:45.000+03:00
Fix: Form.validationErrors disables input validation christianalfoni#239
diff --git a/src/main.js b/src/main.js
@@ -83,7 +83,7 @@ Formsy.Form = React.createClass({
 
   componentDidUpdate: function () {
 
-    if (this.props.validationErrors) {
+    if (this.props.validationErrors && typeof this.props.validationErrors === 'object' && Object.keys(this.props.validationErrors).length > 0) {
       this.setInputValidationErrors(this.props.validationErrors);
     }
 
diff --git a/tests/Element-spec.js b/tests/Element-spec.js
@@ -295,6 +295,29 @@ export default {
 
   },
 
+  'should not override error messages with error messages passed by form if passed eror messages is an empty object': function (test) {
+
+    const TestForm = React.createClass({
+      render() {
+        return (
+          <Formsy.Form validationErrors={{}}>
+            <TestInput name="A" validations={{
+              isEmail: true
+            }} validationError="bar2" validationErrors={{isEmail: 'bar3'}} value="foo"/>
+          </Formsy.Form>
+        );
+      }
+    });
+    const form = TestUtils.renderIntoDocument(<TestForm/>);
+
+    const inputComponent = TestUtils.findRenderedComponentWithType(form, TestInput);
+    test.equal(inputComponent.getErrorMessage(), 'bar3');
+
+    test.done();
+
+  },
+
+
   'should override all error messages with error messages passed by form': function (test) {
 
     const TestForm = React.createClass({

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ Formsy.Form = React.createClass({`
`83`	`83`
`84`	`84`	`componentDidUpdate: function () {`
`85`	`85`
`86`		`- if (this.props.validationErrors) {`
	`86`	`+ if (this.props.validationErrors && typeof this.props.validationErrors === 'object' && Object.keys(this.props.validationErrors).length > 0) {`
`87`	`87`	`this.setInputValidationErrors(this.props.validationErrors);`
`88`	`88`	`}`
`89`	`89`