Model.exists() should return a query instead of a promise

[vkarpov15]

Re: #8075 , Model.exists() is very inconsistent with other helper functions.

[felixveysseyre]

I cannot agree more with @vkarpov15

With current implementation, there is no way to use exists in a transaction context 😭

[vkarpov15]

@felixveysseyre there is an options parameter to exists(), you should be able to do Model.exists(filter, { session }). Does that not work for you?

[felixveysseyre]

@vkarpov15 Thanks for your answer 🙃

Looking into the sources, I have been able to find the options parameters you are talking about.

I've been tricked by the outdated documentation and the wrong types 😭

[vkarpov15]

Making a note of this for #8108

[AbdelrahmanHafez]

@vkarpov15
This will make it so that Model.exists(...) won't trigger pre-findOne hooks, correct?

If so, this should fix #8739

[AbdelrahmanHafez]

Should revert #8764 once this is released.

[vkarpov15]

@AbdelrahmanHafez actually exists() will still trigger pre('findOne') hooks. Unless you happen to change query op before executing.