Punch In
Punch In

Reputation: 39

Sequelize.js/Node.js/Express.js: Tasks.findAll()returns a TypeError:Cannot read property 'findAll' of undefined

code is supposed to return a a JSON object with empty tasks when requesting /tasks instead it returns a message error- TypeError: cannot read property 'findAll' of undefined. The source of errors as per the message comes from routes >tasks.js see below for screenshots or/and live code on sandbox.

Project Folder: enter image description here

sandbox

some codes: src>models >tasks.js

module.exports = (sequelize, DataType) => {
  const Tasks = sequelize.define(
    "Tasks",
    {
      id: {
        type: DataType.INTEGER,
        primaryKey: true,
        autoIncrement: true
      },
      title: {
        type: DataType.STRING,
        allowNull: false,
        validate: {
          notEmpty: true
        }
      },
      done: {
        type: DataType.BOOLEAN,
        allowNull: false,
        defaultValue: false
      }
    },
    {
      classMethods: {
        associate: models => {
          Tasks.belongsTo(models.Users);
        }
      }
    }
  );
  return Tasks;
};

src>routes>tasks.js

module.exports = app => {
  const Tasks = app.db.models.tasks;
  app.get("/tasks", (req, res) => {
    Tasks.findAll({}).then(tasks => {//source of error as per error message
      res.json({ tasks: tasks });
    });
  });
};

src >db.js

var path = require("path");

var fs = require("fs");

var Sequelize = require("sequelize");
//const config = require("./libs/config.js");

var sequelize = null;
let db = null;

module.exports = app => {
  if (!db) {
    const config = app.libs.config;
    sequelize = new Sequelize(
      config.database,
      config.username,
      config.password,
      config.params
    );
    db = {
      sequelize,
      Sequelize,
      models: {}
    };
    const dir = path.join(__dirname, "models");
    fs.readdirSync(dir).forEach(file => {
      const modelDir = path.join(dir, file);
      const model = sequelize.import(modelDir);
      db.models[model.name] = model;
    });
    Object.keys(db.models).forEach(key => {
      db.models[key].options.classMethods.associate(db.models);
    });
  }
  return db;
};

src>index.js

var express = require("express");
var consign = require("consign");
var app = express();

consign({ cwd: "src" })
  .include("./libs/config.js")
  .then("db.js")
  .then("./libs")
  .then("./routes")
  .into(app);

Upvotes: 2

Views: 117

Answers (1)

Tobino
Tobino

Reputation: 942

On file routes/tasks.js line 2, add a capital on task;

  const Tasks = app.db.models.Tasks;

Then it should works.

Upvotes: 3

Related Questions