Issue 80213 - missing lines in Net charts
Summary: missing lines in Net 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
: P2 Trivial (vote)
Target Milestone: ---
Assignee: christian.guenther
QA Contact: issues@graphics
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2007-07-31 11:15 UTC by Armin Le Grand
Modified: 2013-02-24 21:21 UTC (History)
3 users (show)

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

Attachments
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 Armin Le Grand 2007-07-31 11:15:14 UTC 
AW: When using the net chart type, glrid and data lines are missing for the last
segment. This looks like a open/Closed polygon mismatch stuff.
Comment 1 Armin Le Grand 2007-07-31 11:19:41 UTC 
AW: Indeed, for some reason i removed the explicit correction of a closed
polygon (adding the first point as last) to an open one in
SdrPathObj::ImpForceKind(), probably because i wanted to avoid the
basegfx::tools::checkClosed usage there.
Now this needs to be replaced with a pair of
basegfx::tools::openWithGeometryChange/basegfx::tools::closeWithGeometryChange
which does what we need: Adding/removing last points, changing open/closed state
and eventually correcting added control vectors. That symmetrical pair of
tooling methods was not available before aw051.
AW: Tested if all works well with usage of
openWithGeometryChange/closeWithGeometryChange, it does. Stopping aw052, this
task is best to be added there.
Comment 2 Armin Le Grand 2007-07-31 11:22:30 UTC 
AW: Okay, done. Checked in. Adding IHA.
Comment 3 Armin Le Grand 2007-07-31 12:24:16 UTC 
AW->CGU: Please verify. To check, open any office app, add chart, chaneg chart
type to net, choose a net with some lines.
Comment 4 IngridvdM 2007-07-31 13:28:51 UTC 
set to prio 2 as a whole chart type is defect and no workaround available
Comment 5 IngridvdM 2007-07-31 13:31:48 UTC 
added keyword regression
Comment 6 kla 2007-07-31 13:56:30 UTC 
added me to cc
Comment 7 christian.guenther 2007-08-02 10:14:34 UTC 
CGU: Verified in cws aw052
Comment 8 christian.guenther 2007-08-13 13:16:43 UTC 
CGU: Integrated in src680m225