fix issue mwunsch#49

QiaoBuTang · Apr 8, 2015 · f5833d8 · f5833d8
1 parent 74df673
commit f5833d8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/main/scala/com/gilt/handlebars/scala/helper/HelperOptionsBuilder.scala b/src/main/scala/com/gilt/handlebars/scala/helper/HelperOptionsBuilder.scala
@@ -54,6 +54,8 @@ trait HelperOptions[T] {
    */
   def inverse(binding: Binding[T], extraData: Map[String, Binding[T]]): String
 
+  def hasInverse: Boolean
+
   /**
    * Look up a path in the the current context. The one in which the helper was called.
    * @param path The path to lookup in the context. e.g., ../name
@@ -111,6 +113,8 @@ class HelperOptionsBuilder[T](context: Context[T],
       }
     }
 
+    val hasInverse = inverseNode.isDefined
+
     def lookup(path: String): Binding[T] = {
       HandlebarsGrammar.path(path).map { identifier =>
         context.lookup(identifier, Seq()).binding

diff --git a/src/main/scala/com/gilt/handlebars/scala/helper/IfHelper.scala b/src/main/scala/com/gilt/handlebars/scala/helper/IfHelper.scala
@@ -8,6 +8,7 @@ class IfHelper[T] extends Helper[T] with Loggable {
     if (options.argument(0).isTruthy)
       options.visit(binding)
     else
-      options.inverse(binding)
+      if (options.hasInverse) options.inverse(binding)
+      else ""
   }
 }