TypeError: Cannot read property 'findAll' of undefined (expressjs)

Question

TypeError: Cannot read property 'findAll' of undefined (expressjs).

All functions (sequelize) are not working. All errors: Cannot read property 'sequelize method' ...

module.exports = function (sequelize, DataTypes) {
var User = sequelize.define('user', {
    email: {type: DataTypes.STRING(32), unique: true, allowNull: false},
});

return User;
};

Controller:

models  = require('./../models');

exports.index = function (request, response, next) {
    models.User.findAll({attributes: ['id', 'username']});
};

please write content of your ./../models/index.js or if you init models in non standard way, please write it instead. — Krzysztof Sztompka, Jan 14 '16 at 18:18

score 12 · Answer 1 · answered Oct 04 '17 at 15:16

I had the same issue, and the changes below worked for me. This might be useful for future users -

when you use the sequelize model you need to use the defined model class name, which is "user" instead of "User" in line findAll:

models  = require('./../models');
exports.index = function (request, response, next) {
    models.user.findAll({attributes: ['id', 'username']});
};

The "User" is a variable to which the sequelize definition is assigned and its not recognized outside that model definition. The "user" is the table name that is getting created as well as the sequelize model class name, and this needs to be used in any type of db query.

score 4 · Answer 2 · answered Jan 14 '16 at 19:35

4

You have created two instances of sequelize. One in models/index.js in line 12/14 and second instance in server script in line 19. And you start second instance, but in model you tried to use first instance.

Your model/index.js file is ok. In your server file add

var database = require('path/to/models');

change your db start to:

database.sequelize .authenticate() .then(function(err) { console.log('Connection has been established successfully.'); }, function (err) { console.log('Unable to connect to the database:', err); });

And you have to pass 'database' object to your controllers instead of models = require('./../models'); and from your controllers you have access to your model : database.User

answered Jan 14 '16 at 19:35

Krzysztof Sztompka

7,066
4
33
48

Hm, ReferenceError: database is not defined at exports.index (c:\webserver\data\htdocs\newcloudy\controllers\user.js:5:5) http://pastebin.com/tS89jbBp – Designlenta Gogo Jan 14 '16 at 19:44
how you pass database object from server script to controller? It looks like there is some problem with it – Krzysztof Sztompka Jan 14 '16 at 19:47
var database = require('./../app.js'); and TypeError: Cannot read property 'findAll' of undefined. – Designlenta Gogo Jan 14 '16 at 19:51
No, you can't do it like that. You run your controllers in server script. You have to pass database instance to your controllers in server script. Please check this http://stackoverflow.com/a/33096610/4138339 and 'second edit'. – Krzysztof Sztompka Jan 14 '16 at 19:53

George Mathew · Answer 3 · 2017-02-07T20:44:12.363

Your model/index.js looks fine. In your controller try findAll() method inside sequelize.sync().then(function () {

Here is my approach to the problem

nb: instead of models/index.js i have a similar db.js file inside config folder which having the dbconnection scripts and sequelize object.

My userController.js will look like (working code) :

var db = require('./../config/db'),
seq = db.seq,
Sequelize = db.Sequelize;

module.exports = function(app) {
    app.get('/getUsers',function(req,res){
        var Users = require('../models/UserModel')(app); //since i am binding to a single object - app
        seq.sync().then(function () {
            Users.findAll({
               attributes: ['usr_code', 'usr_name']
           }).then(function (users) {
                    users.forEach(function(user,index,arr){
                        console.log(user.usr_code);
                    });
                });
        });
    });

}

Hope this helps you. :)

score 0 · Answer 4 · edited Jan 02 '21 at 11:51

0

I also had the same issue, you need to check your table name that's it.

edited Jan 02 '21 at 11:51

Eric Aya

69,473
35
181
253

answered Jan 02 '21 at 11:17

Arshad

333
4
10

score 0 · Answer 5 · answered Jun 01 '21 at 07:30

0

you may have this issue when relationships or associations are not defined with db connection.

answered Jun 01 '21 at 07:30

Mohammed Tariq

13
1
5

score 0 · Answer 6 · answered Jul 29 '22 at 19:02

0

This error is because you intend to use Sequelise in the controller before fully connecting it to the database

try use .then or async await

answered Jul 29 '22 at 19:02

hassan zarghami

175
6

score 0 · Answer 7 · answered Sep 30 '22 at 02:20

0

I just had the same problem, a few minutes ago.

This is how I imported the models:

const {product: Product, cartItem: CartItem} = require('../models');

answered Sep 30 '22 at 02:20

Reinier Garcia

1,002
1
11
19

TypeError: Cannot read property 'findAll' of undefined (expressjs)

7 Answers7

Linked