tslint - polish rule (support multiple declarations)

build/lib/tslint/abstractGlobalsRule.js

@@ -19,19 +19,24 @@ class AbstractGlobalsRuleWalker extends Lint.RuleWalker {
             const checker = this.program.getTypeChecker();
             const symbol = checker.getSymbolAtLocation(node);
             if (symbol) {
-                const valueDeclaration = symbol.valueDeclaration;
-                if (valueDeclaration) {
-                    const parent = valueDeclaration.parent;
+                const declarations = symbol.declarations;
+                if (Array.isArray(declarations) && symbol.declarations.some(declaration => {
+                    if (declaration) {
+                        const parent = declaration.parent;
                         if (parent) {
                             const sourceFile = parent.getSourceFile();
                             if (sourceFile) {
                                 const fileName = sourceFile.fileName;
                                 if (fileName && fileName.indexOf(this.getDefinitionPattern()) >= 0) {
-                                this.addFailureAtNode(node, `Cannot use global '${node.text}' in '${this._config.target}'`);
+                                    return true;
                                 }
                             }
                         }
                     }
+                    return false;
+                })) {
+                    this.addFailureAtNode(node, `!!Cannot use global '${node.text}' in '${this._config.target}'`);
+                }
             }
         }
         super.visitIdentifier(node);

build/lib/tslint/abstractGlobalsRule.ts

@@ -30,18 +30,24 @@ export abstract class AbstractGlobalsRuleWalker extends Lint.RuleWalker {
 			const checker = this.program.getTypeChecker();
 			const symbol = checker.getSymbolAtLocation(node);
 			if (symbol) {
-				const valueDeclaration = symbol.valueDeclaration;
-				if (valueDeclaration) {
-					const parent = valueDeclaration.parent;
+				const declarations = symbol.declarations;
+				if (Array.isArray(declarations) && symbol.declarations.some(declaration => {
+					if (declaration) {
+						const parent = declaration.parent;
 						if (parent) {
 							const sourceFile = parent.getSourceFile();
 							if (sourceFile) {
 								const fileName = sourceFile.fileName;
 								if (fileName && fileName.indexOf(this.getDefinitionPattern()) >= 0) {
-								this.addFailureAtNode(node, `Cannot use global '${node.text}' in '${this._config.target}'`);
+									return true;
+								}
 							}
 						}
 					}
+
+					return false;
+				})) {
+					this.addFailureAtNode(node, `Cannot use global '${node.text}' in '${this._config.target}'`);
 				}
 			}
 		}