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

Karl Pauls resolved SLING-9959.
-------------------------------
    Resolution: Fixed

> SystemUser.getPath must reveal the path of the original user node
> -----------------------------------------------------------------
>
>                 Key: SLING-9959
>                 URL: https://issues.apache.org/jira/browse/SLING-9959
>             Project: Sling
>          Issue Type: Bug
>          Components: Content-Package to Feature Model Converter
>            Reporter: Angela Schreiber
>            Assignee: Karl Pauls
>            Priority: Major
>             Fix For: Content-Package to Feature Model Converter 1.0.26
>
>
> -found the following repo-init statement to create system users:-
> -[https://github.com/apache/sling-org-apache-sling-feature-cpconverter/blob/master/src/main/java/org/apache/sling/feature/cpconverter/acl/DefaultAclManager.java#L109]-
> -which states:-
> {code:java}
> formatter.format("create service user %s with path %s%n", systemUser.getId(), 
> systemUser.getPath());
> {code}
> -now.... i might be mistaken, but if my reading of the {{SystemUserParser}} 
> is correct, the path defined with the {{SystemUser}} instance actually 
> reflects the path of the system user node itself:-
> -[https://github.com/apache/sling-org-apache-sling-feature-cpconverter/blob/master/src/main/java/org/apache/sling/feature/cpconverter/handlers/SystemUsersEntryHandler.java#L70-L75]-
> -if that was true the repo-init statements would be wrong, because the path 
> argument doesn't specify that path of the user node itself but rather the 
> intermediate path to be used.-
> -so the statement should probably read:-
> {code:java}
> formatter.format("create service user %s with path %s%n", systemUser.getId(), 
> systemUser.getParent().getPath());
> {code}
>  



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

Reply via email to