Closed (fixed)
Project:
Open Flash Chart API
Version:
5.x-2.2
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
24 Nov 2007 at 20:48 UTC
Updated:
10 Dec 2007 at 05:21 UTC
Jump to comment: Most recent file
There's a bug in the API which prevents multiple series from working. I don't know how to officially patch things, but this seems to be working for me:
in open_flash_chart_api.module, replace line 106 with these two:
original:
$tmp .= $this->format_output($output_type,'values_'. $num, $data);
fixed:
$valueNum = $num+1; // fix naming convention to allow multiple series
$tmp .= $this->format_output($output_type,'values_'. $valueNum, $data);
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | multi_data_line.patch | 756 bytes | redndahead |
Comments
Comment #1
sprugman commentedOops. I meant line 906.
Comment #2
redndahead commentedCan you explain what multiple series means? Are you saying two sets of data on the same graph or two graphs? Can you give me some example code? I expect that multiple graphs may have issues and I have filed a bug to consider how to deal with it. But on line 909 the $num++; should take care of doing the incrementing.
I'll try some testing to see what I can and can not get to work.
Comment #3
redndahead commentedI found the bug and attached is the patch file. Only 2 lines of code need to be changed. It was a misread of the OFC spec.
Comment #4
redndahead commentedCommitted code.
Comment #5
(not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.