Uploaded image for project: '[Read Only] - Hippo Repository'
  1. [Read Only] - Hippo Repository
  2. REPO-906

Remove (unused) ValueMap from API, simplify and unit test RepositoryMapImpl

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Normal
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.26.00
    • Component/s: None
    • Labels:
    • Sprint:
      Sprint 76

      Description

      The value map seems to offer little value and indeed is not used in the repository itself.

      The repository map has a lot of intricate logic for hierarchy resolution and query based configuration. AFAICS this logic is not used. It is certainly not subject to unit tests. It also does not respect the basic Map contract. (the #entrySet does not contain properties, while they are resolved by #get, for instance)

      My proposal is to simplify the repository map implementation, such that it does implement a (read-only) Map.

      Let's deprecate the ValueMap and remove it in a later release.

        Attachments

          Activity

            People

            Assignee:
            fvlankvelt Frank van Lankvelt (Inactive)
            Reporter:
            fvlankvelt Frank van Lankvelt (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: