0 Replies Latest reply on Jun 15, 2008 9:37 AM by timfox

    Use the positive for boolean configuration parameters

    timfox

      In our config we were mixing and matching both positive and negatives for configuration parameters e.g.:

      isSSLEnabled
      isInVMDisabled

      This is inconsistent and confusing for the user.

      I have changed these now so they are all expressed in the positive:

      isSSLEnabled
      isInVMOptimisationEnabled

      I have also done some refactoring of the configuration - ConfigurationImpl, FileConfiguration and ConnectionParamsImpl to remove duplicated config, fixed settings which had no defaults, made sure defaults are only in one place (they were in many before) and fixed some settings that weren't being loaded from the config.

      I've also added complete tests for these classes.