[MFCOMMONS-2] ExporterActionListener should be dependant of UIData instead of HtmlDataTable

• Type: Bug
• Status: Patch Available
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 1.0.1-SNAPSHOT
• Component/s: myfaces-commons-components
• Environment: myfaces 1.2.4 + richfaces 3.2.2.SR1 + jetty
• Assignee: Unassigned
• Reporter: Gilles Demarty
• Created: 08/Dec/08 16:45
• Updated: 09/Dec/08 08:59

Description

Right now, the component <mc:exporterActionListener /> cast the for attribute as a htmlDataTable. It does not allow other implementations of UIData to be used with this component (e.g. richfaces one). Is there any reasons to do that ?

Changing all the htmlDataTable to UIData in those classes provides a satisfying solution without any visible drawbacks.

By the way, in ExporterActionListenr, the cast cast test that throws the correct message is thrown after the cast, thus the message return in the web page is a ClassCastException, not a RuntimeException