Bug 491357 - [fix available] CRM 1892074 Oowriter inserts an arbitrary <TD> tag in table on html import/export
Summary: [fix available] CRM 1892074 Oowriter inserts an arbitrary <TD> tag in table o...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: openoffice.org
Version: 5.3
Hardware: All
OS: Linux
medium
medium
Target Milestone: rc
: ---
Assignee: David Tardon
QA Contact: desktop-bugs@redhat.com
URL:
Whiteboard:
Depends On:
Blocks: 499522
TreeView+ depends on / blocked
 
Reported: 2009-03-20 15:41 UTC by Alan Matsuoka
Modified: 2018-10-27 14:24 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-03-30 08:45:16 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
openoffice.org-fix-html-import-export-of-merged-cells.patch (884 bytes, patch)
2009-03-20 15:41 UTC, Alan Matsuoka
no flags Details | Diff
10-html_test.htmldl (731 bytes, application/octet-stream)
2009-03-20 15:42 UTC, Alan Matsuoka
no flags Details
sosreport-j.luginsland.1892074-686161-98355c.tar.bz2 (1.10 MB, application/x-bzip2)
2009-03-20 15:43 UTC, Alan Matsuoka
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2010:0274 0 normal SHIPPED_LIVE openoffice.org bug fix and enhancement update 2010-03-29 13:52:09 UTC

Description Alan Matsuoka 2009-03-20 15:41:26 UTC
Created attachment 336061 [details]
openoffice.org-fix-html-import-export-of-merged-cells.patch

Description of problem:

$subject. This seems to have been fixed upstream and in the latest Fedora-10. I've included a likely fix from upstream:

http://sw.openoffice.org/source/browse/sw/sw/source/filter/html/htmltab.cxx?rev=1.28&r1=1.28&r2=1.28.96.1&sortby=date

The patch is adapted to the latest Openoffice.org in RHEL-5.3.

Attached is also a test html file that could be used to reproduce the error.

How reproducible:

Always, for html tables that have merged cells.

Steps to Reproduce:

* Open 10-html_test.html in oowriter
* Save file as 10-test2.html

Actual results:

A blank <td></td> is inserted.

Expected results:

File sould be unchanged.

Additional info:

Attached files:
1) openoffice.org-fix-html-import-export-of-merged-cells.patch: Likely fix from upstream
2) 10-html_test.html: test case

Comment 1 Alan Matsuoka 2009-03-20 15:42:10 UTC
Created attachment 336062 [details]
10-html_test.htmldl

Comment 2 Alan Matsuoka 2009-03-20 15:43:05 UTC
Created attachment 336063 [details]
sosreport-j.luginsland.1892074-686161-98355c.tar.bz2

Comment 3 Caolan McNamara 2009-03-23 10:09:39 UTC
caolanm->dtardon: Can you see if that suggested patch fixes the reported problem, and if it does then we can ack it for 5.5, which is the next errata, not the current one so its more than likely we'll be rebasing to a new OOo version in 5.5, but one never knows for sure...

Comment 4 David Tardon 2009-03-26 08:20:58 UTC
The problem is in how openoffice.org handles rowspanned cells during export to HTML. It correctly generates a cell with rowspan attribute on the first line of the rowspan, but then (not correctly) generates another cell to each line the rowspan covers. That causes the described empty <td></td> pair in the output and misaligned table. The suggested patch fixes the problem.

Comment 6 koka xiong 2009-12-29 02:41:10 UTC
Verified with openoffice.org-3.1.1-19.1.el5
on 
Red Hat Enterprise Linux 5Client - i386, x86_64
Red Hat Enterprise Linux 5Server-DPAS - i386, x86_64
steps:
1.Open 10-html_test.html in oowriter
2.Save file as 10-test2.html
3.View html source, no blank <TD></TD> is inserted.

Verified passed.

Comment 8 errata-xmlrpc 2010-03-30 08:45:16 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2010-0274.html


Note You need to log in before you can comment on or make changes to this bug.