Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#13313 closed defect (fixed)

plot2d/Grid.js exception (null reference)

Reported by: bill Owned by: Eugene Lazutkin
Priority: high Milestone: 1.7
Component: Charting Version: 1.7.0b1
Keywords: Cc:
Blocked By: Blocking:

Description

On loading demos/mobileCharting/demo.html, in Grid.js, it fails on the line:

dojo.forEach(ticks.major, function(tick){

(and similar lines) because ticks is null. If you skip those exceptions (which show up in firebug), the page still loads.

Attachments (1)

dojox.charting.plot2d.Grid.patch (4.2 KB) - added by cjolif 8 years ago.
patch to avoid Grid exceptions (IBM, CCLA)

Download all attachments as: .zip

Change History (4)

comment:1 Changed 8 years ago by cjolif

I think you get that exception because you are running with Firebug with "break on all errors" as there errors are normally caught in try/catch block in Grid. So this is not strictly a "bug" as the behavior in a regular runtime environment should be ok. However I agree it would be better to manage that differently and will prepare a patch for that.

Changed 8 years ago by cjolif

patch to avoid Grid exceptions (IBM, CCLA)

comment:2 Changed 8 years ago by Chris Mitchell

Resolution: fixed
Status: newclosed

(In [25758]) fixes #13313 plot2d.Grid.js exception. null reference \!strict

comment:3 Changed 8 years ago by Chris Mitchell

Milestone: tbd1.7
Note: See TracTickets for help on using tickets.