Issue 119872

Summary: [From Symphony]all drawing objects lost aftering saving to another ppt by AOO
Product: Impress Reporter: Li Feng Wang <phoenix.wanglf>
Component: editingAssignee: AOO issues mailing list <issues>
Status: CLOSED FIXED QA Contact:
Severity: Normal    
Priority: P1 (highest) CC: awf.aoo, lijianyuan1983, liushenf, louqingle, shanzhu33
Version: 3.4.0   
Target Milestone: 4.0.0   
Hardware: PC   
OS: All   
Issue Type: DEFECT Latest Confirmation in: ---
Developer Difficulty: ---
Attachments:
Description Flags
sample file
none
root cause analysis for this issue
none
UT file for this issue
none
Another UT file for this issue
none
Patch for this issue awf.aoo: review+

Description Li Feng Wang 2012-06-07 08:43:06 UTC
Created attachment 78120 [details]
sample file

build:
AOO3.4 r1327774

steps:
1. Launch AOO, open the attached .ppt.
2. Save the file as *.ppt
3. Reopen the saved ppt file in MS 2003 Powerpoint.

Defects:
all drawing objects lost

Notes: If reopen the saved ppt file with AOO, this problem does not exist
Comment 1 Jianyuan Li 2012-07-09 02:19:20 UTC
I am investigating this issue.
Comment 2 Jianyuan Li 2012-07-10 03:21:21 UTC
Root cause:
When import, incorrect resolving for custom shape "pSegmentInfo" causes incorrect "command" and "count" is set into document model. When export, incorrect content from import is exported. For example, there is a pSegmentInfo with value "0xB000". According to the incorrect logic, path escape command will be resolved as "msopathEscapeExtension(0x00)". But actually it is "msopathEscapeSmoothLine(0x10)".
Solution:
Resolve the pSegmentInfo according to specification[MS-ODRAW].pdf. See the detail analysis for pSegmentInfo in attachment "root cause.odt".
Comment 3 Jianyuan Li 2012-07-10 03:22:27 UTC
Created attachment 78604 [details]
root cause analysis for this issue
Comment 4 Jianyuan Li 2012-07-10 03:23:33 UTC
Created attachment 78605 [details]
UT file for this issue
Comment 5 Jianyuan Li 2012-07-10 03:24:08 UTC
Created attachment 78606 [details]
Another UT file for this issue
Comment 6 Jianyuan Li 2012-07-10 06:15:37 UTC
Created attachment 78611 [details]
Patch for this issue
Comment 7 SVN Robot 2012-07-10 11:01:06 UTC
"af" committed SVN revision 1359598:
#i119872# Fixed import of custom shapes from PPT.
Comment 8 Andre 2012-07-10 11:01:28 UTC
Comment on attachment 78611 [details]
Patch for this issue

Reviewed and tested: OK
Comment 9 Andre 2012-07-10 11:04:10 UTC
Setting status to fixed.
Comment 10 Jianyuan Li 2012-07-10 11:09:46 UTC
Thanks Andre.
Comment 11 louqle 2012-08-20 03:20:33 UTC
verified on XP SP3 and Ubuntu 12.04 against trunk build 1374181
Comment 12 louqle 2012-08-23 00:58:21 UTC
verified in xp, Ubuntu, win7,mac,win7-64,Vista,suse,redhat, SLED 11 SP1 64 bit, all pass. Close this bug.
Comment 13 Jianyuan Li 2012-08-23 02:49:09 UTC
*** Issue 119879 has been marked as a duplicate of this issue. ***
Comment 14 Jianyuan Li 2012-08-24 09:23:09 UTC
*** Issue 119778 has been marked as a duplicate of this issue. ***
Comment 15 Shenfeng Liu 2012-10-02 09:51:58 UTC
Set Target Milestone to AOO 3.5.0 for PM purpose.