Mongoose pre.save() 异步中间件无法创建记录

Question

我正在使用keystone@0.2.32。 我想将帖子类别更改为树形结构。 下面的代码运行良好，除了当我创建一个类别时，它进入了死锁：

var keystone = require('keystone'),
	Types = keystone.Field.Types;
/**
 * PostCategory Model
 * ==================
 */
var PostCategory = new keystone.List('PostCategory', {
	autokey: { from: 'name', path: 'key', unique: true }
});
PostCategory.add({
	name: { type: String, required: true },
	parent: { type: Types.Relationship, ref: 'PostCategory' },
	parentTree: { type: Types.Relationship, ref: 'PostCategory', many: true }
});
PostCategory.relationship({ ref: 'Post', path: 'categories' });
PostCategory.scanTree = function(item, obj, done) {
	if(item.parent){
		PostCategory.model.find().where('_id', item.parent).exec(function(err, cats) {
			if(cats.length){
				obj.parentTree.push(cats[0]);
				PostCategory.scanTree(cats[0], obj, done);
			}
		});
	}else{
		done();
	}
}
PostCategory.schema.pre('save', true, function (next, done) { //Parallel middleware, waiting done to be call
	if (this.isModified('parent')) {
        this.parentTree = [];
		if(this.parent != null){
			this.parentTree.push(this.parent);
			PostCategory.scanTree(this, this, done);
		}else
			process.nextTick(done);
    }else
		process.nextTick(done); //here is deadlock.
    next();
});
PostCategory.defaultColumns = 'name, parentTree';
PostCategory.register();

非常感谢。

Answer 1

正如我解释的问题所解释的那样，您在此处登录Keystone：https://github.com/keystonejs/keystone/issues/759

这似乎是猫鼬中的一个可重现的错误，它会阻止中间件在以下情况下解析：

• 并行中间件运行，执行查询，后跟
• 执行查询的串行中间件运行

将 Keystone 的autokey中间件更改为以并行模式运行可能会导致其他用例中的错误，因此无法完成。答案是在串行模式下实现 parentTree 中间件，而不是并行模式。

另外，我注意到的其他一些事情：

• 中间件中存在一个错误，其中第一个父项被添加到数组中两次。
• scanTree方法最好作为 schama 上的方法实现
• 可以将 findById 方法用于更简单的父查询

架构方法如下所示：

PostCategory.schema.methods.addParents = function(target, done) {
    if (this.parent) {
        PostCategory.model.findById(this.parent, function(err, parent) {
            if (parent) {
                target.parentTree.push(parent.id);
                parent.addParents(target, done);
            }
        });
    } else {
        done();
    }
}

固定的中间件如下所示：

PostCategory.schema.pre('save', function(done) {
    if (this.isModified('parent')) {
        this.parentTree = [];
        if (this.parent != null) {
            PostCategory.scanTree(this, this, done);
        } else {
            process.nextTick(done);
        }
    } else {
        process.nextTick(done);
    }
});

Answer 2

我认为这是

Keystone.js的错误。我已经更改了架构插件.js 104 行

从

this.schema.pre('save', function(next) {

自

this.schema.pre('save', true, function(next, done) {

并从第 124 行更改为以下内容，

		// if has a value and is unmodified or fixed, don't update it
		if ((!modified || autokey.fixed) && this.get(autokey.path)) {
			process.nextTick(done);
			return next();
		}
		var newKey = utils.slug(values.join(' ')) || this.id;
		if (autokey.unique) {
			r = getUniqueKey(this, newKey, done);
			next();
			return r;
		} else {
			this.set(autokey.path, newKey);
			process.nextTick(done);
			return next();
		}

它有效。