Details
-
Type: Bug
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: 1.7RC1, 1.8.1, 2.0.0
-
Fix Version/s: 2.0.1
-
Component/s: ICE-Components
-
Labels:None
-
Environment:All
-
ICEsoft Forum Reference:
Description
Upon examination of com.icesoft.faces.component.commandsortheader.CommandSortHeader.broadcast(-), it appears that HtmlDataTable.setSortColumn(-) and HtmlDataTable.setSortAscending(-) are not being called in the right phase, and are also being called after the commandSortHeader's actionListener.
We should decide if we want to follow the typical EditableValueHolder paradigm for HtmlDataTable's sortColumn and sortAscending properties, and make them UpdatableProperties that CommandSortHeader sets on decode ( queueEvent(-) ), or if we want to follow the UICommand paradigm for CommandSortHeader, and have it use the immediate property, which would necessitate setting HtmlDataTable's sortColumn and sortAscending properties immediately before the ActionEvent is broadcast.
We should decide if we want to follow the typical EditableValueHolder paradigm for HtmlDataTable's sortColumn and sortAscending properties, and make them UpdatableProperties that CommandSortHeader sets on decode ( queueEvent(-) ), or if we want to follow the UICommand paradigm for CommandSortHeader, and have it use the immediate property, which would necessitate setting HtmlDataTable's sortColumn and sortAscending properties immediately before the ActionEvent is broadcast.
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #24002 | Thu Feb 24 13:54:31 MST 2011 | mark.collette | |
Files Changed | ||||
MODIFY
/icefaces2/trunk/icefaces/samples/compat/component-showcase/src/main/webapp/WEB-INF/includes/examples/custom/dataTable-commandSortHeader.xhtml
MODIFY /icefaces2/trunk/icefaces/samples/compat/component-showcase/src/main/java/org/icefaces/application/showcase/view/bean/examples/component/commandSortHeader/SortHeaderModel.java MODIFY /icefaces2/trunk/icefaces/samples/compat/component-showcase/src/main/webapp/WEB-INF/faces-config.xml |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #24001 | Thu Feb 24 13:51:52 MST 2011 | mark.collette | |
Files Changed | ||||
MODIFY
/icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/commandsortheader/CommandSortHeader.java
|