[ https://issues.apache.org/jira/browse/OAK-285?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13443220#comment-13443220 ]
Michael Dürig commented on OAK-285: ----------------------------------- Yes I know. But it buries validating somewhat which uses to confuse people. Also it does not guarantee that validating happens only when editing is done. > Split CommitEditor into CommitEditor and Validator interfaces > ------------------------------------------------------------- > > Key: OAK-285 > URL: https://issues.apache.org/jira/browse/OAK-285 > Project: Jackrabbit Oak > Issue Type: Improvement > Components: core > Reporter: Michael Dürig > > Having {{ValidatingEditor}} extend {{CommitEditor}} is a constant source of > confusion since the former does not edit at all. Since it also makes sense to > run validation only after all editing has taken place, I suggest to introduce > a new interface {{CommitValidator}} which covers the validating aspect of a > commit. Alternatively, we could of course also re-use the {{Observer}} > interface here. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira