Added support for <foreach>. Tested and works.

jbeard4 · Jul 20, 2012 · ec8108c · ec8108c
1 parent 134d158
commit ec8108c
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/lib/core/util/action-code-generator.js b/lib/core/util/action-code-generator.js
@@ -26,7 +26,7 @@ var codeGenerators = {
     },
 
     "assign" : function(attributes,children){
-        return attributes.location + " = " + attributes.expr;
+        return attributes.location + " = " + attributes.expr + ";";
     },
 
     "if" : function(attributes,children){
@@ -108,6 +108,20 @@ var codeGenerators = {
             "delay: " + (attributes.delayexpr ? attributes.delayexpr : getDelayInMs(attributes.delay)) + ",\n" + 
             "sendId: " + (attributes.idlocation ? attributes.idlocation : JSON.stringify(attributes.id)) + "\n" + 
         "});";
+    },
+
+    "foreach" : function(attributes,children){
+        var isIndexDefined = attributes.index,
+            index = attributes.index || "$i",        //FIXME: the index variable could shadow the datamodel. We should pick a unique temperorary variable name
+            item = attributes.item,
+            arr = attributes.array;
+
+        return "(function(){\n" + 
+            "for(" + (isIndexDefined  ? "" : "var " + index + " = 0") + "; " + index + " < " + arr + ".length; " + index + "++){\n" + 
+                item + " = " + arr + "[" + index + "];\n" + 
+                children.filter(Array.isArray).map(_generateCode).join("\n;;\n") + 
+            "\n}\n" + 
+        "})();";
     }
 };
 

diff --git a/test/scxml-test-framework b/test/scxml-test-framework