adobe/brackets

Function getTagInfo has 102 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function getTagInfo(editor, constPos, isHtmlMode) {
        // We're going to be changing pos a lot, but we don't want to mess up
        // the pos the caller passed in so we use extend to make a safe copy of it.
        var pos = $.extend({}, constPos),
            ctx = TokenUtils.getInitialContext(editor._codeMirror, pos),

File HTMLUtils.js has 310 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 _extractAttrVal has 35 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function _extractAttrVal(ctx) {
        var attrValue = ctx.token.string,
            startChar = attrValue.charAt(0),
            endChar = attrValue.charAt(attrValue.length - 1),
            offset = TokenUtils.offsetInToken(ctx),

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

Open

    function getTagAttributes(editor, pos) {
        var attrs       = [],
            backwardCtx = TokenUtils.getInitialContext(editor._codeMirror, pos),
            forwardCtx  = $.extend({}, backwardCtx);

Function findBlocks has 32 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function findBlocks(editor, modeName) {
        // Start scanning from beginning of file
        var ctx = TokenUtils.getInitialContext(editor._codeMirror, {line: 0, ch: 0}),
            blocks = [],
            currentBlock = null,

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

Open

    function _getTagInfoStartingFromAttrValue(ctx) {
        // Assume we in the attr value
        // and validate that by going backwards
        var attrInfo = _extractAttrVal(ctx),
            attrVal = attrInfo.val,

Function createTagInfo has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

    function createTagInfo(tokenType, offset, tagName, attrName, attrValue, valueAssigned, quoteChar, hasEndQuote) {

Avoid deeply nested control flow statements.

Open

                    } else if (forwardCtx.token.type === "error") {
                        if (forwardCtx.token.string.indexOf("<") === 0 || forwardCtx.token.string.indexOf(">") === 0) {
                            break;
                        }
                        // If we type the first letter of the next attribute, it comes as an error

Avoid deeply nested control flow statements.

Open

                        if (offset === 0) {
                            TokenUtils.moveNextToken(ctx);
                        }

Avoid deeply nested control flow statements.

Open

                        if (forwardCtx.token.string.indexOf("<") === 0) {
                            break;
                        }

Avoid deeply nested control flow statements.

Open

                        if (ctx.token.string === "</") {
                            tokenType = CLOSING_TAG;
                            offset -= 2;
                        } else {
                            offset = 0;

Consider simplifying this complex logical expression.

Open

            if (isPriorAttr &&
                    (!ctx.token.type ||
                    (ctx.token.type && ctx.token.type !== "attribute" &&
                        ctx.token.type.indexOf("error") === -1 &&
                        ctx.token.string.indexOf("<") !== -1))) {

Avoid too many return statements within this function.

Open

                return createTagInfo();

Avoid too many return statements within this function.

Open

                    return createTagInfo();

Avoid too many return statements within this function.

Open

                    return createTagInfo();

Avoid too many return statements within this function.

Open

                return createTagInfo(tokenType, offset, _extractTagName(ctx));

Avoid too many return statements within this function.

Open

                return createTagInfo();

Avoid too many return statements within this function.

Open

                    return createTagInfo();

Avoid too many return statements within this function.

Open

        return createTagInfo(ATTR_NAME, offset, tagName, attrName, attrVal, true, quoteChar, hasEndQuote);

Avoid too many return statements within this function.

Open

                        return createTagInfo(ATTR_NAME, 0, tagInfo.tagName);

Avoid too many return statements within this function.

Open

                    return createTagInfo();

Avoid too many return statements within this function.

Open

                        return createTagInfo();

Avoid too many return statements within this function.

Open

        return tagInfo;

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

Open

            var testPos = {ch: ctx.pos.ch + 1, line: ctx.pos.line},
                testToken = editor._codeMirror.getTokenAt(testPos, true);

• Read up
  Read up

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

Open

                            (tagPrefixedRegExp.test(ctx.token.type) &&
                            (ctx.token.string === ">" || ctx.token.string === "/>" ||
                                ctx.token.string === "</"))) {

• Read up
  Read up