add support for template repeat

lib/src/analyzer.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /**
  * Part of the template compilation that concerns with extracting information
  * from the HTML parse tree.
  */
 library analyzer;
 
@@ skipping 85 matching lines @@
     var info = null;
     if (node.tagName == 'script') {
       // We already extracted script tags in previous phase.
       return;
     }
 
     if (node.tagName == 'template'
         || node.attributes.containsKey('template')
         || node.attributes.containsKey('if')
         || node.attributes.containsKey('instantiate')
-        || node.attributes.containsKey('iterate')) {
+        || node.attributes.containsKey('iterate')
+        || node.attributes.containsKey('repeat')) {
       // template tags, conditionals and iteration are handled specially.
       info = _createTemplateInfo(node);
     }
 
     // TODO(jmesserly): it would be nice not to create infos for text or
     // elements that don't need data binding. Ideally, we would visit our
     // child nodes and get their infos, and if any of them need data binding,
     // we create an ElementInfo for ourselves and return it, otherwise we just
     // return null.
     if (info == null) {
@@ skipping 160 matching lines @@
       _currentInfo.usedComponents[component] = true;
       return component;
     }
     return null;
   }
 
   TemplateInfo _createTemplateInfo(Element node) {
     if (node.tagName != 'template' &&
         !node.attributes.containsKey('template')) {
       _messages.warning('template attribute is required when using if, '
-          'instantiate, or iterate attributes.',
+          'instantiate, repeat, or iterate attributes.',
           node.sourceSpan);
     }
 
     var instantiate = node.attributes['instantiate'];
     var condition = node.attributes['if'];
     if (instantiate != null) {
       if (instantiate.startsWith('if ')) {
         if (condition != null) {
           _messages.warning(
               'another condition was already defined on this element.',
               node.sourceSpan);
         } else {
           condition = instantiate.substring(3);
         }
       }
     }
+
+    // TODO(jmesserly): deprecate iterate.
     var iterate = node.attributes['iterate'];
+    var repeat = node.attributes['repeat'];
+    if (repeat == null) {
+      repeat = iterate;
+    } else if (iterate != null) {
+      _messages.warning('template cannot have both iterate and repeat. '
+          'iterate attribute will be ignored.', node.sourceSpan);
+      iterate = null;
+    }
 
     // Note: we issue warnings instead of errors because the spirit of HTML and
     // Dart is to be forgiving.
-    if (condition != null && iterate != null) {
+    if (condition != null && repeat != null) {
       _messages.warning('template cannot have both iteration and conditional '
           'attributes', node.sourceSpan);
       return null;
     }
 
     if (node.parent != null && node.parent.tagName == 'element' &&
-        (condition != null || iterate != null)) {
+        (condition != null || repeat != null)) {
 
       // TODO(jmesserly): would be cool if we could just refactor this, or offer
       // a quick fix in the Editor.
       var example = new Element.html('<element><template><template>');
       node.parent.attributes.forEach((k, v) { example.attributes[k] = v; });
       var nestedTemplate = example.nodes.first.nodes.first;
       node.attributes.forEach((k, v) { nestedTemplate.attributes[k] = v; });
 
       _messages.warning('the <template> of a custom element does not support '
-          '"if" or "iterate". However, you can create another template node '
-          'that is a child node, for example:\n'
+          '"if", "iterate" or "repeat". However, you can create another '
+          'template node that is a child node, for example:\n'
           '${example.outerHtml}',
           node.parent.sourceSpan);
       return null;
     }
 
     if (condition != null) {
       var result = new TemplateInfo(node, _parent, ifCondition: condition);
       result.removeAttributes.add('if');
       result.removeAttributes.add('instantiate');
       if (node.tagName == 'template') {
         return node.nodes.length > 0 ? result : null;
       }
 
-      result.removeAttributes.add('template');
+      _createTemplateAttributePlaceholder(node, result);
+      return result;
 
-      // TODO(jmesserly): if-conditions in attributes require injecting a
-      // placeholder node, and a real node which is a clone. We should
-      // consider a design where we show/hide the node instead (with care
-      // taken not to evaluate hidden bindings). That is more along the lines
-      // of AngularJS, and would have a cleaner DOM. See issue #142.
-      var contentNode = node.clone();
-      // Clear out the original attributes. This is nice to have, but
-      // necessary for ID because of issue #141.
-      node.attributes.clear();
-      contentNode.nodes.addAll(node.nodes);
+    } else if (repeat != null) {
+      var match = new RegExp(r"(.*) in (.*)").firstMatch(repeat);
+      if (match == null) {
+        _messages.warning('template iterate/repeat must be of the form: '
+            'repeat="variable in list", where "variable" is your variable name '
+            'and "list" is the list of items.',
+            node.sourceSpan);
+        return null;
+      }
 
-      // Create a new ElementInfo that is a child of "result" -- the
-      // placeholder node. This will become result.contentInfo.
-      visitElementInfo(_createElementInfo(contentNode, result));
-      return result;
-    } else if (iterate != null) {
-      var match = new RegExp(r"(.*) in (.*)").firstMatch(iterate);
-      if (match != null) {
-        if (node.nodes.length == 0) return null;
-        var result = new TemplateInfo(node, _parent, loopVariable: match[1],
-            loopItems: match[2]);
-        result.removeAttributes.add('iterate');
-        if (node.tagName != 'template') result.removeAttributes.add('template');
+      if (node.nodes.length == 0) return null;
+      var result = new TemplateInfo(node, _parent, loopVariable: match[1],
+          loopItems: match[2], isRepeat: iterate == null);
+      result.removeAttributes.add('iterate');
+      result.removeAttributes.add('repeat');
+      if (node.tagName == 'template') {
         return result;
       }
-      _messages.warning('template iterate must be of the form: '
-          'iterate="variable in list", where "variable" is your variable name '
-          'and "list" is the list of items.',
-          node.sourceSpan);
+
+      if (!result.isRepeat) {
+        result.removeAttributes.add('template');
+        // TODO(jmesserly): deprecate this? I think you want "template repeat"
+        // most of the time, but "template iterate" seems useful sometimes.
+        // (Native <template> element parsing would make both obsolete, though.)
+        return result;
+      }
+
+      _createTemplateAttributePlaceholder(node, result);
+      return result;
     }
+
     return null;
   }
 
+  // TODO(jmesserly): if and repeat in attributes require injecting a
+  // placeholder node, and a real node which is a clone. We should
+  // consider a design where we show/hide the node instead (with care
+  // taken not to evaluate hidden bindings). That is more along the lines
+  // of AngularJS, and would have a cleaner DOM. See issue #142.
+  void _createTemplateAttributePlaceholder(Element node, TemplateInfo result) {
+    result.removeAttributes.add('template');
+    var contentNode = node.clone();
+    node.attributes.clear();
+    contentNode.nodes.addAll(node.nodes);
+
+    // Create a new ElementInfo that is a child of "result" -- the
+    // placeholder node. This will become result.contentInfo.
+    visitElementInfo(_createElementInfo(contentNode, result));
+  }
+
   void visitAttribute(ElementInfo info, String name, String value) {
     if (name.startsWith('on')) {
       _readEventHandler(info, name, value);
       return;
     } else if (name.startsWith('bind-')) {
       // Strip leading "bind-" and make camel case.
       var fieldName = toCamelCase(name.substring(5));
       if (_readTwoWayBinding(info, fieldName, value)) {
         info.removeAttributes.add(name);
       }
@@ skipping 628 matching lines @@
     if (start == null) moveNext();
     if (start < length) {
       do {
         bindings.add(binding);
         content.add(textContent);
       } while (moveNext());
     }
     content.add(textContent);
   }
 }