ICEfaces
  1. ICEfaces
  2. ICE-2024

Row selector attribute multiple = false not enforced across multiple pages

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6
    • Fix Version/s: 1.6.2, 1.7DR#1, 1.7
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      When using row selection within a datatable that has a paginator, if multiple = false for the row selector, one row is selectable per paginator page. However, this should be enforced across multiple pages. For example, if a user selects a row on the first page, paginates forward and then selects a new row, the previously selected row is not being unselected.

        Activity

        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Yip Ng [ yip.ng ]
        Ken Fyten made changes -
        Fix Version/s 1.7 [ 10080 ]
        yip.ng made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #14991 Fri Oct 19 15:27:07 MDT 2007 yip.ng ICE-2024
        Changed to clear row selection for the whole table, not just the currently displayed page.
        Files Changed
        Commit graph MODIFY /icefaces/branches/icefaces-1.6/icefaces/component/src/com/icesoft/faces/component/ext/RowSelector.java
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Ken Fyten made changes -
        Fix Version/s 1.6.2 [ 10111 ]
        Ken Fyten made changes -
        Fix Version/s 1.7DR#1 [ 10100 ]
        Fix Version/s 1.7 [ 10080 ]
        yip.ng made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        yip.ng added a comment -

        Original code logic would only clear selections on the currently displayed page. Added/Changed code to retrieve the rest of the table rows and clear them all.

        Show
        yip.ng added a comment - Original code logic would only clear selections on the currently displayed page. Added/Changed code to retrieve the rest of the table rows and clear them all.
        Ken Fyten made changes -
        Assignee Adnan Durrani [ adnan.durrani ] Yip Ng [ yip.ng ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #14835 Wed Sep 19 15:19:10 MDT 2007 yip.ng ICE-2024
        Changed to clear row selection for the whole table, not just the currently displayed page.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/RowSelector.java
        Ken Fyten made changes -
        Fix Version/s 1.7 [ 10080 ]
        Assignee Yip Ng [ yip.ng ] Adnan Durrani [ adnan.durrani ]
        Adnan Durrani made changes -
        Assignee Adnan Durrani [ adnan.durrani ] Yip Ng [ yip.ng ]
        Ken Fyten made changes -
        Assignee Adnan Durrani [ adnan.durrani ]
        tjohnson made changes -
        Attachment JIRA-2024.rar [ 10639 ]
        Hide
        tjohnson added a comment -

        Example application demonstrating the issue. NOTE: The build.xml and build.properties files will need to be modified based on your configuration.

        Show
        tjohnson added a comment - Example application demonstrating the issue. NOTE: The build.xml and build.properties files will need to be modified based on your configuration.
        tjohnson made changes -
        Field Original Value New Value
        Support Case References https://www.icesoft.ca:4443/supportilla/show_bug.cgi?id=4246
        ICEfaces Forum Reference https://www.icesoft.ca:4443/supportilla/show_bug.cgi?id=4246 http://www.icefaces.org/JForum/posts/list/5437.page#24172
        Description When having a paginator that encloses a datatable which then encloses a rowselector. If multiple = false for the row selector, only one row is selectable per paginator page but it should be enforced across all pages. When using row selection within a datatable that has a paginator, if multiple = false for the row selector, one row is selectable per paginator page. However, this should be enforced across multiple pages. For example, if a user selects a row on the first page, paginates forward and then selects a new row, the previously selected row is not being unselected.
        Tyler Johnson created issue -

          People

          • Assignee:
            Unassigned
            Reporter:
            Tyler Johnson
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: