Fix lint issues

Author: taxilian

lib/document.js

@@ -2079,8 +2079,6 @@ function _evaluateRequiredFunctions(doc) {
  */
 
 function _getPathsToValidate(doc) {
-  let i;
-  let len;
   const skipSchemaValidators = {};
 
   _evaluateRequiredFunctions(doc);
@@ -2110,13 +2108,13 @@ function _getPathsToValidate(doc) {
       // subdoc
       for (const p of paths) {
         if (p === null || p.startsWith(subdoc.$basePath + '.')) {
-            paths.delete(p);
+          paths.delete(p);
         }
       }
 
       if (doc.isModified(subdoc.$basePath, modifiedPaths) &&
-              !doc.isDirectModified(subdoc.$basePath) &&
-              !doc.$isDefault(subdoc.$basePath)) {
+            !doc.isDirectModified(subdoc.$basePath) &&
+            !doc.$isDefault(subdoc.$basePath)) {
         paths.add(subdoc.$basePath);
 
         skipSchemaValidators[subdoc.$basePath] = true;
@@ -2174,8 +2172,8 @@ function _getPathsToValidate(doc) {
     // be validated on their own when we call `validate()` on the subdoc itself.
     // Re: gh-8468
     if (doc.schema.singleNestedPaths.hasOwnProperty(path)) {
-        paths.delete(path);
-        continue;
+      paths.delete(path);
+      continue;
     }
     const _pathType = doc.schema.path(path);
     if (!_pathType || !_pathType.$isSchemaMap) {
@@ -2214,10 +2212,10 @@ Document.prototype.$__validate = function(pathsToValidate, options, callback) {
       ('validateModifiedOnly' in options);
 
   const saveDeepStacktrace = (
-        options && 
-        (typeof options === 'object') &&
-        !!options['deepStackTrace'])
-     || !!this.schema.options.deepStackTrace;
+    options &&
+      (typeof options === 'object') &&
+      !!options['deepStackTrace'])
+      || !!this.schema.options.deepStackTrace;
   const parentStack = saveDeepStacktrace && options &&
       (typeof options === 'object') &&
       options.parentStack || [];
@@ -2308,8 +2306,8 @@ Document.prototype.$__validate = function(pathsToValidate, options, callback) {
     validated[path] = true;
     total++;
 
-    let stackToHere = saveDeepStacktrace ?
-        [(new Error().stack), path].concat(parentStack) : void 0;
+    const stackToHere = saveDeepStacktrace ?
+      [(new Error().stack), path].concat(parentStack) : void 0;
 
     process.nextTick(function() {
       const p = _this.schema.path(path);

lib/error/parallelValidate.js

@@ -18,8 +18,8 @@ function ParallelValidateError(doc, opts) {
   MongooseError.call(this, msg + doc._id);
   this.name = 'ParallelValidateError';
   if (opts && opts.parentStack) {
-      // Provide a full async stack, most recent first
-      this.stack = this.stack + "\n\n" + opts.parentStack.join('\n\n');
+    // Provide a full async stack, most recent first
+    this.stack = this.stack + '\n\n' + opts.parentStack.join('\n\n');
   }
   // You need to know to look for this, but having it can be very helpful
   // for tracking down issues when combined with the deepStackTrace schema

lib/schema.js

@@ -404,7 +404,7 @@ Schema.prototype.defaultOptions = function(options) {
     capped: false, // { size, max, autoIndexId }
     // Setting to true may help with debugging but will have performance
     // consequences
-    deepStackTrace: false, 
+    deepStackTrace: false,
     versionKey: '__v',
     discriminatorKey: '__t',
     minimize: true,

lib/schema/SingleNestedPath.js

@@ -238,13 +238,13 @@ SingleNestedPath.prototype.doValidate = function(value, fn, scope, options) {
       value = new Constructor(value, null, scope);
     }
     try {
-        return value.validate(fn);
+      return value.validate(fn);
     } catch (err) {
-        // Save the parent stack on the error
-        if (options.parentStack) {
-            err.parentStack = options.parentStack;
-        }
-        throw err;
+      // Save the parent stack on the error
+      if (options.parentStack) {
+        err.parentStack = options.parentStack;
+      }
+      throw err;
     }
   }
   const parentStack = options && options.parentStack;
@@ -258,8 +258,8 @@ SingleNestedPath.prototype.doValidate = function(value, fn, scope, options) {
     }
 
     value.validate(fn, {
-        deepStackTrace: !!parentStack,
-        parentStack: parentStack,
+      deepStackTrace: !!parentStack,
+      parentStack: parentStack,
     });
   }, scope, options);
 };

lib/schema/documentarray.js

@@ -252,9 +252,9 @@ DocumentArrayPath.prototype.doValidate = function(array, fn, scope, options) {
       }
 
       doc.$__validate(null, {
-          parentStack: options && options.parentStack,
-          deepStackTrace: !!(options && options.parentStack)
-        }, callback);
+        parentStack: options && options.parentStack,
+        deepStackTrace: !!(options && options.parentStack)
+      }, callback);
     }
   }
 };

lib/schematype.js

@@ -1061,7 +1061,7 @@ SchemaType.prototype.doValidate = function(value, fn, scope, options) {
       err = new ErrorConstructor(validatorProperties);
       err[validatorErrorSymbol] = true;
       if (options && options.parentStack) {
-          error.parentStack = options.parentStack;
+        err.parentStack = options.parentStack;
       }
       immediate(function() {
         fn(err);