Sign upLogin

adobe/brackets

View on GitHub
Star

Showing 1,485 of 3,294 total issues

Avoid deeply nested control flow statements.
Open

                            if ( !(unmatched[i] || setMatched[i]) ) {
                                setMatched[i] = pop.call( results );
                            }
Severity: Major
Found in src/thirdparty/jquery-2.1.3.js - About 45 mins to fix

    Avoid deeply nested control flow statements.
    Open

                        if ( !selector ) {
                        push.apply( results, seed );
                        return results;
                    }
    Severity: Major
    Found in src/thirdparty/jquery-2.1.3.js - About 45 mins to fix

      Avoid deeply nested control flow statements.
      Open

                              if ( (event.result = ret) === false ) {
                            event.preventDefault();
                            event.stopPropagation();
                        }
      Severity: Major
      Found in src/thirdparty/jquery-2.1.3.js - About 45 mins to fix

        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
        Severity: Major
        Found in src/thirdparty/jquery-2.1.3.js - About 45 mins to fix

          Avoid deeply nested control flow statements.
          Open

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

                    } else {
          Severity: Major
          Found in src/thirdparty/jquery-2.1.3.js - About 45 mins to fix

            Avoid deeply nested control flow statements.
            Open

                            } else if ( copy !== undefined ) {
                    target[ name ] = copy;
                }
            Severity: Major
            Found in src/thirdparty/jquery-2.1.3.js - About 45 mins to fix

              Avoid deeply nested control flow statements.
              Open

                                      if ( hasScripts ) {
                            // Support: QtWebKit
                            // jQuery.merge because push.apply(_, arraylike) throws
                            jQuery.merge( scripts, getAll( node, "script" ) );
                        }
              Severity: Major
              Found in src/thirdparty/jquery-2.1.3.js - About 45 mins to fix

                Avoid deeply nested control flow statements.
                Open

                                    if ( conv && s[ "throws" ] ) {
                        response = conv( response );
                    } else {
                        try {
                            response = conv( response );
                Severity: Major
                Found in src/thirdparty/jquery-2.1.3.js - About 45 mins to fix

                  Avoid deeply nested control flow statements.
                  Open

                                      for ( conv2 in converters ) {

                        // If conv2 outputs current
                        tmp = conv2.split( " " );
                        if ( tmp[ 1 ] === current ) {
                  Severity: Major
                  Found in src/thirdparty/jquery-2.1.3.js - About 45 mins to fix

                    Avoid deeply nested control flow statements.
                    Open

                                            if ( matches[ sel ] ) {
                            matches.push( handleObj );
                        }
                    Severity: Major
                    Found in src/thirdparty/jquery-2.1.3.js - About 45 mins to fix

                      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 ];
                                    }
                      Severity: Major
                      Found in src/thirdparty/jquery-2.1.3.js - About 45 mins to fix

                        Avoid deeply nested control flow statements.
                        Open

                                  if (options.debug)
            console.error('globstar swallow a segment, and continue')
                        Severity: Major
                        Found in src/thirdparty/globmatch.js - About 45 mins to fix

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

                              function addIndicator(id, indicator, visible, style, tooltip, insertBefore) {
                          Severity: Minor
                          Found in src/widgets/StatusBar.js - About 45 mins to fix

                            Avoid deeply nested control flow statements.
                            Open

                                            if (_hasButton(this, DIALOG_BTN_DONTSAVE)) {
                    buttonId = DIALOG_BTN_DONTSAVE;
                }
                            Severity: Major
                            Found in src/widgets/Dialogs.js - About 45 mins to fix

                              Avoid deeply nested control flow statements.
                              Open

                                              if (_hasButton(this, DIALOG_BTN_DONTSAVE)) {
                    buttonId = DIALOG_BTN_DONTSAVE;
                }
                              Severity: Major
                              Found in src/widgets/Dialogs.js - About 45 mins to fix

                                Avoid deeply nested control flow statements.
                                Open

                                                if (this._highlightIndex === null || this._highlightIndex === 0) {
                    this._highlightIndex = this._displayedResults.length - 1;
                } else {
                    this._highlightIndex--;
                }
                                Severity: Major
                                Found in src/search/QuickSearchField.js - About 45 mins to fix

                                  Avoid deeply nested control flow statements.
                                  Open

                                              } else if (e.metaKey && (e.which === KeyEvent.DOM_VK_PERIOD)) {
                buttonId = DIALOG_BTN_CANCEL;
            }
                                  Severity: Major
                                  Found in src/widgets/Dialogs.js - About 45 mins to fix

                                    Consider simplifying this complex logical expression.
                                    Open

                                            if (!(event.ctrlKey || event.altKey || event.metaKey) &&
                (event.keyCode === KeyEvent.DOM_VK_ENTER ||
                 event.keyCode === KeyEvent.DOM_VK_RETURN ||
                 event.keyCode === KeyEvent.DOM_VK_TAB)) {
            lastChar = String.fromCharCode(event.keyCode);
                                    Severity: Major
                                    Found in src/editor/CodeHintManager.js - About 40 mins to fix

                                      Consider simplifying this complex logical expression.
                                      Open

                                              if (inst == $.datepicker._curInst && $.datepicker._datepickerShowing && inst.input &&
                // #6694 - don't focus the input if it's already focused
                // this breaks the change event in IE
                inst.input.is(':visible') && !inst.input.is(':disabled') && inst.input[0] != document.activeElement)
            inst.input.focus();
                                      Severity: Major
                                      Found in src/extensions/default/JavaScriptQuickEdit/unittest-files/jquery-ui/ui/jquery.ui.datepicker.js - About 40 mins to fix

                                        Consider simplifying this complex logical expression.
                                        Open

                                                    if ( ( this.options.values.length === 2 && this.options.range === true ) &&
                    ( ( index === 0 && newVal > otherVal) || ( index === 1 && newVal < otherVal ) )
                ) {
                newVal = otherVal;
            }
                                        Severity: Major
                                        Found in src/extensions/default/JavaScriptQuickEdit/unittest-files/jquery-ui/ui/jquery.ui.slider.js - About 40 mins to fix
                                          Severity
                                          Category
                                          Status
                                          Source
                                          Language