[jira] [Commented] (JCRVLT-551) Set jcr:uuid to value from package when overwriting referenceable node

["Konrad Windszus (Jira)" <ji...@apache.org>]

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

Konrad Windszus commented on JCRVLT-551:
----------------------------------------

Also using {{ImportUUIDBehavior.IMPORT_UUID_COLLISION_REMOVE_EXISTING}} is dangerous when the node did not exist before: https://github.com/apache/jackrabbit-filevault/blob/ac74458a0226eb503d96cf2b238cec78b6f36dc2/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXImporter.java#L1022. I would rather recommend to always use https://docs.adobe.com/content/docs/en/spec/jcr/2.0/11_Import.html#11.8.4%20Throw%20on%20Identifier%20Collision.

> Set jcr:uuid to value from package when overwriting referenceable node
> ----------------------------------------------------------------------
>
>                 Key: JCRVLT-551
>                 URL: https://issues.apache.org/jira/browse/JCRVLT-551
>             Project: Jackrabbit FileVault
>          Issue Type: Improvement
>          Components: vlt
>    Affects Versions: 3.5.0
>            Reporter: Konrad Windszus
>            Priority: Major
>
> Currently the code at https://github.com/apache/jackrabbit-filevault/blob/ac74458a0226eb503d96cf2b238cec78b6f36dc2/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXImporter.java#L809 does not try to adjust the {{jcr:uuid}} property in case a referenceable node is imported from a package which already exists with a different {{jcr:uuid}} in the repository.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)