[seam-issues] [JBoss JIRA] (JBSEAM-5131) Seam ExcelExporter

[Bobby Lawrence (JIRA)]

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

Bobby Lawrence commented on JBSEAM-5131:
----------------------------------------

It might also help if the "processOutputs" method and some of the others like "processColumn" were protected instead of private so that the class could be extended and implementations could provide enhanced functionality
                
> Seam ExcelExporter
> ------------------
>
>                 Key: JBSEAM-5131
>                 URL: https://issues.jboss.org/browse/JBSEAM-5131
>             Project: Seam 2
>          Issue Type: Bug
>          Components: Excel
>    Affects Versions: 2.3.0.Final
>            Reporter: Bobby Lawrence
>            Priority: Minor
>
> The org.jboss.seam.excel.exporter.ExcelExporter component is great, but it doesn't apply converters to objects is puts into Excel spreadsheet cells.
> The "processOutputs" method simply sets the value of the UIOutput in the cell via:
> cell.setValue(output.getValue());
> It would be much more helpful if it called any converter on the output before something like:
> Object outputValue = output.getValue();
> Converter converter = output.getConverter();
> if (converter != null){
>      outputValue = converter.getAsString(
>                  FacesContext.getCurrentInstance(), output, outputValue);
> }
> cell.setValue(outputValue);

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira