[GitHub] [sling-org-apache-sling-distribution-core] tmaret commented on a diff in pull request #66: SLING-11560 : Vault error when importing a node with the same name and UUID as a pre-existing non-sibling node

[GitBox <gi...@apache.org>]

tmaret commented on code in PR #66:
URL: https://github.com/apache/sling-org-apache-sling-distribution-core/pull/66#discussion_r975017294


##########
src/main/java/org/apache/sling/distribution/serialization/impl/vlt/FileVaultContentSerializer.java:
##########
@@ -131,7 +133,7 @@ public void importFromStream(ResourceResolver resourceResolver, InputStream inpu
         Archive archive = null;
         try {
             session = getSession(resourceResolver);
-            ImportOptions importOptions = VltUtils.getImportOptions(aclHandling, cugHandling, importMode, autosaveThreshold, strict);
+            ImportOptions importOptions = VltUtils.getImportOptions(aclHandling, cugHandling, importMode, autosaveThreshold, strict, idConflictPolicy);
             ErrorListener errorListener = new ErrorListener();
             importOptions.setListener(errorListener);
             Importer importer = new Importer(importOptions);

Review Comment:
   Agreed with @kwin point. We'd need to reuse the existing configuration as a base.
   
   @kwin can we get the configuration through an API ? 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@sling.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org