web-dev-qa-db-fra.com

Créez un modèle avec le crochet beforeCreate

J'ai défini mon crochet avant de créer comme suit:

module.exports = function (sequelize, DataTypes) {
  var userSchema = sequelize.define('User', {
  // define...
  });
  userSchema.beforeCreate(function (model) {
    debug('Info: ' + 'Storing the password');    
    model.generateHash(model.password, function (err, encrypted) {
      debug('Info: ' + 'getting ' + encrypted);

      model.password = encrypted;
      debug('Info: ' + 'password now is: ' + model.password);
      // done;
    });
  });
};

et quand je crée un modèle

  User.create({
    name:           req.body.name.trim(),
    email:          req.body.email.toLowerCase(),
    password:       req.body.password,
    verifyToken:    verifyToken,
    verified:       verified
  }).then(function (user) {
    debug('Info: ' + 'after, the password is ' + user.password);    
  }).catch(function (err) {
    // catch something
  });

Maintenant, ce que j'en retire c'est

Info: Storing the password +6ms
Info: hashing password 123123 +0ms    // debug info calling generateHash()
Executing (default): INSERT INTO "Users" ("id","email","password","name","verified","verifyToken","updatedAt","createdAt") VALUES (DEFAULT,'[email protected]','123123','wwx',true,NULL,'2015-07-15 09:55:59.537 +00:00','2015-07-15 09:55:59.537 +00:00') RETURNING *;

Info: getting $2a$10$6jJMvvevCvRDp5E7wK9MNuSRKjFpieGnO2WrETMFBKXm9p4Tz6VC. +0ms
Info: password now is: $2a$10$6jJMvvevCvRDp5E7wK9MNuSRKjFpieGnO2WrETMFBKXm9p4Tz6VC. +0ms
Info: after, the password is 123123 +3ms

Il semble que chaque partie du code fonctionne. La création d'un schéma utilisateur invoquera beforeCreate, qui génère correctement le code de hachage pour le mot de passe .... sauf qu'il n'a pas écrit dans la base de données!

Je suis certain qu'il me manque un morceau de code très important et évident, mais je n'arrive pas à trouver où est le problème (aghh). Toute aide appréciée!

node.jssequelize.js
20
Alon

Les hooks sont appelés de manière asynchrone dans Sequelize, vous devez donc appeler le rappel de fin lorsque vous avez terminé:

userSchema.beforeCreate(function(model, options, cb) {
  debug('Info: ' + 'Storing the password');    
  model.generateHash(model.password, function(err, encrypted) {
    if (err) return cb(err);
    debug('Info: ' + 'getting ' + encrypted);

    model.password = encrypted;
    debug('Info: ' + 'password now is: ' + model.password);
    return cb(null, options);
  });
});

(Alternativement, vous pouvez retourner une promesse du crochet)

34
robertklep

Pour les versions plus récentes de Sequelize, les crochets n'ont plus de fonctions de rappel mais promettent. Par conséquent, le code ressemblerait davantage à ce qui suit:

userSchema.beforeCreate(function(model, options) {
    debug('Info: ' + 'Storing the password');

    return new Promise ((resolve, reject) => {
        model.generateHash(model.password, function(err, encrypted) {
            if (err) return reject(err);
            debug('Info: ' + 'getting ' + encrypted);

            model.password = encrypted;
            debug('Info: ' + 'password now is: ' + model.password);
            return resolve(model, options);
        });
    });
});
16
Rene