ast-scope - npm Package Compare versions

Comparing version 0.1.1 to 0.1.2

lib/reference.js

@@ -13,3 +13,3 @@ var Variable = require('./variable');
   this.scope = scope;
-  this.variable = scope.resolveVariable(Variable.extractName(node));
+  this.variable = scope.resolveVariable(Variable.extractId(node));
   if (this.variable) {
@@ -23,5 +23,6 @@ this.variable.references.push(this);
   case 'Identifier':
-  case 'MemberExpression':
   case 'ThisExpression':
     return true;
+  case 'MemberExpression':
+    return Reference.isRequired(node.object);
   case 'CallExpression':
@@ -28,0 +29,0 @@ case 'NewExpression':

lib/variable.js

@@ -15,15 +15,14 @@
 
-Variable.extractName = function(node) {
+Variable.extractId = function(node) {
   switch (node.type) {
   case 'Identifier':
-    return node.name;
   case 'ThisExpression':
-    return 'this';
+    return node;
   case 'MemberExpression':
-    return Variable.extractName(node.object);
+    return Variable.extractId(node.object);
   case 'CallExpression':
   case 'NewExpression':
-    return Variable.extractName(node.callee);
+    return Variable.extractId(node.callee);
   }
   throw new Error('Invalid node type: ' + node.type);
 };

package.json

 {
   "name": "ast-scope",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "A JavaScript AST scope analyzer",
@@ -5,0 +5,0 @@ "keywords": [

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

11028

increased by0.13%

No dependency changes