Blame view

node_modules/eslint/lib/rules/no-catch-shadow.js 2.14 KB
f7563de62   Palak Handa   first commit
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
  /**
   * @fileoverview Rule to flag variable leak in CatchClauses in IE 8 and earlier
   * @author Ian Christian Myers
   */
  
  "use strict";
  
  //------------------------------------------------------------------------------
  // Requirements
  //------------------------------------------------------------------------------
  
  const astUtils = require("../ast-utils");
  
  //------------------------------------------------------------------------------
  // Rule Definition
  //------------------------------------------------------------------------------
  
  module.exports = {
      meta: {
          docs: {
              description: "disallow `catch` clause parameters from shadowing variables in the outer scope",
              category: "Variables",
              recommended: false
          },
  
          schema: []
      },
  
      create(context) {
  
          //--------------------------------------------------------------------------
          // Helpers
          //--------------------------------------------------------------------------
  
          /**
           * Check if the parameters are been shadowed
           * @param {Object} scope current scope
           * @param {string} name parameter name
           * @returns {boolean} True is its been shadowed
           */
          function paramIsShadowing(scope, name) {
              return astUtils.getVariableByName(scope, name) !== null;
          }
  
          //--------------------------------------------------------------------------
          // Public API
          //--------------------------------------------------------------------------
  
          return {
  
              CatchClause(node) {
                  let scope = context.getScope();
  
                  // When blockBindings is enabled, CatchClause creates its own scope
                  // so start from one upper scope to exclude the current node
                  if (scope.block === node) {
                      scope = scope.upper;
                  }
  
                  if (paramIsShadowing(scope, node.param.name)) {
                      context.report({ node, message: "Value of '{{name}}' may be overwritten in IE 8 and earlier.", data: { name: node.param.name } });
                  }
              }
          };
  
      }
  };