Skip to content
Permalink
Browse files

Support for .foo(Function) and testing. TODO: More tests

  • Loading branch information
wycats committed Jul 12, 2009
1 parent e8eff25 commit 7d7a960035b7431887581c9cef09eaaef28e3d27
Showing with 171 additions and 79 deletions.
  1. +15 −5 src/attributes.js
  2. +8 −0 src/manipulation.js
  3. +148 −74 test/unit/manipulation.js
@@ -82,19 +82,29 @@ jQuery.fn.extend({
return undefined;
}

// Typecast once if the value is a number
if ( typeof value === "number" )
value += '';

var val = value;

return this.each(function(){
if(jQuery.isFunction(value)) {
val = value.call(this);
// Typecast each time if the value is a Function and the appended
// value is therefore different each time.
if( typeof val === "number" ) val += '';
}

if ( this.nodeType != 1 )
return;

if ( jQuery.isArray(value) && /radio|checkbox/.test( this.type ) )
this.checked = (jQuery.inArray(this.value, value) >= 0 ||
jQuery.inArray(this.name, value) >= 0);
if ( jQuery.isArray(val) && /radio|checkbox/.test( this.type ) )
this.checked = (jQuery.inArray(this.value, val) >= 0 ||
jQuery.inArray(this.name, val) >= 0);

else if ( jQuery.nodeName( this, "select" ) ) {
var values = jQuery.makeArray(value);
var values = jQuery.makeArray(val);

jQuery( "option", this ).each(function(){
this.selected = (jQuery.inArray( this.value, values ) >= 0 ||
@@ -105,7 +115,7 @@ jQuery.fn.extend({
this.selectedIndex = -1;

} else
this.value = value;
this.value = val;
});
}
});
@@ -138,6 +138,14 @@ jQuery.fn.extend({

domManip: function( args, table, callback ) {
var fragment, scripts, cacheable, cached, cacheresults, first;
var value = args[0];

if ( jQuery.isFunction(value) ) {
return this.each(function() {
args[0] = value.call(this);
return jQuery(this).domManip( args, table, callback );
});
};

if ( this[0] ) {
if ( args.length === 1 && typeof args[0] === "string" && args[0].length < 512 && args[0].indexOf("<option") < 0 ) {

0 comments on commit 7d7a960

Please sign in to comment.
You can’t perform that action at this time.