[
https://issues.apache.org/jira/browse/JCRVLT-415?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17045305#comment-17045305
]
Konrad Windszus edited comment on JCRVLT-415 at 2/26/20 9:40 AM:
-----------------------------------------------------------------
IMHO the validation error message is correct, as this package will never modify
an existing {{/test/foo}} node. This is rather an issue of the exporter that
the primaryType is not included for nt:folders!. Just check by first installing
my package [^test2.zip] and afterwards install yours. You can see that test2
sets the nodetype of {{/test/foo}} to {{nt:unstructured}} with the arbitrary
property {{test}}. Neither the nodetype is modified not the property {{test}}
is removed once you install your package [^test.zip] afterwards. This is
really dangerous! Compare also with JCRVLT-251.
was (Author: kwin):
IMHO the validation error message is correct, as this package will never modify
an existing {{/test/foo}} node. This is rather an issue of the exporter that
the primaryType is not included for nt:folders!. Just check by first installing
my package [^test2.zip] and afterwards install yours. You can see that test2
sets the nodetype of {{/test/foo}} to {{nt:unstructured}} with the arbitrary
property {{test}}. Neither the nodetype is modified not the property {{test}}
is removed once you install your package [^test.zip] afterwards. This is
really dangerous!
> jackrabbit-emptyelements validator reports error for nested folders
> -------------------------------------------------------------------
>
> Key: JCRVLT-415
> URL: https://issues.apache.org/jira/browse/JCRVLT-415
> Project: Jackrabbit FileVault
> Issue Type: Bug
> Components: vlt
> Affects Versions: 3.4.2
> Reporter: Csaba Varga
> Priority: Minor
> Attachments: test.zip, test2.zip
>
>
> When an nt:folder node is present under an orderable node, the
> "jackrabbit-emptyelements" validator will report an issue (breaking the build
> with the default settings of the package Maven plugin). I believe this
> shouldn't be reported as an issue because the AEM Package Manager can
> generate packages like this. For example, see the attached test.zip file,
> which was exported on an out-of-the-box AEM 6.5 instance with dummy nodes.
> To reproduce, save test.zip somewhere, then invoke the validation Maven goal
> in the command like like this:
> {{mvn org.apache.jackrabbit:filevault-package-maven-plugin:validate-package
> -Dvault.packageToValidate=test.zip}}
> test.zip contains the following very simple node hierarchy:
> {{/}}
> {{ test (nt:unstructured)}}
> {{ foo (nt:folder)}}
> {{ bar (nt:folder)}}
> {{ baz (sling:OrderedFolder)}}
> The error I'm getting is:
> {{[ERROR] ValidationViolation: "jackrabbit-emptyelements: Found empty nodes:
> '/test/foo' (in 'test\.content.xml') (used for ordering only) which are
> included in the filter with mode=merge. Rather use the according
> include/exclude patterns."}}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
