attributeDescription should be per-field, not per-validator

[lolmaus]

It is burdensome to pass attributeDescription to every validator of a field:

const Validations = buildValidations({
  name: [
    validator('presence', {
      presence: true,
      attributeDescription: 'Course name (language)'
    }),
    validator('length', {
      min: 1,
      attributeDescription: 'Course name (language)'
    }),
    validator('no-whitespace-around', {
      attributeDescription: 'Course name (language)'
    })
  ]
});

It is obvious that conceptually attributeDescription is an entity that belongs to the whole field and should be shared among all validators.

I suggest the following API:

/**
 * Top level method that will ultimately return a mixin with all CP validations
 * @param  {Object} validations  Validation rules
 * @param  {Object} commonOptions  Options to be includd into every validator unless defined explicitly
 * @return {Ember.Mixin}
 */
export default function buildValidations(validations) {

which would drastically simplify the above example:

const Validations = buildValidations({
  name: [
    validator('presence', true),
    validator('length', { min: 1 }),
    validator('no-whitespace-around')
  ]
},
{
  attributeDescription: 'Course name (language)'
});

[offirgolan]

Thanks for the suggestion. I definitely had something like this in my head for future implementation. Ill take a look into this more when I have some time.

[offirgolan]

This will be available once we release 2.0! Thanks again for the suggestion 😄