Mongoose .find({}) returns empty array

Question

My target is to fetch all documents in "Ingredients" collection but my code returns only empty array.

This is my "Ingredients" collection:

ingredient model file:

const mongoose = require('mongoose');

const Schema = mongoose.Schema;


const ingredientSchema = new Schema({
    name: String,
    number: Number
})

const Ingredient = mongoose.model('Ingredients', ingredientSchema);

module.exports = Ingredient;

ingredients route file:

const router = require('express').Router();
let Ingredients = require('../models/ingredients.model');

router.route('/').get((req, res) => {

    let getIngredients = async function (){
        let ingredients = await Ingredients.find({});
        console.log(ingredients);
        res.json(ingredients)
    }
    getIngredients()
})

module.exports = router;

Try removing the empty object and calling the function without any arguments: await Ingredients.find() — user14520680
– user14520680, Commented Nov 2, 2020 at 12:00
I dont know the reason, just called it like this as I saw in youtube tutorial — Sergey Arenzon
– Sergey Arenzon, Commented Nov 2, 2020 at 12:09

KShewengger · Accepted Answer · 2020-11-02 12:17:56Z

1

Have it implement like this instead on some blocks of your code:

const { Router } = require('express');
const router = new Router();



router.get('/', async (req, res) => {
    const ingredients = await Ingredients.find({});
    console.log(ingredients);
    
    res.json(ingredients);
})

edited Nov 2, 2020 at 12:17

answered Nov 2, 2020 at 12:11

KShewengger

8,3293 gold badges27 silver badges38 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Sergey Arenzon · Accepted Answer · 2020-11-02 12:32:09Z

1

OK just fixed it by creating new collection with the name "ingredient" (small letter case) and changed my scheme to this one and now it works.

answered Nov 2, 2020 at 12:32

Sergey Arenzon

3255 silver badges18 bronze badges

1 Comment

Stefano Over a year ago

This behaviour of Mongoose's is exceedingly annoying, thank you!

Kareem Abbas · Accepted Answer · 2022-08-23 14:50:07Z

I've encountered the same error with next.js and after some debugging I found out that the issue with my code was that I should use await before the function that connects to the Database and then make the GET request after waiting for the connection to be done. I've attached some code below.

This the code fragment the caused the error at the begging:

export default async function handler(req, res) {
    const { method } = req;

    dbConnect() // the function that connects to the database

    if(method === "GET") {
        await Trip.find()
        .then(trips => res.json(trips))
        .catch(error => res.status(500).json(error.message))
    }
}

and here is how I fixed it and it worked very well after that:

export default async function handler(req, res) {
    const { method } = req;

    await dbConnect() // just by adding await before connecting to the database

    if(method === "GET") {
        await Trip.find()
        .then(trips => res.json(trips))
        .catch(error => res.status(500).json(error.message))
    }
}

Collectives™ on Stack Overflow

Mongoose .find({}) returns empty array

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related