backlogs/redmine_backlogs

File jqplot.pyramidRenderer.js has 344 lines of code (exceeds 250 allowed). Consider refactoring.

Open

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

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

Open

    $.jqplot.PyramidRenderer.prototype.draw = function(ctx, gridData, options) {
        var i;
        // Ughhh, have to make a copy of options b/c it may be modified later.
        var opts = $.extend({}, options);
        var shadow = (opts.shadow != undefined) ? opts.shadow : this.shadow;

Function init has 49 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    $.jqplot.PyramidRenderer.prototype.init = function(options, plot) {
        options = options || {};
        this._type = 'pyramid';
        // Group: Properties
        //

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

Open

    $.jqplot.PyramidRenderer.prototype.setGridData = function(plot) {
        // recalculate the grid data
        var xp = this._xaxis.series_u2p;
        var yp = this._yaxis.series_u2p;
        var data = this._plotData;

Function makeGridData has 30 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    $.jqplot.PyramidRenderer.prototype.makeGridData = function(data, plot) {
        // recalculate the grid data
        var xp = this._xaxis.series_u2p;
        var yp = this._yaxis.series_u2p;
        var gd = [];

Avoid deeply nested control flow statements.

Open

                        if (this.useNegativeColors) {
                            opts.fillStyle = negativeColors.next();
                        }
                        else {
                            opts.fillStyle = positiveColors.next();

Avoid deeply nested control flow statements.

Open

                        if (shadow) {
                            this.renderer.shadowRenderer.draw(ctx, points);
                        }

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

    function handleMove(ev, gridpos, datapos, neighbor, plot) {
        if (neighbor) {
            var ins = [neighbor.seriesIndex, neighbor.pointIndex, neighbor.data];
            var evt1 = jQuery.Event('jqplotDataMouseOver');
            evt1.pageX = ev.pageX;

• Read up
  Read up

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

Open

    function postPlotDraw() {
        // Memory Leaks patch    
        if (this.plugins.pyramidRenderer && this.plugins.pyramidRenderer.highlightCanvas) {

            this.plugins.pyramidRenderer.highlightCanvas.resetCanvas();

• Read up
  Read up

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

Open

    function unhighlight (plot) {
        var canvas = plot.plugins.pyramidRenderer.highlightCanvas;
        canvas._ctx.clearRect(0,0, canvas._ctx.canvas.width, canvas._ctx.canvas.height);
        for (var i=0; i<plot.series.length; i++) {
            plot.series[i]._highlightedPoint = null;

• Read up
  Read up

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

Open

        if (shadow_offset == null) {
            // scale the shadowOffset to the width of the line.
            if (this.lineWidth > 2.5) {
                shadow_offset = 1.25 * (1 + (Math.atan((this.lineWidth/2.5))/0.785398163 - 1)*0.6);
                // var shadow_offset = this.shadowOffset;

• Read up
  Read up

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

Open

        if (this._yaxis.name === 'yMidAxis' && this.side === 'right') {
            this._xnudge = this._xaxis.max/2000.0;
            adjust = true;
        }

• Read up
  Read up

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

Open

        if (this._yaxis.name === 'yMidAxis' && this.side === 'right') {
            this._xnudge = this._xaxis.max/2000.0;
            adjust = true;
        }

• Read up
  Read up

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

Open

                if (this._plotData[i][1] < 0) {
                    if (this.varyBarColor && !this._stack) {
                        if (this.useNegativeColors) {
                            opts.fillStyle = negativeColors.next();
                        }

• Read up
  Read up

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

Open

        for (i = 0; i < l; i++) {
            if (data[i][1] < 0) {
                this.side = 'left';
            }
        }

• Read up
  Read up

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

Open

        for (i = 0; i < l; i++) {
            if (data[i][1] < 0) {
                this.side = 'left';
            }
        }

• Read up
  Read up

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

Open

            if (this.fill) {
                this.barWidth = (paxis._offsets.min - paxis._offsets.max) / nvals - this.barPadding + temp;
            }
            else {
                this.barWidth = (paxis._offsets.min - paxis._offsets.max) / nvals;

• Read up
  Read up

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

Open

        if (paxis.name == 'xaxis' || paxis.name == 'x2axis') {
            this.barWidth = (paxis._offsets.max - paxis._offsets.min) / nvals - this.barPadding + temp;
        }
        else {
            if (this.fill) {

• Read up
  Read up