Bug 100202 - FILEOPEN: Paragraph White Fill added to cell in DOCX import
Summary: FILEOPEN: Paragraph White Fill added to cell in DOCX import
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
5.2.0.0.alpha0+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bisected, filter:docx, regression
Depends on:
Blocks:
 
Reported: 2016-06-03 03:45 UTC by Luke
Modified: 2016-08-19 18:44 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
Screenshot (left-to-right) 5.1, 5.1 round-tripped, 5.3 (740.83 KB, image/png)
2016-06-03 04:10 UTC, Luke
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Luke 2016-06-03 03:45:38 UTC
Steps to reproduce:
1. Open attachment 121892 [details] with 5.1 or early build of LO 5.2
2. Open attachment 121892 [details] with 5.3 or an recent build of LO 5.2
3. Compare

Note that since eae2331f83bd58bacccd898d60f6c5f54856c036, some cells in the table are being set to a white fill instead of being left transparent. 


result of git bisect:
$ git bisect good eae2331f83bd58bacccd898d60f6c5f54856c036 is the first bad commit
commit eae2331f83bd58bacccd898d60f6c5f54856c036
Author: Miklos Vajna <vmiklos@collabora.co.uk>
Date:   Tue Mar 29 14:03:00 2016 +0200

    tdf#98882 DOCX import: set default para properties on the Standard para style
Comment 1 Luke 2016-06-03 03:58:17 UTC
Miklos,
Roundtripping this file in 5.1(Bug 97090) causes the same issue as importing with 5.3. Last time I saw something like this with Calc it was an issue of our importer expecting 2007 transitional OOXML and our exporter generating strict OOXML or vice versa. Could this be another 2007 vs strict OOXML issue?

Mike,
Was this file created in Word 2007?
Comment 2 Luke 2016-06-03 04:10:28 UTC
Created attachment 125467 [details]
Screenshot (left-to-right) 5.1, 5.1 round-tripped, 5.3
Comment 3 Buovjaga 2016-06-09 17:09:34 UTC
Yes.

Arch Linux 64-bit, KDE Plasma 5
Version: 5.3.0.0.alpha0+
Build ID: 2d2a33934ecb952433a635ce5dab76cb2837b8a0
CPU Threads: 8; OS Version: Linux 4.6; UI Render: default; 
Locale: fi-FI (fi_FI.UTF-8)
Built on June 9th 2016
Comment 4 Luke 2016-08-19 18:44:05 UTC
Justin Luth fixed this issue and the associated export issue in Bug 97090 with 

http://cgit.freedesktop.org/libreoffice/core/commit/?id=2a174c2f6fd58a31eb150b84de83e5ba1c4d3fed