Merge pull request #612 from jccazeaux/nested-if-dont-correctly-rebind

Set bound to false when unbinding 'if' binder

Author: benadamstyles

spec/rivets/binders.js

@@ -199,6 +199,24 @@ describe("Rivets.binders", function() {
       // 1 for the comment placeholder
       Should(fragment.childNodes.length).be.exactly(1);
     });
+
+    it("rebindes nested if", function() {
+      var nestedEl = document.createElement("div")
+      nestedEl.setAttribute("rv-if", "data.showNested");
+      nestedEl.innerHTML = "{ data.countNested }";
+      el.appendChild(nestedEl);
+
+      var view = rivets.bind(fragment, model);
+
+      model.data.countNested = "1";
+      model.data.showNested = true;
+      Should(nestedEl.innerHTML).be.exactly("1");
+      model.data.show = false;
+      model.data.show = true;
+      model.data.countNested = "42";
+
+      Should(nestedEl.innerHTML).be.exactly("42");
+    });
   });
 
   describe("Custom binder with no attribute value", function() {

src/binders.coffee

@@ -111,7 +111,9 @@ Rivets.public.binders.if =
       el.parentNode.removeChild el
 
   unbind: ->
-    @nested?.unbind()
+    if @nested
+      @nested.unbind()
+      @bound = false
 
   routine: (el, value) ->
     if !!value is not @bound