adobe/brackets

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

Open

    function _markTags(cm, node) {
        node.children.forEach(function (childNode) {
            if (childNode.isElement()) {
                _markTags(cm, childNode);
            }

• Read up
  Read up

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

Open

            if ( val == null ) {
                val = "";

            } else if ( typeof val === "number" ) {
                val += "";

• Read up
  Read up

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

Open

            if ( val == null ) {
                val = "";
            } else if ( typeof val === "number" ) {
                val += "";
            } else if ( jQuery.isArray( val ) ) {

• Read up
  Read up

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

Open

        if ( one === 1 ) {
            origFn = fn;
            fn = function( event ) {
                // Can use an empty set, since event contains the info
                jQuery().off( event );

• Read up
  Read up

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

Open

        var prev = (this._canAdjustMonth(inst, -1, drawYear, drawMonth) ?
            '<a class="ui-datepicker-prev ui-corner-all" data-handler="prev" data-event="click"' +
            ' title="' + prevText + '"><span class="ui-icon ui-icon-circle-triangle-' + ( isRTL ? 'e' : 'w') + '">' + prevText + '</span></a>' :
            (hideIfNoPrevNext ? '' : '<a class="ui-datepicker-prev ui-corner-all ui-state-disabled" title="'+ prevText +'"><span class="ui-icon ui-icon-circle-triangle-' + ( isRTL ? 'e' : 'w') + '">' + prevText + '</span></a>'));

• Read up
  Read up

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

Open

            it("should change nothing when cursor at end", function () {  // (bug #1314)
                testEditor.setCursorPos({ line: 5, ch: 8 });    // cursor between end of existing attribute name ("id") and =
                selectHint(HTMLCodeHints.attrHintProvider, "id");
                expect(testDocument.getLine(5)).toBe("  <h1 id='foo'>Heading</h1>");
                expectCursorAt({ line: 5, ch: 8 });            // cursor stays at end of attr name

• Read up
  Read up

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

Open

            it("should insert a new attribute before the existing one that starts at cursor", function () {  // (bug #1312)
                testEditor.setCursorPos({ line: 5, ch: 6 });    // cursor between space and start of existing attribute name ("id")
                selectHint(HTMLCodeHints.attrHintProvider, "class");
                expect(testDocument.getLine(5)).toBe("  <h1 class=\"\"id='foo'>Heading</h1>");
                expectCursorAt({ line: 5, ch: 13 });            // cursor between two double-quotes

• Read up
  Read up

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

Open

        if ( one === 1 ) {
            origFn = fn;
            fn = function( event ) {
                // Can use an empty set, since event contains the info
                jQuery().off( event );

• Read up
  Read up

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

Open

        var next = (this._canAdjustMonth(inst, +1, drawYear, drawMonth) ?
            '<a class="ui-datepicker-next ui-corner-all" data-handler="next" data-event="click"' +
            ' title="' + nextText + '"><span class="ui-icon ui-icon-circle-triangle-' + ( isRTL ? 'w' : 'e') + '">' + nextText + '</span></a>' :
            (hideIfNoPrevNext ? '' : '<a class="ui-datepicker-next ui-corner-all ui-state-disabled" title="'+ nextText + '"><span class="ui-icon ui-icon-circle-triangle-' + ( isRTL ? 'w' : 'e') + '">' + nextText + '</span></a>'));

• Read up
  Read up

Function _create has 76 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    _create: function() {
        var accordionId = this.accordionId = "ui-accordion-" +
                (this.element.attr( "id" ) || ++uid),
            options = this.options;

Function parseFloat has 76 lines of code (exceeds 25 allowed). Consider refactoring.

Open

Globalize.parseFloat = function( value, radix, cultureSelector ) {
    // radix argument is optional
    if ( typeof radix !== "number" ) {
        cultureSelector = radix;
        radix = 10;

File ExtensionManagerViewModel.js has 292 lines of code (exceeds 250 allowed). Consider refactoring.

Open

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

Open

    DOMEditHandler.prototype.apply = function (edits) {
        var targetID,
            targetElement,
            childElement,
            self = this;

Function doJumpToDef has 75 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        JSJumpToDefProvider.prototype.doJumpToDef = function () {
            var offset,
                handleJumpResponse;

Function Datepicker has 75 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function Datepicker() {
    this.debug = false; // Change this to true to start debugging
    this._curInst = null; // The current instance in use
    this._keyEvent = false; // If the last event was a key event
    this._disabledInputs = []; // List of date picker inputs that have been disabled

Consider simplifying this complex logical expression.

Open

if ( "getBoundingClientRect" in document.documentElement ) {
    getOffset = function( elem, doc, docElem, box ) {
        try {
            box = elem.getBoundingClientRect();
        } catch(e) {}

Function _parseSelector has 75 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        function _parseSelector(start, level) {

            currentSelector = "";
            selectorStartChar = start;
            selectorStartLine = line;

Function Sizzle has 75 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function Sizzle( selector, context, results, seed ) {
    var match, elem, m, nodeType,
        // QSA vars
        i, groups, old, nid, newContext, newSelector;

Function getContextInfo has 75 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function getContextInfo(editor, constPos, requireParent, requireNextToken) {
        var pos, ctx, ctxPrev, ctxNext, offset, keyName, valueName, parentKeyName,
            isArray, exclusionList, hasColon, hasComma, hasBracket, shouldReplace;

        pos = $.extend({}, constPos);

Function _keydownHook has 75 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    InlineMenu.prototype._keydownHook = function (event) {
        var keyCode,
            self = this;

        // positive distance rotates down; negative distance rotates up