[ 
https://issues.apache.org/jira/browse/SLING-5455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15123677#comment-15123677
 ] 

Robert Munteanu commented on SLING-5455:
----------------------------------------

Adding it to the API bundle sounds fine to me - I initially did that but 
switched to a new bundle on [~bdelacretaz]'s suggestion. So I would be OK with 
adding it back to the API bundle, if that's something everyone can live with.

About the package name, I think .util is fine. We can also consider .path and 
.text and consolidate other utilities in there. For instance, we can add [the 
Trie that we have in the validation 
module|https://github.com/apache/sling/blob/e8264c737e8fbb456279be4ec34fa4c1f6128938/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/util/Trie.java]
 .

> Add helper class to construct valid paths
> -----------------------------------------
>
>                 Key: SLING-5455
>                 URL: https://issues.apache.org/jira/browse/SLING-5455
>             Project: Sling
>          Issue Type: New Feature
>          Components: Extensions
>            Reporter: Robert Munteanu
>            Assignee: Robert Munteanu
>            Priority: Minor
>             Fix For: Commons 1.0.0
>
>
> When working with paths it's easy to combine two valid paths into an invalid 
> one. For instance
> * '/parent/' + '/child' -> '/parent//child'
> * '/parent' + 'child' -> '/parentchild'
> We should add a simple helper class which allows to build paths, with a 
> fluent syntax.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to