Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | Rhombus are reflected | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Draw | Reporter: | efa <efa> | ||||||
Component: | open-import | Assignee: | AOO issues mailing list <issues> | ||||||
Status: | CLOSED FIXED | QA Contact: | Prachi <prachi27jain> | ||||||
Severity: | Major | ||||||||
Priority: | P3 | CC: | Armin.Le.Grand, fanyuzhen, issues, jsc, rb.henschel | ||||||
Version: | 4.0.0 | Keywords: | regression | ||||||
Target Milestone: | 4.0.1 | Flags: | jsc:
4.0.1_release_blocker+
|
||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Issue Type: | DEFECT | Latest Confirmation in: | --- | ||||||
Developer Difficulty: | --- | ||||||||
Attachments: |
|
Description
efa
2013-09-04 13:49:59 UTC
Created attachment 81449 [details]
.odg file correctly seen with OO3.x
Created attachment 81450 [details]
export of OO3.x rendering of the same file
It is an import problem. You see the same error with a file created in AOO4.0. Import is OK in r1453677, import fails in r1466899. ALG: Indeed, can reproduce. Simple way: - Start AOO400 - draw a rectangle (CustomShape, not regular) - shear (e.g. 60) - save, reload -> shear is at -60, object is 'flipped', keeping it's upper edge but shearing to the opposite side. - loads correctly in AOO341 ALG: Thanks, Regina! Looks as if we looked at the same time ;-) Definitely has to do with TRGetBaseGeometry/TRSetBaseGeometry and how it handles shear; in SdrTextObj it is *not* converted/mirrored at all, in SdrObjCustomShape it is mirrored in TRSetBaseGeometry (this seems to be the error, #121932# is involved). To make sure I will have to check what the API and the ODF im/export uses... ALG: Adding regression keyword ALG: CHecked and also made sure that in AOO341 the mirroring of fShearX in SdrObjCustomShape::TRSetBaseGeometry was not there. The fix for #121932# here was wrong, the trunk version does not correct the mirrored shear values, neither at the object level, nor on the API or XML level. Taking back the mirroring of the shear angle (maybe I was at aw080 in my thoughts here ;-( ) ALG: Comitted fix for trunk, set regression flag for AOO401, done for AOO410 ALG:@efa: Many thanks for finding and reporting this! "alg" committed SVN revision 1520057 into trunk: i123181 Corrected mirroring of shear angle in SdrObjCustomShape::TRSetBaseGeo... approve showstopper request ALG: Checked with AOO401 branch and comitted, done. "alg" committed SVN revision 1521006 into branches/AOO401: i123181 Corrected mirroring of shear angle in SdrObjCustomShape::TRSetBaseGeo... |