Mongoose: pre、postミドルウェアはfindByIdAndUpdateで実行されません

意図的に。 フックを呼び出すためのドキュメントがあります。

訂正、フックを呼び出すためのドキュメントはありません。

そう？ pre、postミドルウェアを呼び出したい場合は、最初のアプローチを使用する必要がありますか？

はい、それは正しいです。 Model.update、findByIdAndUpdate、findOneAndUpdate、findOneAndRemove、findByIdAndRemoveは、データベースで直接実行されるすべてのコマンドです。

これはガイドで明確に説明する必要があります。特に、同じページで検証について話していて、 findByIdAndUpdateを「より良い」と説明している場合はそうです。

「より良い」リンクをクリックすると、の完全なドキュメントに移動します。
検証などを説明する方法。

プルリクエストを送信して、より良いと思うものを追加してください。 これは
そうするのはとても簡単です：
https://github.com/LearnBoost/mongoose/blob/master/CONTRIBUTING.md

2012年10月31日水曜日午後6時3分、Jesse [email protected]書き込み：

これは、ガイドhttp://mongoosejs.com/docs/documents.htmlで明確に説明する必要があります。
特に、同じページで検証について話している場合は、
findByIdAndUpdateを「より良い」と表現する

—
このメールに直接返信するか、Gi tHubhttps：//github.com/LearnBoost/mongoose/issues/964#issuecomment-9967865で表示してください。

アーロン
@aaronheckmann https://twitter.com/#!/aaronheckmann

ミドルウェアドキュメントにメモを追加

プルリクエスト： https ：//github.com/LearnBoost/mongoose/pull/1750

こんにちは、

問題が解決したことは知っていますが、答えに完全に満足しているわけではありません。 findOneAndUpdateおよび他の同様の操作のために、少なくとも保存後または更新後のミドルウェアがあるべきではありませんか？ ドキュメントが返されるので大丈夫のようです。 プレミドルウェアやModel.updateには適していません、私は同意します。

これにより、サポートできるはずの一部の操作を現在認識していないmongoosasticなどのプラグインの機能が大幅に向上します。

ミドルウェアがない場合、プラグインで更新後の操作を管理する方法について誰かが考えていますか？

ありがとう

@albanm特定のメソッドはマングースを完全にバイパスするため、ミドルウェアフックを取得しません。 AFAIK、フックを実行させる唯一の方法は、上記のように別々のfind()とsave()の呼び出しを使用することです。

私はそれを理解し、それは理にかなっています。 特定のメソッドの前にフェッチがないため、プレミドルウェアは問題外です。 しかし、それでも、mongooseは、更新されたドキュメントを返し、いくつかのポストフックをトリガーする更新操作をラップできる必要があります。

@albanmは正しいと思います。同じ関数を使用すると、同じ関数が必要になる場合があります。 これらの「直接更新」メソッドを、フックが存在するかどうかを確認するためのインターセプトでラップするのはどうですか？ フックが存在する場合はそれを使用するか、そうでない場合は元の更新メソッドを呼び出します。

+1

+1

+1

+1

：+1：

+1

：-1：
機能要求を受け取りますが、ミドルウェアを使用する時間が長くなると、データベース更新スクリプトや通常の操作では日常的ではないその他の項目を呼び出すときに、ミドルウェアを頻繁にバイパスする必要があります。 静的メソッドを使用することにより、上記の機能要求をすでに実装しているカスタムラッパーを非常に簡単に作成できます。 マングースは現在バージョン4のアイデアを受け入れているため、この規模のAPIの変更がv3で発生する可能性はほとんどありません。 これをv4ディスカッションに移してください。

ただし、保存呼び出しでミドルウェアを無効にできた場合は、：+ 1：になります。 別の関数から渡されたマングースオブジェクトを見つけて、単に.saveを実行したいことがよくあります。この状況では、新しいクエリを作成するよりも.saveを実行する方が望ましいです。 これが可能であれば、それを指摘してください

いずれにせよ、素晴らしいライブラリ。 素晴らしいメンテナへの称賛。 それなしで私がどのように動作するかわかりません。

+1

これらのメソッドのフックを追加する簡単な方法は、埋め込み関数を上書きすることです。

_update = UserModel.update
UserModel.update = (conditions, update) ->
  update.updatedAt or= new Date
  _update.apply this, arguments

その後、mongooseを呼び出すたびに、データのupdatedAtキーが修正されます。

このモデルのリンボを試してみてください。 これはマングースモデルの単純なラッパーであり、すべてのスキーマへの静的/メソッド/上書きのバインドをサポートし、rpcメソッドを呼び出してmongodbをクエリします。

+1、この機能が必要です...

私はその理由をある程度理解していますが、アトミック更新のフックがないため、実際にはMongooseはやや無意味になっています。 アトミック更新を使用すると、検証やデフォルトなどが実行されないため、ODMを使用する目的全体が無効になります。 検索/保存を使用すると機能しますが、これが常に使用されるという保証はありますか？

