Issue 119567

Summary: [From Symphony]WMF graphic size changed too small to see if saved by AOO
Product: Writer Reporter: Yan Ji <yanji.yj>
Component: save-exportAssignee: Oliver-Rainer Wittmann <orw>
Status: CLOSED FIXED QA Contact:
Severity: Major    
Priority: P3 CC: bjdujing, liushenf, orw, zjchencdl
Version: 3.4.0Keywords: ms_interoperability
Target Milestone: 4.0.0   
Hardware: PC   
OS: All   
Issue Type: DEFECT Latest Confirmation in: ---
Developer Difficulty: ---
Description Flags
patch for i119567 orw: review+

Description Yan Ji 2012-05-30 06:29:41 UTC
Created attachment 77786 [details]

Build: AOO3.4
1. Open sample file
2. Save it as another .doc file
3. Open the saved file in AOO or MS Office

In AOO: It looks like the grpahic is lost, but actually it still there which width changed to 0.02 and height to 0.39 make the graphic too small to see.
In MS Office: There is dialog popup "The dimentions after cropping are too samll or too large"
Comment 1 zjchen 2012-07-03 13:29:58 UTC
Open the saved file in AOO, select the invisible picture, noticed there are several improper parameters in crop tab:

scale - width is 1%, height is 7%, while original document should be 7% and 8%,
image size - width is 0.02'' height is 0.39'',while original document should be  5.83'' and 0.40'

put the value of scale/image size as original sample file, the cropped graphic can be displayed properly.
Comment 2 zjchen 2012-07-03 13:59:38 UTC
The problem is that the graphic in sample file is a special WMF picture, and its PicHeader parameters is not correctly on export. 

see <SwWW8WrGrf::WritePICFHeader(..)> in "main\sw\filter\source\wrtww8gr.cxx" line 604
when the original size of WMF picture is too big to fit, the final size will be replaced with Layout size, so that the scaling factor will be calculated based on inappropriate size. 
replace writting the inappropriate scaling value into stream with default operation when the original size of WMF picture is too big to fit
Comment 3 zjchen 2012-07-03 14:13:59 UTC
Created attachment 78566 [details]
patch for i119567
Comment 4 Oliver-Rainer Wittmann 2012-07-05 13:29:56 UTC
taking over to review the attached patch
Comment 5 Oliver-Rainer Wittmann 2012-07-05 14:51:36 UTC
Comment on attachment 78566 [details]
patch for i119567

review done -> patch looks good and solves the problem.
I will apply it on trunk
Comment 6 Oliver-Rainer Wittmann 2012-07-06 08:09:13 UTC
I have applied the patch on trunk, revision 1358065
I took the freedom to apply minor adjustments before committing it.
Comment 7 zjchen 2012-07-08 13:24:22 UTC
Thanks for your review and modification. It looks much better:)
Comment 8 Du Jing 2012-08-20 09:01:49 UTC
verified on the AOO3.5