ICEfaces
  1. ICEfaces
  2. ICE-7382

ACE:column's sortFunction attribute does not activate the display of the up/down arrows.

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.1-Beta
          • Fix Version/s: 3.0.RC1, 3.0
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            Tomcat v7, JSF 2, ICEFaces 2.1.0 beta
          • Assignee Priority:
            P3
          • Affects:
            Documentation (User Guide, Ref. Guide, etc.)

            Description

            ACE:column's sortFunction attribute does not activate the display of the up/down arrows. Therefore, the sort method can not be called/tested. I have discussed this with Nils and he confirmed the arrows should be displayed. Also, the TLD does not specify the method signature required for the sort.

              Activity

              Hide
              Permalink
              Nils Lundquist added a comment -

              My discussion with Pierre was going off the details from the documentation. Turns out that sortFunction isn't equivalent to sortBy, both must be used. SortFunction merely allows the user to define their own relationship between the objects expressed by sortBy, rather than implementing Comparable for the objects themselves, they can value bind a Comparator<Object> to sortFunction.

              Tested this functionality and it works.

              Show
              Nils Lundquist added a comment - My discussion with Pierre was going off the details from the documentation. Turns out that sortFunction isn't equivalent to sortBy, both must be used. SortFunction merely allows the user to define their own relationship between the objects expressed by sortBy, rather than implementing Comparable for the objects themselves, they can value bind a Comparator<Object> to sortFunction. Tested this functionality and it works.

                People

                • Assignee:
                  Nils Lundquist
                  Reporter:
                  Pierre Masse
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: