Bug 81893

Summary: FILEOPEN: RTF doesnt retain table alignment
Product: LibreOffice Reporter: Jay Philips <philipz85>
Component: WriterAssignee: Michael Stahl <mst.fdo>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: medium CC: mst.fdo, vmiklos
Version: 3.5.0 releaseKeywords: regression
Hardware: Other   
OS: All   
See Also: https://bugs.freedesktop.org/show_bug.cgi?id=81892
Whiteboard: filter:rtf rtf_filter target:4.4.0 target:4.3.1
i915 platform: i915 features:
Attachments: LibO 4.3.1 VS Word 2010

Description Jay Philips 2014-07-30 03:52:49 UTC 
Created attachment 103667 [details]
LibO 4.3.1 VS Word 2010

After converting < http://www.microsoft.com/investor/reports/ar13/docs/2013_Annual_Report.docx > into an RTF file in word 2010, opening the RTF file in LibO 4.3.1 results in the incorrect import of a table alignment on page 8.

Steps:
1) Open attachment 103664 [details]
2) Scroll to page 2

Expected: table to be aligned center rather than left

This is a regression as it didnt happen in 3.3.0. Last tested version that had the issue is 3.6.7.
Comment 1 Michael Stahl 2014-07-30 14:00:10 UTC 
alignment is Center, spacing is 2.29 cm left / right in OOo 3.4 beta
alignment is From Left, spacing is 4.59 cm left / 0 right since 3.5.0

(LO 3.4.6 is ... very broken)

it's the new RTF filter...
Comment 2 Michael Stahl 2014-08-01 09:07:00 UTC 
fixed on master
Comment 3 Commit Notification 2014-08-01 09:08:28 UTC 
Michael Stahl committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=008d1187a9f3b36d6cbf31742632d237afecf1bd

fdo#81893: writerfilter: RTF import: \qc before \page was lost



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 4 Commit Notification 2014-08-01 16:18:11 UTC 
Michael Stahl committed a patch related to this issue.
It has been pushed to "libreoffice-4-3":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=22688ede28db14e5947cf73e85fcd2156e26ec5e&h=libreoffice-4-3

fdo#81893: writerfilter: RTF import: \qc before \page was lost


It will be available in LibreOffice 4.3.1.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 5 Michael Stahl 2014-08-04 11:06:14 UTC 
oops - it looks like i mixed up the issue ids in the commit message - see bug 81893 for the commit that fixes this bug

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.