Problem trying to post a record to a nested collection

[simax]

I'm having a problem trying to post a record to a nested collection

This is my schema:

var mongoose = require('mongoose'),
    Schema = mongoose.Schema,
    objectId = mongoose.Schema.ObjectId;

var EmployeeSchema = new Schema({
    firstname: {
        type: String,
        required: true,
        trim: true
    },
    lastname: {
        type: String,
        required: true,
        trim: true
    },
    email: {
        type: String,
        required: true,
        trim: true,
        index: true,
        validate: /\b[a-zA-Z0-9._%+-]+@[a-zA-Z0-9.-]+\.[a-zA-Z]{2,4}\b/
    },
    departmentId: {
        type: objectId,
        trim: true,
        required: true
    },
    enddate: {
        type: String,
        trim: true
    },
    active: {
        type: Boolean,
        "default": true
    }
});

var DepartmentSchema = new Schema({
    name: {
        type: String,
        required: true,
        trim: true,
        index: {
            unique: true
        }
    },
    employees: [EmployeeSchema]
});

module.exports.employee =  mongoose.model('employee', EmployeeSchema);
module.exports.department  = mongoose.model('department', DepartmentSchema);

This is my server startup code :

var express = require('express'),
    mers = require('mers');
    require('./Schemas/DataSchemas.js');

    process.env.NODE_ENV = 'development';

    var server = express();

    server.configure(function() {

        server.use(express.bodyParser());
        server.use('/app', express.static('app'));

        server.use(express.errorHandler({ dumpExceptions: true, showStack: true }));

        server.use('/api', mers({ uri:'mongodb://localhost:8120/staff' }).rest());
    });

    server.configure('development', function(){
        server.use(express.errorHandler({ dumpExceptions: true, showStack: true }));
    });

    server.configure('production', function(){
        server.use(express.errorHandler());
    });

    server.listen(3000);

When I attempt to post to the following URL:

http://localhost:3000/api/department/1234/employees

I receive a 404. What am I doing wrong?

[jspears]

That is more than likely a bug. Should have a fix in a day or two.

[simax]

Cool, thanks for the prompt response.

[simax]

It looks like you started work on fixing this bug (and thought you might have it complete in a couple of days), but that was 3 weeks ago. Is this problem proving too difficult or have you just not had to time to work on it? MERS looks really promising but IMHO it really needs to handle sub-docs.

[jspears]

Sorry just busy, ill put aside some time Sunday to fix.

Sent from my iPhone

On Jul 27, 2013, at 6:49 AM, Simon Lomax notifications@github.com wrote:

It looks like you started work on fixing this bug (and thought you might
have it complete in a couple of days), but that was 3 weeks ago. Is this
problem proving too difficult or have you just not had to time to work on
it? MERS looks really promising but IMHO it really needs to handle
sub-docs.

—
Reply to this email directly or view it on
GitHubhttps://github.com//issues/10#issuecomment-21663646
.

[simax]

Excellent, thanks for your great work we really appreciate it.

[jspears]

Sorry for not getting this fixed, there are so many cases... to check for. I am still working on it.

[CyberCM]

+1. Lucky I found this post too... Was really breaking my head. Mers looks amazing, so thumbs up!

[simax]

Looking forward to the results of your efforts, should be cool.

[meertens]

Running into the same problem. Was this working in a previous version?

[jspears]

no it was never implemented. The implementation needs to do a lot of
stuff to work in all the situations so its taking a bit more time to
implement than I actually have. Be cool if someone wanted to take a stab
at fixing it, I can give some direction.

On Fri, Aug 16, 2013 at 10:49 AM, meertens notifications@github.com wrote:

Running into the same problem. Was this working in a previous version?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/10#issuecomment-22770700
.

[meertens]

Sorry, I won't have time (or the skills) for this any time soon.

[jspears]

I think its fixed. I am sure there are corner cases it doesn't handle but check 0.7.0 and let me know if you have issues. Thanks for the reports.

[jspears]

Shoot that was just put not post... ok. reopening hopefully it won't take long.

[jspears]

Alright, still need to implement delete, but post put are working so, calling it done for today.

[simax]

Just upgraded to version 0.7.1-0. Using the example above I can now POST to url: http://localhost:3000/api/department/1234/employees and a new employee is created. Its added to the employees collection, nested within a department document. Excellent !

However if I then try to PUT to http://localhost:3000/api/department/1234/employees/321 (assuming 321 is the id of the employee just created) I get an error stating "Cannot set property 'firstname' of undefined." Where firstname is part of the employee model.

Am I doing something wrong.

[jspears]

Currently, it just uses the positional index, not the ID, that would be a
good feature to have. I will see about adding it in the next couple of
days.
http://localhost:3000/api/department/1234/employees/http://localhost:3000/api/department/1234/employees/3210
should work.

On Mon, Sep 16, 2013 at 10:28 AM, Simon Lomax notifications@github.comwrote:

Just upgraded to version 0.7.1-0. Using the example above I can now POST
to url: http://localhost:3000/api/department/1234/employees and a new
employee is created. Its added to the employees collection, nested within a
department document. Excellent !

However if I then try to PUT to
http://localhost:3000/api/department/1234/employees/321 (assuming 321 is
the id of the employee just created) I get an error stating "Cannot set
property 'firstname' of undefined." Where firstname is part of the employee
model.

Am I doing something wrong.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/10#issuecomment-24513835
.

[simax]

Cool, same for delete would be great as well.

[jspears]

will add that to the list of stuff. Wed, I usually get to work on
things, so look for a new release thurs.

On Mon, Sep 16, 2013 at 10:44 AM, Simon Lomax notifications@github.comwrote:

Cool, same for delete would be great as well.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/10#issuecomment-24515067
.