engineyard/todo

Function dirCheck has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

function dirCheck( dir, cur, doneName, checkSet, nodeCheck, isXML ) {

Avoid deeply nested control flow statements.

Open

                        if ( name.indexOf( "data-" ) === 0 ) {
                            name = jQuery.camelCase( name.substring(5) );

                            dataAttr( this[0], name, data[ name ] );
                        }

Avoid deeply nested control flow statements.

Open

                        if ( elem && elem.parentNode ) {
                            // Handle the case where IE and Opera return items
                            // by name instead of ID
                            if ( elem.id === match[3] ) {
                                return makeArray( [ elem ], extra );

Avoid deeply nested control flow statements.

Open

                        if ( ret[r] === ret[n] ) {
                            ret.splice(n--, 1);
                            break;
                        }

Avoid deeply nested control flow statements.

Open

                                    if ( xml && xml.documentElement /* #4958 */ ) {
                                        responses.xml = xml;
                                    }

Function inspectPrefiltersOrTransports has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

function inspectPrefiltersOrTransports( structure, options, originalOptions, jqXHR,
        dataType /* internal */, inspected /* internal */ ) {

Avoid deeply nested control flow statements.

Open

                        if ( match.jquery ? match.index( cur ) > -1 : jQuery( cur ).is( match ) ) {
                            ret.push({ selector: selector, elem: cur, level: level });
                        }

Avoid deeply nested control flow statements.

Open

                    if ( tmp[ 0 ] === prev || tmp[ 0 ] === "*" ) {
                        conv2 = converters[ tmp[1] + " " + current ];
                        if ( conv2 ) {
                            conv1 = converters[ conv1 ];
                            if ( conv1 === true ) {

Avoid deeply nested control flow statements.

Open

                                    if ( xhrOnUnloadAbort ) {
                                        delete xhrCallbacks[ handle ];
                                    }

Consider simplifying this complex logical expression.

Open

        if ( !seed && parts.length > 1 && context.nodeType === 9 && !contextXML &&
                Expr.match.ID.test(parts[0]) && !Expr.match.ID.test(parts[parts.length - 1]) ) {

            ret = Sizzle.find( parts.shift(), context, contextXML );
            context = ret.expr ?

Consider simplifying this complex logical expression.

Open

if ( jQuery.expr && jQuery.expr.filters ) {
    jQuery.expr.filters.hidden = function( elem ) {
        var width = elem.offsetWidth,
            height = elem.offsetHeight;

Consider simplifying this complex logical expression.

Open

if ( !jQuery.support.opacity ) {
    jQuery.cssHooks.opacity = {
        get: function( elem, computed ) {
            // IE uses filters for opacity
            return ropacity.test( (computed && elem.currentStyle ? elem.currentStyle.filter : elem.style.filter) || "" ) ?

Consider simplifying this complex logical expression.

Open

                if ( (e.keyCode === 13 && !jQuery.nodeName( elem, "textarea" ) ) ||
                    (e.keyCode === 32 && (type === "checkbox" || type === "radio")) ||
                    type === "select-multiple" ) {
                    testChange.call( this, e );
                }

Consider simplifying this complex logical expression.

Open

        if ( (!id || (pvt && id && !cache[ id ][ internalKey ])) && getByName && data === undefined ) {
            return;
        }

Consider simplifying this complex logical expression.

Open

    if ( event.liveFired === this || !events || !events.live || event.target.disabled || event.button && event.type === "click" ) {
        return;
    }

Consider simplifying this complex logical expression.

Open

        } else if ( typeof value === "string" && !rnocache.test( value ) &&
            (jQuery.support.leadingWhitespace || !rleadingWhitespace.test( value )) &&
            !wrapMap[ (rtagName.exec( value ) || ["", ""])[1].toLowerCase() ] ) {

            value = value.replace(rxhtmlTag, "<$1></$2>");

Consider simplifying this complex logical expression.

Open

        if ( s.crossDomain == null ) {
            parts = rurl.exec( s.url.toLowerCase() );
            s.crossDomain = !!( parts &&
                ( parts[ 1 ] != ajaxLocParts[ 1 ] || parts[ 2 ] != ajaxLocParts[ 2 ] ||
                    ( parts[ 3 ] || ( parts[ 1 ] === "http:" ? 80 : 443 ) ) !=

Function PSEUDO has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

        PSEUDO: function( match, curLoop, inplace, result, not ) {

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            for ( ; i < length; i++ ) {
                value = callback( elems[ i ], i, arg );

                if ( value != null ) {
                    ret[ ret.length ] = value;

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            for ( key in elems ) {
                value = callback( elems[ key ], key, arg );

                if ( value != null ) {
                    ret[ ret.length ] = value;

• Read up
  Read up