sequelize beforeSave hook not firing

Question

I've generated models with sequelize-auto, and need to use a beforeSave hook (see here). The hook isn't firing as far as I can tell. sequelize version ^4.20.1, sequelize-auto version ^0.4.29, express version ~4.15.5. Can anyone help?

module.exports = function(sequelize, DataTypes) {
  return sequelize.define('trad', {
    id: {
      type: DataTypes.INTEGER,
      allowNull: false,
      primaryKey: true,
      autoIncrement: true
    },
    geom: {
      type: DataTypes.GEOMETRY('POINT', 4326),
      allowNull: true
    },
    ...
  }, {
    hooks: {
      beforeSave: (instance, options) => {
        console.log('Saving geom: ' + instance.geom);
        if (instance.geom && !instance.geom.crs) {
          instance.geom.crs = {
            type: 'name',
            properties: {
              name: 'EPSG:4326'
            }
          };
        }
      }
    },
    tableName: 'trad',
    timestamps: false,
  });
};

Here's the code from the PUT request:

// Update (PUT)
router.put('/table/:table/:id', function(req, res, next) {
  db.resolveTableName( req )
  .then( table => {
    const primaryKey = table.primaryKeyAttributes[0];
    var where = {};
    where[primaryKey] =  req.params.id;
    console.log('Put - pkey: ' + primaryKey);

    auth.authMethodTable( req )
    .then( function() {
      table.update( req.body, {
        where: where,
        returning: true,
        plain: true
      })
      .then( data => {
        res.status(200).json( data[1].dataValues );
      })
      .catch( function (error ) {
        res.status(500).json( error );
      });
    })
    .catch( function( error ) {
      res.status(401).json('Unauthorized');
    });
  })
  .catch( function(e) {
    res.status(400).json('Bad request');
  });
});

Answer 1

The beforeSave hook fires for individual model instances, but not for bulk updates unless specified. In your case, you have one of two options:

(1) Pass individualHooks to your query:

table.update( req.body, {
  where: where,
  returning: true,
  individualHooks: true
  plain: true
})

(2) Fetch the model instance before updating:

table.findById(req.params.id)
  .then(function(instance) {
    instance.update(req.body, {
      returning: true
      plain: true
    })
  })