Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | Connectors wrongly connected in .xls | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Calc | Reporter: | oharboe <oyvind.harboe> | ||||||||||
Component: | open-import | Assignee: | AOO issues mailing list <issues> | ||||||||||
Status: | CONFIRMED --- | QA Contact: | |||||||||||
Severity: | Normal | ||||||||||||
Priority: | P3 | CC: | Armin.Le.Grand, awf.aoo, elish, issues, raal, rainerbielefeld_ooo_qa | ||||||||||
Version: | OOO310m9 | Keywords: | crash | ||||||||||
Target Milestone: | --- | ||||||||||||
Hardware: | PC | ||||||||||||
OS: | All | ||||||||||||
See Also: | https://issues.apache.org/ooo/show_bug.cgi?id=76373 | ||||||||||||
Issue Type: | DEFECT | Latest Confirmation in: | 4.1.0-dev | ||||||||||
Developer Difficulty: | --- | ||||||||||||
Attachments: |
|
Description
oharboe
2009-09-30 21:42:18 UTC
Created attachment 65067 [details]
Original file with formatting problems
Created attachment 65068 [details]
Expected result when viewing with Microsoft Excel Viewer 2003
Looking at the expected result it should be trivial to pick something that's "not right" with the formatting... The trick with this bug is probably to break out the bug report into many smaller reports... I can confirm within version DEV300m60 duplicate of issue 76407 ? see text boxes "person som oppdager", "fabriksjef" Attachment 65067 [details] opens OK in Excel 2010.
Crash with
AOO410m1(Build:9750) - Rev. 1552994
Rev.1552994
Win 7
I can reproduce the crash. It is a stack overflow due to this loop: svxcore.dll!SdrObject::SetObjectItemSet(const SfxItemSet & rSet) Line 2021 C++ svxcore.dll!SdrObjCustomShape::AdaptTextMinSize() Line 1763 C++ svxcore.dll!SdrObjCustomShape::NbcSetSnapRect(const Rectangle & rRect) Line 1777 C++ svxcore.dll!SdrObjCustomShape::SetSnapRect(const Rectangle & rRect) Line 1787 C++ svxcore.dll!SdrObjCustomShape::SetVerticalWriting(unsigned char bVertical) Line 2608 C++ svxcore.dll!sdr::properties::TextProperties::ItemChange(const unsigned short nWhich, const SfxPoolItem * pNewItem) Line 175 C++ svxcore.dll!sdr::properties::CustomShapeProperties::ItemChange(const unsigned short nWhich, const SfxPoolItem * pNewItem) Line 177 C++ svxcore.dll!sdr::properties::DefaultProperties::SetObjectItemSet(const SfxItemSet & rSet) Line 167 C++ svxcore.dll!SdrObject::SetObjectItemSet(const SfxItemSet & rSet) Line 2021 C++ The loop seems to play ping-pong with the verical writing mode. SetVerticalWriting is called alternating with true and false. "af" committed SVN revision 1559155 into trunk: 105491: Switched update of vertical flag and setting the item set to avoid in... Fixed the crash when loading the document. In SdrObjCustomShape::SetVerticalWriting(sal_Bool) the shape's item set was updated before the (explicitly stored) vertical flag was updated. During the update of the item set the old value of the flag was copied by some other object. Then the vertical flag was updated. In the following clean up step the copied old value of the flag was used and led to another round of setting the vertical flag. Fixed by switching the order of setting the item set and setting the flag value. I almost forgot that the crash was not the primary problem of the bug. Reopening for the formatting problem. Regarding the formatting problems: The only problem that I can see is that some arrows are routed differently in OpenOffice than they are in Excel. Removing keyword regression because neither bug description nor comments indicate that this is a regression. Setting importance to normal because I can not see big differences except in the routing of some connectors. That probably could be handled better but I am not sure that the actual routing is not implementation dependent. Created attachment 82817 [details]
Screenshot comparison
The main problems I see when I open sample document <<2009-09-30 23:43 CEST, oharboe >> are
(a) wrong text orientation in Drawing objects, fixed in 4.1.0 (yellow ellipse)
I tried to find a DUP, but it's too expensive
(b) connectors completely messed up, still a problem (red ellipse)
So I think we enjoy the fix for the text orientation problem and reduce this one to the connectors problem
Reduced as announced. There are at least 2 possibly related Bugs: "Bug 76373 - Problem of XLS file with connector" "Bug 111863 - Arrow connectors are not imported correctly from .xls" Bug 111863 is not related @Rainer: Can you explain what I am supposed to see in the attached file? Regarding the layouting of the connectors: this may not be a bug at all. It may be just a different interpretation of the OOXML spec. I have to check the spec to know which one it is. Created attachment 82829 [details] Screenshot comparison New (In reply to Andre from comment #16) Well, this simply was a wrong attachment :-( @Rainer: I suspected a wrong attachment, but it did have connectors in it, so I was not sure :-) Thanks for the new one. Adding me to CC. The algorithm for lay-outing connectors is probably different in MS and its not published somewhere (I remember Sven telling this). There is a path data for the originally lay-outed connector in MS files normally that should be loaded and not touched in this case. The error may have to do with unintentionally destroying this by for some reason re-lay-outing these connectors. The state of connectors is not good in the source, I did some enhancements in aw080 already, thus maybe one for me, at least staying in cc. |