Out-of-bounds index error patch

Share RGraph:   Share or Like RGraph on Facebook!

« Back to message list

Enter your email address to get email updates on this topic. You can stop receiving updates by clicking the link in the update email messages.

Posted by Jan Dirk on 31st March 2017
Hi,

I am using mobx (see mobx.js.org) together with RGraph.
Because of mobx, I noticed that RGraph.common.line.js does have an issue with accessing an array with an out of bound index at line 745.

After changing the file with the patch

743c743
< for (var i=0; i<this.coords[index].length; ++i,++this.tooltipsSequentialIndex) {
---
> for (var i=0; i<this.coords[index].length && this.tooltipsSequentialIndex < prop.tooltips.length; ++i,++this.tooltipsSequentialIndex) {

this issue got solved.


Regards,

Jan Dirk
Posted by Richard on 1st April 2017
Hi there,

Thanks for that - I've added that in to the Line chart.


Richard

Add a reply




« Back to message list