Was mache ich falsch in dieser Mongoose einzigartigen Pre-Save-Validierung?

Question

Ich versuche ein Benutzermodell zu erstellen, das einen eindeutigen Benutzernamen hat. Hier ist der Code für sie:

var mongoose = require("mongoose"); 

var Schema = mongoose.Schema; 

var UserSchema = new Schema({ 
    username: String, 
    password: String, 
}); 

UserSchema.virtual("password_confirmation").get(function() { 
    return this.pw_conf; 
}).set(function(value) { 
    this.pw_conf = value; 
}); 

UserSchema.path("username").required(true); 
UserSchema.path("password").required(true); 

UserSchema.pre("save",function(next, done) { 
    var self = this; 
    mongoose.models["User"].findOne({username : self.username},function(err, user) { 
     if(user) { 
      self.invalidate("user","username must be unique"); 
     } 
     done(); 
    }); 
    next(); 
}); 

UserSchema.pre("save",function(next) { 
    if(this.pw_conf !== this.password) { 
     next(new Error("Must specify the password confirmation")); 
    } 
    else { 
     next(); 
    } 
}); 

module.exports = mongoose.model("User",UserSchema); 

ich auch, wenn die Einzigartigkeit Werke zu sehen, die Prüfung:

var User = require("./users"), 
    mongoose = require("mongoose"); 
var u = new User(); 

mongoose.connect('mongodb://localhost/my_database'); 

u.username = "me"; 
u.password = "password"; 
u.password_confirmation = "password"; 
u.save(function(err) { 
    if(err) { 
     console.log(err); 
    } 
    mongoose.disconnect(); 
}); 

Problem ist, ist es nicht. Jedes Mal, wenn ich den Code ausführe, erhalte ich ein neues Objekt. Ich bin mir bewusst, dass es wahrscheinlich andere Möglichkeiten gibt, Einzigartigkeit zu gewährleisten, aber ich würde es gerne so machen. Sollte ich nicht anrufen done, nachdem ich das Ergebnis der findOne Methode behandeln? Rufe ich next falsch?

Answer 1

verwenden parallel middleware (mit next und done Parameter), müssen Sie true als zweiten Parameter zu übergeben.

Darüber hinaus gibt es zwei Möglichkeiten:

Ihr self.invalidate Anruf "username" statt "user" werden verweisen sollte. Wenn das nicht hilft, können Sie die Dinge nicht explizit durch einen Fehler Objekt done vorbei, wenn Sie den Speichervorgang abbrechen wollen:

UserSchema.pre("save", true, function(next, done) { 
    var self = this; 
    mongoose.models["User"].findOne({username: self.username}, function(err, user) { 
     if(err) { 
      done(err); 
     } else if(user) { 
      self.invalidate("username", "username must be unique"); 
      done(new Error("username must be unique")); 
     } else { 
      done(); 
     } 
    }); 
    next(); 
}); 

Answer 2

http://mongoosejs.com/docs/api.html#schematype_SchemaType-unique ist der Weg zu gehen. Es verwendet tatsächliche MongoDb-Indizes, um sicherzustellen, dass Ihr Feld eindeutig ist. Keine Notwendigkeit für .pre Middleware.

Viel Spaß!

Answer 3

Haben Sie darüber nachgedacht mit einem asynchronen Validator fangen den Fehler?

UserSchema.path('username').validate(function (value, done) { 
    User.count({ username: value }, function (error, count) { 
    // Return false if an error is thrown or count > 0 
    done(!(error || count)); 
    }); 
}, 'unique'); 

Answer 4

Es gibt ein wirklich gutes Plugin für Mungo, das wirklich einfach zu installieren und zu verwenden ist. Die Dokumentation ist hervorragend und es hat für mich zum ersten Mal funktioniert.

Es gibt jedoch ein Problem beim erneuten Speichern.

https://npmjs.org/package/mongoose-unique-validator