fix: look up symbol directly rather than via type

alexeagle · alexeagle · commit aca7001f1179 · 2016-06-08T10:52:56.000-07:00
Summary: fixes #151 Reviewers: evmar Subscribers: typescript-eng Differential Revision: https://reviews.angular.io/D145
diff --git a/.arcconfig b/.arcconfig
@@ -0,0 +1,4 @@
+{
+  "base": "git:origin/master",
+  "phabricator.uri" : "https://reviews.angular.io/"
+}
diff --git a/src/decorator-annotator.ts b/src/decorator-annotator.ts
@@ -57,7 +57,7 @@ class ClassRewriter extends Rewriter {
       if (param.type) {
         // param has a type provided, e.g. "foo: Bar".
         // Verify that "Bar" is a value (e.g. a constructor) and not just a type.
-        let sym = this.typeChecker.getTypeAtLocation(param.type).getSymbol();
+        let sym = this.typeChecker.getSymbolAtLocation(param.type);
         if (sym && (sym.flags & ts.SymbolFlags.Value)) {
           paramCtor = new TypeTranslator(this.typeChecker, param.type).symbolToString(sym);
         }

-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +{
 +  "base": "git:origin/master",
 +  "phabricator.uri" : "https://reviews.angular.io/"
 +}
Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ class ClassRewriter extends Rewriter {`
`57`	`57`	`if (param.type) {`
`58`	`58`	`// param has a type provided, e.g. "foo: Bar".`
`59`	`59`	`// Verify that "Bar" is a value (e.g. a constructor) and not just a type.`
`60`		`- let sym = this.typeChecker.getTypeAtLocation(param.type).getSymbol();`
	`60`	`+ let sym = this.typeChecker.getSymbolAtLocation(param.type);`
`61`	`61`	`if (sym && (sym.flags & ts.SymbolFlags.Value)) {`
`62`	`62`	`paramCtor = new TypeTranslator(this.typeChecker, param.type).symbolToString(sym);`
`63`	`63`	`}`