Skip to content

Commit c051b37

Browse files
authored
Merge pull request #68 from FullstackAcademy/fixUserHook
Fixed beforeCreate hook for user
2 parents 78a045e + aafbfa7 commit c051b37

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

generated/server/db/models/user.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,10 @@ module.exports = db.define('user', {
4646
},
4747
hooks: {
4848
beforeCreate: function (user) {
49-
user.salt = user.Model.generateSalt();
50-
user.password = user.Model.encryptPassword(user.password, user.salt);
49+
if (user.changed('password')) {
50+
user.salt = user.Model.generateSalt();
51+
user.password = user.Model.encryptPassword(user.password, user.salt);
52+
}
5153
},
5254
beforeUpdate: function (user) {
5355
if (user.changed('password')) {

0 commit comments

Comments
 (0)