strict: true for embedded documents

[timfinity]

I've noticed that strict: true doesn't work for embedded documents. I wasn't sure if this was by design or not.

A quick example:

var Comments = new Schema({
    title     : String
  , body      : String
  , date      : Date
}, {strict: true});

var BlogPost = new Schema({
  , title     : String
  , body      : String
  , comments  : [Comments]
}, {strict: true});

var Post = mongoose.model('BlogPost', BlogPost);

var post = new Post();

post.title = 'Saves';
post.fake = 'Doesnt save';

post.comments.push({
  title: 'Saves',
  fake: 'Shouldnt save but does'
});

post.save(function(err){

});

[aheckmann]

well that sucks

[albrow]

An important note:

You have to put child schema before the parent schema for strict mode to work properly with subdocuments. For this example, that means putting the Comments declaration before the BlogPost declaration. If you don't, strict mode will not work for subdocuments.

Hopefully this will help people avoid the same problem I just ran into. Perhaps a note in the documentation would be helpful? In general, javascript doesn't care what order you declare functions and classes, so I found this behavior a little surprising.

[offminded]

So it seems mongoose is setting strict mode to true for all but for some reason it is failing to enforce it when it comes to subdocuments.

Even when I put child schema before parent schema it is not working for me, I guess I will post an issue with an example.

[vkarpov15]

@offminded please do