Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.
For bugs related to Red Hat Enterprise Linux 5 product line. The current stable release is 5.10. For Red Hat Enterprise Linux 6 and above, please visit Red Hat JIRA https://issues.redhat.com/secure/CreateIssue!default.jspa?pid=12332745 to report new issues.

Bug 491357

Summary: [fix available] CRM 1892074 Oowriter inserts an arbitrary <TD> tag in table on html import/export
Product: Red Hat Enterprise Linux 5 Reporter: Alan Matsuoka <alanm>
Component: openoffice.orgAssignee: David Tardon <dtardon>
Status: CLOSED ERRATA QA Contact: desktop-bugs <desktop-bugs>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.3CC: caolanm, kxiong, tao
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-03-30 08:45:16 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 499522    
Attachments:
Description Flags
openoffice.org-fix-html-import-export-of-merged-cells.patch
none
10-html_test.htmldl
none
sosreport-j.luginsland.1892074-686161-98355c.tar.bz2 none

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