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

        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().
        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.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: