adobe/brackets

View on GitHub

Showing 1,485 of 3,294 total issues

Function showInfoBar has 51 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    function showInfoBar(msgObj) {
        var jsonToMustache = generateJsonForMustache(msgObj),
            $infoBarElement = $(Mustache.render(InfoBarHtml, jsonToMustache));

        cleanInfoBar(); //Remove an already existing info bar, if any
Severity: Major
Found in src/widgets/infobar.js - About 2 hrs to fix

    Consider simplifying this complex logical expression.
    Open

            if(!(/^(document|window|parent)$/).test(o.containment) && o.containment.constructor != Array) {
                var c = $(o.containment);
                var ce = c[0]; if(!ce) return;
                var co = c.offset();
                var over = ($(ce).css("overflow") != 'hidden');

      Function animateClass has 50 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

      $.effects.animateClass = function( value, duration, easing, callback ) {
          var o = $.speed( duration, easing, callback );
      
          return this.queue( function() {
              var animated = $( this ),

        Function _mouseCapture has 50 lines of code (exceeds 25 allowed). Consider refactoring.
        Open

            _mouseCapture: function( event ) {
                var position, normValue, distance, closestHandle, index, allowed, offset, mouseOverHandle,
                    that = this,
                    o = this.options;
        
        

          Consider simplifying this complex logical expression.
          Open

              if ( args.length === 1 && typeof first === "string" && first.length < 512 && doc === document &&
                  first.charAt(0) === "<" && !rnocache.test( first ) &&
                  (jQuery.support.checkClone || !rchecked.test( first )) &&
                  (jQuery.support.html5Clone || !rnoshimcache.test( first )) ) {
          
          

            Consider simplifying this complex logical expression.
            Open

                            if ( ( mappedTypes || origType === handleObj.origType ) &&
                                 ( !handler || handler.guid === handleObj.guid ) &&
                                 ( !namespaces || namespaces.test( handleObj.namespace ) ) &&
                                 ( !selector || selector === handleObj.selector || selector === "**" && handleObj.selector ) ) {
                                eventType.splice( j--, 1 );

              Function handleExtractToFunction has 50 lines of code (exceeds 25 allowed). Consider refactoring.
              Open

                  function handleExtractToFunction() {
                      var editor = EditorManager.getActiveEditor();
                      var result = new $.Deferred(); // used only for testing purpose
              
                      if (editor.getSelections().length > 1) {
              Severity: Minor
              Found in src/extensions/default/JavaScriptRefactoring/ExtractToFunction.js - About 2 hrs to fix

                Function step has 50 lines of code (exceeds 25 allowed). Consider refactoring.
                Open

                    step: function( gotoEnd ) {
                        var p, n, complete,
                            t = fxNow || createFxNow(),
                            done = true,
                            elem = this.elem,

                  Function _handleRefactor has 50 lines of code (exceeds 25 allowed). Consider refactoring.
                  Open

                      function _handleRefactor(functionName) {
                          var eventName, eventType = "";
                          
                          switch (functionName) {
                          case REFACTOR_RENAME:
                  Severity: Minor
                  Found in src/extensions/default/JavaScriptRefactoring/main.js - About 2 hrs to fix

                    Function initTypingSpeedLogging has 50 lines of code (exceeds 25 allowed). Consider refactoring.
                    Open

                        function initTypingSpeedLogging() {
                            var editor = null,
                                inputField = null,
                                inProgress = false;
                    
                    
                    Severity: Minor
                    Found in src/extensions/samples/TypingSpeedLogger/main.js - About 2 hrs to fix

                      Function init has 50 lines of code (exceeds 25 allowed). Consider refactoring.
                      Open

                      function init(domainManager) {
                          if (!domainManager.hasDomain("fileWatcher")) {
                              domainManager.registerDomain("fileWatcher", {major: 0, minor: 1});
                          }
                      
                      
                      Severity: Minor
                      Found in src/filesystem/impls/appshell/node/FileWatcherDomain.js - About 2 hrs to fix

                        Function remove has 50 lines of code (exceeds 25 allowed). Consider refactoring.
                        Open

                            remove: function( elem, types, handler, selector, mappedTypes ) {
                        
                                var j, origCount, tmp,
                                    events, t, handleObj,
                                    special, handlers, type, namespaces, origType,
                        Severity: Minor
                        Found in src/thirdparty/jquery-2.1.3.js - About 2 hrs to fix

                          Function _getFunctionEndOffset has 50 lines of code (exceeds 25 allowed). Consider refactoring.
                          Open

                              function _getFunctionEndOffset(text, offsetStart) {
                                  var mode = CodeMirror.getMode({}, "javascript");
                                  var state = CodeMirror.startState(mode), stream, style, token;
                                  var curOffset = offsetStart, length = text.length, blockCount = 0, lineStart;
                                  var foundStartBrace = false;
                          Severity: Minor
                          Found in src/language/JSUtils.js - About 2 hrs to fix

                            Consider simplifying this complex logical expression.
                            Open

                                            if ( ( mappedTypes || origType === handleObj.origType ) &&
                                                ( !handler || handler.guid === handleObj.guid ) &&
                                                ( !tmp || tmp.test( handleObj.namespace ) ) &&
                                                ( !selector || selector === handleObj.selector || selector === "**" && handleObj.selector ) ) {
                                                handlers.splice( j, 1 );
                            Severity: Critical
                            Found in src/thirdparty/jquery-2.1.3.js - About 2 hrs to fix

                              Function _initTest has 50 lines of code (exceeds 25 allowed). Consider refactoring.
                              Open

                                  function _initTest() {
                                      // TODO: (issue #265) Make sure the "test" object is not included in final builds
                                      // All modules that need to be tested from the context of the application
                                      // must to be added to this object. The unit tests cannot just pull
                                      // in the modules since they would run in context of the unit test window,
                              Severity: Minor
                              Found in src/brackets.js - About 2 hrs to fix

                                Consider simplifying this complex logical expression.
                                Open

                                    if (
                                      !isIterable(b) ||
                                      a.size !== undefined && b.size !== undefined && a.size !== b.size ||
                                      a.__hash !== undefined && b.__hash !== undefined && a.__hash !== b.__hash ||
                                      isKeyed(a) !== isKeyed(b) ||
                                Severity: Critical
                                Found in src/thirdparty/immutable.js - About 2 hrs to fix

                                  Function _startServerAndGetTransports has 50 lines of code (exceeds 25 allowed). Consider refactoring.
                                  Open

                                      function _startServerAndGetTransports(communication, processArgs, isRuntime) {
                                          return new Promise(function (resolve, reject) {
                                              var serverProcess = null,
                                                  result = null,
                                                  protocolTransport = null,
                                  Severity: Minor
                                  Found in src/languageTools/LanguageClient/ServerUtils.js - About 2 hrs to fix

                                    Function _requestTernServer has 49 lines of code (exceeds 25 allowed). Consider refactoring.
                                    Open

                                    function _requestTernServer(commandConfig) {
                                        var file, text, offset,
                                            request = commandConfig,
                                            type = request.type;
                                        if (config.debug) {
                                    Severity: Minor
                                    Found in src/JSUtils/node/TernNodeDomain.js - About 1 hr to fix

                                      Function advance has 49 lines of code (exceeds 25 allowed). Consider refactoring.
                                      Open

                                          function advance(id, t) {
                                              switch (token.id) {
                                              case '(number)':
                                                  if (nexttoken.id === '.') {
                                                      warning("A dot following a number can be confused with a decimal point.", token);

                                        Function blind has 49 lines of code (exceeds 25 allowed). Consider refactoring.
                                        Open

                                        $.effects.effect.blind = function( o, done ) {
                                            // Create element
                                            var el = $( this ),
                                                props = [ "position", "top", "bottom", "left", "right", "height", "width" ],
                                                mode = $.effects.setMode( el, o.mode || "hide" ),
                                          Severity
                                          Category
                                          Status
                                          Source
                                          Language