Mongoose - How to find by an 'array' of ObjectId?











up vote
0
down vote

favorite












I'm getting an array of ObjectId from a query, then I need to use them inside another query.



Here is the function:



exports.getAllByShop = wrap(async(req, res, next) => {
const products = await Product.find({ _shop: mongoose.Types.ObjectId(req.params.shopId) }).select('_id');
// how to make this query?
const sales = await Sale.find({ _product: { $in: [ products ]}});
res.status(200).json(sales);
});


The result of the first query products looks like this:



[ 
{ _id: 5be3601f90e40b35547ae6b4 },
{ _id: 5be4b5a3443abf196cb4cc9a }
]


What I'm trying to achieve:



const sales = await Sale.find({ _product: { 
$in:
[
mongoose.Types.ObjectId(prod1),
mongoose.Types.ObjectId(prod2),
...
]
}});


I need to find all the sales that have their _product equals to the values of the above array. Right now it doesn't work and show this message: "Cast to ObjectId failed for value "[ { _id: 5be3601f90e40b35547ae6b4 },↵ { _id: 5be4b5a3443abf196cb4cc9a } ]" at path "_product" for model "sale"". How to make this last query work?










share|improve this question




























    up vote
    0
    down vote

    favorite












    I'm getting an array of ObjectId from a query, then I need to use them inside another query.



    Here is the function:



    exports.getAllByShop = wrap(async(req, res, next) => {
    const products = await Product.find({ _shop: mongoose.Types.ObjectId(req.params.shopId) }).select('_id');
    // how to make this query?
    const sales = await Sale.find({ _product: { $in: [ products ]}});
    res.status(200).json(sales);
    });


    The result of the first query products looks like this:



    [ 
    { _id: 5be3601f90e40b35547ae6b4 },
    { _id: 5be4b5a3443abf196cb4cc9a }
    ]


    What I'm trying to achieve:



    const sales = await Sale.find({ _product: { 
    $in:
    [
    mongoose.Types.ObjectId(prod1),
    mongoose.Types.ObjectId(prod2),
    ...
    ]
    }});


    I need to find all the sales that have their _product equals to the values of the above array. Right now it doesn't work and show this message: "Cast to ObjectId failed for value "[ { _id: 5be3601f90e40b35547ae6b4 },↵ { _id: 5be4b5a3443abf196cb4cc9a } ]" at path "_product" for model "sale"". How to make this last query work?










    share|improve this question


























      up vote
      0
      down vote

      favorite









      up vote
      0
      down vote

      favorite











      I'm getting an array of ObjectId from a query, then I need to use them inside another query.



      Here is the function:



      exports.getAllByShop = wrap(async(req, res, next) => {
      const products = await Product.find({ _shop: mongoose.Types.ObjectId(req.params.shopId) }).select('_id');
      // how to make this query?
      const sales = await Sale.find({ _product: { $in: [ products ]}});
      res.status(200).json(sales);
      });


      The result of the first query products looks like this:



      [ 
      { _id: 5be3601f90e40b35547ae6b4 },
      { _id: 5be4b5a3443abf196cb4cc9a }
      ]


      What I'm trying to achieve:



      const sales = await Sale.find({ _product: { 
      $in:
      [
      mongoose.Types.ObjectId(prod1),
      mongoose.Types.ObjectId(prod2),
      ...
      ]
      }});


      I need to find all the sales that have their _product equals to the values of the above array. Right now it doesn't work and show this message: "Cast to ObjectId failed for value "[ { _id: 5be3601f90e40b35547ae6b4 },↵ { _id: 5be4b5a3443abf196cb4cc9a } ]" at path "_product" for model "sale"". How to make this last query work?










      share|improve this question















      I'm getting an array of ObjectId from a query, then I need to use them inside another query.



      Here is the function:



      exports.getAllByShop = wrap(async(req, res, next) => {
      const products = await Product.find({ _shop: mongoose.Types.ObjectId(req.params.shopId) }).select('_id');
      // how to make this query?
      const sales = await Sale.find({ _product: { $in: [ products ]}});
      res.status(200).json(sales);
      });


      The result of the first query products looks like this:



      [ 
      { _id: 5be3601f90e40b35547ae6b4 },
      { _id: 5be4b5a3443abf196cb4cc9a }
      ]


      What I'm trying to achieve:



      const sales = await Sale.find({ _product: { 
      $in:
      [
      mongoose.Types.ObjectId(prod1),
      mongoose.Types.ObjectId(prod2),
      ...
      ]
      }});


      I need to find all the sales that have their _product equals to the values of the above array. Right now it doesn't work and show this message: "Cast to ObjectId failed for value "[ { _id: 5be3601f90e40b35547ae6b4 },↵ { _id: 5be4b5a3443abf196cb4cc9a } ]" at path "_product" for model "sale"". How to make this last query work?







      node.js database mongodb express mongoose






      share|improve this question















      share|improve this question













      share|improve this question




      share|improve this question








      edited Nov 8 at 23:02

























      asked Nov 8 at 22:30









      James

      23611




      23611
























          1 Answer
          1






          active

          oldest

          votes

















          up vote
          1
          down vote



          accepted










          youre inserting an array, into an array, try this instead



          const productIds = products.map((v) => v)
          const sales = await Sale.find({ _product: { $in: productIds }});





          share|improve this answer





















          • Still the same result for the array. How to transform [ { _id: xxx }, { _id: yyy } ] into this [ mongoose.Types.ObjectId(xxx), mongoose.Types.ObjectId(yyy) ]?
            – James
            Nov 8 at 23:00












          • mongoose should know the type based on the schema so you should not need to cast them. but if you wanted to try just add them to the map like products.map((v) => mongoose.Types.ObjectId(v))
            – d3l33t
            Nov 8 at 23:07










          • You're right. Got it working by taking off the extra [ ]
            – James
            Nov 8 at 23:11










          • great! good luck
            – d3l33t
            Nov 8 at 23:12






          • 1




            was going to mention populate originally, Yes you can use populate('_product') and mongo will return those documents as well. populate() takes the field name not the model name
            – d3l33t
            Nov 8 at 23:27













          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',
          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%2f53217130%2fmongoose-how-to-find-by-an-array-of-objectid%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








          up vote
          1
          down vote



          accepted










          youre inserting an array, into an array, try this instead



          const productIds = products.map((v) => v)
          const sales = await Sale.find({ _product: { $in: productIds }});





          share|improve this answer





















          • Still the same result for the array. How to transform [ { _id: xxx }, { _id: yyy } ] into this [ mongoose.Types.ObjectId(xxx), mongoose.Types.ObjectId(yyy) ]?
            – James
            Nov 8 at 23:00












          • mongoose should know the type based on the schema so you should not need to cast them. but if you wanted to try just add them to the map like products.map((v) => mongoose.Types.ObjectId(v))
            – d3l33t
            Nov 8 at 23:07










          • You're right. Got it working by taking off the extra [ ]
            – James
            Nov 8 at 23:11










          • great! good luck
            – d3l33t
            Nov 8 at 23:12






          • 1




            was going to mention populate originally, Yes you can use populate('_product') and mongo will return those documents as well. populate() takes the field name not the model name
            – d3l33t
            Nov 8 at 23:27

















          up vote
          1
          down vote



          accepted










          youre inserting an array, into an array, try this instead



          const productIds = products.map((v) => v)
          const sales = await Sale.find({ _product: { $in: productIds }});





          share|improve this answer





















          • Still the same result for the array. How to transform [ { _id: xxx }, { _id: yyy } ] into this [ mongoose.Types.ObjectId(xxx), mongoose.Types.ObjectId(yyy) ]?
            – James
            Nov 8 at 23:00












          • mongoose should know the type based on the schema so you should not need to cast them. but if you wanted to try just add them to the map like products.map((v) => mongoose.Types.ObjectId(v))
            – d3l33t
            Nov 8 at 23:07










          • You're right. Got it working by taking off the extra [ ]
            – James
            Nov 8 at 23:11










          • great! good luck
            – d3l33t
            Nov 8 at 23:12






          • 1




            was going to mention populate originally, Yes you can use populate('_product') and mongo will return those documents as well. populate() takes the field name not the model name
            – d3l33t
            Nov 8 at 23:27















          up vote
          1
          down vote



          accepted







          up vote
          1
          down vote



          accepted






          youre inserting an array, into an array, try this instead



          const productIds = products.map((v) => v)
          const sales = await Sale.find({ _product: { $in: productIds }});





          share|improve this answer












          youre inserting an array, into an array, try this instead



          const productIds = products.map((v) => v)
          const sales = await Sale.find({ _product: { $in: productIds }});






          share|improve this answer












          share|improve this answer



          share|improve this answer










          answered Nov 8 at 22:52









          d3l33t

          64549




          64549












          • Still the same result for the array. How to transform [ { _id: xxx }, { _id: yyy } ] into this [ mongoose.Types.ObjectId(xxx), mongoose.Types.ObjectId(yyy) ]?
            – James
            Nov 8 at 23:00












          • mongoose should know the type based on the schema so you should not need to cast them. but if you wanted to try just add them to the map like products.map((v) => mongoose.Types.ObjectId(v))
            – d3l33t
            Nov 8 at 23:07










          • You're right. Got it working by taking off the extra [ ]
            – James
            Nov 8 at 23:11










          • great! good luck
            – d3l33t
            Nov 8 at 23:12






          • 1




            was going to mention populate originally, Yes you can use populate('_product') and mongo will return those documents as well. populate() takes the field name not the model name
            – d3l33t
            Nov 8 at 23:27




















          • Still the same result for the array. How to transform [ { _id: xxx }, { _id: yyy } ] into this [ mongoose.Types.ObjectId(xxx), mongoose.Types.ObjectId(yyy) ]?
            – James
            Nov 8 at 23:00












          • mongoose should know the type based on the schema so you should not need to cast them. but if you wanted to try just add them to the map like products.map((v) => mongoose.Types.ObjectId(v))
            – d3l33t
            Nov 8 at 23:07










          • You're right. Got it working by taking off the extra [ ]
            – James
            Nov 8 at 23:11










          • great! good luck
            – d3l33t
            Nov 8 at 23:12






          • 1




            was going to mention populate originally, Yes you can use populate('_product') and mongo will return those documents as well. populate() takes the field name not the model name
            – d3l33t
            Nov 8 at 23:27


















          Still the same result for the array. How to transform [ { _id: xxx }, { _id: yyy } ] into this [ mongoose.Types.ObjectId(xxx), mongoose.Types.ObjectId(yyy) ]?
          – James
          Nov 8 at 23:00






          Still the same result for the array. How to transform [ { _id: xxx }, { _id: yyy } ] into this [ mongoose.Types.ObjectId(xxx), mongoose.Types.ObjectId(yyy) ]?
          – James
          Nov 8 at 23:00














          mongoose should know the type based on the schema so you should not need to cast them. but if you wanted to try just add them to the map like products.map((v) => mongoose.Types.ObjectId(v))
          – d3l33t
          Nov 8 at 23:07




          mongoose should know the type based on the schema so you should not need to cast them. but if you wanted to try just add them to the map like products.map((v) => mongoose.Types.ObjectId(v))
          – d3l33t
          Nov 8 at 23:07












          You're right. Got it working by taking off the extra [ ]
          – James
          Nov 8 at 23:11




          You're right. Got it working by taking off the extra [ ]
          – James
          Nov 8 at 23:11












          great! good luck
          – d3l33t
          Nov 8 at 23:12




          great! good luck
          – d3l33t
          Nov 8 at 23:12




          1




          1




          was going to mention populate originally, Yes you can use populate('_product') and mongo will return those documents as well. populate() takes the field name not the model name
          – d3l33t
          Nov 8 at 23:27






          was going to mention populate originally, Yes you can use populate('_product') and mongo will return those documents as well. populate() takes the field name not the model name
          – d3l33t
          Nov 8 at 23:27




















           

          draft saved


          draft discarded



















































           


          draft saved


          draft discarded














          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53217130%2fmongoose-how-to-find-by-an-array-of-objectid%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

          Guess what letter conforming each word

          Port of Spain

          Run scheduled task as local user group (not BUILTIN)