さらに、通常、不可分操作ではないため、検索/保存は避けようとします。 MongoDBは、強力なアトミッククエリおよび更新機能を提供することにより、トランザクションサポートの欠如を補います。 したがって、これらのアトミック操作を使用しますが、ミドルウェアのサポートがない場合、MongooseはネイティブのMongoClientよりも多くの価値を提供しません。

http://aaronheckmann.tumblr.com/post/48943525537/mongoose-v3-part-1-versioningの例でさえ、 updateを使用するため、ミドルウェアをバイパスします。 バージョン管理またはミドルウェアを適切に使用できますが、両方を組み合わせることはできませんか？ 本当に、それを持っていることのポイントはどこにありますか？

私は技術的な理由さえ完全には理解していません： update ＆coの場合。 データベース操作をラップアラウンドします。実際に更新を行う前に検証/カスタマイズを実行できるように、呼び出しをインターセプトしてクエリオブジェクトを渡すことができないのはなぜですか。

@joerx +1で十分です.. :)しかし、あなたの推論は完璧です。

3.9.xブランチは、 findとfindOneのプリフックとポストフックをサポートしています。 findOneAndUpdateとupdateのサポートを簡単に追加できるはずです。

この機能は統合されていますか？

したがって、 pre('findOneAndUpdate') $フックとpost('findOneAndUpdate')フックはマスターにあり、更新フックはまだありません。 これらのいずれかを含むリリースはまだありません。

では、.update（）の後に事前保存がトリガーされるのでしょうか？

いいえ。 Query.update()用に別のupdate()フックがあります。 保存フックはクエリフックとは異なります。

@ vkarpov15アップデートフックのサポートドキュメントにリンクしていただけますか？ pre('findOneAndUpdate')とpost('findOneAndUpdate')がいつリリースされるかについて何か言いたいことはありますか？

@karlstanton use 4.0.0-rc2、 npm install mongoose@unstable :)

'use strict';

var Promise  = require('bluebird');
var mongoose = Promise.promisifyAll(require('mongoose'));

var counterSchema = new mongoose.Schema({
    total: {
        type:    Number,
        default: 0
    }
});

counterSchema.post('findOneAndUpdate', function (doc) {
    console.log(doc.total);
});

var Counter = mongoose.model('Counter', counterSchema);

Promise.coroutine(function *() {
    yield mongoose.connectAsync(process.env.MONGODB_URI);
    console.log('Connected');
    let counter = yield Counter.createAsync({});
    console.log(`${counter.total}`);
    for (let i = 0; i < 10; i++) {
        yield Counter.findOneAndUpdateAsync({ _id: counter.id }, { $inc: { total: 1} });
    }
})();

0
0
1
2
3
4
5
6
7
8
9

一歩遅れているようです。 私は何が欠けていますか？

AH、 newフラグのデフォルトの重大な変更。 基礎となるドライバーとの一貫性については何か言いたいことがありますが、特にこの新しいフックを検討する場合、それは本当に直感に反していると言わざるを得ません。

@neverfox :)

ええ@neverfox良いキャッチ。 この変更はリリースノートに記載されており、＃2262で変更された理由について詳しく説明しています。

その理由は、mongodbノードドライバー、mongodbシェル、実際のmongodbサーバーなどでnewがデフォルトでfalseであり、ラッパーレイヤーが非標準のデフォルトを設定すると、開発者の生活が困難になるためです。 これをひどく間違っているモジュールの私の標準的な例は、 gulp-uglifyです。これは、uglify-jsのデフォルトの束をオーバーライドします。

問題は解決されているようですが、バージョン4.0.2の機能はまだ不安定なバージョンのままですか？ Scema.pre（ 'update'）を使用したfindOneAndUpdateまたは4.0.2バージョンを使用したSchema.pre（ 'findOneAndUpdate'）では実行されないようです。 関数に渡さなければならないものがありませんか？

プレフック@CaptainStaplerzをどのように宣言していますか？

findOneAndUpdateミドルウェアフックは4.0.2で利用できますか？ これを使用するために3.8から最新のマングース4.0.2にアップグレードしましたが、middlware Document.schema.post（ 'findOneAndUpdate'、関数（doc）がsave（）やremove（）のようにトリガーされません

@honitusはあなたのコードを見せてくれます

@ vkarpov15-迅速な対応に感謝します。

blog.controller、js

// Updates an existing Blog in the DB, adds comment
exports.update = function(req, res) {

Blog.findOneAndUpdate(
     {"_id": req.body._id,},
      {"$push": {"comments": buildComment}},
     {safe: true, upsert: true}, function (err, workspace) {
      if (err) {
         return handleError(res, err);
       }
       return res.send(200);
      }
   );

}

