Use getAttributeNode for ^on attributes in IE6/7 to avoid anonymous function wrapper...

diff --git a/src/attributes.js b/src/attributes.js
index 0eac08a6f069f29b1bc826373350cebefcf21e73..a4cf76959316e2b4992b484a1dc0c5a369d77694 100644 (file)
--- a/src/attributes.js
+++ b/src/attributes.js
@@ -7,7 +7,7 @@ var rclass = /[\n\t\r]/g,
        rfocusable = /^(?:button|input|object|select|textarea)$/i,
        rclickable = /^a(?:rea)?$/i,
        rboolean = /^(?:autofocus|autoplay|async|checked|controls|defer|disabled|hidden|loop|multiple|open|readonly|required|scoped|selected)$/i,
-       rinvalidChar = /\:/,
+       rinvalidChar = /\:|^on/,
        formHook, boolHook;
 
 jQuery.fn.extend({

diff --git a/test/unit/attributes.js b/test/unit/attributes.js
index 8f2eb723703427abd7074e72cf0717ce94f347c5..9ecf2360b8ea732d8c21dcb35b37c8745d12d75b 100644 (file)
--- a/test/unit/attributes.js
+++ b/test/unit/attributes.js
@@ -40,7 +40,7 @@ test("jQuery.attrFix/jQuery.propFix integrity test", function() {
 });
 
 test("attr(String)", function() {
-       expect(39);
+       expect(40);
 
        equals( jQuery("#text1").attr("type"), "text", "Check for type attribute" );
        equals( jQuery("#radio1").attr("type"), "radio", "Check for type attribute" );
@@ -119,7 +119,10 @@ test("attr(String)", function() {
        ok( jQuery().attr("doesntexist") === undefined, "Make sure undefined is returned when no element is there." );
 
        var $form = jQuery("<form class='something'></form>").appendTo("#qunit-fixture");
-       equal( $form.attr("class"), "something", "Retrieve the class attribute on a form" );
+       equal( $form.attr("class"), "something", "Retrieve the class attribute on a form." );
+
+       var $a = jQuery("<a href='#' onclick='something()'>Click</a>").appendTo("#qunit-fixture");
+       equal( $a.attr("onclick"), "something()", "Retrieve ^on attribute without anonymous function wrapper." );
 });
 
 if ( !isLocal ) {