[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-3830) e:cell doesn't support null values

[Eric Jung (JIRA)]

    [ https://jira.jboss.org/jira/browse/JBSEAM-3830?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12442183#action_12442183 ] 

Eric Jung commented on JBSEAM-3830:
-----------------------------------

I agree my patch wasn't the best place to fix the problem. A more likely place would be UICell.getDataType() or somewhere else, but I didn't understand the flow of things enough to do the patch there. Can you tell me where/how it was fixed? Even a subversion revision number would help. I'd like to pull down the fix and use it locally. By the way, I'm the JExcelAPI SourceForge site owner, Forum Moderator and have contributed to the JExcelAPI source code. You guys have done excellent work with this; thanks so much!

> e:cell doesn't support null values
> ----------------------------------
>
>                 Key: JBSEAM-3830
>                 URL: https://jira.jboss.org/jira/browse/JBSEAM-3830
>             Project: Seam
>          Issue Type: Bug
>          Components: Excel
>            Reporter: Eric Jung
>            Assignee: Nicklas Karlsson
>             Fix For: 2.1.1.GA
>
>         Attachments: JXLHelper.java.diff
>
>
> If the value of an e:cell is null , an exception is thrown due to a lack of null checks in JXLHelper.createCell(). It would be better if the cell value was empty in the case of nulls, just like JSF's and Facelets' <h:outputText value="#{aNull}"/> outputsan empty string when encountering nulls.
> Workaround described below, but it gets very tedious adding null checks for hundreds of cells across multiple worksheets.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira