![]() |
|
|
|
|
Issue Type:
|
![Bug]() Bug
|
|
Assignee:
|
Unassigned |
|
Created:
|
25/Feb/13 10:16 PM
|
|
Description:
|
Countless times, I wrote utility methods, or tests, to validate a usage of some split method. Typically, we need to split comma-separated value input strings.
A quick check on Magnolia-core reveals inconsistent and error-prone usage of several different methods: grep -ri split magnolia-core/
None of these usages really provide a safe output; in some cases we trim the input, but in most cases an input of " a, b,c" will yield different results than "a,b,c", and that is most likely not desired.
Here's a proposal to move a utility class I wrote for the external-indexer module, along its extensive unit tests. If this goes through, you'll certainly want to check the current usages of StringUtils.split() and String.split() !
g...@git.magnolia-cms.com:forge/external-indexing
|
|
Project:
|
Magnolia
|
|
Priority:
|
![Neutral]() Neutral
|
|
Reporter:
|
Grégory Joseph
|
|
|
|
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
|
----------------------------------------------------------------
For list details, see: http://www.magnolia-cms.com/community/mailing-lists.html
Alternatively, use our forums: http://forum.magnolia-cms.com/
To unsubscribe, E-mail to: <dev-list-unsubscr...@magnolia-cms.com>
----------------------------------------------------------------
