Fix for #883
authorJörn Zaefferer <joern.zaefferer@gmail.com>
Wed, 31 Jan 2007 21:36:48 +0000 (21:36 +0000)
committerJörn Zaefferer <joern.zaefferer@gmail.com>
Wed, 31 Jan 2007 21:36:48 +0000 (21:36 +0000)
src/jquery/coreTest.js
src/jquery/jquery.js

index a7b470e..0bb9727 100644 (file)
@@ -102,7 +102,7 @@ test("attr(Hash)", function() {
 });\r
 \r
 test("attr(String, Object)", function() {\r
-       expect(6);\r
+       expect(7);\r
        var div = $("div");\r
        div.attr("foo", "bar");\r
        var pass = true;\r
@@ -110,6 +110,8 @@ test("attr(String, Object)", function() {
          if ( div.get(i).getAttribute('foo') != "bar" ) pass = false;\r
        }\r
        ok( pass, "Set Attribute" );\r
+\r
+       ok( $("#foo").attr({"width": null}), "Try to set an attribute to nothing" );    \r
        \r
        $("#name").attr('name', 'something');\r
        ok( $("#name").attr('name') == 'something', 'Set name attribute' );\r
index 933e974..76f11d7 100644 (file)
@@ -1281,7 +1281,7 @@ jQuery.extend({
                        var exclude = /z-?index|font-?weight|opacity|zoom|line-?height/i;
 
                        // Handle passing in a number to a CSS property
-                       if ( value.constructor == Number && type == "curCSS" && !exclude.test(prop) )
+                       if ( value && value.constructor == Number && type == "curCSS" && !exclude.test(prop) )
                                return value + "px";
 
                        return value;