[JCR-3711] RepositoryChecker versioning cleanup may leave repaired node in invalid type state

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.4.5, 2.6.5, 2.7.3
• Fix Version/s: 2.7.4, 2.4.6, 2.6.6
• Component/s: jackrabbit-core
• Assignee: Julian Reschke
• Reporter: Julian Reschke
• Created: 07/Jan/14 13:08
• Updated: 11/Feb/14 08:57
• Resolved: 07/Jan/14 18:16

Description

When detaching a version history, the repair code removes the mix:versionable mixin. However, it leaves the jcr:uuid property alone. If the effective node type of the repaired node does not allow a jcr:uuid property, the node will be in an inconsistent state.

The existing tests in AutoFixCorruptNode did not uncover this because they use the default node type, nt:unstructured, in which case jcr:uuid falls back to a residual property.

The suggested fix (in addition to test inprovements) is to add the mix:referenceable mixin.