ICEfaces
  1. ICEfaces
  2. ICE-2911

dataTable's sortColumn sortAscending not set at right time

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major 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

      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.

        Activity

        Mark Collette created issue -
        Stefan Zeller made changes -
        Field Original Value New Value
        Salesforce Case []
        Affects Version/s 1.8.1 [ 10170 ]
        Stefan Zeller made changes -
        Salesforce Case []
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 2.0.1 [ 10255 ]
        Assignee Priority P2
        Affects Version/s 2.0.0 [ 10230 ]
        Assignee Mark Collette [ mark.collette ]
        Mark Collette made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Assignee Priority P2
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Mark Collette
            Reporter:
            Mark Collette
          • Votes:
            15 Vote for this issue
            Watchers:
            9 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: