Issue 74660 - orientation of category axes swaps between 2D and 3D for bar charts
Summary: orientation of category axes swaps between 2D and 3D for bar charts
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: http://specs.openoffice.org/chart/wiz...
Keywords:
Depends on: 24614
Blocks: 72764
  Show dependency tree
 
Reported: 2007-02-19 12:50 UTC by matthias.mueller-prove
Modified: 2013-02-24 21:20 UTC (History)
6 users (show)

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


Attachments
patch_080814 (1.26 KB, text/plain)
2008-08-14 08:06 UTC, weiz
no flags Details
patch_080814 (1.26 KB, text/plain)
2008-08-14 08:06 UTC, weiz
no flags Details
example doc showing the import problem (37.15 KB, application/vnd.oasis.opendocument.spreadsheet)
2008-08-22 10:53 UTC, IngridvdM
no flags Details
patch_080814 (7.44 KB, patch)
2008-08-29 04:48 UTC, weiz
no flags Details | Diff
example created with OOo 2.3 (25.63 KB, application/vnd.oasis.opendocument.spreadsheet)
2008-08-29 13:14 UTC, IngridvdM
no flags Details
example created with OOo 2.4 (26.05 KB, application/vnd.oasis.opendocument.spreadsheet)
2008-08-29 13:14 UTC, IngridvdM
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description matthias.mueller-prove 2007-02-19 12:50:34 UTC
A 2D bar chart displays the y axis from top to bottom (like the data in the
spreadsheet.)
When this chart is changed to 3D Look the y axis is swapped from bottom to top.
This is a surprise for the user.

a) the orientation should stay the same between 2D and 3D.
b) the axes orientation should be configured
c) Chart1 charts should migrate properly

// needs to be spec'ed as well.
Comment 1 matthias.mueller-prove 2007-02-19 12:51:24 UTC
target 2.x
Comment 2 IngridvdM 2007-02-23 19:40:37 UTC
accepted, but I would rate the current behaviour as defect.
Comment 3 IngridvdM 2007-09-11 16:19:03 UTC
changing target
Comment 4 IngridvdM 2008-01-16 13:48:59 UTC
changed target due to limited resources
Comment 5 IngridvdM 2008-06-04 12:54:29 UTC
changed target due to limited resources
Comment 6 IngridvdM 2008-07-22 14:21:21 UTC
reset to new
Comment 7 weiz 2008-08-14 08:05:08 UTC
@iha: The patch is finished, I didn't change the template code, but disable 
some codes at DiagramHelper.cxx, as the scale orientation of 2d bar chart is 
changed to "AxisOrientation_REVERSE" at function "DiagramHelper::setVertical", 
I think it's unnecessary. 
Please let me know your suggestions. Thank you!
Comment 8 weiz 2008-08-14 08:06:36 UTC
Created attachment 55767 [details]
patch_080814
Comment 9 weiz 2008-08-14 08:06:42 UTC
Created attachment 55768 [details]
patch_080814
Comment 10 IngridvdM 2008-08-22 10:52:01 UTC
@weiz: Removing the code in DiagramHelper::setVertical is correct but not
sufficient. There is a problem with old files now which are created with
OpenOffice versions that did not have the feature of reverse axis direction -
for example with OpenOffice 2.2 or older. I'll attach an example file.
Comment 11 IngridvdM 2008-08-22 10:53:33 UTC
Created attachment 55942 [details]
example doc showing the import problem
Comment 12 IngridvdM 2008-08-22 10:56:42 UTC
Problem with the patch: Load the attached doc with a version containing the
attached patch and press F9 -> Within the upper right chart the direction of the
vertical axis switches.
Comment 13 weiz 2008-08-29 04:47:32 UTC
@iha: the final attachment is the patch for solve the error from the old 
versions. Please let me know your suggestions, Thanks!
Comment 14 weiz 2008-08-29 04:48:33 UTC
Created attachment 56078 [details]
patch_080814
Comment 15 IngridvdM 2008-08-29 13:12:22 UTC
@weiz, thanks for the patch. In principle that is the right way to fix the
import of old files. But there were still two problems:
1) Categories are not shown anymore at the axes. That is because you created a
new ScaleData and set it at the axes instead of using the existing ScaleData,
thus the axis type switched from category to value. I corrected that before I
checked in the changes.
2) Files created with OpenOffice.org 2.3 were loaded wrong, as the feature to
reverse axes was introduced in version OpenOffice.org 2.4. I added an additional
method isDocumentGeneratedWithOpenOfficeOlderThan2_4() to SchXMLTools to detect
the files correctly that need a conversion.

Checked all in to CWS calcchartrtl01.
Comment 16 IngridvdM 2008-08-29 13:14:08 UTC
Created attachment 56091 [details]
example created with OOo 2.3
Comment 17 IngridvdM 2008-08-29 13:14:58 UTC
Created attachment 56092 [details]
example created with OOo 2.4
Comment 18 IngridvdM 2008-11-20 11:20:05 UTC
I migrated the fixes to CWS rtlchart02.
Comment 19 IngridvdM 2008-11-20 11:24:31 UTC
@Thomas, please verify in CWS rtlchart02.
The vertical axes in bar charts now points from bottom to top per default when a
new chart is created. The direction does not switch anymore when switching
between 2D and 3D.
Please also ensure that the direction of the axes does not change when loading
the attached example documents.
Comment 20 kla 2008-12-11 12:10:17 UTC
seen ok in cws rtlchart02 -> verified
Comment 21 vladimir_hitekschool 2009-04-01 01:20:41 UTC
Issue 74660 has been fixed in master version OOo-dev 3.1 .0 (OOO310m7 
Build:9393) for Windows XP and can be closed.
Comment 22 oleghitekschool 2009-04-01 05:36:01 UTC
Checked and closed by HitekSchool Group 1 as a part of Issue Hunting Party 
regression check.