BaseModel.find callback issue about hapi-mongo-models HOT 5 CLOSED

Yeah we need that callback to be sent through the resultFactory. Would you be willing to create a gist that reproduces the error so I can help debug?

from hapi-mongo-models.

Yes, of course, I’ll try to put that together today or tomorrow. Thx for the reply!

from hapi-mongo-models.

I prepared a repo instead, with two example calls which reproduce the issue: https://github.com/edimoldovan/hapi-mongo-models-example

If you do this:

curl -X POST -H "Content-Type: application/json; charset=utf-8" "127.0.0.1:8000/api/users" -d '{"email":"[email protected]","password":"bla","name":"name”}’

a user will be inserted, but not returned in the response. Also, if you hit this url:

http://localhost:8000/api/users

you will see that the data array contains the correct number of objects, only they are empty.

If I do the change I mentioned above, all this suddenly becomes ok.

from hapi-mongo-models.

Thanks so much. That made it much easier to help figure out why the objects were empty for you.

Your model was missing a constructor. When we create a new instance of a model we pass the instance data into it's constructor.

Here is what I did to fix your example.

$ npm install object-assign --save

Then in your model I added a constructor:

var Joi = require('joi');
var BaseModel = require('hapi-mongo-models').BaseModel;
var ObjectAssign = require('object-assign');

var User = BaseModel.extend({
    constructor: function (attrs) {

        ObjectAssign(this, attrs);
    }
});

User._collection = 'users'; // the mongo collection name

User.schema = Joi.object().keys({
    name: Joi.string(),
    image: Joi.string(),
    password: Joi.string().required(),
    email: Joi.string().email().required()
});

module.exports = User;

Then both GET and POST worked as expected.

from hapi-mongo-models.

Works very well, thank you for your help. The readme update would also be welcome, thate is where I copied my code from. Suppose will be done later.

from hapi-mongo-models.