blog.socket.js

/**
 * Broadcast updates to client when the model changes
 */

'use strict';

var Blog= require('./blog.model');

exports.register = function(socket) {
//SAVE WORKS
  Blog.schema.post('save', function (doc) {
    onSave(socket, doc);
  });

// IS NOT TRIGGERED :(
 Blog.schema.post('findOneAndUpdate', function (doc) {
    onComment(socket, doc);
  });

  Blog.schema.post('remove', function (doc) {
    onRemove(socket, doc);
  });
}

//SAVE WORKS when a new blog is created
function onSave(socket, doc, cb) {
  socket.emit('blog:save', doc);
}

// IS NOT TRIGGERED :(
function onComment(socket, doc, cb) {
  socket.emit('blog:findOneAndUpdate', doc);
}

function onRemove(socket, doc, cb) {
  socket.emit('blog:remove', doc);
}

@ vkarpov15再開していただきありがとうございます。 これは、findOneAndUpdateのフックが4.0.2になく、4.0.3に含めることを計画していることを意味しますか？

@ vkarpov15フックを宣言するコードは次のとおりです。

...
var TodoSchema = new mongoose.Schema({
  name: {type: String, required: true},
  note: String,
  completed: {type: Boolean, default: false},
  updatedAt: {type: Date, default: Date.now},
  user: {
    type: mongoose.Schema.ObjectId,
    ref: 'Users'
  }
});
...
// Not executed
TodoSchema.pre('update', function() {
  console.log('------------->>>>>> update updatedAt')
  this.updatedAt = Date.now();
});
// Not executed
TodoSchema.pre('findOneAndUpdate', function() {
  console.log('------------->>>>>> update updatedAt')
  this.updatedAt = Date.now();
});

そして、これが私がアップデートと呼ぶところです：

...
router.route('/:id')
.put(function(req, res, next) {
  TodoModel.findOneAndUpdate({_id: req.params.id, user: req.user.id}, req.body, {new: true}, function(err, post) {
    if(err) return next(err);
    if(post) {
      res.status(200).json(post);
    }
    else {
      next(newSystemError(errorCodes.TODO_NOT_FOUND, undefined, req.params.id));
    }
  });
});

私の知る限り、 findOneAndUpdate()フックはそこにあるはずですが、機能していない場合はバグです

@CaptainStaplerz試してみてください：

TodoSchema.pre('update', function(next) {
  console.log('------------->>>>>> update updatedAt')
  this.updatedAt = Date.now();
  next();
});

また、 console.logステートメントが実行されているのでしょうか、それとも、予期しない結果をもたらすのはDate.now（）の部分だけでしょうか。

@ vkarpov15
ソースコードを変更し、実装の追加＃ 964https：//github.com/Automattic/mongoose/commit/e98ef98e857965c4b2ae3339fdd7eefd2a5a9913で行った変更を追加しました

今では魅力のように機能します。 だから私は修正がメインにチェックインされていないと思います

@honitusマングース4.0.2を使用していますか？ その変更は、実際には4.0.0以降にコミットされています。

honitus $ npmマングースバージョンを表示
4.0.2

@honitusが間違って行っているのは、モデルのコンパイル後にスキーマにフックを追加しているということです。 Model.schema.pre('remove');は機能しないと予想されます。モデルドキュメントの「最初のモデルのコンパイル」を参照してください。 最初にフックをスキーマにアタッチすると、動作するはずです。これが、コードとテストの違いです。

@CaptainStaplerzコードを再現するために私が見つけることができる唯一の方法は、空の更新を使用することです。 以下のコードは機能します

var mongoose = require('mongoose');
mongoose.set('debug', true);
var util = require('util');

mongoose.connect('mongodb://localhost:27017/gh964');

var TodoSchema = new mongoose.Schema({
  name: {type: String, required: true},
  note: String,
  completed: {type: Boolean, default: false},
  updatedAt: {type: Date, default: Date.now},
  user: {
    type: mongoose.Schema.ObjectId,
    ref: 'Users'
  }
});
TodoSchema.pre('update', function() {
  console.log('------------->>>>>> update updatedAt')
  this.updatedAt = Date.now();
});
TodoSchema.pre('findOneAndUpdate', function() {
  console.log('------------->>>>>> update updatedAt')
  this.updatedAt = Date.now();
});

var Todo = mongoose.model('Todo', TodoSchema);

Todo.update({}, { note: "1" }, function(err) {
  if (err) {
    console.log(err);
  }
  console.log('Done');
  process.exit(0);
});

ただし、更新を空にすると、次のようになります。

var mongoose = require('mongoose');
mongoose.set('debug', true);
var util = require('util');

mongoose.connect('mongodb://localhost:27017/gh964');

