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

PropertyConfiguration's handling of includes depends on the existence of a base path

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9
    • Fix Version/s: 2.0, 1.10
    • Component/s: None
    • Labels:
      None

      Description

      PropertiesConfiguration has a property which controls whether include files are allowed. This property does not have a set method, it is set to true automatically if a base path is set. There may be scenarios in which it makes sense to support include files anyway, e.g. if they are specified as fully qualified file names or URIs or can be retrieved by the standard lookup mechanism.

      So the set method should be public, and it makes sense to set this property to true per default. If a file contains an include property, the files it refers to should propably be loaded.

      Also the get method for the property which controls file inclusion does not conform to the Java Beans specification; returning a boolean, it should start with the prefix is.

        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: