[
http://jira.codehaus.org/browse/DOXIA-295?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=167968#action_167968
]
Lukas Theussl commented on DOXIA-295:
-------------------------------------
It would be good to know how you arrived at this problem, are you using a Doxia
parser or the Sink directly? The point is that a table without cells is not
really a table, e.g. a html table without a <td> element leads to invalid
xhtml.
I think it's logical to require a table to have at least one cell. The javadocs
for table in Sink.java [1] specifies the canonical sequence of events, and
explicitly mentions only tableCaption to be optional.
I am hesitant to fix this as you suggest, as it could potentially hide bugs in
parsers.
[1] http://maven.apache.org/doxia/doxia/doxia-sink-api/apidocs/index.html
> doxia-module-fo fails with a division by 0 error when using FoSink
> ------------------------------------------------------------------
>
> Key: DOXIA-295
> URL: http://jira.codehaus.org/browse/DOXIA-295
> Project: Maven Doxia
> Issue Type: Bug
> Components: Module - FO
> Affects Versions: 1.2
> Environment: Linux / Solairs jdk 1.5
> Build from svn check out Revsion: 749805
> Reporter: Dean Murphy
>
> When using the FoSink to convert from documents containing a table, I am
> running into a division by 0 error on line: 703 in
> org.apache.maven.doxia.module.fo.FoSink.java:
> int percent = 100 / cellCount;
> for ( int i = 0; i < cellCount; i++ )
> {
> sb.append( "<fo:table-column column-width=\"" + percent +
> "%\"/>" );
> sb.append( EOL );
> }
> Simple fix would be to check if cellCount != 0:
> if ( cellCount != 0 ) {
> int percent = 100 / cellCount;
> for ( int i = 0; i < cellCount; i++ )
> {
> sb.append( "<fo:table-column column-width=\"" + percent +
> "%\"/>" );
> sb.append( EOL );
> }
> }
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
