Skip to content

Commit ce14c51

Browse files
authored
Merge pull request #4397 from jyrkive/required-number-with-default
Fix: model with required field that defaults to 0 can't be saved
2 parents eb7ef63 + 12de7af commit ce14c51

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

fields/types/boolean/BooleanType.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ boolean.prototype.validateInput = function (data, callback) {
3636

3737
boolean.prototype.validateRequiredInput = function (item, data, callback) {
3838
var value = this.getValueFromData(data);
39-
var result = (value && value !== 'false') || item.get(this.path) ? true : false;
39+
var result = (value && value !== 'false') || typeof item.get(this.path) === 'boolean' ? true : false;
4040
utils.defer(callback, result);
4141
};
4242

fields/types/number/NumberType.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ number.prototype.validateInput = function (data, callback) {
3939
number.prototype.validateRequiredInput = function (item, data, callback) {
4040
var value = this.getValueFromData(data);
4141
var result = !!(value || typeof value === 'number');
42-
if (value === undefined && item.get(this.path)) {
42+
if (value === undefined && typeof item.get(this.path) === 'number') {
4343
result = true;
4444
}
4545
utils.defer(callback, result);

0 commit comments

Comments
 (0)