You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Konrad Windszus (Jira)" <ji...@apache.org> on 2021/12/10 09:02:00 UTC

[jira] [Updated] (SLING-10318) Content Loader - Fix merge and overwrite options

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

Konrad Windszus updated SLING-10318:
------------------------------------
    Fix Version/s: JCR ContentLoader 2.5.2
                       (was: JCR ContentLoader 2.5.0)

> Content Loader - Fix merge and overwrite options
> ------------------------------------------------
>
>                 Key: SLING-10318
>                 URL: https://issues.apache.org/jira/browse/SLING-10318
>             Project: Sling
>          Issue Type: Improvement
>          Components: JCR
>    Affects Versions: JCR ContentLoader 2.4.0
>            Reporter: Konrad Windszus
>            Priority: Major
>             Fix For: JCR ContentLoader 2.5.2
>
>
> There are 4 boolean flags which influence the import behaviour:
> # overwrite
> # overwriteProperties
> # merge
> # mergeProperties
> The latter two have been added with SLING-8243. Currently the  behaviour differs between JSON and XML Import and also from the Javadoc 
> the differences between {{overwrite}} and {{merge}} are not really clear (https://github.com/apache/sling-org-apache-sling-jcr-contentloader/blob/master/src/main/java/org/apache/sling/jcr/contentloader/ImportOptions.java).
> For me the flag {{merge}} seems a bit misleading, as the javadoc indicates this will replace all old content (nodes and properties or only nodes). Usually when you refer to a merge you also keep some part of the existing content. It seems that {{merge}} should rather be named something like {{replace}}. WDYT?



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