click-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Joseph Schmidt (JIRA)" <j...@apache.org>
Subject [jira] Commented: (CLK-536) setRedirect() to support parameters too.
Date Mon, 27 Apr 2009 20:27:30 GMT

    [ https://issues.apache.org/jira/browse/CLK-536?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12703342#action_12703342
] 

Joseph Schmidt commented on CLK-536:
------------------------------------

Thank you for the patch :).

For the case of a single parameter, I use this pattern now:
setRedirect(DetailPage.class, new HashMap(){{put("customer", id);}});

It not that nice(Java doesn't have a nice syntax for HashMaps like other popular languages),
 but it's one line only, and the IDE helps complete it very quickly.

> setRedirect() to support parameters too.
> ----------------------------------------
>
>                 Key: CLK-536
>                 URL: https://issues.apache.org/jira/browse/CLK-536
>             Project: Click
>          Issue Type: New Feature
>            Reporter: Joseph Schmidt
>         Attachments: setRedirect_with_params.diff
>
>
> Please extend the setRedirect() methods to support parameters too.
> Right now it is possible to use parameters only by manually creating the redirect location(and
using the String version of the method). This is however very error prone.
> Please extend the two existing methods:
> setRedirect(String location) and
> setRedirect(Class pageClass)
> with a "params" parameter, e.g.
> setRedirect(String location, Map params) and 
> setRedirect(Class pageClass, Map params
> to work for the very required case when the destination requires parameters.
> thanks,
> Joseph.

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


Mime
View raw message