raphiz/passwordcards

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

                            if ( !(unmatched[i] || setMatched[i]) ) {
                                setMatched[i] = pop.call( results );
                            }

Avoid deeply nested control flow statements.

Open

                        while ( cur.indexOf( " " + clazz + " " ) >= 0 ) {
                            cur = cur.replace( " " + clazz + " ", " " );
                        }

Avoid deeply nested control flow statements.

Open

                        if ( (elem = matcherOut[i]) ) {
                            // Restore matcherIn since elem is not yet a final match
                            temp.push( (matcherIn[i] = elem) );
                        }

Avoid deeply nested control flow statements.

Open

                        if ( matches[ sel ] ) {
                            matches.push( handleObj );
                        }

Avoid deeply nested control flow statements.

Open

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

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 ( rscriptType.test( node.type || "" ) &&
                            !data_priv.access( node, "globalEval" ) && jQuery.contains( doc, node ) ) {

                            if ( node.src ) {
                                // Optional AJAX dependency, but won't run scripts if not present

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 ( cur.indexOf( " " + clazz + " " ) < 0 ) {
                            cur += clazz + " ";
                        }

Avoid deeply nested control flow statements.

Open

                                if ( ( ofType ? node.nodeName.toLowerCase() === name : node.nodeType === 1 ) && ++diff ) {
                                    // Cache the index of each encountered element
                                    if ( useCache ) {
                                        (node[ expando ] || (node[ expando ] = {}))[ type ] = [ dirruns, diff ];
                                    }

Avoid deeply nested control flow statements.

Open

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

Avoid deeply nested control flow statements.

Open

                                    if ( ofType ? node.nodeName.toLowerCase() === name : node.nodeType === 1 ) {
                                        return false;
                                    }

Avoid deeply nested control flow statements.

Open

                        if ( (event.result = ret) === false ) {
                            event.preventDefault();
                            event.stopPropagation();
                        }

Avoid deeply nested control flow statements.

Open

                        if ( matches[ sel ] === undefined ) {
                            matches[ sel ] = handleObj.needsContext ?
                                jQuery( sel, this ).index( cur ) >= 0 :
                                jQuery.find( sel, this, null, [ cur ] ).length;
                        }

Avoid deeply nested control flow statements.

Open

                    if ( elem && elem.parentNode ) {
                        // Handle the case where IE, Opera, and Webkit return items
                        // by name instead of ID
                        if ( elem.id === m ) {
                            results.push( elem );

Avoid deeply nested control flow statements.

Open

                            if ( (newCache[ 2 ] = matcher( elem, context, xml )) ) {
                                return true;
                            }

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

                        for ( match in context ) {
                            // Properties of context are called as methods if possible
                            if ( jQuery.isFunction( this[ match ] ) ) {
                                this[ match ]( context[ match ] );

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

Open

                cur = elem.nodeType === 1 && ( elem.className ?
                    ( " " + elem.className + " " ).replace( rclass, " " ) :
                    " "
                );

• Read up
  Read up