[
https://issues.apache.org/jira/browse/JCR-2740?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Thomas Mueller updated JCR-2740:
--------------------------------
Attachment: jcr-2740-b.patch
This new patch only fixes the problem when a session attribute is set.
Instead of the ItemManager, it uses the ItemStateManager to check for node
existence.
The new patch fixes the problem when iterating over child nodes, and when
trying to remove a node. That means, it's possible to fix all problems in a
subtree by iterating over it with a session that has this session attribute set.
P.S. I didn't fully test this new patch yet.
> On missing child node, automatically rename entry when trying to add a node
> with the same name
> ----------------------------------------------------------------------------------------------
>
> Key: JCR-2740
> URL: https://issues.apache.org/jira/browse/JCR-2740
> Project: Jackrabbit Content Repository
> Issue Type: New Feature
> Components: jackrabbit-core
> Reporter: Thomas Mueller
> Assignee: Thomas Mueller
> Attachments: jcr-2740-b.patch, jcr-2740.patch
>
>
> If a node points to a non-existing child node (which is a repository
> inconsistency), currently this child node is silently ignored for read
> operations (as far as I can tell). However, when trying to add another child
> node with the same name, an exception is thrown with a message saying a child
> node with this name already exists.
> I suggest to rename the missing child node entry in that case (for example
> add the current date/time, or a random digit until there is no conflict), and
> then continue with adding the new child node. I wouldn't automatically remove
> the bad entry, because the node might "appear" later (after a restore), and
> because removing data from the repository seems wrong.
> It's not a perfect solution, but it might be better than throwing an
> exception and basically preventing changes.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
