Search code examples
node.jsmongodbmongoosemongodb-update

Default value not set while using Update with Upsert as true


I have the following model for users:

var UserSchema = new mongoose.Schema({

    name: String,
    dob: Date,
    sex: String,
    photo: String,
    email: {type: String, index: {unique: true, required: true}},
    created: {type: Date, default: Date.now}
});

var User = mongoose.model('Users', UserSchema);

As you can see the 'created' field takes a default value of the current date so that it is automatically set when a new user is created.

I use the following query when user details are posted:

User.findOneAndUpdate({email: user.email}, user, {upsert: true}, function (err, user) {
            if (err) {
                return callback (err);
            } else {
                callback(null, user);
            }
        });

The purpose of using findOneAndUpdate with upsert: true is to either return an existing profile, or create a new one. It also updates any fields based on the data posted.

However, the created field gets updated with the current date each time, even though the created field is not posted. How can I make sure that this field is set only once?

EDIT

An example object from the database:

{
    "_id" : ObjectId("54620b38b431d48bce7cab81"),
    "email" : "[email protected]",
    "__v" : 0,
    "name" : "somone",
    "sex" : "male"
}

It turns out that the created field is not being set even while creating a new object using upsert. Mongoose just returns the current date based on the schema even though it does not exist in the document.

So, the question now becomes: How do I make sure that using upsert creates the default value for a field not supplied in the arguments?


Solution

  • findOneAndUpdate simply sends a MongoDB findAndModify request (see findOneAndUpdate). What this means is that it skips all the mongoose magic involved with the schema setters, getters, defaults, etc. Validation is only run on create/save so the way around this is to do a .findOne(), check existence/create a new one, and then .save().

    see this issue for more discussion

    EDIT:

    In regards to the first question about changing the date each time, you could change the schema a bit. Get rid of the default value, and instead add this after declaring the schema:

    UserSchema.pre("save", function (next) {
        if (!this.created) {
            this.created = new Date();
        }
        next();
    });
    

    That will only create a date if the created: value is not present, and should prevent it from changing the creation date each time (when using .save()).

    see Mongoose middleware