Uploaded image for project: 'Couchbase Lite'
  1. Couchbase Lite
  2. CBL-1727

Improved naming for AbstractReplicatorConfiguration.ReplicatorType

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Critical
    • Resolution: Fixed
    • 3.0
    • Helium
    • Java, Java-Android
    • Security Level: Public
    • 1

    Description

      Currently, the replicator type is an attribute of AbstractReplicatorConfiguration. There are three problems with this:
      1) it means that AbstractReplicatorConfiguration is public, part of our API, to support Kotlin.
      2) the property, a property of the Replicator, is illogically associated with the ReplicatorConfiguration
      3) the name is ridiculously long.

      Add a new enum, Type, to AbstractReplicator, replacing the old enum and deprecate the old enum.

      Attachments

        Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            blake.meike Blake Meike added a comment -

            Fixed incouchbase-lite-java-ee-root @ 8a0e9be8946c1a0e

            blake.meike Blake Meike added a comment - Fixed incouchbase-lite-java-ee-root @ 8a0e9be8946c1a0e
            blake.meike Blake Meike added a comment - - edited

            This must stay open until the deprecated type is removed, in v4.x

            blake.meike Blake Meike added a comment - - edited This must stay open until the deprecated type is removed, in v4.x
            blake.meike Blake Meike added a comment -

            The new types exist and the old ones have been deprecated.  CBL-3963 tracks removing the deprecated types in the 4.x release

            blake.meike Blake Meike added a comment - The new types exist and the old ones have been deprecated.   CBL-3963 tracks removing the deprecated types in the 4.x release

            People

              blake.meike Blake Meike
              blake.meike Blake Meike
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty