Ensure that text/value bindings treat "null" and "undefined" gracious…

…ly (display in UI as empty string)
knockout · Dec 19, 2010 · 920c3f8 · 920c3f8
1 parent 001ea5a
commit 920c3f8
Show file tree

Hide file tree

Showing 5 changed files with 99 additions and 54 deletions.
diff --git a/build/output/knockout-latest.debug.js b/build/output/knockout-latest.debug.js
@@ -830,8 +830,11 @@ ko.exportSymbol('ko.toJSON', ko.toJSON);(function () {
                         break;
                     }
                 }
-            } else
+            } else {
+                if ((value === null) || (value === undefined))
+                    value = "";
                 element.value = value;
+            }
         }
     };        
 })();
@@ -1221,6 +1224,8 @@ ko.bindingHandlers['selectedOptions'] = {
 ko.bindingHandlers['text'] = {
     'update': function (element, valueAccessor) {
         var value = ko.utils.unwrapObservable(valueAccessor());
+        if ((value === null) || (value === undefined))
+            value = "";
         typeof element.innerText == "string" ? element.innerText = value
                                              : element.textContent = value;
     }