Ticket #8108 (reopened defect)

Opened 7 weeks ago

Last modified 7 weeks ago

straight line graphs with no axes don't show up

Reported by: nazanin Owned by: elazutkin
Priority: normal Milestone: 1.3
Component: Charting Version: 1.2.1
Severity: normal Keywords: straight line graph
Cc: nazanin@…

Description

1. Create a graph with no axes with these values: (0,10), (1,10),(2,10),(3,10),..
Result: the graph doesn't show up since the max y value minus min y value becomes zero and scale which is span/(max - min) will become NaN.

Fixed it with this patch:

--- a/public/javascripts/dojotoolkit/dojox/charting/scaler/primitive.js
+++ b/public/javascripts/dojotoolkit/dojox/charting/scaler/primitive.js
@@ -2,13 +2,17 @@ dojo.provide("dojox.charting.scaler.primitive");
 
 dojox.charting.scaler.primitive = {
        buildScaler: function(/*Number*/ min, /*Number*/ max, /*Number*/ span, /*Object*/ kwArgs){
+         if (min == max)
+           scale = min;
+         else
+           scale = span/(max - min);
                return {
                        bounds: {
                                lower: min,
                                upper: max,
                                from:  min,
                                to:    max,
-                               scale: span / (max - min),
+                               scale: scale,
                                span:  span
                        },
                        scaler: dojox.charting.scaler.primitive

Change History

Changed 7 weeks ago by elazutkin

  • status changed from new to closed
  • resolution set to invalid

Use Dojo 1.2.2 or the trunk. Set min/max parameters on the axis.

Changed 7 weeks ago by nazanin

  • status changed from closed to reopened
  • resolution deleted

well that's the point. This happens when you graph with no axis''' in dojo/dojox/charting/plot2d/Base.js _calc function decides whether the graph has axis or not. So this problems doesn't occur when there are axis.

Changed 7 weeks ago by elazutkin

  • milestone changed from tbd to 1.3

Yep, missed the point. Thank you for reopening.

Note: See TracTickets for help on using tickets.