Coverage decreased (-6.4%) to 93.603% when pulling 098d444 on nishidayuya:add_array_support into ab5ad5a on ansman:master.

Hey guys, when can we expect this to be released? 😃

Is there any update on this?

+1

I think that instead return a object like:

{
   'ary[1].name': 'Cant be blank',
   'ary[1].somethikng': 'Cant be blank'
}

should be:

{
    ary: [
       { name: 'Cant be blank', something: 'Cant be blank' }
    ]
}

or:

{
    ary: {
       1: {
            name: 'Cant be blank',
            something: 'Cant be blank'
         }
    }
}

What you think?

+1

At first glance, without writing a custom formatter, it looks like the minimum we can do is the following.

length: {
  minimum: 1
}

which only checks length and a string would give us a false positive, when we're looking for an array. Is there a widely accepted array formatter pattern that people are using in the meantime?

Something that could validate this kind of pattern:

{
  "id": "123",
  "notes": [
    {
      "date": 2015-10-05T12:00:00,
      "comments": "monty python"
    },
    {
      "date": 2015-10-06T12:00:00,
      "comments": "something else entirely"
    }
  ]
}

definitely looking forward to this PR! thanks :)

edit: example custom validator to validate all values in an array for example above

custom validator:

validate.validators.arrayValidator = function(value, options) {
  if (!Array.isArray(value))
    return 'must be an array';

  value.forEach(function(obj) {
    validate(obj, options);
  });
};

schema:

  notes: {
    length: {
      minimum: 1
    },
    arrayValidator: {
      date: {
        datetime: true
      },
      comments: {
        length: {
          minimum: 10
        }
      }
    },
    presence: true

I'm so sorry. This PR slipped through the cracks it seems, it looks like a good first implementation. If you add some tests I'll try to get it merged ASAP.

@ansman Any update on when this will be released? Thank you.

+1

This would be pretty cool

Its some way how works with var constraints = {} like with array

I have trouble, when is in {} some element which not exists, code does not work

Original:
var constraints = { "addresses.shipping": { presence: true }, "addresses.shipping.street": { format: { // Must be numbers followed by a name pattern: "^[0-9]+ .+$", message: "^The street for the shipping address must be a valid street name" } } };

Will be like:
`
var constraints = {};

// dont care about "exists()"
if(exists("addresses.shipping")){
constraints.push(
"addresses.shipping": {
presence: true
}
);
}

// dont care about "exists()"
if(exists("addresses.shipping.street")){
constraints.push(
"addresses.shipping.street": {
format: {
// Must be numbers followed by a name
pattern: "^[0-9]+ .+$",
message: "^The street for the shipping address must be a valid street name"
}
}
);
}
`

@nishidayuya resolve merge conflicts when you have time :D

any update on when this will be released?

+1 when can we expect this?

I worked around this for now, but it would be great to have this baked in.