From de0ce854e27426cd641e6041d821a03da188706a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jakub=20Nes=CC=8Cetr=CC=8Cil?= <jakub@apiary.io>
Date: Tue, 28 Aug 2012 20:19:44 +0200
Subject: [PATCH] Fixed :else in section when iterating over empty array.

The "No friends" 101 example from dust documentation was actually not
working
---
 lib/dust.js                            | 18 +++++++++++-------
 test/jasmine-test/spec/grammarTests.js |  7 +++++++
 test/test.html                         |  1 +
 3 files changed, 19 insertions(+), 7 deletions(-)

diff --git a/lib/dust.js b/lib/dust.js
index e5341a16..b954a3ed 100644
--- a/lib/dust.js
+++ b/lib/dust.js
@@ -390,14 +390,18 @@ Chunk.prototype.section = function(elem, context, bodies, params) {
   if (dust.isArray(elem)) {
     if (body) {
       var len = elem.length, chunk = this;
-      context.stack.head['$len'] = len;
-      for (var i=0; i<len; i++) {
-        context.stack.head['$idx'] = i;
-        chunk = body(chunk, context.push(elem[i], i, len));
+      if (len > 0) {
+        context.stack.head['$len'] = len;
+        for (var i=0; i<len; i++) {
+          context.stack.head['$idx'] = i;
+          chunk = body(chunk, context.push(elem[i], i, len));
+        }
+        context.stack.head['$idx'] = undefined;
+        context.stack.head['$len'] = undefined;
+        return chunk;
+      } else if (skip) {
+        return skip(this, context);
       }
-      context.stack.head['$idx'] = undefined;
-      context.stack.head['$len'] = undefined;
-      return chunk;
     }
   } else if (elem === true) {
     if (body) return body(this, context);
diff --git a/test/jasmine-test/spec/grammarTests.js b/test/jasmine-test/spec/grammarTests.js
index f95e5fab..51155ae5 100644
--- a/test/jasmine-test/spec/grammarTests.js
+++ b/test/jasmine-test/spec/grammarTests.js
@@ -127,6 +127,13 @@ var grammarTests = [
     expected: "foo, not bar!",
     message:"should test the else block"
   },
+  {
+    name:   "empty_else_block",
+    source: "{#foo}full foo{:else}empty foo{/foo}",
+    context: { foo: []},
+    expected: "empty foo",
+    message: "should test else block when array empty"
+  },
   {
     name:     "conditional",
     source:   "{?tags}\n"                     +
diff --git a/test/test.html b/test/test.html
index e724ea01..fdb191e8 100644
--- a/test/test.html
+++ b/test/test.html
@@ -259,6 +259,7 @@ <h1>{dust}</h1>
 				<option value="25">child_template</option>
 				<option value="26">recursion</option>
 				<option value="27">comments</option>
+				<option value="28">empty_else_block</option>
 			</select></span> or write your own:
 			</p>
 			<textarea id="input-source" class="console"></textarea>