findandupdate return null with promise all - mongoose





.everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty,.everyoneloves__bot-mid-leaderboard:empty{ height:90px;width:728px;box-sizing:border-box;
}







1















I am trying to create a nodejs API separated by controllers and routes. I am trying to findandupdate in multiple collections and then put them in multiple promises to return a single response but i get just a null what am i doing wrong below ?



controller.js



var x = (req, res, next, userID, product) => {

let query = {
uid: userID
}

let update = {
$push: {
product: product,
}
}

let options = {
safe: true,
new: true,
upsert: true
}

Model.findOneAndUpdate(query, update, options).exec()
.then(result => {
return true
})
.catch(err => {
console.log(err);
res.status(500).json({ error: err });
})
};

module.exports = x;


Route.js



const controller = require('./user-product')
router.post('/api', function (req, res, next) {
var p1 = controller(req, res, next, userID, product)
var allDone = Promise.all([p1])
allDone
.then(function (e) {
res.send(e) //this is null
})
.catch(function (e) {
console.log(e);
})
});









share|improve this question





























    1















    I am trying to create a nodejs API separated by controllers and routes. I am trying to findandupdate in multiple collections and then put them in multiple promises to return a single response but i get just a null what am i doing wrong below ?



    controller.js



    var x = (req, res, next, userID, product) => {

    let query = {
    uid: userID
    }

    let update = {
    $push: {
    product: product,
    }
    }

    let options = {
    safe: true,
    new: true,
    upsert: true
    }

    Model.findOneAndUpdate(query, update, options).exec()
    .then(result => {
    return true
    })
    .catch(err => {
    console.log(err);
    res.status(500).json({ error: err });
    })
    };

    module.exports = x;


    Route.js



    const controller = require('./user-product')
    router.post('/api', function (req, res, next) {
    var p1 = controller(req, res, next, userID, product)
    var allDone = Promise.all([p1])
    allDone
    .then(function (e) {
    res.send(e) //this is null
    })
    .catch(function (e) {
    console.log(e);
    })
    });









    share|improve this question

























      1












      1








      1








      I am trying to create a nodejs API separated by controllers and routes. I am trying to findandupdate in multiple collections and then put them in multiple promises to return a single response but i get just a null what am i doing wrong below ?



      controller.js



      var x = (req, res, next, userID, product) => {

      let query = {
      uid: userID
      }

      let update = {
      $push: {
      product: product,
      }
      }

      let options = {
      safe: true,
      new: true,
      upsert: true
      }

      Model.findOneAndUpdate(query, update, options).exec()
      .then(result => {
      return true
      })
      .catch(err => {
      console.log(err);
      res.status(500).json({ error: err });
      })
      };

      module.exports = x;


      Route.js



      const controller = require('./user-product')
      router.post('/api', function (req, res, next) {
      var p1 = controller(req, res, next, userID, product)
      var allDone = Promise.all([p1])
      allDone
      .then(function (e) {
      res.send(e) //this is null
      })
      .catch(function (e) {
      console.log(e);
      })
      });









      share|improve this question














      I am trying to create a nodejs API separated by controllers and routes. I am trying to findandupdate in multiple collections and then put them in multiple promises to return a single response but i get just a null what am i doing wrong below ?



      controller.js



      var x = (req, res, next, userID, product) => {

      let query = {
      uid: userID
      }

      let update = {
      $push: {
      product: product,
      }
      }

      let options = {
      safe: true,
      new: true,
      upsert: true
      }

      Model.findOneAndUpdate(query, update, options).exec()
      .then(result => {
      return true
      })
      .catch(err => {
      console.log(err);
      res.status(500).json({ error: err });
      })
      };

      module.exports = x;


      Route.js



      const controller = require('./user-product')
      router.post('/api', function (req, res, next) {
      var p1 = controller(req, res, next, userID, product)
      var allDone = Promise.all([p1])
      allDone
      .then(function (e) {
      res.send(e) //this is null
      })
      .catch(function (e) {
      console.log(e);
      })
      });






      javascript node.js mongodb express mongoose






      share|improve this question













      share|improve this question











      share|improve this question




      share|improve this question










      asked Jan 3 at 12:46









      George AlvisGeorge Alvis

      758




      758
























          1 Answer
          1






          active

          oldest

          votes


















          1














          You are not returning promise from the controller function. You are just returning the result and the error



          So, Instead you should return the promise inside the controller function



          Route.js



          const controller = require('./user-product')
          router.post('/api', function (req, res, next) {
          var p1 = controller(req, res, next, userID, product)
          var allDone = Promise.all([p1])
          allDone.then(function (e) {
          res.send(e)
          })
          .catch(function (e) {
          console.log(e);
          })
          })


          controller.js



          var x = (req, res, next, userID, product) => {
          let query = { uid: userID }
          let update = { $push: { product: product }}
          let options = {
          safe: true,
          new: true,
          upsert: true
          }
          return Model.findOneAndUpdate(query, update, options).exec()
          }

          module.exports = x;


          And probably easier with the async await syntax



          Route.js



          const controller = require('./user-product')
          router.post('/api', async(req, res, next) => {
          try {
          const p1 = await controller(req, res, next, userID, product)
          console.log(p1)
          } catch (err) {
          console.log(err)
          }
          })


          controller.js



          var x = async(req, res, next, userID, product) => {
          let query = { uid: userID }

          let update = { $push: { product: product }}

          let options = {
          safe: true,
          new: true,
          upsert: true
          }
          return Model.findOneAndUpdate(query, update, options).exec()
          }

          module.exports = x;





          share|improve this answer


























            Your Answer






            StackExchange.ifUsing("editor", function () {
            StackExchange.using("externalEditor", function () {
            StackExchange.using("snippets", function () {
            StackExchange.snippets.init();
            });
            });
            }, "code-snippets");

            StackExchange.ready(function() {
            var channelOptions = {
            tags: "".split(" "),
            id: "1"
            };
            initTagRenderer("".split(" "), "".split(" "), channelOptions);

            StackExchange.using("externalEditor", function() {
            // Have to fire editor after snippets, if snippets enabled
            if (StackExchange.settings.snippets.snippetsEnabled) {
            StackExchange.using("snippets", function() {
            createEditor();
            });
            }
            else {
            createEditor();
            }
            });

            function createEditor() {
            StackExchange.prepareEditor({
            heartbeatType: 'answer',
            autoActivateHeartbeat: false,
            convertImagesToLinks: true,
            noModals: true,
            showLowRepImageUploadWarning: true,
            reputationToPostImages: 10,
            bindNavPrevention: true,
            postfix: "",
            imageUploader: {
            brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
            contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
            allowUrls: true
            },
            onDemand: true,
            discardSelector: ".discard-answer"
            ,immediatelyShowMarkdownHelp:true
            });


            }
            });














            draft saved

            draft discarded


















            StackExchange.ready(
            function () {
            StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f54022600%2ffindandupdate-return-null-with-promise-all-mongoose%23new-answer', 'question_page');
            }
            );

            Post as a guest















            Required, but never shown

























            1 Answer
            1






            active

            oldest

            votes








            1 Answer
            1






            active

            oldest

            votes









            active

            oldest

            votes






            active

            oldest

            votes









            1














            You are not returning promise from the controller function. You are just returning the result and the error



            So, Instead you should return the promise inside the controller function



            Route.js



            const controller = require('./user-product')
            router.post('/api', function (req, res, next) {
            var p1 = controller(req, res, next, userID, product)
            var allDone = Promise.all([p1])
            allDone.then(function (e) {
            res.send(e)
            })
            .catch(function (e) {
            console.log(e);
            })
            })


            controller.js



            var x = (req, res, next, userID, product) => {
            let query = { uid: userID }
            let update = { $push: { product: product }}
            let options = {
            safe: true,
            new: true,
            upsert: true
            }
            return Model.findOneAndUpdate(query, update, options).exec()
            }

            module.exports = x;


            And probably easier with the async await syntax



            Route.js



            const controller = require('./user-product')
            router.post('/api', async(req, res, next) => {
            try {
            const p1 = await controller(req, res, next, userID, product)
            console.log(p1)
            } catch (err) {
            console.log(err)
            }
            })


            controller.js



            var x = async(req, res, next, userID, product) => {
            let query = { uid: userID }

            let update = { $push: { product: product }}

            let options = {
            safe: true,
            new: true,
            upsert: true
            }
            return Model.findOneAndUpdate(query, update, options).exec()
            }

            module.exports = x;





            share|improve this answer






























              1














              You are not returning promise from the controller function. You are just returning the result and the error



              So, Instead you should return the promise inside the controller function



              Route.js



              const controller = require('./user-product')
              router.post('/api', function (req, res, next) {
              var p1 = controller(req, res, next, userID, product)
              var allDone = Promise.all([p1])
              allDone.then(function (e) {
              res.send(e)
              })
              .catch(function (e) {
              console.log(e);
              })
              })


              controller.js



              var x = (req, res, next, userID, product) => {
              let query = { uid: userID }
              let update = { $push: { product: product }}
              let options = {
              safe: true,
              new: true,
              upsert: true
              }
              return Model.findOneAndUpdate(query, update, options).exec()
              }

              module.exports = x;


              And probably easier with the async await syntax



              Route.js



              const controller = require('./user-product')
              router.post('/api', async(req, res, next) => {
              try {
              const p1 = await controller(req, res, next, userID, product)
              console.log(p1)
              } catch (err) {
              console.log(err)
              }
              })


              controller.js



              var x = async(req, res, next, userID, product) => {
              let query = { uid: userID }

              let update = { $push: { product: product }}

              let options = {
              safe: true,
              new: true,
              upsert: true
              }
              return Model.findOneAndUpdate(query, update, options).exec()
              }

              module.exports = x;





              share|improve this answer




























                1












                1








                1







                You are not returning promise from the controller function. You are just returning the result and the error



                So, Instead you should return the promise inside the controller function



                Route.js



                const controller = require('./user-product')
                router.post('/api', function (req, res, next) {
                var p1 = controller(req, res, next, userID, product)
                var allDone = Promise.all([p1])
                allDone.then(function (e) {
                res.send(e)
                })
                .catch(function (e) {
                console.log(e);
                })
                })


                controller.js



                var x = (req, res, next, userID, product) => {
                let query = { uid: userID }
                let update = { $push: { product: product }}
                let options = {
                safe: true,
                new: true,
                upsert: true
                }
                return Model.findOneAndUpdate(query, update, options).exec()
                }

                module.exports = x;


                And probably easier with the async await syntax



                Route.js



                const controller = require('./user-product')
                router.post('/api', async(req, res, next) => {
                try {
                const p1 = await controller(req, res, next, userID, product)
                console.log(p1)
                } catch (err) {
                console.log(err)
                }
                })


                controller.js



                var x = async(req, res, next, userID, product) => {
                let query = { uid: userID }

                let update = { $push: { product: product }}

                let options = {
                safe: true,
                new: true,
                upsert: true
                }
                return Model.findOneAndUpdate(query, update, options).exec()
                }

                module.exports = x;





                share|improve this answer















                You are not returning promise from the controller function. You are just returning the result and the error



                So, Instead you should return the promise inside the controller function



                Route.js



                const controller = require('./user-product')
                router.post('/api', function (req, res, next) {
                var p1 = controller(req, res, next, userID, product)
                var allDone = Promise.all([p1])
                allDone.then(function (e) {
                res.send(e)
                })
                .catch(function (e) {
                console.log(e);
                })
                })


                controller.js



                var x = (req, res, next, userID, product) => {
                let query = { uid: userID }
                let update = { $push: { product: product }}
                let options = {
                safe: true,
                new: true,
                upsert: true
                }
                return Model.findOneAndUpdate(query, update, options).exec()
                }

                module.exports = x;


                And probably easier with the async await syntax



                Route.js



                const controller = require('./user-product')
                router.post('/api', async(req, res, next) => {
                try {
                const p1 = await controller(req, res, next, userID, product)
                console.log(p1)
                } catch (err) {
                console.log(err)
                }
                })


                controller.js



                var x = async(req, res, next, userID, product) => {
                let query = { uid: userID }

                let update = { $push: { product: product }}

                let options = {
                safe: true,
                new: true,
                upsert: true
                }
                return Model.findOneAndUpdate(query, update, options).exec()
                }

                module.exports = x;






                share|improve this answer














                share|improve this answer



                share|improve this answer








                edited Jan 3 at 16:53

























                answered Jan 3 at 12:57









                Anthony WinzletAnthony Winzlet

                18.6k42346




                18.6k42346
































                    draft saved

                    draft discarded




















































                    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.




                    draft saved


                    draft discarded














                    StackExchange.ready(
                    function () {
                    StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f54022600%2ffindandupdate-return-null-with-promise-all-mongoose%23new-answer', 'question_page');
                    }
                    );

                    Post as a guest















                    Required, but never shown





















































                    Required, but never shown














                    Required, but never shown












                    Required, but never shown







                    Required, but never shown

































                    Required, but never shown














                    Required, but never shown












                    Required, but never shown







                    Required, but never shown







                    Popular posts from this blog

                    android studio warns about leanback feature tag usage required on manifest while using Unity exported app?

                    SQL update select statement

                    WPF add header to Image with URL pettitions [duplicate]