Consider whitespace-only fields to fail `required` validation? about backbone-validator HOT 5 CLOSED

To implement this, I have added a new validator notBlank to my project:

Validator.add('notBlank', function(value, expectation) {
    return expectation === false || (_.isString(value) && !value.match(/^[\s\t\r\n]*$/));
}, "Can't be blank");

This matches all whitespace and newline characters. It can be used instead of required: true, as it implies that the string must be present.

I think this is enough to satisfy my use case and keeps required: true as-is, so I'll close this. However, how do you feel about a notBlank validator in the core? It seems like a reasonably common use case.

from backbone-validator.

@fiznool Reasonable validation. I've seen that some validators removes leading/trailing spaces before running validation, but it would be weird:

model.set("name", "     Joe    ", { validate: true });
// validation will run for trimmed value - "Joe", 
// but model will be set with white-spaced value "     Joe    "

So from my point your approach is way better. I'll see if there's a good way to have that as a part of core validators.
Do you think it verbose enough to have { blank: false }?

Thanks!

from backbone-validator.

I'd like to take credit, but all I've really done is 'borrow' the validation from node-validator...

Let me know if you'd like a pull request with the above validator, more than happy to help.

from backbone-validator.

Yeah, it would be great to have PR if possible. Make sure to add couple specs, should be simple: https://github.com/fantactuka/backbone-validator/blob/master/spec/backbone-validator-spec.js#L50

from backbone-validator.

FYI, v0.2.1 contains blank validator, that checks String, Array and Object for emptiness (string is trimmed before validation)

from backbone-validator.