deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

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

Browse source
This commit is contained in:
David Serduke 2007-11-28 16:38:34 +00:00
parent 0a755f6ab3
commit 5c6c99361f
2 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/core.js
View file

@ -396,7 +396,7 @@ jQuery.fn = jQuery.prototype = {
// Everything else, we just grab the value // Everything else, we just grab the value
} else } else
return this[0].value.replace(/\r/g, ""); return (this[0].value || "").replace(/\r/g, "");
} }

4
test/unit/core.js
View file

@ -881,9 +881,11 @@ test("$.extend(Object, Object)", function() {
}); });
test("val()", function() { test("val()", function() {
expect(2); expect(3);
ok( $("#text1").val() == "Test", "Check for value of input element" ); ok( $("#text1").val() == "Test", "Check for value of input element" );
ok( !$("#text1").val() == "", "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() { test("val(String)", function() {