Sign upLogin

adobe/brackets

View on GitHub
Star
src/extensions/default/JavaScriptRefactoring/ExtractToVariable.js

Summary

Maintainability
C
1 day
Test Coverage

Function handleExtractToVariable has 84 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    function handleExtractToVariable() {
        var editor = EditorManager.getActiveEditor();

        if (editor.getSelections().length > 1) {
            editor.displayErrorMessageAtCursor(Strings.ERROR_EXTRACTTO_VARIABLE_MULTICURSORS);
Severity: Major
Found in src/extensions/default/JavaScriptRefactoring/ExtractToVariable.js - About 3 hrs to fix

    Function extract has 46 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

        function extract(scopes, parentStatement, expns, text, insertPosition) {
        var varType          = "var",
            varName          = RefactoringUtils.getUniqueIdentifierName(scopes, "extracted"),
            varDeclaration   = varType + " " + varName + " = " + text + ";\n",
            parentStatementStartPos = session.editor.posFromIndex(parentStatement.start),
    Severity: Minor
    Found in src/extensions/default/JavaScriptRefactoring/ExtractToVariable.js - About 1 hr to fix

      Function extractToVariable has 35 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

          function extractToVariable(ast, start, end, text, scopes) {
        var doc                   = session.editor.document,
            editor = EditorManager.getActiveEditor(),
            parentExpn            = RefactoringUtils.getExpression(ast, start, end, doc.getText()),
            expns                 = [],
      Severity: Minor
      Found in src/extensions/default/JavaScriptRefactoring/ExtractToVariable.js - About 1 hr to fix

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

            function getExpressions(ast, start, end) {
        var expns = [],
            s     = start,
            e     = end,
            expn;
        Severity: Minor
        Found in src/extensions/default/JavaScriptRefactoring/ExtractToVariable.js - About 1 hr to fix

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

              function extractToVariable(ast, start, end, text, scopes) {
          Severity: Minor
          Found in src/extensions/default/JavaScriptRefactoring/ExtractToVariable.js - About 35 mins to fix

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

                function extract(scopes, parentStatement, expns, text, insertPosition) {
            Severity: Minor
            Found in src/extensions/default/JavaScriptRefactoring/ExtractToVariable.js - About 35 mins to fix

              There are no issues that match your filters.

              Category
              Status