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

Fixed #2076 where .val() could return an array instead of undefined when the jquery object was empty. The bug was created during the .val() refactoring for the javascript strict FF ticket.

Browse source
This commit is contained in:
David Serduke 2007-12-18 17:19:33 +00:00
parent fc51e14b81
commit 7ef40fd199
2 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
src/core.js
View file

@ -419,6 +419,7 @@ jQuery.fn = jQuery.prototype = {
} }
return undefined;
} }
return this.each(function(){ return this.each(function(){

3
test/unit/core.js
View file

@ -1029,11 +1029,12 @@ test("$.extend(Object, Object)", function() {
}); });
test("val()", function() { test("val()", function() {
expect(3); expect(4);
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 // ticket #1714 this caused a JS error in IE
ok( $("#first").val() == "", "Check a paragraph element to see if it has a value" ); ok( $("#first").val() == "", "Check a paragraph element to see if it has a value" );
ok( $([]).val() === undefined, "Check an empty jQuery object will return undefined from val" );
}); });
test("val(String)", function() { test("val(String)", function() {