Skip to content
Permalink
Browse files

Fixed #1714 by adding a default empty string if the value is falsey.

  • Loading branch information
davids549 committed Nov 28, 2007
1 parent 0a755f6 commit 5c6c99361f315bfbc5829be3af94499c3a107f5d
Showing with 4 additions and 2 deletions.
  1. +1 −1 src/core.js
  2. +3 −1 test/unit/core.js
@@ -396,7 +396,7 @@ jQuery.fn = jQuery.prototype = {

// Everything else, we just grab the value
} else
return this[0].value.replace(/\r/g, "");
return (this[0].value || "").replace(/\r/g, "");

}

@@ -881,9 +881,11 @@ test("$.extend(Object, Object)", function() {
});

test("val()", function() {
expect(2);
expect(3);
ok( $("#text1").val() == "Test", "Check for value of input element" );
ok( !$("#text1").val() == "", "Check for value of input element" );
// ticket #1714 this caused a JS error in IE
ok( $("#first").val() == "", "Check a paragraph element to see if it has a value" );
});

test("val(String)", function() {

0 comments on commit 5c6c993

Please sign in to comment.
You can’t perform that action at this time.