Merge pull request #13651 from Automattic/IslandRhythms/deprecate-ove…

…rwrite

Island rhythms/deprecate overwrite

lib/query.js

@@ -1683,6 +1683,15 @@ const printRawResultDeprecationWarning = util.deprecate(
   'The `rawResult` option for Mongoose queries is deprecated. Use `includeResultMetadata: false` as a replacement for `rawResult: true`.'
 );
 
+/*!
+ * ignore
+ */
+
+const printOverwriteDeprecationWarning = util.deprecate(
+  function printOverwriteDeprecationWarning() {},
+  'The `overwrite` option for `findOneAndUpdate()` is deprecated. use `findOneAndReplace()` instead.'
+);
+
 /**
  * Sets the [`explain` option](https://www.mongodb.com/docs/manual/reference/method/cursor.explain/),
  * which makes this query return detailed execution stats instead of the actual
@@ -3283,6 +3292,10 @@ Query.prototype.findOneAndUpdate = function(filter, doc, options) {
 
 Query.prototype._findOneAndUpdate = async function _findOneAndUpdate() {
   // For backwards compability with Mongoose 6 re: #13550
+
+  if (this._mongooseOptions.overwrite != null) {
+    printOverwriteDeprecationWarning();
+  }
   if (this._mongooseOptions.overwrite) {
     this.op = 'findOneAndReplace';
     return this._findOneAndReplace();