[ 
https://issues.apache.org/jira/browse/LANG-374?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12543279
 ] 

Sebb commented on LANG-374:
---------------------------

The Lang-374-escapeCsv-v2.patch is different from my previous suggested 
addition, in that it allows the escape character (normally double-quote) to be 
specified. My patch was to allow the delimiter - field separator - (comma) to 
be specified.

Probably both are useful, because spreadsheets allow both to be specified when 
importing data.

Unfortunately the patch is broken, because it does not check whether the escape 
character is present in the string ;-)

I've already reworked my patch to rename the method, and to use the 
containsNone fix already applied (by the way, much neater than my version).

I'll add an optional escape character to the mix, and upload shortly.

> Add escaping for CSV columns to StringEscapeUtils
> -------------------------------------------------
>
>                 Key: LANG-374
>                 URL: https://issues.apache.org/jira/browse/LANG-374
>             Project: Commons Lang
>          Issue Type: Improvement
>    Affects Versions: 2.3
>            Reporter: Niall Pemberton
>            Assignee: Niall Pemberton
>            Priority: Minor
>             Fix For: 2.4
>
>         Attachments: Lang-374-escapeCsv-v2.patch, 
> LANG-374-escapeDelimited.patch, StringEscapeUtils.patch, 
> StringEscapeUtilsTest.patch
>
>
> Add escaping for CSV columns to StringEscapeUtils.
> Further information on CSV columns available here:
>     http://en.wikipedia.org/wiki/Comma-separated_values
>     http://tools.ietf.org/html/rfc4180

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

Reply via email to