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

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.9
    • 2.0, 1.10
    • None
    • 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

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

            Dates

              Created:
              Updated:
              Resolved: