ICEfaces
  1. ICEfaces
  2. ICE-8435

ace:dataTable column sortPriority when value bound can never be set to null

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.2
    • Component/s: None
    • Labels:
      None
    • Environment:
      IF 3.1

      Description

      This is due to an effect of EL, however this wouldn't be an issue, if the value returned instead of null, 0 didn't behave like it was a valid priority. Sort priorities should be indexed at 1, so ignoring 0 should fix this issue.

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #30376 Thu Aug 02 12:33:41 MDT 2012 nils.lundquist ICE-8435 - Altered sortPriority checks to accept 0 as a null value.
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTable.java
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTableHeadRenderer.java
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/column/Column.java
        Nils Lundquist created issue -
        Hide
        Nils Lundquist added a comment -

        Revision #30376
        Committed by nils.lundquist
        A minute ago ICE-8435 - Altered sortPriority checks to accept 0 as a null value.

        Show
        Nils Lundquist added a comment - Revision #30376 Committed by nils.lundquist A minute ago ICE-8435 - Altered sortPriority checks to accept 0 as a null value.
        Nils Lundquist made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Nils Lundquist made changes -
        Assignee Nils Lundquist [ nils.lundquist ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: