DamageStudios/rims

Avoid deeply nested control flow statements.

Open

          if (element.data('params')) data = data + '&' + element.data('params');

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

Open

function on( elem, types, selector, data, fn, one ) {

Avoid deeply nested control flow statements.

Open

                    if ( nodeType === 9 ) {
                        if ( (elem = context.getElementById( m )) ) {

                            // Support: IE, Opera, Webkit
                            // TODO: identify versions

Avoid deeply nested control flow statements.

Open

                        if ( attrs[ i ] ) {
                            name = attrs[ i ].name;
                            if ( name.indexOf( "data-" ) === 0 ) {
                                name = jQuery.camelCase( name.slice( 5 ) );
                                dataAttr( elem, name, data[ name ] );

Avoid deeply nested control flow statements.

Open

                    if ( copyIsArray ) {
                        copyIsArray = false;
                        clone = src && jQuery.isArray( src ) ? src : [];

                    } else {

Avoid deeply nested control flow statements.

Open

                    if ( !selector ) {
                        push.apply( results, seed );
                        return results;
                    }

Avoid deeply nested control flow statements.

Open

                    while ( j-- ) {
                        if ( jQuery.nodeName( ( tbody = elem.childNodes[ j ] ), "tbody" ) &&
                            !tbody.childNodes.length ) {

                            elem.removeChild( tbody );

Avoid deeply nested control flow statements.

Open

                    for ( conv2 in converters ) {

                        // If conv2 outputs current
                        tmp = conv2.split( " " );
                        if ( tmp[ 1 ] === current ) {

Avoid deeply nested control flow statements.

Open

                    if ( hasScripts ) {

                        // Support: Android<4.1, PhantomJS<2
                        // push.apply(_, arraylike) throws on ancient WebKit
                        jQuery.merge( scripts, getAll( node, "script" ) );

Avoid deeply nested control flow statements.

Open

                        if ( !attributes && typeof elem.removeAttribute !== "undefined" ) {
                            elem.removeAttribute( internalKey );

                        // Webkit & Blink performance suffers when deleting properties
                        // from DOM nodes, so set to undefined instead

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

Open

    init: function( elem, options, prop, end, easing, unit ) {

Avoid deeply nested control flow statements.

Open

                    if ( Expr.relative[ tokens[j].type ] ) {
                        break;
                    }

Avoid deeply nested control flow statements.

Open

                        for ( type in data.events ) {
                            if ( special[ type ] ) {
                                jQuery.event.remove( elem, type );

                            // This is a shortcut to avoid jQuery.event.remove's overhead

Avoid deeply nested control flow statements.

Open

                                if ( typeof xhr.responseText === "string" ) {
                                    responses.text = xhr.responseText;
                                }

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

Open

function setMatcher( preFilter, selector, matcher, postFilter, postFinder, postSelector ) {

Avoid deeply nested control flow statements.

Open

                } else if ( match[2] ) {
                    push.apply( results, context.getElementsByTagName( selector ) );
                    return results;

                // Class selector

Avoid deeply nested control flow statements.

Open

                    if ( (nid = context.getAttribute( "id" )) ) {
                        nid = nid.replace( rescape, "\\$&" );
                    } else {
                        context.setAttribute( "id", (nid = expando) );
                    }

Avoid deeply nested control flow statements.

Open

                    if ( rscriptType.test( node.type || "" ) &&
                        !jQuery._data( node, "globalEval" ) &&
                        jQuery.contains( doc, node ) ) {

                        if ( node.src ) {

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

Open

    $document.on('ajax:send.rails', rails.formSubmitSelector, function(event) {
      if (this === event.target) rails.disableFormElements($(this));
    });

• Read up
  Read up

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

Open

    $document.on('ajax:complete.rails', rails.formSubmitSelector, function(event) {
      if (this === event.target) rails.enableFormElements($(this));
    });

• Read up
  Read up