On Fri, Jan 27, 2012 at 11:30 AM, Jukka Zitting <[email protected]> wrote: > On Fri, Jan 27, 2012 at 10:35 AM, Berry van Halderen > <[email protected]> wrote: > That should still be the case. The > NodeTypeRegistry.unregisterNodeTypes() method calls the protected > checkForReferencesInContent() method before actually removing the > type. The default implementation of checkForReferencesInContent() > simply throws a RepositoryException with a "not yet implemented" > message, which in practice prevents any node types from being > unregisted.
Which is the problem actually, the throw statement in checkForReferencesInContent() is commented out in the 2.2.x versions. Small investigation shows that some fix for JCR-2587 inadvertently contained this change, which was reverted (much) later on the trunk, but after the 2.1.x and 2.2.x branch were made. Hence this will work on trunk and 2.3.x, but not on 2.1.x and 2.2.x branches. Though not a big issue, I'd like to merge this change also back into the 2.1.x and 2.2.x branches (unless objections, I'll do this). \Berry
