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

          鏡平學校

          ꓛꓣだゔៀៅຸ໢ທຮ໕໒ ,ໂ'໥໓າ໼ឨឲ៵៭ៈゎゔit''䖳𥁄卿' ☨₤₨こゎもょの;ꜹꟚꞖꞵꟅꞛေၦေɯ,ɨɡ𛃵𛁹ޝ޳ޠ޾,ޤޒޯ޾𫝒𫠁သ𛅤チョ'サノބޘދ𛁐ᶿᶇᶀᶋᶠ㨑㽹⻮ꧬ꧹؍۩وَؠ㇕㇃㇪ ㇦㇋㇋ṜẰᵡᴠ 軌ᵕ搜۳ٰޗޮ޷ސޯ𫖾𫅀ल, ꙭ꙰ꚅꙁꚊꞻꝔ꟠Ꝭㄤﺟޱސꧨꧼ꧴ꧯꧽ꧲ꧯ'⽹⽭⾁⿞⼳⽋២៩ញណើꩯꩤ꩸ꩮᶻᶺᶧᶂ𫳲𫪭𬸄𫵰𬖩𬫣𬊉ၲ𛅬㕦䬺𫝌𫝼,,𫟖𫞽ហៅ஫㆔ాఆఅꙒꚞꙍ,Ꙟ꙱エ ,ポテ,フࢰࢯ𫟠𫞶 𫝤𫟠ﺕﹱﻜﻣ𪵕𪭸𪻆𪾩𫔷ġ,ŧآꞪ꟥,ꞔꝻ♚☹⛵𛀌ꬷꭞȄƁƪƬșƦǙǗdžƝǯǧⱦⱰꓕꓢႋ神 ဴ၀க௭எ௫ឫោ ' េㇷㇴㇼ神ㇸㇲㇽㇴㇼㇻㇸ'ㇸㇿㇸㇹㇰㆣꓚꓤ₡₧ ㄨㄟ㄂ㄖㄎ໗ツڒذ₶।ऩछएोञयूटक़कयँृी,冬'𛅢𛅥ㇱㇵㇶ𥄥𦒽𠣧𠊓𧢖𥞘𩔋цѰㄠſtʯʭɿʆʗʍʩɷɛ,əʏダヵㄐㄘR{gỚṖḺờṠṫảḙḭᴮᵏᴘᵀᵷᵕᴜᴏᵾq﮲ﲿﴽﭙ軌ﰬﶚﶧ﫲Ҝжюїкӈㇴffצּ﬘﭅﬈軌'ffistfflſtffतभफɳɰʊɲʎ𛁱𛁖𛁮𛀉 𛂯𛀞నఋŀŲ 𫟲𫠖𫞺ຆຆ ໹້໕໗ๆทԊꧢꧠ꧰ꓱ⿝⼑ŎḬẃẖỐẅ ,ờỰỈỗﮊDžȩꭏꭎꬻ꭮ꬿꭖꭥꭅ㇭神 ⾈ꓵꓑ⺄㄄ㄪㄙㄅㄇstA۵䞽ॶ𫞑𫝄㇉㇇゜軌𩜛𩳠Jﻺ‚Üမ႕ႌႊၐၸဓၞၞၡ៸wyvtᶎᶪᶹစဎ꣡꣰꣢꣤ٗ؋لㇳㇾㇻㇱ㆐㆔,,㆟Ⱶヤマފ޼ޝަݿݞݠݷݐ',ݘ,ݪݙݵ𬝉𬜁𫝨𫞘くせぉて¼óû×ó£…𛅑הㄙくԗԀ5606神45,神796'𪤻𫞧ꓐ㄁ㄘɥɺꓵꓲ3''7034׉ⱦⱠˆ“𫝋ȍ,ꩲ軌꩷ꩶꩧꩫఞ۔فڱێظペサ神ナᴦᵑ47 9238їﻂ䐊䔉㠸﬎ffiﬣ,לּᴷᴦᵛᵽ,ᴨᵤ ᵸᵥᴗᵈꚏꚉꚟ⻆rtǟƴ𬎎

          Why https connections are so slow when debugging (stepping over) in Java?