在更新后中间件中发现的文档修订号不正确

Question

我有一个猫鼬插件，我用它来增加文档修订号（__v），以及创建修订本身。该插件涵盖了文档Doc.save()中间件功能，以及查询update和findOneAndUpdate中间件功能。

module.exports = ( schema, options ) => {
    _.forEach( [ 'save','update', 'findOneAndUpdate' ], query => {
        // Since the Doc.save() middleware is the only document update middleware function, just isolate that one
        if( query === 'save' )
            schema.pre( query, function( next ) {
                this.increment()
                next()
            } )
        // The rest are query updates
        else
            schema.pre( query, function() {
                this.update( {}, { $inc: { __v: 1 } } )
            })

        // Create revisions for each document update
        schema.post( query, docData => {
            Mongoose.models.Revision.createRevision( {
                docsId: this._id,
                revision: docData.__v, // <-- This is the wrong number. It's one less than it should be
                document: { /* Stuff.. */ }
                // More stuff
            }, ( err, revision ) => {
                // CB Stuff
            })
        })
    })
}

因此，这主要按预期工作。对于文档和查询交互，文档的__v值都会递增，并且还会创建修订文档。我坚持的部分与查询中间件功能有关，update和findOneAndUpdate。即使__v通过 pre 事件在文档中更新，post事件中的this.__v值似乎也看不到更新的值。因此，这意味着将创建修订版本并引用文档的错误修订版本号。

这太奇怪了，因为当我在数据库中查看文档时，__v文档实际上确实会更新，但是当我控制台时.log帖子中的this.__v update..它在更新之前看到了修订号。

对于临时修复，如果它是一个查询 MW 函数，我只是手动递增它：

schema.post( query, docData => {
    Mongoose.models.Revision.createRevision( {
        docsId: this._id,
        revision: ( query === 'save' // Temporary fix..
            ? docData.__v
            : docData.__v+1 ) // Add +1 if its a query function 
        document: { /* Stuff.. */ }
        // More stuff
    }, ( err, revision ) => {
        // CB Stuff
    })
})

但显然，这只是一个创可贴，所以如果有真正的解决方案，那就太好

了

知道吗？

Answer 1

即使__v通过 pre 事件在文档中更新，post事件中的this.__v值似乎也看不到更新的值。

这很可能是因为您的update和findOneAndUpdate中间件没有使用异步回调来等待操作完成，然后再继续（您确实为save中间件实现了）。

所以让它使用回调：

schema.pre( query, function(next) {
  this.update( {}, { $inc: { __v : 1 } }, next);
})