click-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Finn Bock (JIRA)" <>
Subject [jira] Updated: (CLK-558) Binding multi request parameter to Page class field
Date Wed, 30 Jun 2010 19:29:51 GMT


Finn Bock updated CLK-558:

    Attachment: CLK-558-test.patch

Hi Jahid,

Attached a patch to the test suite that tests several cases of multiple parameter.  Some of
them fails with your patch.  I don't know how many of the cases we want to support but I think
we should support arrays of all types that we already supports.

Naoki also mention List<String>, which is possible, but I don't think List<Integer>
is possible, so IMO it would be better to skip collections.

> Binding multi request parameter to Page class field
> ---------------------------------------------------
>                 Key: CLK-558
>                 URL:
>             Project: Click
>          Issue Type: New Feature
>          Components: core
>            Reporter: Naoki Takezoe
>         Attachments: CLK-558-test.patch, CLK-558.patch
> Click can bind request parameters to Page class public fields.
> However it can not bind multi parameter to an array or list field.
> I propose adding multi parameter binding support to 
> ClickServlet#processPageRequestParams()
> For example, here is the HTML which send a multi parameter.
> <input type="checkbox" name="option" value="1"/>
> <input type="checkbox" name="option" value="2"/>
> ...
> Click would bind this parameter to array or list public field
> of the Page class such as...
> public String[] option;
> or
> public List<String> option;

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message