fix for double binding.

billpull · Apr 17, 2015 · 9d809d5 · 9d809d5
1 parent c68e5f7
commit 9d809d5
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/build/knockout-bootstrap.min.js b/build/knockout-bootstrap.min.js
diff --git a/src/knockout-bootstrap.js b/src/knockout-bootstrap.js
@@ -181,6 +181,7 @@ function setupKoBootstrap(koObject, $) {
                     y: button.outerHeight()
                 };
 
+                koObject.cleanNode(popoverEl[0]);
                 if (data) {
                     koObject.applyBindings({template: template, data: data}, popoverEl[0]);
                 }
@@ -261,10 +262,16 @@ function setupKoBootstrap(koObject, $) {
                 modal.html($('#'+template).html());
             }
 
+            // As modal is not destroyed we want the data to be up to date.
+            var original = modal.html();
 
             modal.modal(options);
 
             $element.on('click', function() {
+
+                //replace modal with original template and clean node and initialize with new data.
+                modal.html(original);
+                koObject.cleanNode(modal[0]);
                 if (data) {
                     koObject.applyBindings({
                         template: template,