Skip to content

Commit

Permalink
#503 元々、isValidは現在validな状態かどうか(結果待ちがあるならtrue、結果が待ちが無いなら今までのvalid結果)で、
Browse files Browse the repository at this point in the history
isAllValidが結果がまだ出ていないならnull、今までの結果でvalidでないならfalse、という扱いでした。
この挙動になるように、isValidにnullを持たせるようにした部分は元に戻して、isAllValidについてのコードを修正しました。
  • Loading branch information
fukudayasuo committed Dec 24, 2015
1 parent 8f7bdc9 commit 20b495c
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions hifive/src/main/webapp/src/h5.validation.js
Expand Up @@ -200,7 +200,10 @@
violation: [ev.violation]
};
}
this.validatingProperties.splice(this.validatingProperties.indexOf(name), 1);
this.validatingProperties.splice($.inArray(name, this.validatingProperties), 1);
if (!this.validatingProperties.length) {
this.isAllValid = this.isValid;
}
}

/**
Expand Down Expand Up @@ -1217,7 +1220,6 @@
properties.push(prop);
}
var isValid = !invalidProperties.length;
isValid = isValid && isAsync ? null : isValid;
var validationResult = new ValidationResult({
validProperties: validProperties,
invalidProperties: invalidProperties,
Expand All @@ -1227,8 +1229,8 @@
isAsync: isAsync,
// isValidは現時点でvalidかどうか(非同期でvalidateしているものは関係ない)
isValid: isValid,
// 非同期でvalidateしているものがあって決まっていない時はisAllValidはnull
isAllValid: isAsync ? null : isValid
// 非同期でvalidateしているものがあって現時点でisValid=falseでない(=全部OKかどうか決まっていない)時はisAllValidはnull
isAllValid: isAsync && isValid ? null : false
});

if (isAsync) {
Expand Down

0 comments on commit 20b495c

Please sign in to comment.