adobe/brackets

File QuickOpen.js has 496 lines of code (exceeds 250 allowed). Consider refactoring.

Open

/*
 * Copyright (c) 2012 - 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 showDialog has 38 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    QuickNavigateDialog.prototype.showDialog = function (prefix, initialString) {
        if (this.isOpen) {
            return;
        }
        this.isOpen = true;

Function _filterCallback has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    QuickNavigateDialog.prototype._filterCallback = function (query) {
        // Re-evaluate which plugin is active each time query string changes
        currentPlugin = null;

        // "Go to line" mode is special-cased

Function QuickOpenPlugin has 10 arguments (exceeds 4 allowed). Consider refactoring.

Open

    function QuickOpenPlugin(name, languageIds, done, search, match, itemFocus, itemSelect, resultsFormatter, matcherOptions, label) {

Function highlightMatch has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function highlightMatch(item, matchClass, rangeFilter) {
        var label = item.label || item;
        matchClass = matchClass || "quicksearch-namematch";

        var stringRanges = item.stringRanges;

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

Open

    QuickNavigateDialog.prototype._handleItemSelect = function (selectedItem, query) {

        var doClose = true,
            self = this;

Consider simplifying this complex logical expression.

Open

        if (query.length <= 1 || !regInfo ||
                (regInfo[1] && isNaN(regInfo[1])) ||
                (regInfo[3] && isNaN(regInfo[3]))) {

            return null;

Avoid too many return statements within this function.

Open

        return searchFileList(query, this._filenameMatcher);

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

Open

        DocumentManager.getDocumentForPath(newFilePath)
            .done(function (newDoc) {
                newDoc.on("languageChanged.quickFindDefinition", function () {
                    var changedLanguageId = LanguageManager.getLanguageForPath(newDoc.file.fullPath).getId();
                    _setMenuItemStateForLanguage(changedLanguageId);

• Read up
  Read up

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

Open

        if (_canHandleTrigger("@", plugins)) {
            CommandManager.get(Commands.NAVIGATE_GOTO_DEFINITION).setEnabled(true);
        } else {
            CommandManager.get(Commands.NAVIGATE_GOTO_DEFINITION).setEnabled(false);
        }

• Read up
  Read up

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

Open

        if (_canHandleTrigger("#", plugins)) {
            CommandManager.get(Commands.NAVIGATE_GOTO_DEFINITION_PROJECT).setEnabled(true);
        } else {
            CommandManager.get(Commands.NAVIGATE_GOTO_DEFINITION_PROJECT).setEnabled(false);
        }

• Read up
  Read up