[
https://issues.apache.org/jira/browse/SLING-9959?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Karl Pauls closed SLING-9959.
-----------------------------
> 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.1.0
>
>
> -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)
