[ 
https://issues.apache.org/jira/browse/SLING-10986?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17456252#comment-17456252
 ] 

Konrad Windszus edited comment on SLING-10986 at 12/9/21, 9:12 AM:
-------------------------------------------------------------------

Although a change here will in some cases might have an undesired effect 
(because more node are potentially removed), I would still rather fix the 
implementation than the documentation, because only clearing the nodes below 
target is IMHO unintuitive and requires awkward bundle headers e.g. in case you 
really want to overwrite the path like "/mypath" fully which would force you to 
use {{path:=/}} which also conflicts with the proposal from SLING-10983.


was (Author: kwin):
Although a change here will in some cases might have an undesired effect 
(because more node are potentially removed), I would still rather fix the 
implementation than the documentation, because only clearing the nodes below 
target is IMHO unintuitive and requires awkward bundle headers e.g. in case you 
really want to overwrite the path like "/mypath" fully.

> Sling-Initial-Content overwrite never touches target path but only nodes below
> ------------------------------------------------------------------------------
>
>                 Key: SLING-10986
>                 URL: https://issues.apache.org/jira/browse/SLING-10986
>             Project: Sling
>          Issue Type: Bug
>    Affects Versions: JCR ContentLoader 2.4.2
>            Reporter: Konrad Windszus
>            Assignee: Konrad Windszus
>            Priority: Major
>
> Contrary to the documentation at 
> https://sling.apache.org/documentation/bundles/content-loading-jcr-contentloader.html
>  which states that
> bq. The overwrite directive specifies if content nodes should be overwritten 
> (at the target repository path, which is "/" by default) or just initially 
> added. If this is true, existing nodes are deleted and a new node is created 
> in the same place. 
> only nodes *_below_* the target path are currently touched but never the node 
> at target path. 



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to