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

Angela Schreiber resolved SLING-10388.
--------------------------------------
    Fix Version/s: Content-Package to Feature Model Converter 1.1.2
       Resolution: Fixed

> RepoPath.getParent returns null for top-level paths
> ---------------------------------------------------
>
>                 Key: SLING-10388
>                 URL: https://issues.apache.org/jira/browse/SLING-10388
>             Project: Sling
>          Issue Type: Bug
>          Components: Content-Package to Feature Model Converter
>            Reporter: Angela Schreiber
>            Assignee: Angela Schreiber
>            Priority: Major
>             Fix For: Content-Package to Feature Model Converter 1.1.2
>
>
> [~kpauls], as discussed it seem there exists one (or potentially many) bugs 
> in the {{RepoPath}} class as {{new RepoPath("/apps").getParent()}} returns 
> null, which IMHO is unexpected.
> more things i noticed while trying to understand what's going on:
> - toString for a repository path is "/" and not ""
> - equals method ignores isRepositoryPath field
> - getSegmentCount is not used at all
> - however, getSegments does not exist (but would be better in 
> {{DefaultAclManager.getCreatePath}} than the split/substring on the toString 
> method)
> - startswith(null(: comment states that the root path is null which is not 
> what the code does in the constructor.... startswith(null) should IMO return 
> false



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to