Don't check uniqueness of blank values
This commit is contained in:
parent
2dfef8cc7f
commit
5f1431aa05
|
@ -305,6 +305,7 @@ function validateCustom(attr, conf, err, done) {
|
||||||
* Uniqueness validator
|
* Uniqueness validator
|
||||||
*/
|
*/
|
||||||
function validateUniqueness(attr, conf, err, done) {
|
function validateUniqueness(attr, conf, err, done) {
|
||||||
|
if (blank(this[attr])) return done();
|
||||||
var cond = {where: {}};
|
var cond = {where: {}};
|
||||||
cond.where[attr] = this[attr];
|
cond.where[attr] = this[attr];
|
||||||
|
|
||||||
|
|
|
@ -227,6 +227,21 @@ describe('validations', function () {
|
||||||
done(err);
|
done(err);
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('should skip blank values', function (done) {
|
||||||
|
User.validatesUniquenessOf('email');
|
||||||
|
var u = new User({email: ' '});
|
||||||
|
Boolean(u.isValid(function (valid) {
|
||||||
|
valid.should.be.true;
|
||||||
|
u.save(function () {
|
||||||
|
var u2 = new User({email: null});
|
||||||
|
u2.isValid(function (valid) {
|
||||||
|
valid.should.be.true;
|
||||||
|
done();
|
||||||
|
});
|
||||||
|
});
|
||||||
|
})).should.be.false;
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
describe('format', function () {
|
describe('format', function () {
|
||||||
|
|
Loading…
Reference in New Issue