web-dev-qa-db-ja.com

beforeCreateフックを使用して作成モデルを逐次化する

フックbeforeCreateを次のように定義しました:

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;
    });
  });
};

モデルを作成すると

  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
  });

これから私が得るものは

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

コードのすべての部分が機能しているようです。ユーザースキーマを作成するとbeforeCreateが呼び出され、データベースに書き込まなかった場合を除いて、パスワードのハッシュコードが適切に生成されます。

私は非常に重要で明白なコードが欠けていることを確信していますが、問題がどこにあるのかを見つけることができません(aghh)。助けてくれてありがとう!

20
Alon

フックはSequelizeで非同期に呼び出されるため、完了したら完了コールバックを呼び出す必要があります。

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);
  });
});

(または、フックからプロミスを返すことができます)

34
robertklep

Sequelizeの新しいバージョンでは、フックにコールバック関数はなくなりましたが、約束します。したがって、コードは次のようになります。

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