fix(compiler): create literal property bindings for empty *… directives.

tbosch · tbosch · commit b2dc5c2c7e71 · 2015-10-26T13:55:34.000-07:00
Closes #4916
diff --git a/modules/angular2/src/core/compiler/template_parser.ts b/modules/angular2/src/core/compiler/template_parser.ts
@@ -272,6 +272,7 @@ class TemplateParseVisitor implements HtmlAstVisitor {
                                  targetMatchableAttrs, targetProps);
         } else {
           targetMatchableAttrs.push([dashCaseKey, '']);
+          this._parseLiteralAttr(dashCaseKey, null, attr.sourceInfo, targetProps);
         }
       }
       return true;
diff --git a/modules/angular2/test/core/compiler/template_parser_spec.ts b/modules/angular2/test/core/compiler/template_parser_spec.ts
@@ -670,6 +670,17 @@ There is no directive with "exportAs" set to "dirA" at TestComp > div:nth-child(
                 [ElementAst, 'div', 'TestComp > div:nth-child(0)']
               ]);
         });
+
+        it('should work with *... and empty value', () => {
+          expect(humanizeTemplateAsts(parse('<div *ng-if>', [ngIf])))
+              .toEqual([
+                [EmbeddedTemplateAst, 'TestComp > div:nth-child(0)'],
+                [DirectiveAst, ngIf, 'TestComp > div:nth-child(0)'],
+                [BoundDirectivePropertyAst, 'ngIf', 'null', 'TestComp > div:nth-child(0)[*ng-if=]'],
+                [ElementAst, 'div', 'TestComp > div:nth-child(0)']
+              ]);
+        });
+
       });
 
     });

Original file line number	Diff line number	Diff line change
`@@ -272,6 +272,7 @@ class TemplateParseVisitor implements HtmlAstVisitor {`
`272`	`272`	`targetMatchableAttrs, targetProps);`
`273`	`273`	`} else {`
`274`	`274`	`targetMatchableAttrs.push([dashCaseKey, '']);`
	`275`	`+ this._parseLiteralAttr(dashCaseKey, null, attr.sourceInfo, targetProps);`
`275`	`276`	`}`
`276`	`277`	`}`
`277`	`278`	`return true;`