var TodoSchema = new mongoose.Schema({
  name: {type: String, required: true},
  note: String,
  completed: {type: Boolean, default: false},
  updatedAt: {type: Date, default: Date.now},
  user: {
    type: mongoose.Schema.ObjectId,
    ref: 'Users'
  }
});
TodoSchema.pre('update', function() {
  console.log('------------->>>>>> update updatedAt')
  this.updatedAt = Date.now();
});
TodoSchema.pre('findOneAndUpdate', function() {
  console.log('------------->>>>>> update updatedAt')
  this.updatedAt = Date.now();
});

var Todo = mongoose.model('Todo', TodoSchema);

Todo.update({}, { }, function(err) {
  if (err) {
    console.log(err);
  }
  console.log('Done');
  process.exit(0);
});

更新前フックは実行されなくなりました。 これはあなたが見ているものと一致していますか？

@ vkarpov15
Model.schema.pre（ 'remove'）; 角度フルスタックによって自動的に作成されるスタブの1つです
実装＃964e98ef98を追加しました
下に1行追加するだけです
this._schema.s_.hooks.wrap（ 'findOneAndUpdate'、Query.base.findOneAndUpdate、
4.0.2の1526行目を参照してください-this.model.hooks.wrap（ 'findOneAndUpdate'、Query.base.findOneAndUpdate、

1526行目を-this.schema.s.hooks.wrap（ 'findOneAndUpdate'、Query.base.findOneAndUpdate、
そしてそれは動作します。

これは仕様によるものです。 Mongooseは、モデルのコンパイル後にスキーマフックを変更することを許可しないことになっています（つまり、 mongoose.model()呼び出し）。そのため、これは機能しなくなります。

@ vkarpov15私は今、次のようにフックを機能させることができました。それは愚かなタイプミスか何か他のものだったに違いありません。

TodoSchema.pre('findOneAndUpdate', function(next) {
  console.log('------------->>>>>> update updatedAt: ', this.updatedAt);
  this.updatedAt = Date.now();
  next();
});

ただし、「this」は更新されたモデルを参照していないようです（ただし、「save」フックでは参照していますか？）。したがって、this.updatedAtはundefinedを参照します。

'findOneAndUpdate'フックの 'updatedAt'フィールドを更新するにはどうすればよいですか？

これを明確にするドキュメントをさらに追加する必要があります- findOneAndUpdateを呼び出すと、更新中のドキュメントがメモリに存在しない可能性があるため、 thisオブジェクトは、クエリミドルウェアのドキュメントではなくクエリを参照します。 this.update({ $set: { updatedAt: Date.now() } });お試しください

@ vkarpov15 this.update({ $set: { updatedAt: Date.now() } });を機能させることができませんでしたが、 findOneAndUpdateフックのドキュメントをthis._update['$setOnInsert'].updatedAt=Date.now();で正常に更新できました。

そのような内部状態の調整に依存しないことを強くお勧めします。 this.update()を機能させることができなかったことに、私はかなり驚いています。フックがどのように見えるかを見せていただけますか？

もちろん！ （4.0.2を使用していることに注意してください）

tagSchema.pre('findOneAndUpdate',function(next){
  var self = this;

  //NOTE THAT 'this' in the findOneAndUpdate hook refers to the query, not the document
  //https://github.com/Automattic/mongoose/issues/964

  geoData.country.findOne({'_id':self._update['$setOnInsert'].countryCode}).select('_id name cca2 cca3 ccn3').lean().exec(function(err,country){
    if (err){throw err;}
    if (!country){throw 'no coutnry';}
    self._update['$setOnInsert'].country=country;
    next();
  });
});

アプリの他の場所でドキュメントを初期化するときに明らかにこれを処理できますが、すべてがMongooseに含まれていると便利です。 どんな考えでも歓迎します！

ええ、私は混乱していて、あなたがupdateを使っていると思いました。 以下のスクリプト

var mongoose = require('mongoose');
mongoose.set('debug', true);
var util = require('util');

mongoose.connect('mongodb://localhost:27017/gh964');

var TodoSchema = new mongoose.Schema({
  name: {type: String, required: true},
  note: String,
  completed: {type: Boolean, default: false},
  updatedAt: {type: Date, default: Date.now},
  user: {
    type: mongoose.Schema.ObjectId,
    ref: 'Users'
  }
});
TodoSchema.pre('findOneAndUpdate', function() {
  this.findOneAndUpdate({}, { updatedAt: Date.now() });
});

var Todo = mongoose.model('Todo', TodoSchema);

Todo.findOneAndUpdate({}, { note: "1" }, function(err) {
  if (err) {
    console.log(err);
  }
  console.log('Done');
  process.exit(0);
});

正しく機能し、目的のクエリを実行します。

Mongoose: todos.findAndModify({}) [] { '$set': { note: '1', updatedAt: new Date("Thu, 07 May 2015 20:36:39 GMT") } } { new: false, upsert: false }
Done

だから使ってください

