[ 
https://issues.apache.org/jira/browse/TOMAHAWK-1367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12644684#action_12644684
 ] 

Michael Heinen commented on TOMAHAWK-1367:
------------------------------------------

I don't understand the new status "Won't fix"
The old bug TOMAHAWK-728 was solved (and I caonnot reopen it!). EL expressions 
are now evaluated.
The new bug is now caused by a wrong cast into Integer instead of Long or 
Number.
It is just one line (nr 453)  that has to be fixed in 
org.apache.myfaces.component.html.ext.HtmlDataTable

I would like to provide this simple patch but  
org.apache.myfaces.component.html.ext.HtmlDataTable contains following comment:
// Generated from class 
org.apache.myfaces.component.html.ext.AbstractHtmlDataTable.
// WARNING: This file was automatically generated. Do not edit it directly, or 
you will lose your changes.
Method getNewspaperColumns() is abstract in AbstractHtmlDataTable.
Class 
tomahawk\core\target\myfaces-builder-plugin\main\java\org\apache\myfaces\component\html\ext
 seems to be not part of the svn repository?



> ClassCastException in HtmlDataTable newspaperColumns
> ----------------------------------------------------
>
>                 Key: TOMAHAWK-1367
>                 URL: https://issues.apache.org/jira/browse/TOMAHAWK-1367
>             Project: MyFaces Tomahawk
>          Issue Type: Bug
>          Components: Extended Datatable
>    Affects Versions: 1.1.7
>         Environment: myFaces 1.1.5, tomahawk 1.1.7, richfaces 3.1.5
>            Reporter: Michael Heinen
>            Priority: Critical
>
> Following ClassCast Eception is thrown if I use an EL expression in the 
> newspaperColumns attribute:
> java.lang.ClassCastException: java.lang.Long cannot be cast to 
> java.lang.Integer
>       at 
> org.apache.myfaces.component.html.ext.HtmlDataTable.getNewspaperColumns(HtmlDataTable.java:453)
>       at 
> org.apache.myfaces.renderkit.html.ext.HtmlTableRenderer.getNewspaperColumns(HtmlTableRenderer.java:81)
>       at 
> org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlTableRendererBase.encodeInnerHtml(HtmlTableRendererBase.java:250)
>       at 
> org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlTableRendererBase.encodeChildren(HtmlTableRendererBase.java:140)
>       at 
> org.apache.myfaces.renderkit.html.ext.HtmlTableRenderer.encodeChildren(HtmlTableRenderer.java:280)
>       at 
> javax.faces.component.UIComponentBase.encodeChildren(UIComponentBase.java:527)
>       at 
> org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils.renderChild(RendererUtils.java:415)
>       at 
> org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils.renderChildren(RendererUtils.java:401)
>       at 
> org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils.renderChild(RendererUtils.java:418)
>       at 
> org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils.renderChildren(RendererUtils.java:401)
>       at 
> org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils.renderChild(RendererUtils.java:418)
>       at 
> org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils.renderChildren(RendererUtils.java:401)
>       at 
> org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils.renderChild(RendererUtils.java:418)
>       at 
> org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils.renderChildren(RendererUtils.java:401)
>       at 
> org.apache.myfaces.renderkit.html.ext.HtmlGroupRenderer.encodeEnd(HtmlGroupRenderer.java:71)
>       at 
> javax.faces.component.UIComponentBase.encodeEnd(UIComponentBase.java:539)
>       at 
> org.ajax4jsf.renderkit.RendererBase.renderChild(RendererBase.java:286)
> sample usage:
> <t:dataTable id="l" forceId="true"
>   value="#{MyController.myList}"
>   newspaperOrientation="horizontal"
>   newspaperColumns="#{MyController.showFoo?7:8}"
> Pls see also TOMAHAWK-728 newspaperColumns attribute ignores EL expression.
> I provided a patch there but it is has not been applied and 728 has been set 
> to fixed although it is not working!

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to