adobe/brackets

File WorkingSetView.js has 890 lines of code (exceeds 250 allowed). Consider refactoring.

Open

/*
 * Copyright (c) 2014 - present Adobe Systems Incorporated. All rights reserved.
 *
 * Permission is hereby granted, free of charge, to any person obtaining a
 * copy of this software and associated documentation files (the "Software"),

Function _makeDraggable has 426 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function _makeDraggable($el) {
        var interval,
            sourceFile = $el.data(_FILE_KEY);

        // turn off the "hover-scroll"

Function hitTest has 194 lines of code (exceeds 25 allowed). Consider refactoring.

Open

            function hitTest(e) {
                var pageY = $ghost.offset().top,
                    direction =  e.pageY - lastPageY,
                    result = {
                        where: NOMANSLAND

Function drag has 58 lines of code (exceeds 25 allowed). Consider refactoring.

Open

                function drag(e) {
                    if (!dragged) {
                        initDragging();
                        // sort redraw and scroll shadows
                        //  cause problems during drag so disable them

Function drop has 44 lines of code (exceeds 25 allowed). Consider refactoring.

Open

            function drop() {
                preDropCleanup();
                if (sourceView.paneId === currentView.paneId && startingIndex === $el.index()) {
                    // if the item was dragged but not moved then don't open or close
                    if (!dragged) {

Function _createNewListItem has 30 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    WorkingSetView.prototype._createNewListItem = function (file) {
        var self = this,
            selectedFile = MainViewManager.getCurrentlyViewedFile(this.paneId),
            data = {fullPath: file.fullPath,
                    name: file.name,

Avoid deeply nested control flow statements.

Open

                        if ($next.length) {
                            result = {
                                where: ABOVEVIEW,
                                which: $next
                            };

Avoid deeply nested control flow statements.

Open

                            if (ghostIsBelow($hit)) {
                                return result;
                            }

Avoid deeply nested control flow statements.

Open

                    } else if (mouseIsInTopHalf($view)) {
                        // we're inside the top half of
                        //  a view so prepend to the view we hit
                        result = {
                            where: ABOVEVIEW,

Avoid too many return statements within this function.

Open

                return result;

Avoid too many return statements within this function.

Open

                        return result;

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                function ghostIsAbove($elem) {
                    var top = $elem.offset().top,
                        checkVal = gTop;

                    if (direction > 0) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                function ghostIsBelow($elem) {
                    var top = $elem.offset().top,
                        checkVal = gTop;

                    if (direction > 0) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                onBottomScroller = hasScroller && scrollerBottomArea && ((gTop >= scrollerBottomArea.top && gTop <= scrollerBottomArea.bottom) ||
                                                         (gBottom >= scrollerBottomArea.top && gBottom <= scrollerBottomArea.bottom));

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                onTopScroller = hasScroller && scrollerTopArea && ((gTop >= scrollerTopArea.top && gTop <= scrollerTopArea.bottom)  ||
                                                    (gBottom >= scrollerTopArea.top && gBottom <= scrollerTopArea.bottom));

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

                                    CommandManager
                                        .execute(Commands.FILE_OPEN, {fullPath: sourceFile.fullPath,
                                                                       paneId: currentView.paneId})
                                        .always(function () {
                                            postDropCleanup();

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

                            CommandManager
                                .execute(Commands.FILE_OPEN, {fullPath: sourceFile.fullPath,
                                                               paneId: currentView.paneId})
                                .always(function () {
                                    postDropCleanup();

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                if (dirSplit.length > 3) {
                    displayPaths[io] = dirSplit[0] + "/\u2026/" + dirSplit[dirSplit.length - 1];
                }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        _iconProviders.forEach(function (provider) {
            var icon = provider(data);
            if (icon) {
                $link.prepend($(icon));
            }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        _iconProviders.forEach(function (provider) {
            var icon = provider(data);
            if (icon) {
                $element.prepend($(icon));
            }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            files.forEach(function (file) {
                var $listItem = self._findListItemFromFile(file);
                if ($listItem) {
                    $listItem.remove();
                }

• Read up
  Read up