juneau-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (JUNEAU-49) Support beans as arguments for GET calls using @RemoteableMethod
Date Tue, 30 May 2017 01:35:04 GMT

    [ https://issues.apache.org/jira/browse/JUNEAU-49?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16028697#comment-16028697

ASF GitHub Bot commented on JUNEAU-49:

Github user clr-apache commented on a diff in the pull request:

    --- Diff: juneau-core/src/main/java/org/apache/juneau/internal/StringUtils.java ---
    @@ -618,7 +618,9 @@ public static boolean isEmpty(String s) {
     	 * @return <jk>true</jk> if specified string is <jk>null</jk>
or it's {@link #toString()} method returns an empty string.
     	public static boolean isEmpty(Object s) {
    -		return s == null || s.toString().isEmpty();
    +		if( s == null ) return true;
    +		if( s instanceof List ) return ((List) s).size() == 0;
    --- End diff --
    Of course, I missed the semantic that you want to return true if s is null. So you need
the null check in addition to the equals or isEmpty check.
    Too much javascript for me I guess. :(

> Support beans as arguments for GET calls using @RemoteableMethod
> ----------------------------------------------------------------
>                 Key: JUNEAU-49
>                 URL: https://issues.apache.org/jira/browse/JUNEAU-49
>             Project: Juneau
>          Issue Type: Improvement
>            Reporter: Steve Blackmon
> The standard use case for wrapping a GET call with query parameters in a rest proxy is
to enumerate each of the possible get params in order and attach @query to each.
> However, when the number of possible parameters becomes large and many are optional,
the ability for the proxy to extract multiple params from a bean would make implementation
much more readable.
> Enable pojo arguments annotated @Query("\*") or @QueryIfNE("\*”) in a method annotated
@RemoteableMethod(httpMethod="GET") to serve as a source for multiple parameters (all query
parameters for now)

This message was sent by Atlassian JIRA

View raw message