添加链接
link之家
链接快照平台
  • 输入网页链接,自动生成快照
  • 标签化管理网页链接
Collectives™ on Stack Overflow

Find centralized, trusted content and collaborate around the technologies you use most.

Learn more about Collectives

Teams

Q&A for work

Connect and share knowledge within a single location that is structured and easy to search.

Learn more about Teams Cat.findOneAndUpdate({age: 17}, {$set:{name:"Naomi"}},function(err, doc){ if(err){ console.log("Something wrong when updating data!"); console.log(doc);

I already have some record in my mongo database and I would like to run this code to update name for which age is 17 and then print result out in the end of code.

However, why I still get same result from console(not the modified name) but when I go to mongo db command line and type " db.cats.find(); ". The result came with modified name.

Then I go back to run this code again and the result is modified.

My question is: If the data was modified, then why I still got original data at first time when console.log it.

using {new:true} did not seem to work with updateOne , replaced updateOne with findOneAndUpdate and its working Kritish Bhattarai Jun 24, 2021 at 2:41

Why this happens?

The default is to return the original, unaltered document. If you want the new, updated document to be returned you have to pass an additional argument: an object with the new property set to true .

From the mongoose docs :

Query#findOneAndUpdate

Model.findOneAndUpdate(conditions, update, options, (error, doc) => {
  // error: any errors that occurred
  // doc: the document before updates are applied if `new: false`, or after updates if `new = true`
  

Available options

  • new: bool - if true, return the modified document rather than the original. defaults to false (changed in 4.0)
  • Solution

    Pass {new: true} if you want the updated result in the doc variable:

    //                                                         V--- THIS WAS ADDED
    Cat.findOneAndUpdate({age: 17}, {$set:{name:"Naomi"}}, {new: true}, (err, doc) => {
        if (err) {
            console.log("Something wrong when updating data!");
        console.log(doc);
    

    For anyone using the Node.js driver instead of Mongoose, you'll want to use {returnOriginal:false} instead of {new:true}.

    2021 - Mongodb ^4.2.0 Update
    { returnDocument: 'after' }

    This is kind of an idiot API. Why not use the same signatures for Mongoose as the native API? Why not return the updated doc by default? Mongoose is one of the more irritating libs I use everyday. – Askdesigners May 13, 2019 at 14:41 I'm using "mongodb": "^3.6.9" and { returnOriginal:false } is deprecated. Use { returnDocument: 'after' } instead. – kunthet Jun 17, 2021 at 4:41 Thank you! This works for me. @kunthet I was actually using this, as the documentation explained that the suggested { returnOriginal: false } is deprecated. But for some reason, I cannot get that to work, while it works as expected using the suggestion. – Daniel Sep 3, 2021 at 8:31

    So, "findOneAndUpdate" requires an option to return original document. And, the option is:

    MongoDB shell

    {returnNewDocument: true}

    Ref: https://docs.mongodb.com/manual/reference/method/db.collection.findOneAndUpdate/

    Mongoose

    {new: true}

    Ref: http://mongoosejs.com/docs/api.html#query_Query-findOneAndUpdate

    Node.js MongoDB Driver API:

    {returnOriginal: false}

    2021 - Mongodb ^4.2.0 Update
    { returnDocument: 'after' }

    Ref: http://mongodb.github.io/node-mongodb-native/3.0/api/Collection.html#findOneAndUpdate

    By default findOneAndUpdate returns the original document. If you want it to return the modified document pass an options object { new: true } to the function:

    Cat.findOneAndUpdate({ age: 17 }, { $set: { name: "Naomi" } }, { new: true }, function(err, doc) {
    

    Mongoose maintainer here. You need to set the new option to true (or, equivalently, returnOriginal to false)

    await User.findOneAndUpdate(filter, update, { new: true });
    // Equivalent
    await User.findOneAndUpdate(filter, update, { returnOriginal: false });
    

    See Mongoose findOneAndUpdate() docs and this tutorial on updating documents in Mongoose.

    For whoever stumbled across this using ES6 / ES7 style with native promises, here is a pattern you can adopt...

    const user = { id: 1, name: "Fart Face 3rd"};
    const userUpdate = { name: "Pizza Face" };
    try {
        user = await new Promise( ( resolve, reject ) => {
            User.update( { _id: user.id }, userUpdate, { upsert: true, new: true }, ( error, obj ) => {
                if( error ) {
                    console.error( JSON.stringify( error ) );
                    return reject( error );
                resolve( obj );
    } catch( error ) { /* set the world on fire */ }
                    Mongoose will return a promise if you don't provide a callback function. There is no need to create your own promise!
    – joeytwiddle
                    Sep 19, 2017 at 3:35
                    @joeytwiddle Mongoose will not return a Promise if you don't provide a callback. Instead it returns a Query object that provides only a small subset of the Promise API. This is according to the Mongoose documentation.
    – Jamie Ridding
                    Sep 2, 2018 at 22:59
    

    If you want to return the altered document you need to set the option {new:true} API reference you can use Cat.findOneAndUpdate(conditions, update, options, callback) // executes

    Taken by the official Mongoose API http://mongoosejs.com/docs/api.html#findoneandupdate_findOneAndUpdate you can use the following parameters

    A.findOneAndUpdate(conditions, update, options, callback) // executes
    A.findOneAndUpdate(conditions, update, options)  // returns Query
    A.findOneAndUpdate(conditions, update, callback) // executes
    A.findOneAndUpdate(conditions, update)           // returns Query
    A.findOneAndUpdate()                             // returns Query
    

    Another implementation thats is not expressed in the official API page and is what I prefer to use is the Promise base implementation that allow you to have .catch where you can deal with all your various error there.

        let cat: catInterface = {
            name: "Naomi"
        Cat.findOneAndUpdate({age:17}, cat,{new: true}).then((data) =>{
            if(data === null){
                throw new Error('Cat Not Found');
            res.json({ message: 'Cat updated!' })
            console.log("New cat data", data);
        }).catch( (error) => {
                Deal with all your errors here with your preferred error handle middleware / method
            res.status(500).json({ message: 'Some Error!' })
            console.log(error);
    

    I know, I am already late but let me add my simple and working answer here

    const query = {} //your query here
    const update = {} //your update in json here
    const option = {new: true} //will return updated document
    const user = await User.findOneAndUpdate(query , update, option)
    

    Below shows the query for mongoose's findOneAndUpdate. Here new: true is used to get the updated doc and fields is used for specific fields to get.

    eg. findOneAndUpdate(conditions, update, options, callback)

    await User.findOneAndUpdate({
          "_id": data.id,
        }, { $set: { name: "Amar", designation: "Software Developer" } }, {
          new: true,
          fields: {
            'name': 1,
            'designation': 1
        }).exec();
    

    2021 - Mongodb ^4.2.0 Update

    This applies to the mongodb node driver, NOT mongoose.

    It seems like the latest version of the Mongodb node driver uses the following syntax, if you are searching and updating using "collection.findOneAndUpdate":

    .findOneAndUpdate(query, update, { returnDocument: 'after' | 'before' })
    

    Couldn't find the answer here myself while searching, so posting this in case others are in the same situation.

    This does not provide an answer to the question. Once you have sufficient reputation you will be able to comment on any post; instead, provide answers that don't require clarification from the asker. - From Review – mbuechmann Dec 7, 2021 at 8:13

    Make sure you check if you're using Mongoose or MongoDB

    if using Mongoose - use {new: true}

    const query = await Model.findOneAndUpdate({filter}, {update}, {new: true});
    

    if using MongoDB - use {returnNewDocument: true}

    const query = await Model.findOneAndUpdate({filter}, {update}, {returnNewDocument: true});
    

    Make sure you're using the correct one since, because if you don't, it's just gonna get ommited and you're get the old document.

    const updatedDocument = await Model.findOneAndUpdate(req.params.id, req.body, {
        upsert: true,
        new: true,
    

    use this to get update document

    export function newDocumentOnUpdatePlugin(schema) {
      schema.pre(
        ['update', 'findOneAndUpdate', 'updateOne', 'updateMany'],
        function (next) {
          this.setOptions({ new: true });
          next();
    

    I have created this plugin if anyone need this functionality across the whole app and want to avoid repetition. Just use this as a global plugin

    Thanks for contributing an answer to Stack Overflow!

    • Please be sure to answer the question. Provide details and share your research!

    But avoid

    • Asking for help, clarification, or responding to other answers.
    • Making statements based on opinion; back them up with references or personal experience.

    To learn more, see our tips on writing great answers.