Issue 91212 - XTableChart::getRanges() returns wrong range
Summary: XTableChart::getRanges() returns wrong range
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: chart (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: kla
QA Contact: issues@graphics
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2008-06-30 13:37 UTC by IngridvdM
Modified: 2013-02-24 21:20 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
doc with example macro (14.14 KB, application/vnd.oasis.opendocument.spreadsheet)
2008-06-30 13:39 UTC, IngridvdM 		no flags Details
Patch fixing the problem (1.57 KB, patch)
2008-06-30 14:06 UTC, IngridvdM 		no flags Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description IngridvdM 2008-06-30 13:37:02 UTC 
The API method XTableChart::getRanges() does not ensure that the chart is loaded
and therefor fails sometimes. To reproduce load the attached example document
and press on the 'GetRanges' Button. A message box shows an empty range, what is
wrong.
Double click on the chart and press the Button afterwards will show a range, as
the chart is loaded now.
But it should not be necessary to double click on the chart to get a range via API.
Comment 1 IngridvdM 2008-06-30 13:39:06 UTC 
Created attachment 54839 [details]
doc with example macro
Comment 2 IngridvdM 2008-06-30 13:59:38 UTC 
As a workaround one can call
ChartModel = Chart.EmbeddedObject
just before Chart.getRanges() as this triggers the loading of the chart object.
Comment 3 IngridvdM 2008-06-30 14:06:20 UTC 
Created attachment 54840 [details]
Patch fixing the problem
Comment 4 IngridvdM 2008-07-09 17:08:17 UTC 
In addition the returned range was wrong when labels for columns and rows were
selected.
Both problems are fixed now in CWS chart29.
Comment 5 IngridvdM 2008-07-10 09:11:42 UTC 
@Thomas, please verify in CWS chart29. To check, load the attached bugdoc and
make sure that the macro security allows to run macros. After opening the
document click on the button 'GetRanges'. A message box should appear with
content 'Chart ranges: Columns:2-5  Rows:9-14.
Comment 6 kla 2008-07-14 12:06:48 UTC 
TO adobt a autotest reassign to JSI.
Comment 7 jogi 2008-07-14 15:06:14 UTC 
work started
Comment 8 jogi 2008-07-14 15:08:15 UTC 
changes to ch2_flexible_source_range_selection.inc where instead of three ranges
now one range is being displayed.
Comment 9 jogi 2008-07-14 15:32:08 UTC 
scripts running without any error or warning on Solaris
Comment 10 jogi 2008-07-14 15:32:54 UTC 
@kla
Comment 11 kla 2008-07-16 14:46:45 UTC 
Seen ok in cws chart29 -> verified
Comment 12 kla 2008-08-29 10:35:59 UTC 
Seen ok in current master -> closed