Sign upLogin

adobe/brackets

View on GitHub
Star

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 {
Severity: Major
Found in src/extensions/default/JavaScriptQuickEdit/unittest-files/jquery-ui/ui/jquery.ui.tabs.js - About 40 mins to fix

    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 );
    Severity: Major
    Found in src/extensions/default/JavaScriptQuickEdit/unittest-files/jquery-ui/ui/jquery.ui.dialog.js - About 40 mins to fix

      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 ||
      Severity: Major
      Found in src/extensions/default/JavaScriptQuickEdit/unittest-files/jquery-ui/ui/jquery.ui.tabs.js - About 40 mins to fix

        Consider simplifying this complex logical expression.
        Open

                    if(
                inst.options.greedy
                && !inst.options.disabled
                && inst.options.scope == draggable.options.scope
                && inst.accept.call(inst.element[0], (draggable.currentItem || draggable.element))
        Severity: Major
        Found in src/extensions/default/JavaScriptQuickEdit/unittest-files/jquery-ui/ui/jquery.ui.droppable.js - About 40 mins to fix

          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 ];
          Severity: Major
          Found in src/extensions/default/JavaScriptQuickEdit/unittest-files/jquery-ui/jquery-1.7.2.js - About 40 mins to fix

            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 ?
            Severity: Major
            Found in src/extensions/default/JavaScriptQuickEdit/unittest-files/jquery-ui/jquery-1.7.2.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/extensions/default/JavaScriptQuickEdit/unittest-files/jquery-ui/jquery-1.7.2.js - About 40 mins to fix

                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) || "" ) ?
                Severity: Major
                Found in src/extensions/default/JavaScriptQuickEdit/unittest-files/jquery-ui/jquery-1.7.2.js - About 40 mins to fix

                  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 ((!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 (!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 (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 ( 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 ( 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 (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 _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

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

                                        function _buildKeyDescriptor(hasMacCtrl, hasCtrl, hasAlt, hasShift, key) {
                                    Severity: Minor
                                    Found in src/command/KeyBindingManager.js - About 35 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 _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
                                          Severity
                                          Category
                                          Status
                                          Source
                                          Language