Skip to content

Commit 8226e48

Browse files
authored
Merge pull request feathersjs-ecosystem#470 from antarasi/patch-1
required() of value false throws fieldname is null
2 parents 08bcb91 + f9bbd68 commit 8226e48

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lib/services/required.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ module.exports = function (...fieldNames) {
1414
fieldNames.forEach(name => {
1515
if (!existsByDot(item, name)) throw new errors.BadRequest(`Field ${name} does not exist. (required)`);
1616
const value = getByDot(item, name);
17-
if (!value && value !== 0) throw new errors.BadRequest(`Field ${name} is null. (required)`);
17+
if (!value && value !== 0 && value !== false) throw new errors.BadRequest(`Field ${name} is null. (required)`);
1818
});
1919
});
2020
};

0 commit comments

Comments
 (0)