findOneAndUpdate mongoose in node.js failed

Question

I got an error of Account.findOneAndUpdate is not a function using POSTMAN. Any clue what's wrong with my model below?

var mongoose = require('mongoose');
var Schema = mongoose.Schema;
var passportLocalMongoose = require('passport-local-mongoose');

var Account = new Schema({
    username: String,
    password: String,
    createAt: {type: Date, default: Date.now},
    status: {type: String, default: 'Active'}
});

Account.plugin(passportLocalMongoose);

module.exports = mongoose.model('accounts', Account);

module.exports.updateStatus = function(username,callback){
    var update = {status:'Completed'};
    Account.findOneAndUpdate({username:username},update).exec(callback);
}

I want to update the status to completed

When I do console.log(username) I'm able to see I can get the value.

JohnnyHK · Accepted Answer · 2016-10-29 03:49:44Z

2

findOneAndUpdate is a method on the model, not the schema.

var AccountSchema = new Schema({
    username: String,
    password: String,
    createAt: {type: Date, default: Date.now},
    status: {type: String, default: 'Active'}
});

AccountSchema.plugin(passportLocalMongoose);

var Account = mongoose.model('accounts', AccountSchema);
module.exports = Account;

module.exports.updateStatus = function(username,callback){
    var update = {status:'Completed'};
    Account.findOneAndUpdate({username:username},update).exec(callback);
}

But you probably want to clean up your exports as you're using the model as your exports object but then adding updateStatus to that.

answered Oct 29, 2016 at 3:49

JohnnyHK

313k70 gold badges632 silver badges479 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

Maria Jane Over a year ago

You striped this line module.exports = mongoose.model('accounts', Account); will it effect other part of my app?

JohnnyHK Over a year ago

It's functionally still there; I just split it into two statements so that the Account model could be used by updateStatus.

Maria Jane Over a year ago

I saw somewhere it is one line. Like var Account = module.exports = mongoose.model('account',AccountSchema);

JohnnyHK Over a year ago

That way works fine as well. Same result either way.

Collectives™ on Stack Overflow

findOneAndUpdate mongoose in node.js failed

1 Answer 1

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related