ICEfaces
  1. ICEfaces
  2. ICE-8064

When using the GroupBy attribute of the ace:column without the sortBy attribute, a null pointer exception is thrown.

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 3.0.1
    • Fix Version/s: 3.1.0.BETA2, 3.1
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Icefaces 3.1.0.ALPHA1 at Williams.
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial

      Description

      When using the GroupBy attribute of the ace:column without the sortBy attribute, a null pointer exception is thrown. The groupBy should be used for sorting when the sortBy is not present.

      Specifically, when a column is specified in the groupBy, then no sorting controls should be rendered, and a sort is implied.

      It's not clear how the sort order will be determined, and clearly the user will not be able to change the sort order used by the grouping directly with this change.

        Activity

        Pierre Masse created issue -
        Nils Lundquist made changes -
        Field Original Value New Value
        Assignee Nils Lundquist [ nils.lundquist ]
        Ken Fyten made changes -
        Issue Type Bug [ 1 ] Improvement [ 4 ]
        Salesforce Case []
        Fix Version/s 3.1.0.BETA2 [ 10336 ]
        Fix Version/s 3.1 [ 10312 ]
        Affects Version/s 3.0.1 [ 10282 ]
        Ken Fyten made changes -
        Salesforce Case []
        Description When using the GroupBy attribute of the ace:column without the sortBy attribute, a null pointer exception is thrown. The groupBy should be used for sorting when the sortBy is not present. When using the GroupBy attribute of the ace:column without the sortBy attribute, a null pointer exception is thrown. The groupBy should be used for sorting when the sortBy is not present.

        Specifically, when a column is specified in the groupBy, then no sorting controls should be rendered, and a sort is implied.

        It's not clear how the sort order will be determined, and clearly the user will not be able to change the sort order used by the grouping directly with this change.
        Nils Lundquist made changes -
        Salesforce Case []
        Priority Major [ 3 ] Minor [ 4 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #29151 Tue May 22 15:17:40 MDT 2012 nils.lundquist ICE-8064 - setting groupBy for a column now behaves as if sortBy was also identically set for that column. This was previously a app level requirement.
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/column/Column.java
        Hide
        Nils Lundquist added a comment -

        Revision #29151
        Committed by nils.lundquist
        Moments ago
        ICE-8064 - setting groupBy for a column now behaves as if sortBy was also identically set for that column. This was previously a app level requirement.

        Show
        Nils Lundquist added a comment - Revision #29151 Committed by nils.lundquist Moments ago ICE-8064 - setting groupBy for a column now behaves as if sortBy was also identically set for that column. This was previously a app level requirement.
        Nils Lundquist made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Ken Fyten added a comment -

        Need to update the showcase Grouping demo to remove the sortBy usage which is no longer necessary.

        Show
        Ken Fyten added a comment - Need to update the showcase Grouping demo to remove the sortBy usage which is no longer necessary.
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Affects [Sample App./Tutorial]
        Ken Fyten made changes -
        Salesforce Case []
        Affects [Sample App./Tutorial] [Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #29341 Wed Jun 06 12:17:48 MDT 2012 nils.lundquist ICE-8064 - Removed now unneeded sortBy from grouped column showcase example.
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/ace/dataTable/dataTableGrouping.xhtml
        Hide
        Nils Lundquist added a comment -

        Revision #29341
        Committed by nils.lundquist
        Moments ago
        ICE-8064 - Removed now unneeded sortBy from grouped column showcase example.

        Show
        Nils Lundquist added a comment - Revision #29341 Committed by nils.lundquist Moments ago ICE-8064 - Removed now unneeded sortBy from grouped column showcase example.
        Nils Lundquist made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Component/s ACE-Components [ 10050 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: