Issue 119559

Summary: [From Symphony]Vertical letters in "VerticalText" fontwork get horizontal when opening .ppt file
Product: Impress Reporter: Li Feng Wang <phoenix.wanglf>
Component: save-exportAssignee: AOO issues mailing list <issues>
Status: CLOSED FIXED QA Contact:
Severity: Normal    
Priority: P2 CC: debin.lei, doneyourself, hdu, liushenf
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 with fontwork
none
before save screenshot
none
after save screenshot
none
patch for the issue
none
Code fix patch hdu: review+

Description Li Feng Wang 2012-05-30 06:00:04 UTC
Created attachment 77775 [details]
sample file with fontwork

build:
AOO3.4 r1327774

steps:
1. Launch AOo3.4
2. Open the attached .ppt file    
3. Save it to another .ppt file
4. Reopen it with both AOO3.4 and MS
5. Check the fontwork in slide 2.

Defect: 
See the attached snapshots, All Vertical letters in the fontwork change to horizontal after saved by AOO3.4.
Comment 1 Li Feng Wang 2012-05-30 06:02:36 UTC
Created attachment 77776 [details]
before save screenshot
Comment 2 Li Feng Wang 2012-05-30 06:03:15 UTC
Created attachment 77777 [details]
after save screenshot
Comment 3 Lei Debin 2012-06-13 02:21:30 UTC
I will check the issue
Comment 4 Lei Debin 2012-06-13 06:45:52 UTC
Root cause is 
The Fontwork object has no vertical properties. When read the text is rotated in PPT, we will set the vertical property to the paragraph in this object. And when export it to PPT file format, we didn't set this flag to the file again.
So reopen, this property is not set to this object
Here is the solution:
When export one fontwork object, check the outlinerparagraph. If it is vertical, set the 0x20000 flag to TextPathFlags.
Comment 5 Lei Debin 2012-06-13 06:56:42 UTC
Created attachment 78286 [details]
patch for the issue
Comment 6 WangZhe 2012-06-13 08:24:40 UTC
Created attachment 78288 [details]
Code fix patch
Comment 7 hdu@apache.org 2012-06-13 08:58:59 UTC
The two patches look almost identical except for formatting except for the use of the full path vs. the local path.
Comment 8 hdu@apache.org 2012-06-13 09:21:46 UTC
Thanks for the great analysis of the problem and the patch. Applied as rev 1349700.
Comment 9 liuping 2012-06-21 07:27:38 UTC
confirm fixing the bug on ver 1351249 in Win7 ,pass