why not use this simple line in one single method instead of that confusing way to access doc and model!

[Mahmoud-AbouDeghedy]

reviewSchema.post(/^findOneAnd/, async function (doc) {
  await this.model.calcAverageRatings(doc.tour);

instead of

reviewSchema.pre(/^findOneAnd/, async function(next) {
this.r = await this.findOne();
// console.log(this.r);
next();
});

reviewSchema.post(/^findOneAnd/, async function() {
// await this.findOne(); does NOT work here, query has already executed
await this.r.constructor.calcAverageRatings(this.r.tour);

[developedbyjay]

The course is old, that was how it has to be done as at then

[Mahmoud-AbouDeghedy]

TypeError: Cannot read property 'tour' of null\n at model.Query.

i got that error. Don't know what is wrong

which code u tried?

[developedbyjay]

I have resolved the issue.. your code works perfect, thanks

…

On Thu, May 18, 2023, 1:58 PM Mahmoud Ehab AbouDeghedy < ***@***.***> wrote: TypeError: Cannot read property 'tour' of null\n at model.Query. i got that error. Don't know what is wrong which code u tried? — Reply to this email directly, view it on GitHub <#190 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ARMTYXRSO2VRPYKFC7EFHJDXGYMI3ANCNFSM6AAAAAAXHLM6I4> . You are receiving this because you commented.Message ID: ***@***.***>

[lorand-horvath]

The best way is to simply include both post hooks and access the model via doc.constructor, like so:

reviewSchema.post(['save', /^findOneAnd/], async function(doc) {
    // Review === doc.constructor
    const stats = await doc.constructor.calcRatingsAverage(doc.product);
});

I'm not sure what's up with all the other convoluted methods...