Uploaded image for project: 'Commons Configuration'
  1. Commons Configuration
  2. CONFIGURATION-562

Improve API of ExprLookup class

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.10
    • Fix Version/s: 2.0
    • Component/s: Interpolation
    • Labels:
      None

      Description

      ExprLookup can be assigned an AbstractConfiguration object. This is a bit smelly because here a concrete implementation rather than the generic Configuration interface is used.

      In fact, the class only uses the ConfigurationInterpolator and the logger from its associated configuration object. So it would be clearer (and more loosely coupled) to define properties for these objects rather than having an associated, full-blown AbstractConfiguration object.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              oheger Oliver Heger
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: