Uploaded image for project: 'Wicket'
  1. Wicket
  2. WICKET-194

Enhance CheckBoxSelector to Select CheckBox-es in CheckGroup on Server Side

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.2.3
    • Fix Version/s: 1.3.2
    • Component/s: wicket
    • Labels:
      None

      Description

      Enhance CheckBoxSelector to select the items e.g. users, product line items, etc. shown in the
      paged list view on the visible page and update the CheckGroup with these "visible" items on the server side.

        Activity

        Hide
        sshark TH Lim added a comment -

        RoundTripCheckBoxSelector functions very much like CheckBoxSelector. Unlike CheckBoxSelector, it makes a round trip back to the server to update its "checked" status. It will fill the CheckGroup instance with "visible and checked" Check instances when it is clicked.

        So, for example, within Link.onClick() method, you will be able to get all the selected rows from CheckGroup instance instead of an empty set. If you use PageableListView component, the CheckGroup will only contains objects in the visible page.

        Show
        sshark TH Lim added a comment - RoundTripCheckBoxSelector functions very much like CheckBoxSelector. Unlike CheckBoxSelector, it makes a round trip back to the server to update its "checked" status. It will fill the CheckGroup instance with "visible and checked" Check instances when it is clicked. So, for example, within Link.onClick() method, you will be able to get all the selected rows from CheckGroup instance instead of an empty set. If you use PageableListView component, the CheckGroup will only contains objects in the visible page.
        Hide
        sshark TH Lim added a comment -

        Bug fixed

        Show
        sshark TH Lim added a comment - Bug fixed
        Hide
        almaw Alastair Maw added a comment -

        Need to evaluate if we want this for 1.3.0 - sounds like a basic AJAX behaviour, but need to actually look at the code to see what the problem the reporter is trying to solve.

        Show
        almaw Alastair Maw added a comment - Need to evaluate if we want this for 1.3.0 - sounds like a basic AJAX behaviour, but need to actually look at the code to see what the problem the reporter is trying to solve.
        Hide
        ivaynberg Igor Vaynberg added a comment -

        i just dont see the value of doing this. we have a solution that works with javascript without the extra roundtrip to the server. the proposed solution still uses javascript to initiate the roundtrip, so what really is the advantage here?

        Show
        ivaynberg Igor Vaynberg added a comment - i just dont see the value of doing this. we have a solution that works with javascript without the extra roundtrip to the server. the proposed solution still uses javascript to initiate the roundtrip, so what really is the advantage here?

          People

          • Assignee:
            Unassigned
            Reporter:
            sshark TH Lim
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development