TodoSchema.pre('findOneAndUpdate', function() {
  this.findOneAndUpdate({}, { updatedAt: Date.now() });
});

mqueryの内部状態を手動で操作する代わりに、実際に何をしているのかを本当に理解していない限り、これは通常は悪い考えです。

これは機能していますか？ 私のために働くフックだけが「保存」であり、残りは完全に無視されます。 私は4.0.6で実行しています。 ありがとう

@agjsはコード例を提供してください。

UserController.prototype.updateAvatar = function (req, res) {
    return new Promise(function (resolve, reject) {
        CompanyDetails.update({
            _author: req.user._id
        }, {
            avatarPath: req.files.file
        }, function (error, updated) {
            if (error) {
                reject(error);
            } else {
                resolve(updated);
            }
        });
    }).then(function (resolved) {
        res.sendStatus(204).send(updated);
    }).catch(function (error) {
        next(error);
    })

};

CompanyAvatarSchema.pre('update', function (next) {
    console.log('pre save');
    let VirtualModel = this,
        parent = this.ownerDocument(),
        PATH = path.normalize('./public/images/uploads/avatars/' + parent._id);

    mkdirp(PATH, function (error) {
        if (error) {
            next(error);
        } else {
            fs.rename(VirtualModel.path, path.join(PATH, VirtualModel.name), function (error2) {
                if (error2) {
                    next(error2);
                } else {
                    next();
                }
            });
        }
    });

});

model.createとpre-saveを使用して、別のモデルに別のプリフックがあり、正常に機能します。

更新を行うと、フックが表示されず、console.logも表示されません。 findOneAndUpdateも試しましたが、実際には機能しません...面白いことに、このスレッド全体を調べました。いつものように、公式ドキュメントを確認しましたが、そこでも機能すると主張しています。

上記のコードのCompanyAvatarSchemaとCompanyDetailsの関係は何ですか？

会社の詳細には、サブドキュメントとしてCompanyAvatar.schemaがあります

avatarPath: {
        type: [CompanyAvatar.schema],
        required: true
    }

また、事前フックだけでなく、検証も完全に無視されます。 このサブドキュメントは入力されますが、検証とプレフックの両方を無視します。 私はすべてをグーグルで検索し、これも試しましたが、動作する継ぎ目はありません。 テストのためにクエリを変更して、新しい別名var parent = new Parent（）でモデルを作成して呼び出すと、機能します。

CompanyDetails.update()を呼び出していますが、preフックは別のスキーマで定義されています。 クエリミドルウェアは、ネストされたスキーマのpre('update') atmを起動しません。

また、「検証が完全に無視される」場合のより完全なコード例も提供してください。

これは、プロファイルを更新しているユーザーの検証と事前フックのために作成された私の会社のアバタースキーマです（アバターの写真）：

'use strict';

let mongoose = require('mongoose'),
    mkdirp = require('mkdirp'),
    fs = require('fs'),
    path = require('path'),
    Schema = mongoose.Schema;

let CompanyAvatarSchema = new Schema({
    name: String,
    width: Number,
    height: Number,
    size: Number,
    type: String
});


CompanyAvatarSchema.path('type').validate(function (type) {
    return /^image\//.test(type);
}, 'Image type not allowed!');

CompanyAvatarSchema.path('size').validate(function (size) {
    return size < 5;
}, 'Image too big!');



CompanyAvatarSchema.virtual('path').set(function (path) {
    return this._path = path;
}).get(function () {
    return this._path;
});


CompanyAvatarSchema.virtual('public_path').get(function () {
    var parent = this.ownerDocument();
    var PATH = path.normalize('images/uploads/avatars/' + parent._id);
    if (this.name) {
        return path.join(PATH, this.name);
    }
});

CompanyAvatarSchema.set('toJSON', {
    getters: true
});

CompanyAvatarSchema.pre('findOneAndUpdate', function (next) {
    console.log('pre save');
    let VirtualModel = this,
        parent = this.ownerDocument(),
        PATH = path.normalize('./public/images/uploads/avatars/' + parent._id);

    mkdirp(PATH, function (error) {
        if (error) {
            next(error);
        } else {
            fs.rename(VirtualModel.path, path.join(PATH, VirtualModel.name), function (error2) {
                if (error2) {
                    next(error2);
                } else {
                    next();
                }
            });
        }
    });

});


let runValidatorsPlugin = function (schema, options) {
    schema.pre('findOneAndUpdate', function (next) {
        this.options.runValidators = true;
        next();
    });
};

CompanyAvatarSchema.plugin(runValidatorsPlugin);

let CompanyAvatar = mongoose.model('CompanyAvatar', CompanyAvatarSchema);
module.exports = CompanyAvatar;

これがcompany_detailsスキーマで、company_avatarはサブドキュメントです。

let CompanyDetailsSchema = new mongoose.Schema({
    _author: [{
        type: Schema.Types.ObjectId,
        ref: 'CompanyAccount'
    }],
    company_name: {
        type: String,
        es_indexed: true,
        es_boost: 2.0
    },
    contact_email: {
        type: String,
        es_indexed: true
    },
    website: {
        type: String,
        es_indexed: true
    },
    country: {
        type: String,
        es_indexed: true
    },
    industry: {
        type: String,
        es_indexed: true
    },
    address: {
        type: String,
        es_indexed: true
    },
    about: {
        type: String,
        es_indexed: true
    },
    avatarPath: {
        type: [CompanyAvatar.schema],

    }
});

そして、これがアップデートプロファイルコントローラーと、このアップデートを実行する前に検証/フックする必要があるavatarPathです。

UserController.prototype.updateAvatar = function (req, res, next) {
    let updates = {
        $set: {
            avatarPath: req.files.file
        }
    };
    return new Promise(function (resolve, reject) {
        CompanyDetails.findOneAndUpdate({
            _author: req.user._id
        }, updates, function (error) {
            if (error) {
                reject(error);
            } else {
                resolve('done');
            }
        });
    }).then(function () {
        res.sendStatus(204);
    }).catch(function (error) {
        next(error);
    });

};

基本的に、私のmongodbにはreq.files.fileのフィールドが入力されますが、それ以外の場合、検証は無視され、フックは機能しません。

問題は、pre（ 'findOneAndUpdate'）ミドルウェアがネストされたスキーマで定義されていることです。 現在、mongooseはトップレベルスキーマのクエリミドルウェアのみを起動するため、 CompanyDetailsSchemaで定義されたミドルウェアは次のように起動されます：＃3125

+1

@ vkarpov15わかりました

TodoSchema.pre('findOneAndUpdate', function() {
  this.findOneAndUpdate({}, { updatedAt: Date.now() });
});

プロパティをハード値に設定するように機能しますが、パスワードをハッシュするなど、プロパティをどのように_読み取り、変更_しますか？

TodoSchema.pre('findOneAndUpdate', function() {
  this.findOneAndUpdate({}, { password: hashPassword(.....?....) });
});

何かご意見は？ これはかなり一般的なユースケースですよね？ または、人々は通常、find（）を見つけてから、save（）を分離しますか？

TodoSchema.pre('findOneAndUpdate', function() {
  this.findOneAndUpdate({}, { password: hashPassword(this.getUpdate().$set.password) });
});

@willemmulderで動作するはずです。

@ vkarpov15完璧、ありがとう！ 私は今夜​​それを試みます。 それはpre('update')でも機能するはずですよね？

@ vkarpov15だから、私はそれを使ってみました

schema.pre('update', function(next) {
this.update({}, { $set : { password: bcrypt.hashSync(this.getUpdate().$set.password) } });
next();
});

そして、console.log thisを取得すると、

_update: { '$set': { password: '$2a$10$CjLYwXFtx0I94Ij0SImk0O32cyQwsShKnWh1248BpYsJLIHh7jb66', postalAddress: [Object], permissions: [Object], firstName: 'Willem', lastName: 'Mulder', email: '...@...', _id: 55ed4e8b6de4ff183c1f98e8 } },

これは問題ないように見えますが（直前にそのプロパティを設定しようとしたこともあります）、最終的には、ハッシュ値をデータベースに書き込むのではなく、単に「raw」値を書き込みます。 何か試すことができますか？

変だね。 require('mongoose').set('debug', true);を使用してマングースデバッグモードを有効にしてみて、データベースに送信されるクエリが何であるかを確認してください。

提案をありがとう。 ちょうどそれをしました：

私はこれを実行します：

schema.pre('update', function(next) {
    this.update({}, { password: bcrypt.hashSync(this.getUpdate().$set.password) } );
    console.log(this.getUpdate());
    next();
});

これはconsole.logにこれを返します

{ '$set':
   { password: '$2a$10$I1oXet30Cl5RUcVMxm3GEOeTFOLFmPWaQvXbr6Z5368zbfpA8nFEK',
     postalAddress: { street: '', houseNumber: '', zipCode: '', city: '', country: '' },
     permissions: [ '' ],
     __v: 0,
     lastName: '',
     firstName: '',
     email: '[email protected]',
     _id: 563b0410bd07ce2030eda26d } }

そしてこれはマングースのデバッグ用です

Mongoose: users.update({ _id: ObjectId("563b0410bd07ce2030eda26d") }) { '$set': { password: 'test', postalAddress: { street: '', houseNumber: '', zipCode: '', city: '', country: '' }, permissions: [ '\u001b[32m\'\'\u001b[39m' ], __v: 0, lastName: '', firstName: '', email: '[email protected]', _id: ObjectId("563b0410bd07ce2030eda26d") } } { overwrite: false, strict: true, multi: false, upsert: false, safe: true }
Mongoose: users.findOne({ _id: ObjectId("563b0410bd07ce2030eda26d") }) { fields: { password: 0 } }

どんな手掛かり？

わからない、追跡する新しい問題を開いた。

@ vkarpov15ありがとう、他の問題を追跡します。

@ vkarpov15preフックで進行中のクエリのオプションを設定する正しい方法は次のようになると思います。

  finishSchema.pre('findOneAndUpdate', function (next) {
    this.setOptions({
      new: true,
      runValidators: true
    });
    this.update({}, {
      lastEdited: Date.now()
    });
    next();
  });

ただし、ドキュメントhttp://mongoosejs.com/docs/api.html#query_Query -setOptionsには、これらのオプションについては記載されていません。 これがハックな解決策であると考えられる場合、より適切なものは何でしょうか？

これはドキュメントの問題です。説明するコードは、一見すると機能するように見えます。

そのための別の問題を開くことができますか？

@ vkarpov15はい、動作します。 はっきりしていなかったと思います。

setOptionsはnewとrunValidatorsを正しく適用します。私は、 setOptionsを介してこれらのオプションを設定する方がthis.optionsよりも優先されるかどうかを尋ねていました。

setOptions()が望ましいIMOですが、両方とも機能するはずです。 またはあなたはただすることができます

this.update({}, { lastEdited: Date.now() }, { new: true, runValidators: true });

schema.pre('update', function(next) {
this.update({}, { $set : { password: bcrypt.hashSync(this.getUpdate().$set.password) } });
next();
});

これにより、update（）を呼び出すたびにパスワードが更新されます。 したがって、他のプロパティの値、つまり名前や年齢を変更するだけで、パスワードも更新されますが、これは正しくありません！？

@nlonguitそうなると思います。 ただし、 thisを介して更新されるフィールドにアクセスでき、次のようなことができます。

if (this._fields.password) { // <- I'm sure about this one, check in debugger the properties of this 
    this.update({}, { $set : { password: bcrypt.hashSync(this.getUpdate().$set.password) } });
}

if (this._update.$set.password) { this.update({}, { $set: { password: bcrypt.hashSync(this.getUpdate().$set.password)} }); }

このコードは私にとってうまく機能しています。 ありがとう@akoskm

findByIdAndUpdateのプリフックも追加できるのではないかと思います。 両方のフックを利用できると便利です。

@mickyginger findOneAndUpdateのフックがあり、 findByIdAndUpdate()は内部でfindOneAndUpdate()を呼び出すだけです

私はこれをこのように行い、機能しています。findByIdを実行し、フィールドを更新せずに保存してから、findByIdAndUpdateメソッドを使用します。

