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

Fixed #1070 by converting all setAttribute() values to a string which is what all browsers but IE did. This will bring IE in line with the others and fix the bug.

Browse source
This commit is contained in:
David Serduke 2007-11-28 23:13:59 +00:00
parent 3ae5fbc16a
commit ed7608d8ee
2 changed files with 11 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/core.js
View file

@ -1042,7 +1042,8 @@ jQuery.extend({
if ( name == "type" && jQuery.nodeName( elem, "input" ) && elem.parentNode )
throw "type property can't be changed";
elem.setAttribute( name, value );
// convert the value to a string (all browsers do this but IE) see #1070
elem.setAttribute( name, "" + value );
}
if ( jQuery.browser.msie && /href|src/.test( name ) && !jQuery.isXMLDoc( elem ) )

10
test/unit/core.js
View file

@ -276,7 +276,7 @@ test("attr(Hash)", function() {
});
test("attr(String, Object)", function() {
expect(13);
expect(16);
var div = $("div");
div.attr("foo", "bar");
var pass = true;
@ -302,6 +302,14 @@ test("attr(String, Object)", function() {
$("#name").attr('maxLength', '10');
ok( document.getElementById('name').maxLength == '10', 'Set maxlength attribute' );
// for #1070
$("#name").attr('someAttr', '0');
equals( $("#name").attr('someAttr'), '0', 'Set attribute to a string of "0"' );
$("#name").attr('someAttr', 0);
equals( $("#name").attr('someAttr'), 0, 'Set attribute to the number 0' );
$("#name").attr('someAttr', 1);
equals( $("#name").attr('someAttr'), 1, 'Set attribute to the number 1' );
reset();
var type = $("#check2").attr('type');