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

[configuration] Instance specific list delimiter

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • Nightly Builds
    • 1.4
    • None
    • None
    • Operating System: All
      Platform: All

    • 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

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

              Dates

                Created:
                Updated:
                Resolved: