Uploaded image for project: 'Jackrabbit Oak'
  1. Jackrabbit Oak
  2. OAK-3522

DefaultSyncContext exposes internal path-utility method

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

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.2.19, 1.3.11, 1.4
    • auth-external
    • None

    Description

      the DefaultSyncContext defines a public static method joinPaths, which IMHO should not be exposed as it is unrelated to the functionality defined by the DefaultSyncContext.

      i would therefore suggest to make it private (and increase the package version to 2.0.0)... we can then still decide to move this utility to PathUtils in oak-commons.

      Attachments

        1. OAK-3522.patch
          2 kB
          Angela Schreiber
        2. OAK-3522_3.patch
          6 kB
          Angela Schreiber
        3. OAK-3522_2.patch
          6 kB
          Angela Schreiber

        Issue Links

        is related to

        Improvement - An improvement or enhancement to an existing feature or task. OAK-2948 Expose DefaultSyncHandler

        • Major - Major loss of function.
        • Closed

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

          People

            angela Angela Schreiber
            angela Angela Schreiber
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment