Bug 42938 - FILEOPEN: RTF import forgets background colors in nested table cells
Summary: FILEOPEN: RTF import forgets background colors in nested table cells
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
(earliest affected)
Master old -3.6
Hardware: All All
: medium normal
Assignee: Miklos Vajna
Depends on: 42858
  Show dependency treegraph
Reported: 2011-11-14 21:07 UTC by Jean-Baptiste Faure
Modified: 2012-03-20 22:43 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:

Partial patch. (729 bytes, text/plain)
2011-12-18 15:52 UTC, Miklos Vajna

Note You need to log in before you can comment on or make changes to this bug.
Description Jean-Baptiste Faure 2011-11-14 21:07:09 UTC
This bug report comes from bug 42858. To reproduce open the bugdoc https://bugs.freedesktop.org/attachment.cgi?id=53466 with the master.
The table seems correctly rendered but the background colors are missing in most of the cells.
To see what we should obtain, open https://bugs.freedesktop.org/attachment.cgi?id=53525

I do not know if it is linked, but cell borders seems not correctly set too.

Best regards. JBF
Comment 1 Jean-Baptiste Faure 2011-11-14 21:10:46 UTC
Assigned to Miklos. Please feel free to reassign to default if you can't handle this bug.

Thank you.
Comment 2 Miklos Vajna 2011-11-15 05:50:18 UTC
Thanks, will have a look.
Comment 3 Miklos Vajna 2011-12-18 15:52:00 UTC
Created attachment 54563 [details]
Partial patch.

I tried to fix this tonight, so far I only have a partial solution. The attached patch makes the first column (i.e. Jan/May/Sep) be imported propertly, but other months now aren't imported at all: DomainMapperTableHandler::endTable() results in 'Exception during table creation' (without an exception message).

To be continued.
Comment 4 Miklos Vajna 2011-12-18 15:55:08 UTC
Cedric, just in case this exception rings any bell, please let me know. :)

Comment 5 Cédric Bosdonnat 2011-12-19 00:48:17 UTC
(In reply to comment #4)
> Cedric, just in case this exception rings any bell, please let me know. :)

Well, that can be anything... the only way to know what's happening there is to debug the convertToTable call to see where the exception is thrown. Having interesting messages for UNO exceptions is an easy hack... and that method is full of exceptions like that ;)
Comment 6 Jean-Baptiste Faure 2012-03-19 22:56:41 UTC
Hi Miklos, Cédric, 

It seems that the color problem is fixed in LO 3.5.1. I do not know which commit changed the behaviour. That said, cell borders are still missing. If I find a bug report for this cell borders problem only, I will close this one as fixed.
If you can provide the commit ID which fix the color problem, please do it.

Side note: if I save odt file (https://bugs.freedesktop.org/attachment.cgi?id=53525) in RTF format with LO 3.5.1 the table is not rendered correctly. It seems that RTF exporter does not know how to manage tables inside a table. Surely not linked bug.

Best regards. JBF
Comment 7 Jean-Baptiste Faure 2012-03-20 22:42:48 UTC
For the cell borders problem, please follows it in the dedicated bug 44715.
So closing this bug as initial problem (background cell color) is fixed.

Will file a new bug report for table inside a table export problem if not already filed.

Best regards. JBF
Comment 8 Jean-Baptiste Faure 2012-03-20 22:43:21 UTC