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

[configuration] Instance specific list delimiter

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: Nightly Builds
    • Fix Version/s: 1.4
    • Component/s: None
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Bugzilla Id:
      35828

      Description

      The delimiter used to split the properties is declared as a static field in
      AbstractConfiguration. That means we cannot have two instances of the same
      concrete implementation of AbstractConfiguration using a different delimiter.

      To adress this issue I suggest the following changes:

      • rename the private DELIMITER field into DEFAULT_LIST_DELIMITER
      • add the static methods setDefaultListDelimiter and getDefaultListDelimiter
      • deprecate get/setDelimiter in AbstractConfiguration in favor of
        get/setDefaultListDelimiter
      • add a non static field listDelimiter initialized with DEFAULT_LIST_DELIMITER
      • add the getListDelimiter and setListDelimiter methods to access the
        listDelimiter field
      • call getListDelimiter() instead of getDelimiter()

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                ebourg Emmanuel Bourg
              • Votes:
                1 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: