[OAK-10784] o.a.j.o.plugins.migration.version.VersionableEditor should create the version storage node, if needed

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.64.0
• Component/s: core
• Labels: candidate_oak_1_22
• Assignee: Manfred Baedke
• Reporter: Manfred Baedke
• Created: 02/May/24 13:15
• Updated: 27/May/24 19:51
• Resolved: 08/May/24 10:33

Description

The method VersionableEditor#createEmptyHistory() is called if the migration of versions is disabled, but the node type of a migrated node strictly requires it to be versionable, because it inherits from mix:versionable (so that mix:versionable can't simply be removed). This may fail if the version storage node has not been explicitly created before the migration. Just creating the necessary node would be a more straightforward behavior.
Also we should change the name of that method, see OAK-10783.

Issue Links

• relates to: OAK-10783 VersionableEditor: misleading WARN message when creating version history
• links to: GitHub Pull Request #1438