Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 343241

Summary: DBCS3.7 Fail to compare MS Word's docx format document files
Product: [Eclipse Project] Platform Reporter: Kentaroh Noji <kennoji>
Component: CompareAssignee: Tomasz Zarna <tomasz.zarna>
Status: VERIFIED FIXED QA Contact:
Severity: major    
Priority: P3 CC: camle, kazm, kitlo, pwebster, Szymon.Brandys
Version: 4.1Flags: Szymon.Brandys: review+
Target Milestone: 3.7 RC1   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
screem capture
none
Fix v01
none
mylyn/context/zip
none
Good result with I20110511-0815 build (screen capture) none

Description Kentaroh Noji CLA 2011-04-19 03:48:08 EDT
Build Identifier: I20110329-2200

This problem is found and reported in DBCS enablement test, but the problem may be not caused by DBCS. 
Comparing two MS word documents which have .docx file extension, Text compare editor shows garbled characters in the editor. When the MS word documents are .doc file extension, no problem. 
JDK version: java full version "JRE 1.6.0 IBM Windows 32 build pwi3260sr9fp1-20110208_03 (SR9 FP1)"
OS: Windows XP SP3 (Japanese)
MS Word: MS Office Professional Plus 2010, MS word version is 14.0.4760.1000 (32 bit)  


Reproducible: Always

Steps to Reproduce:
1.Create two MS word document files and save the files. Ensure that file extension is .docx
2. Create a simple project in Eclipse
3. Copy the two MS word document files and paste them in the project. 
4. Select the two MS word document files in project explorer, and open context menu. Compare > each other
Comment 1 Kentaroh Noji CLA 2011-04-19 03:51:48 EDT
Created attachment 193550 [details]
screem capture
Comment 2 Kentaroh Noji CLA 2011-05-10 02:59:55 EDT
Are there any updates?
Comment 3 Tomasz Zarna CLA 2011-05-10 05:35:43 EDT
Created attachment 195187 [details]
Fix v01

Adding .docx file extension fixed the issue. From now on, files with the new extension are recognized as Word Documents and compared using WordMergeViewer.
Comment 4 Tomasz Zarna CLA 2011-05-10 05:35:45 EDT
Created attachment 195188 [details]
mylyn/context/zip
Comment 5 Tomasz Zarna CLA 2011-05-10 05:37:03 EDT
Szymon, let me know what do you think about it. It's a straightforward fix, safe for RC1 imho.
Comment 6 Szymon Brandys CLA 2011-05-10 05:37:54 EDT
Looks good.
Comment 7 Tomasz Zarna CLA 2011-05-10 06:57:48 EDT
Thanks, applied to HEAD. Available in builds >=I20110510-0800.
Comment 8 Kentaroh Noji CLA 2011-05-11 22:18:40 EDT
I verified the fix worked fine with I20110511-0815 build.
Comment 9 Kentaroh Noji CLA 2011-05-11 22:24:33 EDT
Created attachment 195453 [details]
Good result with I20110511-0815 build (screen capture)
Comment 10 Tomasz Zarna CLA 2011-05-12 04:45:13 EDT
Thanks for the quick verification.