Uploaded image for project: 'Maven Doxia'
  1. Maven Doxia
  2. DOXIA-733

SinkWrapper must allow overwriting legacy Doxia 1.0 methods

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Invalid
    • 2.0.0-M9
    • None
    • Core
    • None

    Description

      Currently SinkWrapper extends AbstractSink. The latter declares lots of Doxia 1.0 methods as final though (and just delegates to the Doxia 1.1/2.0 equivalent).
      While this is fine for regular Sink implementations it does not work for a SinkWrapper (which is not involved when the delegation takes place).

      Therefore those methods must not be final in SinkWrapper.

      Attachments

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            kwin Konrad Windszus
            kwin Konrad Windszus
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment