Merge pull request #967 from mleanos/user-model-validate-bug

[bug] Solves User model validation issue with tests [fixes #966]
meanjs · Oct 10, 2015 · 19aea2f · 19aea2f
2 parents 9c78c63 + 288f0b4
commit 19aea2f
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/modules/users/server/models/user.server.model.js b/modules/users/server/models/user.server.model.js
@@ -116,7 +116,7 @@ UserSchema.pre('save', function (next) {
  * Hook a pre validate method to test the local password
  */
 UserSchema.pre('validate', function (next) {
-  if (this.provider === 'local' && this.password) {
+  if (this.provider === 'local' && this.password && this.isModified('password')) {
     var result = owasp.test(this.password);
     if (result.errors.length) {
       var error = result.errors.join(' ');