dbModel.findById(barId, function (err, bar) {
        if (bar) {

            bar.save(function (err) {
                if (err) throw err;
            });
        }
    });
    dbModel.findByIdAndUpdate(barId, {$set:req.body}, function (err, bar) {
        if (err) throw err;
        res.send('Updated');
    });`

配列の長さを持つようにプロパティを設定しようとしています。

schema.post('findOneAndUpdate', function(result) {
    console.log(result.comments.length);
    this.findOneAndUpdate({}, { totalNumberOfComments: result.comments.length });
});

正しい長さがログに記録されますが、クエリでtotalNumberOfCommentsが設定されることはなく、フィールドは0のままです（スキーマがデフォルトを参照しているため：0）。

フックの最後でconsole.log(this)を実行すると、 queryに次のものが含まれていることがわかります。

_update: { '$push': { comments: [Object] }, totalNumberOfComments: 27 }

デバッグモードをオンにすると、クエリがMongooseによってログに記録されることはありません。

私が間違っていることはありますか、それともこれはバグですか？

@zilions this.findOneAndUpdate({}, { totalNumberOfComments: result.comments.length }).exec();は実際にクエリを実行する必要があります:)投稿保存フックが別の投稿保存フックをトリガーするため、ここで無限再帰が発生することに注意してください。

@ vkarpov15ああ、そうだね！ 次に、代わりにthis.update({} {....}).exec()を使用できます:)
ただし、これを使用すると、 findOneAndUpdateの元の更新も実行されますが、 totalNumberOfCommentsフィールドが完全に設定されます。

例えば：

Post.findOneAndUpdate({_id: fj394hri3hfj}, {$push: {comments: myNewComment}})

次のフックをトリガーします。

schema.post('findOneAndUpdate', function(result) {
    this.update({}, {
        totalNumberOfComments: result.comments.length
    }).exec();
}));

ただし、フックは$pushからcomments $に再びmyNewCommentになるため、重複したエントリが作成されます。

技術的に同じクエリを実行しているため-

schema.post('findOneAndUpdate', function(result) {
    this.update({}, {
        totalNumberOfComments: result.comments.length
    }).exec();
}));

本質的にと同じです

var query = Post.findOneAndUpdate({_id: fj394hri3hfj}, {$push: {comments: myNewComment}});
query.update({}, {
        totalNumberOfComments: result.comments.length
    }).exec();
query.findOneAndUpdate().exec();

新しいクエリを最初から作成する場合は、

schema.post('findOneAndUpdate', function(result) {
    this.model.update({}, { // <--- `this.model` gives you access to the `Post` model
        totalNumberOfComments: result.comments.length
    }).exec();
}));

事前保存フックに触れないでください。

router.put('/:id', jsonParser, function(req, res, next) {

  currentCollection.findByIdAndUpdate(req.params.id, req.body, function (err, item) {
    if (err) {
        res.status(404);
        return res.json({'error': 'Server Error', 'trace': err});
    }
    item.save(); // <=== this is were you save your data again which triggers the pre hook :)
    res.status(200); 
    return res.json({'message': 'Saved successfully'});
  });
});

モデルを定義し、 preフックを定義する順序が重要であることがわかりました。 実演させてください：

動作しません：

// Create Model
let model = Database.Connection.model(`UserModel`, this._schema, `users`);

// Attach Pre Hook
this._schema.pre(`findOneAndUpdate`, function(next) {
    console.log('pre update');
    return next();
});

動作します：

// Attach Pre Hook
this._schema.pre(`findOneAndUpdate`, function(next) {
    console.log('pre update');
    return next();
});

// Create Model
let model = Database.Connection.model(`UserModel`, this._schema, `users`);

これが誰かに役立つことを願っています！

@ nicky-lenaersと同じことを知りました。

'safe'で問題なく動作します。 'delete' 。 モデルの定義後にフックを定義する場合など。

モデルの定義後に'findOneAndUpdate'フックを定義する回避策はありますか？

@ albert-92現時点ではありません

かつてのようなものを手に入れようとしている人のために

SCHEMA.pre('validate', function(done) {
    // and here use something like 
    this.yourNestedElement 
    // to change a value or maybe create a hashed character    
    done();
});

これはうまくいくはずです

SCHEMA.pre('findOneAndUpdate', function(done){
    this._update.yourNestedElement
    done();
});

コレクション内のドキュメントを更新するための投稿フックを取得できません。

`module.exports = function（mongoose）{
var mySchema = mongoose.Schema（{
id：{タイプ：数値、インデックス：{一意：true}}、
field1：{タイプ：文字列}、
field2：{タイプ：文字列}
}、{
コレクション：「mySchema」、
versionKey：false
}）;

mySchema.post('findOneAndUpdate', function (result) {
    this.model.update({}, {
        field2: 'New Value'
    }).exec();
});
return mySchema;

} `

mySchema.findOneAndUpdate({id: 1}, {field1: 'test'}, {new: true});

コレクション内のフィールドを{id：1、field1： 'test'）に設定しますが、{id：1、field1： 'test'、field2： 'New Value'}である必要があります
何が間違っているのかわからない

これを行うことで、findOneAndUpdateの結果を変更することができます
mySchema.post('findOneAndUpdate', function (result) { result.field2 = 'something' });

モデルにすでに存在する要素でモデルを更新しようとしている可能性があります。 または、間違って選択している可能性があります。 mySchema.postに「this」を印刷してみてください。 また、投稿にdone（）またはnext（）が含まれていないようです。 私はこのテーマについてあまり知識がありませんが、これを印刷することで、少なくともあなたが何を扱っているかについてのアイデアが得られることは知っています。

モデル内の既存のドキュメントを変更するための更新のポイントではありませんか？

これはクエリオブジェクトです

私が理解している限り、ポストフックで完了したり次のことをしたりする必要はありません。

これは、オブジェクトのモデルではなくスキーマであるthis.model.updateがあります。 私は思う..それはあなたが使用しなければならないことを意味します

mySchema.post('findOneAndUpdate', function (result) {
    this.model.update({}, {
        $set: { field2: 'New Value'}
    }).exec();
});
return mySchema;

これは、モデル内でモデル関数を呼び出すのとは少し逆のようです。 与えられた「this」オブジェクトの一部を使用できるからです。 findOneAndUpdateを呼び出してから、その上で別のモデル関数を呼び出すのではなく、単にfindOneAndUpdateを使用する方がよい場合があります。
マネージャーで

var data = yourNewData;
self.findOneAndUpdate({id: something._id}, data, {safe: false, new: true})
    .exec()
    .then(resolve)
    .catch(reject);

上記の例では、this._updateを使用しました。これは、これから使用する必要のある更新オブジェクトであるためです。

$ setを使ってみました。 それでもコレクション内のドキュメントは変更されません。

利用可能なすべてのプリフックとポストフックはどこにありますか？

ミドルウェアドキュメントhttp://mongoosejs.com/docs/middleware.html

+1