geddy / model

Datastore-agnostic ORM in JavaScript
265 stars 55 forks source link

object datatype validate() now parses JSON strings #124

Closed der-On closed 10 years ago

der-On commented 10 years ago

As object.serialize() does a JSON.stringify() it must also do a JSON.parse() on validation to be able to load JSON strings from SQL adapters.

mde commented 10 years ago

Could you include some tests for this? I'll be happy to merge it.

der-On commented 10 years ago

Here we go with a unit test.

ben-ng commented 10 years ago

Awesome, this looks like a fix for #107 ?

Will merge in once travis goes green.

ben-ng commented 10 years ago

Merged, thanks! I needed this one too.