From b73ccfe98d5c552de7eeae80160f683ede987466 Mon Sep 17 00:00:00 2001
From: Andrii Dieiev <illusionmh@gmail.com>
Date: Mon, 10 Oct 2016 02:32:07 +0300
Subject: [PATCH] Workaround for #1400. Excludes imported properties from error

---
 src/rules/noUseBeforeDeclareRule.ts           | 15 ++++++++++++++-
 test/rules/no-use-before-declare/test.ts.lint |  9 +++++++++
 2 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/src/rules/noUseBeforeDeclareRule.ts b/src/rules/noUseBeforeDeclareRule.ts
index 1f126607356..b9504b80a04 100644
--- a/src/rules/noUseBeforeDeclareRule.ts
+++ b/src/rules/noUseBeforeDeclareRule.ts
@@ -46,6 +46,8 @@ export class Rule extends Lint.Rules.AbstractRule {
 type VisitedVariables = {[varName: string]: boolean};
 
 class NoUseBeforeDeclareWalker extends Lint.ScopeAwareRuleWalker<VisitedVariables> {
+    private importedPropertiesPositions: number[] = [];
+
     constructor(sourceFile: ts.SourceFile, options: Lint.IOptions, private languageService: ts.LanguageService) {
         super(sourceFile, options);
     }
@@ -89,6 +91,9 @@ class NoUseBeforeDeclareWalker extends Lint.ScopeAwareRuleWalker<VisitedVariable
 
     public visitNamedImports(node: ts.NamedImports) {
         for (let namedImport of node.elements) {
+            if (namedImport.propertyName != null) {
+                this.saveImportedPropertiesPositions(namedImport.propertyName.getStart());
+            }
             this.validateUsageForVariable(namedImport.name.text, namedImport.name.getStart());
         }
         super.visitNamedImports(node);
@@ -120,7 +125,7 @@ class NoUseBeforeDeclareWalker extends Lint.ScopeAwareRuleWalker<VisitedVariable
             for (let highlight of highlights) {
                 for (let highlightSpan of highlight.highlightSpans) {
                     const referencePosition = highlightSpan.textSpan.start;
-                    if (referencePosition < position) {
+                    if (referencePosition < position && !this.isImportedPropertyName(referencePosition)) {
                         const failureString = Rule.FAILURE_STRING_PREFIX + name + Rule.FAILURE_STRING_POSTFIX;
                         this.addFailure(this.createFailure(referencePosition, name.length, failureString));
                     }
@@ -128,4 +133,12 @@ class NoUseBeforeDeclareWalker extends Lint.ScopeAwareRuleWalker<VisitedVariable
             }
         }
     }
+
+    private saveImportedPropertiesPositions(position: number): void {
+        this.importedPropertiesPositions.push(position);
+    }
+
+    private isImportedPropertyName(position: number): boolean {
+        return this.importedPropertiesPositions.indexOf(position) !== -1;
+    }
 }
diff --git a/test/rules/no-use-before-declare/test.ts.lint b/test/rules/no-use-before-declare/test.ts.lint
index 2bb7e7feff3..d98b31ddf96 100644
--- a/test/rules/no-use-before-declare/test.ts.lint
+++ b/test/rules/no-use-before-declare/test.ts.lint
@@ -72,3 +72,12 @@ function a() {
     var b = ({c: dog}) => { dog++; };
     b({ c: 5 });
 }
+
+import { ITest as ITest0 } from './ImportRegularAlias';
+import {/*ensure comments works*/ ITest as ITest1 } from './ImportAliasWithComment';
+import {
+    ITest as ITest2
+} from './ImportWithLineBreaks';
+import {First, ITest as ITest3 } from './ImportAliasSecond';
+
+import ITest from './InterfaceFile';