]> source.dussan.org Git - jquery.git/commitdiff
Disabled the passthrough .attr(method_name) functionality. You can now use it if...
authorjeresig <jeresig@gmail.com>
Fri, 18 Dec 2009 17:41:53 +0000 (12:41 -0500)
committerjeresig <jeresig@gmail.com>
Fri, 18 Dec 2009 17:41:53 +0000 (12:41 -0500)
src/attributes.js
src/core.js
test/unit/attributes.js
test/unit/core.js

index 90f3062ed29e76bb17dbf6c4a8bb59723e779268..e7e5d378da8b886a42a96db88250f1ff9a193a43 100644 (file)
@@ -232,13 +232,13 @@ jQuery.extend({
                offset: true
        },
                
-       attr: function( elem, name, value ) {
+       attr: function( elem, name, value, pass ) {
                // don't set attributes on text and comment nodes
                if ( !elem || elem.nodeType === 3 || elem.nodeType === 8 ) {
                        return undefined;
                }
 
-               if ( name in jQuery.attrFn && value !== undefined ) {
+               if ( pass && name in jQuery.attrFn ) {
                        return jQuery(elem)[name](value);
                }
 
index bc48e5d47dc3dd95b9cdecf706d5e2030c6ec4bd..0c0d5a2e1d291c1e451abcc3352e76bfdf0c514e 100644 (file)
@@ -84,7 +84,13 @@ jQuery.fn = jQuery.prototype = {
                                        ret = rsingleTag.exec( selector );
 
                                        if ( ret ) {
-                                               selector = [ doc.createElement( ret[1] ) ];
+                                               if ( jQuery.isPlainObject( context ) ) {
+                                                       selector = [ document.createElement( ret[1] ) ];
+                                                       jQuery.fn.attr.call( selector, context, true );
+
+                                               } else {
+                                                       selector = [ doc.createElement( ret[1] ) ];
+                                               }
 
                                        } else {
                                                ret = buildFragment( [ match[1] ], [ doc ] );
@@ -687,13 +693,13 @@ function evalScript( i, elem ) {
 
 // Mutifunctional method to get and set values to a collection
 // The value/s can be optionally by executed if its a function
-function access( elems, key, value, exec, fn ) {
+function access( elems, key, value, exec, fn, pass ) {
        var length = elems.length;
        
        // Setting many attributes
        if ( typeof key === "object" ) {
                for ( var k in key ) {
-                       access( elems, k, key[k], exec, fn );
+                       access( elems, k, key[k], exec, fn, value );
                }
                return elems;
        }
@@ -704,7 +710,7 @@ function access( elems, key, value, exec, fn ) {
                exec = exec && jQuery.isFunction(value);
                
                for ( var i = 0; i < length; i++ ) {
-                       fn( elems[i], key, exec ? value.call( elems[i], i ) : value );
+                       fn( elems[i], key, exec ? value.call( elems[i], i ) : value, pass );
                }
                
                return elems;
index cb489804dda9b20db1eda52534c9f8c1ce30be74..2a3d1e1f29dad6081db3b6c557a812a8c999cd2d 100644 (file)
@@ -5,7 +5,7 @@ var functionReturningObj = function(value) { return (function() { return value;
 
 test("attr(String)", function() {
        expect(28);
-       
+
        // This one sometimes fails randomly ?!
        equals( jQuery('#text1').attr('value'), "Test", 'Check for value attribute' );
        
@@ -190,16 +190,16 @@ test("attr(jquery_method)", function(){
                elem = $elem[0];
        
        // one at a time        
-       $elem.attr('html', 'foo');
+       $elem.attr({'html': 'foo'}, true);
        equals( elem.innerHTML, 'foo', 'attr(html)');
        
-       $elem.attr('text', 'bar');
+       $elem.attr({'text': 'bar'}, true);
        equals( elem.innerHTML, 'bar', 'attr(text)');
        
-       $elem.attr('css', {color:'red'});
+       $elem.attr({'css': {color:'red'}}, true);
        ok( /^(#ff0000|red)$/i.test(elem.style.color), 'attr(css)');
        
-       $elem.attr('height', 10);
+       $elem.attr({'height': 10}, true);
        equals( elem.style.height, '10px', 'attr(height)');
        
        // Multiple attributes
@@ -207,7 +207,7 @@ test("attr(jquery_method)", function(){
        $elem.attr({
                width:10,
                css:{ paddingLeft:1, paddingRight:1 }
-       });
+       }, true);
        
        equals( elem.style.width, '10px', 'attr({...})');
        equals( elem.style.paddingLeft, '1px', 'attr({...})');
index 1e03c96c0e51606cd0b2e9c8d760080c7d422ec3..1888e555ada7381c3f5a4db70d5dcd7b5c158a18 100644 (file)
@@ -12,7 +12,7 @@ test("Basic requirements", function() {
 });
 
 test("jQuery()", function() {
-       expect(15);
+       expect(22);
 
        // Basic constructor's behavior
 
@@ -62,6 +62,22 @@ test("jQuery()", function() {
        equals( jQuery([1,2,3]).get(1), 2, "Test passing an array to the factory" );
 
        equals( jQuery(document.body).get(0), jQuery('body').get(0), "Test passing an html node to the factory" );
+
+       var elem = jQuery("<div/>", {
+                width: 10,
+                css: { paddingLeft:1, paddingRight:1 },
+               text: "test",
+               "class": "test2",
+               id: "test3"
+        });
+
+        equals( elem[0].style.width, '10px', 'jQuery() quick setter width');
+        equals( elem[0].style.paddingLeft, '1px', 'jQuery quick setter css');
+        equals( elem[0].style.paddingRight, '1px', 'jQuery quick setter css');
+        equals( elem[0].childNodes.length, 1, 'jQuery quick setter text');
+        equals( elem[0].firstChild.nodeValue, "test", 'jQuery quick setter text');
+        equals( elem[0].className, "test2", 'jQuery() quick setter class');
+        equals( elem[0].id, "test3", 'jQuery() quick setter id');
 });
 
 test("selector state", function() {