backlogs/redmine_backlogs

File jqplot.canvasOverlay.js has 632 lines of code (exceeds 250 allowed). Consider refactoring.

Open

/**
 * jqPlot
 * Pure JavaScript plotting plugin using jQuery
 *
 * Version: 1.0.0

Function draw has 224 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    $.jqplot.CanvasOverlay.prototype.draw = function(plot) {
        var obj, 
            objs = this.objects,
            mr = this.markerRenderer,
            start,

Function handleMove has 89 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function handleMove(ev, gridpos, datapos, neighbor, plot) {
        var co = plot.plugins.canvasOverlay;
        var objs = co.objects;
        var l = objs.length;
        var obj, haveHighlight=false;

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

Open

    function showTooltip(plot, obj, gridpos, datapos) {
        var co = plot.plugins.canvasOverlay;
        var elem = co._tooltipElem;

        var opts = obj.options, x, y;

Function CanvasOverlay has 40 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    $.jqplot.CanvasOverlay = function(opts){
        var options = opts || {};
        this.options = {
            show: $.jqplot.config.enablePlugins,
            deferDraw: false

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

Open

    function LineBase() {
        this.uid = null;
        this.type = null;
        this.gridStart = null;
        this.gridStop = null;

Avoid deeply nested control flow statements.

Open

                            if (obj.options.y != null) {
                                mr.style = 'line';
                                opts.closePath = false;
                                var xaxis = plot.axes[obj.options.xaxis],
                                    xstart,

Avoid deeply nested control flow statements.

Open

                            if (obj.options.y != null) {
                                mr.style = 'line';
                                opts.closePath = false;
                                var xaxis = plot.axes[obj.options.xaxis],
                                    xstart,

Avoid deeply nested control flow statements.

Open

                            if (obj.options.x != null) {
                                mr.style = 'line';
                                opts.closePath = false;
                                var yaxis = plot.axes[obj.options.yaxis],
                                    ystart,

Avoid deeply nested control flow statements.

Open

                    if (obj.fadeTooltip) {
                        elem.fadeOut(obj.tooltipFadeSpeed);
                    }
                    else {
                        elem.hide();

Avoid deeply nested control flow statements.

Open

                else if (n) {
                    switch (obj.type) {
                        case 'line':
                            showTooltip(plot, obj, [gridpos.x, gridpos.y], datapos);
                            break;

Avoid deeply nested control flow statements.

Open

                    switch (obj.type) {
                        case 'line':
                            showTooltip(plot, obj, [gridpos.x, gridpos.y], datapos);
                            break;

Avoid deeply nested control flow statements.

Open

                            if (obj.options.x != null) {
                                mr.style = 'line';
                                opts.closePath = false;
                                var yaxis = plot.axes[obj.options.yaxis],
                                    ystart,

Avoid deeply nested control flow statements.

Open

                            for (var i=0; i<dashPat.length; i++) {
                                dashPatLen += dashPat[i];
                            }

Avoid deeply nested control flow statements.

Open

                            for (var i=0; i<dashPat.length; i++) {
                                dashPatLen += dashPat[i];
                            }

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

Open

    function handleMove(ev, gridpos, datapos, neighbor, plot) {

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

Open

                        case 'horizontalLine':
                            
                            // style and shadow properties should be set before
                            // every draw of marker renderer.
                            if (obj.options.y != null) {

• Read up
  Read up

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

Open

                        case 'verticalLine':
                            
                            // style and shadow properties should be set before
                            // every draw of marker renderer.
                            if (obj.options.x != null) {

• Read up
  Read up

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

Open

                    switch (obj.type) {
                        case 'line':
                            showTooltip(plot, obj, [gridpos.x, gridpos.y], datapos);
                            break;

• Read up
  Read up

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

Open

                    switch (obj.type) {
                        case 'line':
                            showTooltip(plot, obj, [gridpos.x, gridpos.y], datapos);
                            break;

• Read up
  Read up

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

Open

    $.jqplot.CanvasOverlay.prototype.addLine = function(opts) {
        var line = new Line(opts);
        line.uid = objCounter++;
        this.objects.push(line);
        this.objectNames.push(line.options.name);

• Read up
  Read up

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

Open

    $.jqplot.CanvasOverlay.prototype.addDashedHorizontalLine = function(opts) {
        var line = new DashedHorizontalLine(opts);
        line.uid = objCounter++;
        this.objects.push(line);
        this.objectNames.push(line.options.name);

• Read up
  Read up

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

Open

    $.jqplot.CanvasOverlay.prototype.addDashedVerticalLine = function(opts) {
        var line = new DashedVerticalLine(opts);
        line.uid = objCounter++;
        this.objects.push(line);
        this.objectNames.push(line.options.name);

• Read up
  Read up

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

Open

    $.jqplot.CanvasOverlay.prototype.addHorizontalLine = function(opts) {
        var line = new HorizontalLine(opts);
        line.uid = objCounter++;
        this.objects.push(line);
        this.objectNames.push(line.options.name);

• Read up
  Read up

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

Open

    $.jqplot.CanvasOverlay.prototype.addVerticalLine = function(opts) {
        var line = new VerticalLine(opts);
        line.uid = objCounter++;
        this.objects.push(line);
        this.objectNames.push(line.options.name);

• Read up
  Read up

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

Open

                            start = [plot.axes[obj.options.xaxis].series_u2p(obj.options.start[0]), plot.axes[obj.options.yaxis].series_u2p(obj.options.start[1])];

• Read up
  Read up

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

Open

                            stop = [plot.axes[obj.options.xaxis].series_u2p(obj.options.stop[0]), plot.axes[obj.options.yaxis].series_u2p(obj.options.stop[1])];

• Read up
  Read up

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

Open

    $.jqplot.CanvasOverlay.postPlotInit = function (target, data, opts) {
        var options = opts || {};
        // add a canvasOverlay attribute to the plot
        this.plugins.canvasOverlay = new $.jqplot.CanvasOverlay(options.canvasOverlay);     
    };

• Read up
  Read up

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

Open

        var opts = {
            y: null,
            xmin: null,
            xmax: null,
            xOffset: '6px', // number or string.  Number interpreted as units, string as pixels.

• Read up
  Read up

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

Open

        var opts = {
            x: null,
            ymin: null,
            ymax: null,
            yOffset: '6px', // number or string.  Number interpreted as units, string as pixels.

• Read up
  Read up

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

Open

        if (opts.fadeTooltip) {
            // Fix for stacked up animations.  Thnanks Trevor!
            elem.stop(true,true).fadeIn(opts.tooltipFadeSpeed);
        }
        else {

• Read up
  Read up