fix(transformer): Put paramater data in the same order as the reflected version.

jakemac53 · jakemac53 · commit 2b45bd2a63d4 · 2015-07-06T15:48:00.000-07:00
Previously it would be [@Inject(#thing), Thing], but it should be [Thing, @Inject(#thing)].
diff --git a/modules/angular2/src/transform/directive_processor/rewriter.dart b/modules/angular2/src/transform/directive_processor/rewriter.dart
@@ -243,9 +243,7 @@ class CreateNgDepsVisitor extends Object with SimpleAstVisitor<Object> {
     writer.print('..registerFunction(');
     node.name.accept(this);
     writer.print(''', {'parameters': const [''');
-    var parameters = node.childEntities
-        .firstWhere((child) => child is FunctionExpression).parameters;
-    parameters.accept(_paramsVisitor);
+    node.functionExpression.parameters.accept(_paramsVisitor);
     writer.print('''], 'annotations': ''');
     node.metadata.accept(_metaVisitor);
     writer.print('})');
diff --git a/modules/angular2/src/transform/directive_processor/visitors.dart b/modules/angular2/src/transform/directive_processor/visitors.dart
@@ -46,18 +46,9 @@ class _CtorTransformVisitor extends ToSourceVisitor {
   /// `_withParameterNames` is true, this method outputs `node`'s identifier.
   Object _visitNormalFormalParameter(
       NodeList<Annotation> metadata, TypeName type, SimpleIdentifier name) {
-    if (_withParameterAnnotations && metadata != null) {
-      assert(_withParameterTypes);
-      for (var i = 0, iLen = metadata.length; i < iLen; ++i) {
-        if (i != 0) {
-          writer.print(', ');
-        }
-        metadata[i].accept(this);
-      }
-      writer.print(type != null && metadata.isNotEmpty ? ', ' : '');
-    }
     var needCompileTimeConstants = !_withParameterNames;
-    if (_withParameterTypes && type != null) {
+    var needType = _withParameterTypes && type != null;
+    if (needType) {
       _visitNodeWithSuffix(type.name, ' ');
       if (!needCompileTimeConstants) {
         // Types with arguments are not compile-time constants.
@@ -67,6 +58,15 @@ class _CtorTransformVisitor extends ToSourceVisitor {
     if (_withParameterNames) {
       _visitNode(name);
     }
+    if (_withParameterAnnotations && metadata != null) {
+      assert(_withParameterTypes);
+      for (var i = 0, iLen = metadata.length; i < iLen; ++i) {
+        if (i != 0 || needType) {
+          writer.print(', ');
+        }
+        metadata[i].accept(this);
+      }
+    }
     return null;
   }
 
diff --git a/modules/angular2/test/transform/directive_processor/parameter_metadata/expected/soup.ng_deps.dart b/modules/angular2/test/transform/directive_processor/parameter_metadata/expected/soup.ng_deps.dart
@@ -11,7 +11,7 @@ void initReflector(reflector) {
     ..registerType(SoupComponent, {
       'factory':
           (String description, salt) => new SoupComponent(description, salt),
-      'parameters': const [const [Tasty, String], const [const Inject(Salt)]],
+      'parameters': const [const [String, Tasty], const [const Inject(Salt)]],
       'annotations': const [const Component(selector: '[soup]')]
     });
 }

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ void initReflector(reflector) {`
`11`	`11`	`..registerType(SoupComponent, {`
`12`	`12`	`'factory':`
`13`	`13`	`(String description, salt) => new SoupComponent(description, salt),`
`14`		`- 'parameters': const [const [Tasty, String], const [const Inject(Salt)]],`
	`14`	`+ 'parameters': const [const [String, Tasty], const [const Inject(Salt)]],`
`15`	`15`	`'annotations': const [const Component(selector: '[soup]')]`
`16`	`16`	`});`
`17`	`17`	`}`