Default values

[tamtakoe]

Often I need to set custom message (or other fields) for all validators. Now I have to write every time:

myField1: {
    email: {
        message: 'invalidEmail'
    }
}
...

myField2: {
    email: {
        message: 'invalidEmail'
    }
}
...

It is better:

validators.defaults.email.message = 'invalidEmail';
...

myField1: {
    email: true
}
...

myField2: {
    email: true
}

[ansman]

Great idea! I'll make sure to implement this when I'm back from my vacation

On Fri, Dec 26, 2014 at 6:08 PM, tamtakoe notifications@github.com
wrote:

Often I need to set custom message (or other fields) for all validators. Now I have to write every time:

myField1: {
    email: {
        message: 'invalidEmail'
    }
}
...
myField2: {
    email: {
        message: 'invalidEmail'
    }
}
...

It is better:

validators.defaults.email.message = 'invalidEmail';
...
myField1: {
    email: true
}
...
myField2: {
    email: true
}

---

Reply to this email directly or view it on GitHub:
#16

[ansman]

Pushed an implementation of this in d06bd13

Please have a look and see what you think, you can just close the issue if you're satisfied :)

[tamtakoe]

Good! But it only for error params, not for options (f.e. dateOnly). There is best practice if you make object with default options (f.e. defaults) and merge it with options.

Using lodash defaults:

var defaults = {
    presence: ...
    ...
}
validators.defaults = defaults; //user can change it

...
//presence validator
_.defaults(options, defaults.presence);

[Jokero]

I think it should be possible to set default value of any validator option. So I agree with @tamtakoe 👍

[ansman]

Sure, sounds like a resonable feature request.

[ansman]

Done, now you should be able to specify both default messages and default options. Please try it out and let me know what you think.

[Jokero]

Thanks! 👍 I tried to use default messages and it's working.
Added comment 1b65319#commitcomment-9617489.
See please #10 (comment) (it's about promises)

[tamtakoe]

It works! it may be rename options to more popular defaults
https://github.com/twbs/bootstrap/blob/master/js/popover.js#L24
http://docs.ractivejs.org/latest/ractive-defaults
http://stackoverflow.com/questions/8905507/jquery-plugin-defaults-and-options
But it is not critical.

Thank you for your work!