ICEfaces
  1. ICEfaces
  2. ICE-8292

ace:dataTable does not show the full set of rows when the paginator is turned off

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 3.1.0.RC1
    • Fix Version/s: 3.1.0.RC1, 3.1
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces 3.1.0.BETA2-02112
    • Assignee Priority:
      P1

      Description

      This is a regression bug. A data table does not show the full set of rows when the paginator is turned off, the rows in pages before the current one are not displayed.

        Activity

        Ken Fyten created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Salesforce Case []
        Assignee Priority P1
        Assignee Nils Lundquist [ nils.lundquist ]
        Ken Fyten made changes -
        Salesforce Case [5007000000MIY6r]
        Hide
        Nils Lundquist added a comment -

        In general, the UIData first property is not controlled by the component itself, it's expected that the application will control first and rows to implement functionality like pagination, range display, etc.

        The component provides an integrated paginator that breaks this expectation somewhat. However, this paginator still just acts to give the client control over the first and rows properties, without app bindings. The component doesn't adjust these properties without obvious prompts from the client .

        If disabling the paginator was to forcibly reset the state of first / rows / page, it would be a departure from the current expectation that the component doesn't make subtle internal adjustments to the state of these properties. An application that wishes to reduce the displayed data to only the current top row should be able to just disable pagination and set rows to 1.

        Cases where removing pagination entirely is desired should disable pagination and call dataTable.resetPagination().

        Show
        Nils Lundquist added a comment - In general, the UIData first property is not controlled by the component itself, it's expected that the application will control first and rows to implement functionality like pagination, range display, etc. The component provides an integrated paginator that breaks this expectation somewhat. However, this paginator still just acts to give the client control over the first and rows properties, without app bindings. The component doesn't adjust these properties without obvious prompts from the client . If disabling the paginator was to forcibly reset the state of first / rows / page, it would be a departure from the current expectation that the component doesn't make subtle internal adjustments to the state of these properties. An application that wishes to reduce the displayed data to only the current top row should be able to just disable pagination and set rows to 1. Cases where removing pagination entirely is desired should disable pagination and call dataTable.resetPagination().
        Nils Lundquist made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Won't Fix [ 2 ]
        Ken Fyten made changes -
        Security Private [ 10001 ]
        Arran Mccullough made changes -
        Security Private [ 10001 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #31425 Thu Oct 11 13:07:50 MDT 2012 nils.lundquist ICE-8292 - Clarify behaviour of table position when pagination is dynamically disabled.
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTableMeta.java
        Hide
        Leo Milhouse added a comment - - edited

        This is very serious and should be more prominent in the docs and showcase!!! With big bold faces and letters in red.
        I've been using the remove pagination trick to cope for the removal of the "All" option in the # of rows to display, and since all our tables are pretty big (over 10k rows), I've never noticed that if you're navigating through it and then disable the pagination, it'll get stuck with whatever's left, and not the whole resultset. Good thing I was rummaging through the forum to see what got fixed for 3.2 and stumbled upon this. Please raise the awareness of other uses about this, I'm sure a lot of other people are oblivious of this issue.

        Show
        Leo Milhouse added a comment - - edited This is very serious and should be more prominent in the docs and showcase!!! With big bold faces and letters in red. I've been using the remove pagination trick to cope for the removal of the "All" option in the # of rows to display, and since all our tables are pretty big (over 10k rows), I've never noticed that if you're navigating through it and then disable the pagination, it'll get stuck with whatever's left, and not the whole resultset. Good thing I was rummaging through the forum to see what got fixed for 3.2 and stumbled upon this. Please raise the awareness of other uses about this, I'm sure a lot of other people are oblivious of this issue.
        Hide
        Nils Lundquist added a comment - - edited

        I'll add a comment to the TLD of the pagination property clarifying that disabling it doesn't alter the current table position.

        Show
        Nils Lundquist added a comment - - edited I'll add a comment to the TLD of the pagination property clarifying that disabling it doesn't alter the current table position.
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: