adobe/brackets

File MainViewManager.js has 884 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 _loadViewState has 77 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function _loadViewState(e) {
        // file root is appended for each project
        var panes,
            promises = [],
            context = { location : { scope: "user",

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

Open

    function _open(paneId, file, optionsIn) {
        var result = new $.Deferred(),
            options = optionsIn || {};

        function doPostOpenActivation() {

Function _saveViewState has 34 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function _saveViewState() {
        function _computeSplitPercentage() {
            var available,
                used;

Function _initialLayout has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function _initialLayout(forceRefresh) {
        var panes = Object.keys(_panes),
            size = 100 / panes.length;

        _.forEach(_panes, function (pane) {

Function _mergePanes has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function _mergePanes() {
        if (_panes.hasOwnProperty(SECOND_PANE)) {

            var firstPane = _panes[FIRST_PANE],
                secondPane = _panes[SECOND_PANE],

Function _createPaneIfNecessary has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function _createPaneIfNecessary(paneId) {
        var newPane;

        if (!_panes.hasOwnProperty(paneId)) {
            newPane = new Pane(paneId, _$el);

Avoid deeply nested control flow statements.

Open

                        if (pane.id !== _activePaneId) {
                            // activate the pane which will put focus in the pane's doc
                            setActivePaneId(pane.id);
                            // reset the focus to the inline editor
                            current.focus();

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

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

Open

        uniqueFileList.forEach(function (file) {
            if (_findFileInMRUList(pane.id, file) !== -1) {
                console.log(file.fullPath + " duplicated in mru list");
            }
            _mruList.push(_makeMRUListEntry(file, pane.id));

• Read up
  Read up

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

Open

                    fileList.forEach(function (file) {
                        if (_findFileInMRUList(pane.id, file) !== -1) {
                            console.log(file.fullPath + " duplicated in mru list");
                        }
                        _mruList.push(_makeMRUListEntry(file, pane.id));

• Read up
  Read up

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

Open

    function _swapWorkingSetListIndexes(paneId, index1, index2) {
        var pane = _getPane(paneId);

        pane.swapViewListIndexes(index1, index2);
        exports.trigger("workingSetSort", pane.id);

• Read up
  Read up

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

Open

    function _moveWorkingSetItem(paneId, fromIndex, toIndex) {
        var pane = _getPane(paneId);

        pane.moveWorkingSetItem(fromIndex, toIndex);
        exports.trigger("workingSetSort", pane.id);

• Read up
  Read up

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

Open

        do {
            index = _.findIndex(_mruList, compare);
            if (index !== -1) {
                _mruList.splice(index, 1);
            }

• Read up
  Read up

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

Open

        do {
            index = _.findIndex(_mruList, compare);
            if (index !== -1) {
                _mruList.splice(index, 1);
            }

• Read up
  Read up

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

Open

        return _.findIndex(_mruList, function (record) {
            return (record.file.fullPath === file.fullPath && record.paneId === paneId);
        });

• Read up
  Read up

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

Open

        var index = _.findIndex(_mruList, function (record) {
            return (record.file.fullPath === file.fullPath && record.paneId !== paneId);
        });

• Read up
  Read up