Issue 12392 - Modify -> Combine needlessly converts curves to polygons.
Summary: Modify -> Combine needlessly converts curves to polygons.
Status: CLOSED FIXED
Alias: None
Product: Draw
Classification: Application
Component: code (show other issues)
Version: 644
Hardware: PC Linux, all
: P3 Trivial (vote)
Target Milestone: OOo 1.1 Beta2
Assignee: wolframgarten
QA Contact: issues@graphics
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-03-16 13:50 UTC by behrends
Modified: 2003-09-08 16:52 UTC (History)
1 user (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 behrends 2003-03-16 13:50:40 UTC 
In OOo 644m1 and 644m4, Modify -> Combine implicitly converts curved objects to
polygons. This has not been the case in OOo 1.0.*, and since OOo supports a
PolyPolygonBezier type, it appears to be a needless loss in quality and
information (especially in light of issue #12391, but it also makes manipulating
the individual segments of a curve much harder once they have turned into a
sequence of polygon segments).
Comment 1 wolframgarten 2003-04-01 09:38:28 UTC 
Set to new.
Comment 2 wolframgarten 2003-04-01 09:39:42 UTC 
Indeed this has changed. 
Reassigned to Armin. Please add a comment about this. Thanks.
Comment 3 Armin Le Grand 2003-04-01 12:17:12 UTC 
AW: Connect (from the other bug) does the same error, this will be 
the same reason internally (high degree of probability), thus i will 
handle this one in this bug, too.
Comment 4 Armin Le Grand 2003-04-01 16:53:06 UTC 
AW: SdrEditView::CombineMarkedObjects(...) i added a 
ConvertMarkedToPolyObj(...) call for #105899#. That was too strong 
and really converts all objects to polygons, loosing curve 
information. This can without neglecting the previous fix be replaced 
with ConvertMarkedToPathObj(...).
I also just checked if 'connect' uses CombineMarkedObjects(...), too, 
and it does. Thus, it's (as assumed) the same reason here.
I need to do some more checks, but seems to work.
Comment 5 Armin Le Grand 2003-04-02 11:43:08 UTC 
AW: OK, checks completed. Works as wanted. Checked in.
Comment 6 Armin Le Grand 2003-04-02 11:46:10 UTC 
AW: Changed the target milestone accordingly.
Comment 7 Armin Le Grand 2003-04-17 15:45:08 UTC 
AW->WG: Works in CWS draw9.
Comment 8 wolframgarten 2003-04-23 11:04:34 UTC 
Fixed.
Comment 9 wolframgarten 2003-04-23 11:04:48 UTC 
Verified.
Comment 10 wolframgarten 2003-04-29 10:20:12 UTC 
Tested and closed.