adobe/brackets

View on GitHub

Showing 1,485 of 3,294 total issues

Consider simplifying this complex logical expression.
Open

if ( $.uiBackCompat !== false ) {

    // helper method for a lot of the back compat extensions
    $.ui.tabs.prototype._ui = function( tab, panel ) {
        return {

    Consider simplifying this complex logical expression.
    Open

                if ( instances.length !== 0 && instances[ instances.length - 1 ] === $el &&
                    dialog.options.closeOnEscape && !event.isDefaultPrevented() && event.keyCode &&
                    event.keyCode === $.ui.keyCode.ESCAPE ) {
    
                    dialog.close( event );

      Consider simplifying this complex logical expression.
      Open

              if ( tab.hasClass( "ui-state-disabled" ) ||
                      // tab is already loading
                      tab.hasClass( "ui-tabs-loading" ) ||
                      // can't switch durning an animation
                      this.running ||

        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;
                    }

          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 ( 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 ) ) !=

              Consider simplifying this complex logical expression.
              Open

                              if ( ontype && elem[ type ] && ((type !== "focus" && type !== "blur") || event.target.offsetWidth !== 0) && !jQuery.isWindow( elem ) ) {
              
                                  // Don't re-trigger an onFOO event when we call its FOO() method
                                  old = elem[ ontype ];
              
              

                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 ((rows < 1) || (rows > 2) || (columns < 1) || (columns > 2) || (columns === 2 && rows === 2)) {
                              console.error("setLayoutScheme unsupported layout " + rows + ", " + columns);
                              return false;
                          }
                  Severity: Major
                  Found in src/view/MainViewManager.js - About 40 mins to fix

                    Consider simplifying this complex logical expression.
                    Open

                                if (isPriorAttr &&
                                        (!ctx.token.type ||
                                        (ctx.token.type && ctx.token.type !== "attribute" &&
                                            ctx.token.type.indexOf("error") === -1 &&
                                            ctx.token.string.indexOf("<") !== -1))) {
                    Severity: Major
                    Found in src/language/HTMLUtils.js - About 40 mins to fix

                      Consider simplifying this complex logical expression.
                      Open

                              if ((!selector && !foundChars && !isPreprocessorDoc) ||
                                      (isPreprocessorDoc && (ctx.token.string === "" || /\s+/.test(ctx.token.string)))) {
                                  if (TokenUtils.moveNextToken(ctx) && ctx.token.type !== "comment" && _hasNonWhitespace(ctx.token.string)) {
                                      foundChars = true;
                                      ctx = TokenUtils.getInitialContext(cm, $.extend({}, pos));
                      Severity: Major
                      Found in src/language/CSSUtils.js - About 40 mins to fix

                        Consider simplifying this complex logical expression.
                        Open

                                    if ( matcher[ expando ] ) {
                                        // Find the next relative operator (if any) for proper handling
                                        j = ++i;
                                        for ( ; j < len; j++ ) {
                                            if ( Expr.relative[ tokens[j].type ] ) {
                        Severity: Major
                        Found in src/thirdparty/jquery-2.1.3.js - About 40 mins to fix

                          Consider simplifying this complex logical expression.
                          Open

                              if ( support.matchesSelector && documentIsHTML &&
                                  ( !rbuggyMatches || !rbuggyMatches.test( expr ) ) &&
                                  ( !rbuggyQSA     || !rbuggyQSA.test( expr ) ) ) {
                          
                                  try {
                          Severity: Major
                          Found in src/thirdparty/jquery-2.1.3.js - About 40 mins to fix

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

                              Consider simplifying this complex logical expression.
                              Open

                                      if (!language._setId(id) || !language._setName(name) ||
                                              (blockComment && !language.setBlockCommentSyntax(blockComment[0], blockComment[1])) ||
                                              (lineComment && !language.setLineCommentSyntax(lineComment))) {
                                          result.reject();
                                          return result.promise();
                              Severity: Major
                              Found in src/language/LanguageManager.js - About 40 mins to fix

                                Consider simplifying this complex logical expression.
                                Open

                                        if (query.length <= 1 || !regInfo ||
                                                (regInfo[1] && isNaN(regInfo[1])) ||
                                                (regInfo[3] && isNaN(regInfo[3]))) {
                                
                                            return null;
                                Severity: Major
                                Found in src/search/QuickOpen.js - About 40 mins to fix

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

                                      Editor.prototype.setSelection = function (start, end, center, centerOptions, origin) {
                                  Severity: Minor
                                  Found in src/editor/Editor.js - About 35 mins to fix

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

                                    function _cmdUpdate(packagePath, destinationDirectory, options, callback, pCallback) {
                                    Severity: Minor
                                    Found in src/extensibility/node/ExtensionManagerDomain.js - About 35 mins to fix

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

                                      function _cmdDownloadFile(downloadId, url, proxy, callback, pCallback) {
                                      Severity: Minor
                                      Found in src/extensibility/node/ExtensionManagerDomain.js - About 35 mins to fix

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

                                            function _getLineCommentPrefixSuffixEdit(editor, prefix, suffix, lineSel, command) {
                                        Severity: Minor
                                        Found in src/editor/EditorCommandHandlers.js - About 35 mins to fix
                                          Severity
                                          Category
                                          Status
                                          Source
                                          Language