angelakuo/citydogshare

View on GitHub

Showing 2,136 of 3,976 total issues

Avoid deeply nested control flow statements.
Open

                    while ( i-- ) {
                        groups[i] = nidselect + " " + toSelector( groups[i] );
                    }

    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 ] );

      Function Order has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
      Open

      getJasmineRequireObj().Order = function() {
        function Order(options) {
          this.random = 'random' in options ? options.random : true;
          var seed = this.seed = options.seed || generateSeed();
          this.sort = this.random ? randomOrder : naturalOrder;

      Cognitive Complexity

      Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

      A method's cognitive complexity is based on a few simple rules:

      • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
      • Code is considered more complex for each "break in the linear flow of the code"
      • Code is considered more complex when "flow breaking structures are nested"

      Further reading

      Avoid deeply nested control flow statements.
      Open

                              if ( nid === expando ) {
                                  context.removeAttribute( "id" );
                              }

        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

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

            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 ( ( ofType ?
                                                      node.nodeName.toLowerCase() === name :
                                                      node.nodeType === 1 ) &&
                                                      ++diff ) {
              
              

                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 ( conv && s[ "throws" ] ) { // jscs:ignore requireDotNotation
                                          response = conv( response );
                                      } else {
                                          try {
                                              response = conv( response );

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

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

                      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

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

                          Avoid deeply nested control flow statements.
                          Open

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

                            Avoid deeply nested control flow statements.
                            Open

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

                              Avoid deeply nested control flow statements.
                              Open

                                                              if ( !status && options.isLocal && !options.crossDomain ) {
                                                                  status = responses.text ? 200 : 404;
                              
                                                              // IE - #1450: sometimes returns 1223 when it should be 204
                                                              } else if ( status === 1223 ) {

                                Avoid deeply nested control flow statements.
                                Open

                                            if (obj instanceof Assign) {
                                              defaultValue = obj.value;
                                              obj = obj.variable;
                                            }

                                  Avoid deeply nested control flow statements.
                                  Open

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

                                    Avoid deeply nested control flow statements.
                                    Open

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

                                      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" ) );
                                        Severity
                                        Category
                                        Status
                                        Source
                                        Language