mongoose 6.2.9 → 6.3.0

package/lib/model.js

@@ -26,6 +26,7 @@ const applyQueryMiddleware = require('./helpers/query/applyQueryMiddleware');
 const applyHooks = require('./helpers/model/applyHooks');
 const applyMethods = require('./helpers/model/applyMethods');
 const applyProjection = require('./helpers/projection/applyProjection');
+const applySchemaCollation = require('./helpers/indexes/applySchemaCollation');
 const applyStaticHooks = require('./helpers/model/applyStaticHooks');
 const applyStatics = require('./helpers/model/applyStatics');
 const applyWriteConcern = require('./helpers/schema/applyWriteConcern');
@@ -82,7 +83,7 @@ const saveToObjectOptions = Object.assign({}, internalToObjectOptions, {
  * [`connection.model()`](./api.html#connection_Connection-model) functions
  * create subclasses of `mongoose.Model` as shown below.
  *
- * ####Example:
+ * #### Example:
  *
  *     // `UserModel` is a "Model", a subclass of `mongoose.Model`.
  *     const UserModel = mongoose.model('User', new Schema({ name: String }));
@@ -201,12 +202,12 @@ Model.prototype.baseModelName;
  * Event emitter that reports any errors that occurred. Useful for global error
  * handling.
  *
- * ####Example:
+ * #### Example:
  *
  *     MyModel.events.on('error', err => console.log(err.message));
  *
  *     // Prints a 'CastError' because of the above handler
- *     await MyModel.findOne({ _id: 'notanid' }).catch(noop);
+ *     await MyModel.findOne({ _id: 'Not a valid ObjectId' }).catch(noop);
  *
  * @api public
  * @fires error whenever any query or model function errors
@@ -343,7 +344,8 @@ Model.prototype.$__handleSave = function(options, callback) {
   } else {
     const optionsWithCustomValues = Object.assign({}, options, saveOptions);
     const where = this.$__where();
-    if (this.$__schema.options.optimisticConcurrency) {
+    const optimisticConcurrency = this.$__schema.options.optimisticConcurrency;
+    if (optimisticConcurrency) {
       const key = this.$__schema.options.versionKey;
       const val = this.$__getValue(key);
       if (val != null) {
@@ -397,7 +399,7 @@ Model.prototype.$__save = function(options, callback) {
         }
       }
 
-      const versionBump = this.$__.version || this.$__schema.options.optimisticConcurrency;
+      const versionBump = this.$__.version;
       // was this an update that required a version bump?
       if (versionBump && !this.$__.inserting) {
         const doIncrement = VERSION_INC === (VERSION_INC & this.$__.version);
@@ -452,7 +454,7 @@ function generateVersionError(doc, modifiedPaths) {
  * Saves this document by inserting a new document into the database if [document.isNew](/docs/api.html#document_Document-isNew) is `true`,
  * or sends an [updateOne](/docs/api.html#document_Document-updateOne) operation with just the modified paths if `isNew` is `false`.
  *
- * ####Example:
+ * #### Example:
  *
  *     product.sold = Date.now();
  *     product = await product.save();
@@ -460,7 +462,7 @@ function generateVersionError(doc, modifiedPaths) {
  * If save is successful, the returned promise will fulfill with the document
  * saved.
  *
- * ####Example:
+ * #### Example:
  *
  *     const newProduct = await product.save();
  *     newProduct === product; // true
@@ -575,7 +577,6 @@ function operand(self, where, delta, data, val, op) {
   if (VERSION_ALL === (VERSION_ALL & self.$__.version)) return;
 
   if (self.$__schema.options.optimisticConcurrency) {
-    self.$__.version = VERSION_ALL;
     return;
   }
 
@@ -698,6 +699,12 @@ function handleAtomics(self, where, delta, data, value) {
 
 Model.prototype.$__delta = function() {
   const dirty = this.$__dirty();
+
+  const optimisticConcurrency = this.$__schema.options.optimisticConcurrency;
+  if (optimisticConcurrency) {
+    this.$__.version = dirty.length ? VERSION_ALL : VERSION_WHERE;
+  }
+
   if (!dirty.length && VERSION_ALL !== this.$__.version) {
     return;
   }
@@ -883,7 +890,7 @@ Model.prototype.$__version = function(where, delta) {
 /**
  * Signal that we desire an increment of this documents version.
  *
- * ####Example:
+ * #### Example:
  *
  *     Model.findById(id, function (err, doc) {
  *       doc.increment();
@@ -927,7 +934,7 @@ Model.prototype.$__where = function _where(where) {
 /**
  * Removes this document from the db.
  *
- * ####Example:
+ * #### Example:
  *     product.remove(function (err, product) {
  *       if (err) return handleError(err);
  *       Product.findById(product._id, function (err, product) {
@@ -938,7 +945,7 @@ Model.prototype.$__where = function _where(where) {
  *
  * As an extra measure of flow control, remove will return a Promise (bound to `fn` if passed) so it could be chained, or hooked to receive errors
  *
- * ####Example:
+ * #### Example:
  *     product.remove().then(function (product) {
  *        ...
  *     }).catch(function (err) {
@@ -985,7 +992,7 @@ Model.prototype.delete = Model.prototype.remove;
 /**
  * Removes this document from the db. Equivalent to `.remove()`.
  *
- * ####Example:
+ * #### Example:
  *     product = await product.deleteOne();
  *     await Product.findById(product._id); // null
  *
@@ -1054,7 +1061,7 @@ Model.prototype.$__deleteOne = Model.prototype.$__remove;
 /**
  * Returns another Model instance.
  *
- * ####Example:
+ * #### Example:
  *
  *     const doc = new Tank;
  *     doc.model('User').findById(id, callback);
@@ -1074,7 +1081,7 @@ Model.prototype.model = function model(name) {
  * Under the hood, `MyModel.exists({ answer: 42 })` is equivalent to
  * `MyModel.findOne({ answer: 42 }).select({ _id: 1 }).lean()`
  *
- * ####Example:
+ * #### Example:
  *     await Character.deleteMany({});
  *     await Character.create({ name: 'Jean-Luc Picard' });
  *
@@ -1113,7 +1120,7 @@ Model.exists = function exists(filter, options, callback) {
 /**
  * Adds a discriminator type.
  *
- * ####Example:
+ * #### Example:
  *
  *     function BaseSchema() {
  *       Schema.apply(this, arguments);
@@ -1242,7 +1249,7 @@ for (const i in EventEmitter.prototype) {
  * to get back a promise that will resolve when your indexes are finished
  * building as an alternative to [`MyModel.on('index')`](/docs/guide.html#indexes)
  *
- * ####Example:
+ * #### Example:
  *
  *     const eventSchema = new Schema({ thing: { type: 'string', unique: true }})
  *     // This calls `Event.init()` implicitly, so you don't need to call
@@ -1328,7 +1335,7 @@ Model.init = function init(callback) {
  * Note 2: You don't have to call this if your schema contains index or unique field.
  * In that case, just use `Model.init()`
  *
- * ####Example:
+ * #### Example:
  *
  *     const userSchema = new Schema({ name: String })
  *     const User = mongoose.model('User', userSchema);
@@ -1410,7 +1417,7 @@ Model.createCollection = function createCollection(options, callback) {
  * See the [introductory blog post](https://thecodebarbarian.com/whats-new-in-mongoose-5-2-syncindexes)
  * for more information.
  *
- * ####Example:
+ * #### Example:
  *
  *     const schema = new Schema({ name: { type: String, unique: true } });
  *     const Customer = mongoose.model('Customer', schema);
@@ -1556,6 +1563,7 @@ Model.cleanIndexes = function cleanIndexes(callback) {
 
         for (const [schemaIndexKeysObject, schemaIndexOptions] of schemaIndexes) {
           const options = decorateDiscriminatorIndexOptions(this.schema, utils.clone(schemaIndexOptions));
+          applySchemaCollation(schemaIndexKeysObject, options, this.schema.options);
 
           if (isIndexEqual(schemaIndexKeysObject, options, dbIndex)) {
             found = true;
@@ -1628,7 +1636,7 @@ Model.listIndexes = function init(callback) {
  * Sends `createIndex` commands to mongo for each index declared in the schema.
  * The `createIndex` commands are sent in series.
  *
- * ####Example:
+ * #### Example:
  *
  *     Event.ensureIndexes(function (err) {
  *       if (err) return handleError(err);
@@ -1636,7 +1644,7 @@ Model.listIndexes = function init(callback) {
  *
  * After completion, an `index` event is emitted on this `Model` passing an error if one occurred.
  *
- * ####Example:
+ * #### Example:
  *
  *     const eventSchema = new Schema({ thing: { type: 'string', unique: true }})
  *     const Event = mongoose.model('Event', eventSchema);
@@ -1774,26 +1782,17 @@ function _ensureIndexes(model, options, callback) {
 
     const indexFields = utils.clone(index[0]);
     const indexOptions = utils.clone(index[1]);
-    let isTextIndex = false;
-    for (const key of Object.keys(indexFields)) {
-      if (indexFields[key] === 'text') {
-        isTextIndex = true;
-      }
-    }
+
     delete indexOptions._autoIndex;
     decorateDiscriminatorIndexOptions(model.schema, indexOptions);
     applyWriteConcern(model.schema, indexOptions);
+    applySchemaCollation(indexFields, indexOptions, model.schema.options);
 
     indexSingleStart(indexFields, options);
 
     if ('background' in options) {
       indexOptions.background = options.background;
     }
-    if (model.schema.options.hasOwnProperty('collation') &&
-        !indexOptions.hasOwnProperty('collation') &&
-        !isTextIndex) {
-      indexOptions.collation = model.schema.options.collation;
-    }
 
     model.collection.createIndex(indexFields, indexOptions, utils.tick(function(err, name) {
       indexSingleDone(err, indexFields, indexOptions, name);
@@ -1873,7 +1872,7 @@ Model.discriminators;
 /**
  * Translate any aliases fields/conditions so the final query or document object is pure
  *
- * ####Example:
+ * #### Example:
  *
  *     Character
  *       .find(Character.translateAliases({
@@ -1881,7 +1880,7 @@ Model.discriminators;
  *       })
  *       .exec(function(err, characters) {})
  *
- * ####Note:
+ * #### Note:
  * Only translate arguments of object type anything else is returned raw
  *
  * @param {Object} fields fields/conditions that may contain aliased keys
@@ -1970,12 +1969,12 @@ Model.translateAliases = function translateAliases(fields) {
  *
  * This method is deprecated. See [Deprecation Warnings](../deprecations.html#remove) for details.
  *
- * ####Example:
+ * #### Example:
  *
  *     const res = await Character.remove({ name: 'Eddard Stark' });
  *     res.deletedCount; // Number of documents removed
  *
- * ####Note:
+ * #### Note:
  *
  * This method sends a remove command directly to MongoDB, no Mongoose documents
  * are involved. Because no Mongoose documents are involved, Mongoose does
@@ -2017,11 +2016,11 @@ Model.remove = function remove(conditions, options, callback) {
  * Behaves like `remove()`, but deletes at most one document regardless of the
  * `single` option.
  *
- * ####Example:
+ * #### Example:
  *
  *     await Character.deleteOne({ name: 'Eddard Stark' }); // returns {deletedCount: 1}
  *
- * ####Note:
+ * #### Note:
  *
  * This function triggers `deleteOne` query hooks. Read the
  * [middleware docs](/docs/middleware.html#naming) to learn more.
@@ -2060,11 +2059,11 @@ Model.deleteOne = function deleteOne(conditions, options, callback) {
  * Behaves like `remove()`, but deletes all documents that match `conditions`
  * regardless of the `single` option.
  *
- * ####Example:
+ * #### Example:
  *
  *     await Character.deleteMany({ name: /Stark/, age: { $gte: 18 } }); // returns {deletedCount: x} where x is the number of documents deleted.
  *
- * ####Note:
+ * #### Note:
  *
  * This function triggers `deleteMany` query hooks. Read the
  * [middleware docs](/docs/middleware.html#naming) to learn more.
@@ -2103,7 +2102,7 @@ Model.deleteMany = function deleteMany(conditions, options, callback) {
  * See our [query casting tutorial](/docs/tutorials/query_casting.html) for
  * more information on how Mongoose casts `filter`.
  *
- * ####Examples:
+ * #### Examples:
  *
  *     // find all documents
  *     await MyModel.find({});
@@ -2179,7 +2178,7 @@ Model.find = function find(conditions, projection, options, callback) {
  * to `findOne({})` and return arbitrary documents. However, mongoose
  * translates `findById(undefined)` into `findOne({ _id: null })`.
  *
- * ####Example:
+ * #### Example:
  *
  *     // Find the adventure with the given `id`, or `null` if not found
  *     await Adventure.findById(id).exec();
@@ -2222,7 +2221,7 @@ Model.findById = function findById(id, projection, options, callback) {
  * mongoose will send an empty `findOne` command to MongoDB, which will return
  * an arbitrary document. If you're querying by `_id`, use `findById()` instead.
  *
- * ####Example:
+ * #### Example:
  *
  *     // Find one adventure whose `country` is 'Croatia', otherwise `null`
  *     await Adventure.findOne({ country: 'Croatia' }).exec();
@@ -2278,7 +2277,7 @@ Model.findOne = function findOne(conditions, projection, options, callback) {
  * `estimatedDocumentCount()` uses collection metadata rather than scanning
  * the entire collection.
  *
- * ####Example:
+ * #### Example:
  *
  *     const numAdventures = await Adventure.estimatedDocumentCount();
  *
@@ -2301,7 +2300,7 @@ Model.estimatedDocumentCount = function estimatedDocumentCount(options, callback
 /**
  * Counts number of documents matching `filter` in a database collection.
  *
- * ####Example:
+ * #### Example:
  *
  *     Adventure.countDocuments({ type: 'jungle' }, function (err, count) {
  *       console.log('there are %d jungle adventures', count);
@@ -2356,7 +2355,7 @@ Model.countDocuments = function countDocuments(conditions, options, callback) {
  * a collection, e.g. `count({})`, use the [`estimatedDocumentCount()` function](/docs/api.html#model_Model.estimatedDocumentCount)
  * instead. Otherwise, use the [`countDocuments()`](/docs/api.html#model_Model.countDocuments) function instead.
  *
- * ####Example:
+ * #### Example:
  *
  *     const count = await Adventure.count({ type: 'jungle' });
  *     console.log('there are %d jungle adventures', count);
@@ -2388,7 +2387,7 @@ Model.count = function count(conditions, callback) {
  *
  * Passing a `callback` executes the query.
  *
- * ####Example
+ * #### Example
  *
  *     Link.distinct('url', { clicks: {$gt: 100}}, function (err, result) {
  *       if (err) return handleError(err);
@@ -2480,7 +2479,7 @@ Model.$where = function $where() {
  *
  * Finds a matching document, updates it according to the `update` arg, passing any `options`, and returns the found document (if any) to the callback. The query executes if `callback` is passed else a Query object is returned.
  *
- * ####Options:
+ * #### Options:
  *
  * - `new`: bool - if true, return the modified document rather than the original. defaults to false (changed in 4.0)
  * - `upsert`: bool - creates the object if it doesn't exist. defaults to false.
@@ -2493,7 +2492,7 @@ Model.$where = function $where() {
  * - `rawResult`: if true, returns the [raw result from the MongoDB driver](https://mongodb.github.io/node-mongodb-native/4.3/interfaces/ModifyResult.html)
  * - `strict`: overwrites the schema's [strict mode option](https://mongoosejs.com/docs/guide.html#strict) for this update
  *
- * ####Examples:
+ * #### Examples:
  *
  *     A.findOneAndUpdate(conditions, update, options, callback) // executes
  *     A.findOneAndUpdate(conditions, update, options)  // returns Query
@@ -2501,11 +2500,11 @@ Model.$where = function $where() {
  *     A.findOneAndUpdate(conditions, update)           // returns Query
  *     A.findOneAndUpdate()                             // returns Query
  *
- * ####Note:
+ * #### Note:
  *
  * All top level update keys which are not `atomic` operation names are treated as set operations:
  *
- * ####Example:
+ * #### Example:
  *
  *     const query = { name: 'borne' };
  *     Model.findOneAndUpdate(query, { name: 'jason bourne' }, options, callback)
@@ -2515,7 +2514,7 @@ Model.$where = function $where() {
  *
  * This helps prevent accidentally overwriting your document with `{ name: 'jason bourne' }`.
  *
- * ####Note:
+ * #### Note:
  *
  * `findOneAndX` and `findByIdAndX` functions support limited validation that
  * you can enable by setting the `runValidators` option.
@@ -2618,7 +2617,7 @@ function _decorateUpdateWithVersionKey(update, options, versionKey) {
  *
  * - `findOneAndUpdate()`
  *
- * ####Options:
+ * #### Options:
  *
  * - `new`: bool - true to return the modified document rather than the original. defaults to false
  * - `upsert`: bool - creates the object if it doesn't exist. defaults to false.
@@ -2629,7 +2628,7 @@ function _decorateUpdateWithVersionKey(update, options, versionKey) {
  * - `rawResult`: if true, returns the [raw result from the MongoDB driver](https://mongodb.github.io/node-mongodb-native/4.3/interfaces/ModifyResult.html)
  * - `strict`: overwrites the schema's [strict mode option](https://mongoosejs.com/docs/guide.html#strict) for this update
  *
- * ####Examples:
+ * #### Examples:
  *
  *     A.findByIdAndUpdate(id, update, options, callback) // executes
  *     A.findByIdAndUpdate(id, update, options)  // returns Query
@@ -2637,11 +2636,11 @@ function _decorateUpdateWithVersionKey(update, options, versionKey) {
  *     A.findByIdAndUpdate(id, update)           // returns Query
  *     A.findByIdAndUpdate()                     // returns Query
  *
- * ####Note:
+ * #### Note:
  *
  * All top level update keys which are not `atomic` operation names are treated as set operations:
  *
- * ####Example:
+ * #### Example:
  *
  *     Model.findByIdAndUpdate(id, { name: 'jason bourne' }, options, callback)
  *
@@ -2650,7 +2649,7 @@ function _decorateUpdateWithVersionKey(update, options, versionKey) {
  *
  * This helps prevent accidentally overwriting your document with `{ name: 'jason bourne' }`.
  *
- * ####Note:
+ * #### Note:
  *
  * `findOneAndX` and `findByIdAndX` functions support limited validation. You can
  * enable validation by setting the `runValidators` option.
@@ -2721,7 +2720,7 @@ Model.findByIdAndUpdate = function(id, update, options, callback) {
  * this distinction is purely pedantic. You should use `findOneAndDelete()`
  * unless you have a good reason not to.
  *
- * ####Options:
+ * #### Options:
  *
  * - `sort`: if multiple docs are found by the conditions, sets the sort order to choose which doc to update
  * - `maxTimeMS`: puts a time limit on the query - requires mongodb >= 2.6.0
@@ -2730,7 +2729,7 @@ Model.findByIdAndUpdate = function(id, update, options, callback) {
  * - `rawResult`: if true, returns the [raw result from the MongoDB driver](https://mongodb.github.io/node-mongodb-native/4.3/interfaces/ModifyResult.html)
  * - `strict`: overwrites the schema's [strict mode option](https://mongoosejs.com/docs/guide.html#strict) for this update
  *
- * ####Examples:
+ * #### Examples:
  *
  *     A.findOneAndDelete(conditions, options, callback) // executes
  *     A.findOneAndDelete(conditions, options)  // return Query
@@ -2834,7 +2833,7 @@ Model.findByIdAndDelete = function(id, options, callback) {
  *
  * - `findOneAndReplace()`
  *
- * ####Options:
+ * #### Options:
  *
  * - `sort`: if multiple docs are found by the conditions, sets the sort order to choose which doc to update
  * - `maxTimeMS`: puts a time limit on the query - requires mongodb >= 2.6.0
@@ -2843,7 +2842,7 @@ Model.findByIdAndDelete = function(id, options, callback) {
  * - `rawResult`: if true, returns the [raw result from the MongoDB driver](https://mongodb.github.io/node-mongodb-native/4.3/interfaces/ModifyResult.html)
  * - `strict`: overwrites the schema's [strict mode option](https://mongoosejs.com/docs/guide.html#strict) for this update
  *
- * ####Examples:
+ * #### Examples:
  *
  *     A.findOneAndReplace(filter, replacement, options, callback) // executes
  *     A.findOneAndReplace(filter, replacement, options)  // return Query
@@ -2913,7 +2912,7 @@ Model.findOneAndReplace = function(filter, replacement, options, callback) {
  *
  * - `findOneAndRemove()`
  *
- * ####Options:
+ * #### Options:
  *
  * - `sort`: if multiple docs are found by the conditions, sets the sort order to choose which doc to update
  * - `maxTimeMS`: puts a time limit on the query - requires mongodb >= 2.6.0
@@ -2922,7 +2921,7 @@ Model.findOneAndReplace = function(filter, replacement, options, callback) {
  * - `rawResult`: if true, returns the [raw result from the MongoDB driver](https://mongodb.github.io/node-mongodb-native/4.3/interfaces/ModifyResult.html)
  * - `strict`: overwrites the schema's [strict mode option](https://mongoosejs.com/docs/guide.html#strict) for this update
  *
- * ####Examples:
+ * #### Examples:
  *
  *     A.findOneAndRemove(conditions, options, callback) // executes
  *     A.findOneAndRemove(conditions, options)  // return Query
@@ -2993,14 +2992,14 @@ Model.findOneAndRemove = function(conditions, options, callback) {
  *
  * - `findOneAndRemove()`
  *
- * ####Options:
+ * #### Options:
  *
  * - `sort`: if multiple docs are found by the conditions, sets the sort order to choose which doc to update
  * - `select`: sets the document fields to return
  * - `rawResult`: if true, returns the [raw result from the MongoDB driver](https://mongodb.github.io/node-mongodb-native/4.3/interfaces/ModifyResult.html)
  * - `strict`: overwrites the schema's [strict mode option](https://mongoosejs.com/docs/guide.html#strict) for this update
  *
- * ####Examples:
+ * #### Examples:
  *
  *     A.findByIdAndRemove(id, options, callback) // executes
  *     A.findByIdAndRemove(id, options)  // return Query
@@ -3043,7 +3042,7 @@ Model.findByIdAndRemove = function(id, options, callback) {
  *
  * - `save()`
  *
- * ####Example:
+ * #### Example:
  *
  *     // Insert one new `Character` document
  *     await Character.create({ name: 'Jean-Luc Picard' });
@@ -3195,7 +3194,7 @@ Model.create = function create(doc, options, callback) {
  * - 'end': Emitted if the underlying stream is closed
  * - 'close': Emitted if the underlying stream is closed
  *
- * ####Example:
+ * #### Example:
  *
  *     const doc = await Person.create({ name: 'Ned Stark' });
  *     const changeStream = Person.watch().on('change', change => console.log(change));
@@ -3244,7 +3243,7 @@ Model.watch = function(pipeline, options) {
  *
  * This function does not trigger any middleware.
  *
- * ####Example:
+ * #### Example:
  *
  *     const session = await Person.startSession();
  *     let doc = await Person.findOne({ name: 'Ned Stark' }, null, { session });
@@ -3285,7 +3284,7 @@ Model.startSession = function() {
  *
  * - `insertMany()`
  *
- * ####Example:
+ * #### Example:
  *
  *     const arr = [{ name: 'Star Wars' }, { name: 'The Empire Strikes Back' }];
  *     Movies.insertMany(arr, function(error, docs) {});
@@ -3505,7 +3504,7 @@ function _setIsNew(doc, val) {
  * If you need to trigger
  * `save()` middleware for every document use [`create()`](https://mongoosejs.com/docs/api.html#model_Model.create) instead.
  *
- * ####Example:
+ * #### Example:
  *
  *     Character.bulkWrite([
  *       {
@@ -3761,7 +3760,7 @@ Model.buildBulkWriteOperations = function buildBulkWriteOperations(documents, op
  * Shortcut for creating a new Document from existing raw data, pre-saved in the DB.
  * The document returned has no paths marked as modified initially.
  *
- * ####Example:
+ * #### Example:
  *
  *     // hydrate previous data into a Mongoose document
  *     const mongooseCandy = Candy.hydrate({ _id: '54108337212ffb6d459f854c', type: 'jelly bean' });
@@ -3796,7 +3795,7 @@ Model.hydrate = function(obj, projection) {
  *
  * This method is deprecated. See [Deprecation Warnings](../deprecations.html#update) for details.
  *
- * ####Examples:
+ * #### Examples:
  *
  *     MyModel.update({ age: { $gt: 18 } }, { oldEnough: true }, fn);
  *
@@ -3806,7 +3805,7 @@ Model.hydrate = function(obj, projection) {
  *     // had `ferret` set to `true`, `nModified` will be 0.
  *     res.nModified;
  *
- * ####Valid options:
+ * #### Valid options:
  *
  *  - `strict` (boolean): overrides the [schema-level `strict` option](/docs/guide.html#strict) for this update
  *  - `upsert` (boolean): whether to create the doc if it doesn't match (false)
@@ -3824,11 +3823,11 @@ Model.hydrate = function(obj, projection) {
  * - `err` is the error if any occurred
  * - `rawResponse` is the full response from Mongo
  *
- * ####Note:
+ * #### Note:
  *
  * All top level keys which are not `atomic` operation names are treated as set operations:
  *
- * ####Example:
+ * #### Example:
  *
  *     const query = { name: 'borne' };
  *     Model.update(query, { name: 'jason bourne' }, options, callback);
@@ -3839,7 +3838,7 @@ Model.hydrate = function(obj, projection) {
  *
  * This helps prevent accidentally overwriting all documents in your collection with `{ name: 'jason bourne' }`.
  *
- * ####Note:
+ * #### Note:
  *
  * Be careful to not use an existing model instance for the update clause (this won't work and can cause weird behavior like infinite loops). Also, ensure that the update clause does not have an _id property, which causes Mongo to return a "Mod on _id not allowed" error.
  *
@@ -3880,7 +3879,7 @@ Model.update = function update(conditions, doc, options, callback) {
  * **Note** updateMany will _not_ fire update middleware. Use `pre('updateMany')`
  * and `post('updateMany')` instead.
  *
- * ####Example:
+ * #### Example:
  *     const res = await Person.updateMany({ name: /Stark$/ }, { isDeleted: true });
  *     res.matchedCount; // Number of documents matched
  *     res.modifiedCount; // Number of documents modified
@@ -3920,7 +3919,7 @@ Model.updateMany = function updateMany(conditions, doc, options, callback) {
  * - MongoDB will update _only_ the first document that matches `filter` regardless of the value of the `multi` option.
  * - Use `replaceOne()` if you want to overwrite an entire document rather than using atomic operators like `$set`.
  *
- * ####Example:
+ * #### Example:
  *     const res = await Person.updateOne({ name: 'Jean-Luc Picard' }, { ship: 'USS Enterprise' });
  *     res.matchedCount; // Number of documents matched
  *     res.modifiedCount; // Number of documents modified
@@ -3957,7 +3956,7 @@ Model.updateOne = function updateOne(conditions, doc, options, callback) {
  * Same as `update()`, except MongoDB replace the existing document with the
  * given document (no atomic operators like `$set`).
  *
- * ####Example:
+ * #### Example:
  *     const res = await Person.replaceOne({ _id: 24601 }, { name: 'Jean Valjean' });
  *     res.matchedCount; // Number of documents matched
  *     res.modifiedCount; // Number of documents modified
@@ -4028,7 +4027,7 @@ function _update(model, op, conditions, doc, options, callback) {
  *
  * This function does not trigger any middleware.
  *
- * ####Example:
+ * #### Example:
  *
  *     const o = {};
  *     // `map()` and `reduce()` are run on the MongoDB server, not Node.js,
@@ -4039,7 +4038,7 @@ function _update(model, op, conditions, doc, options, callback) {
  *       console.log(results)
  *     })
  *
- * ####Other options:
+ * #### Other options:
  *
  * - `query` {Object} query filter object.
  * - `sort` {Object} sort input objects using this key
@@ -4052,7 +4051,7 @@ function _update(model, op, conditions, doc, options, callback) {
  * - `readPreference` {String}
  * - `out*` {Object, default: {inline:1}} sets the output target for the map reduce job.
  *
- * ####* out options:
+ * #### * out options:
  *
  * - `{inline:1}` the results are returned in an array
  * - `{replace: 'collectionName'}` add the results to collectionName: the results replace the collection
@@ -4061,7 +4060,7 @@ function _update(model, op, conditions, doc, options, callback) {
  *
  * If `options.out` is set to `replace`, `merge`, or `reduce`, a Model instance is returned that can be used for further querying. Queries run against this model are all executed with the [`lean` option](/docs/tutorials/lean.html); meaning only the js object is returned and no Mongoose magic is applied (getters, setters, etc).
  *
- * ####Example:
+ * #### Example:
  *
  *     const o = {};
  *     // You can also define `map()` and `reduce()` as strings if your
@@ -4154,7 +4153,7 @@ Model.mapReduce = function mapReduce(o, callback) {
  *
  * - `aggregate()`
  *
- * ####Example:
+ * #### Example:
  *
  *     // Find the max balance of all accounts
  *     const res = await Users.aggregate([
@@ -4171,7 +4170,7 @@ Model.mapReduce = function mapReduce(o, callback) {
  *       exec();
  *     console.log(res); // [ { maxBalance: 98 } ]
  *
- * ####NOTE:
+ * #### Note:
  *
  * - Mongoose does **not** cast aggregation pipelines to the model's schema because `$project` and `$group` operators allow redefining the "shape" of the documents at any stage of the pipeline, which may leave documents in an incompatible format. You can use the [mongoose-cast-aggregation plugin](https://github.com/AbdelrahmanHafez/mongoose-cast-aggregation) to enable minimal casting for aggregation pipelines.
  * - The documents returned are plain javascript objects, not mongoose documents (since any shape of document can be returned).
@@ -4213,7 +4212,6 @@ Model.aggregate = function aggregate(pipeline, options, callback) {
 
   const aggregate = new Aggregate(pipeline || []);
   aggregate.model(this);
-
   if (options != null) {
     aggregate.option(options);
   }
@@ -4233,7 +4231,7 @@ Model.aggregate = function aggregate(pipeline, options, callback) {
  * Casts and validates the given object against this model's schema, passing the
  * given `context` to custom validators.
  *
- * ####Example:
+ * #### Example:
  *
  *     const Model = mongoose.model('Test', Schema({
  *       name: { type: String, required: true },
@@ -4367,7 +4365,7 @@ Model.validate = function validate(obj, pathsToValidate, context, callback) {
  * Changed in Mongoose 6: the model you call `populate()` on should be the
  * "local field" model, **not** the "foreign field" model.
  *
- * ####Available top-level options:
+ * #### Available top-level options:
  *
  * - path: space delimited path(s) to populate
  * - select: optional fields to select
@@ -4377,7 +4375,7 @@ Model.validate = function validate(obj, pathsToValidate, context, callback) {
  * - justOne: optional boolean, if true Mongoose will always set `path` to an array. Inferred from schema by default.
  * - strictPopulate: optional boolean, set to `false` to allow populating paths that aren't in the schema.
  *
- * ####Examples:
+ * #### Examples:
  *
  *     const Dog = mongoose.model('Dog', new Schema({ name: String, breed: String }));
  *     const Person = mongoose.model('Person', new Schema({
@@ -4810,7 +4808,6 @@ Model.compile = function compile(name, schema, collectionName, connection, base)
     o[schema.options.versionKey] = Number;
     schema.add(o);
   }
-
   let model;
   if (typeof name === 'function' && name.prototype instanceof Model) {
     model = name;
@@ -4854,6 +4851,7 @@ Model.compile = function compile(name, schema, collectionName, connection, base)
   model.model = function model(name) {
     return this.db.model(name);
   };
+
   model.db = connection;
   model.prototype.db = connection;
   model.prototype[modelDbSymbol] = connection;
@@ -5027,7 +5025,7 @@ Model.$wrapCallback = function(callback) {
  * Helper for console.log. Given a model named 'MyModel', returns the string
  * `'Model { MyModel }'`.
  *
- * ####Example:
+ * #### Example:
  *
  *     const MyModel = mongoose.model('Test', Schema({ name: String }));
  *     MyModel.inspect(); // 'Model { Test }'