Skip to content

Commit 5262658

Browse files
author
Joe Alves
committed
Centralizing redundant hook function dealing with password/salt for user
1 parent ad89f39 commit 5262658

File tree

1 file changed

+9
-12
lines changed

1 file changed

+9
-12
lines changed

generated/server/db/models/user.js

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -45,17 +45,14 @@ module.exports = db.define('user', {
4545
}
4646
},
4747
hooks: {
48-
beforeCreate: function (user) {
49-
if (user.changed('password')) {
50-
user.salt = user.Model.generateSalt();
51-
user.password = user.Model.encryptPassword(user.password, user.salt);
52-
}
53-
},
54-
beforeUpdate: function (user) {
55-
if (user.changed('password')) {
56-
user.salt = user.Model.generateSalt();
57-
user.password = user.Model.encryptPassword(user.password, user.salt);
58-
}
59-
}
48+
beforeCreate: setSaltAndPassword,
49+
beforeUpdate: setSaltAndPassword
6050
}
6151
});
52+
53+
function setSaltAndPassword(user) {
54+
if (user.changed('password')) {
55+
user.salt = user.Model.generateSalt();
56+
user.password = user.Model.encryptPassword(user.password, user.salt);
57+
}
58+
}

0 commit comments

Comments
 (0)