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

Stojce Dimski updated HTTPCLIENT-746:
-------------------------------------

    Attachment: URLEncodedUtils2.zip

I took a step back end refactored URLEncodingUtils in a separated module using 
a NameValuePair instead of Map<String, List<String>> for the sake of 
consistency... In alleged patch you will find also a unit tests for all those 
methods...
Cleaned up URLUtils contains just a methods for URI manipulation and I would 
propose rename in URIUtils, at this moment URIUtils methods are used in just 2 
classes in total of 5 points of code so the change would be very easy...
I also adjusted UrlEncodedFormEntity, and would like to propose a move of this 
class together with 'mime brother' into a package o.a.h.client.entity instead 
of leaving it in non intuitive package 'methods'...


> substitute for URLUtils.java
> ----------------------------
>
>                 Key: HTTPCLIENT-746
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-746
>             Project: HttpComponents HttpClient
>          Issue Type: Improvement
>          Components: HttpClient
>            Reporter: Stojce Dimski
>            Priority: Minor
>             Fix For: 4.0 Alpha 4
>
>         Attachments: urlEncoded.patch, UrlEncodedUtils.zip, 
> URLEncodedUtils2.zip
>
>
> I would like to contribute a substitute for existing URLUtils.java... 
> UrlEncodedUtils.java offer utility methods for dealing with 'urlencoded' 
> data. Main difference with existing class is that parameters are Map <String, 
> List <String>> instead of NameValue pairs and lack of third party 
> dependencies...  It's partially covered with tests which I will further 
> extend to cover all methods of this utility class.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to