[ 
https://issues.apache.org/jira/browse/HTTPCORE-501?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Gary Gregory updated HTTPCORE-501:
----------------------------------
    Summary: org.apache.hc.core5.net.URLEncodedUtils.parse() should return a 
new ArrayList when there are no query parameters  (was: 
org.apache.http.client.utils.URLEncodedUtils.parse() should return a new 
ArrayList when there are no query parameters)

> org.apache.hc.core5.net.URLEncodedUtils.parse() should return a new ArrayList 
> when there are no query parameters
> ----------------------------------------------------------------------------------------------------------------
>
>                 Key: HTTPCORE-501
>                 URL: https://issues.apache.org/jira/browse/HTTPCORE-501
>             Project: HttpComponents HttpCore
>          Issue Type: Improvement
>          Components: HttpCore
>    Affects Versions: 5.0-beta1
>            Reporter: Gary Gregory
>            Assignee: Gary Gregory
>
> {{org.apache.http.client.utils.URLEncodedUtils.parse(URI, Charset)}} should 
> return a new {{ArrayList}} when there are no query parameters.
> Currently, {{org.apache.http.client.utils.URLEncodedUtils.parse(URI, 
> Charset)}} returns an immutable list through {{Collections.emptyList()}} if 
> there are no params> But if there are parameters, the method returns a 
> mutable list.
> This makes it impossible to write the same code to handle both cases.
> This change will cause the method to return a new {{ArrayList}} if there are 
> no parameters.
> The methods that are directly affected are:
> - {{org.apache.http.client.utils.URLEncodedUtils.parse(URI, Charset)}}
> - {{org.apache.http.client.utils.URLEncodedUtils.parse(HttpEntity)}}
> - {{org.apache.http.client.utils.URLEncodedUtils.parse(String, Charset)}}
> - {{org.apache.http.client.utils.URLEncodedUtils.parse(String, Charset, 
> char...)}}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to