[sling-org-apache-sling-feature-cpconverter] branch master updated: added minor comments

[si...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

simonetripodi pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-feature-cpconverter.git


The following commit(s) were added to refs/heads/master by this push:
     new 82c8603  added minor comments
82c8603 is described below

commit 82c8603ca9321b92395595f42326aa6ee1344049
Author: stripodi <st...@simos-mbp>
AuthorDate: Thu May 9 14:55:13 2019 +0200

    added minor comments
---
 .../org/apache/sling/feature/cpconverter/acl/DefaultAclManager.java | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/main/java/org/apache/sling/feature/cpconverter/acl/DefaultAclManager.java b/src/main/java/org/apache/sling/feature/cpconverter/acl/DefaultAclManager.java
index 62dda31..c4e2d7b 100644
--- a/src/main/java/org/apache/sling/feature/cpconverter/acl/DefaultAclManager.java
+++ b/src/main/java/org/apache/sling/feature/cpconverter/acl/DefaultAclManager.java
@@ -101,7 +101,13 @@ public final class DefaultAclManager implements AclManager {
 
                 if (preProvidedSystemUsers.contains(systemUser)) {
                     List<Acl> authorizations = currentAcls.getValue();
+
+                    // make sure all paths are created first
+
                     addPaths(authorizations, packageAssembler, formatter);
+
+                    // finally add ACLs
+
                     addAclStatement(formatter, systemUser, authorizations);
                 }
             }