Kuali Rice Development
  1. Kuali Rice Development
  2. KULRICE-4585

The Rice config system should "trim" whitespace off of "config.location" import statements

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0.3
    • Component/s: Configuration
    • Labels:
      None
    • Similar issues:
      KULRICE-4586The KNS ModuleConfiguration should "trim" whitespace off of incoming data dictionary and database (OJB) file references
      KULRICE-4398Rule Attribute Import does not properly trim input values
      KULRICE-5858Trim the prefix value of whitespace
      KULRICE-3296Unused import statements in KIM API classes create circular dependency for build
      KULRICE-12699Change theme builder JS compiler to use WHITESPACE_ONLY option
      KULRICE-4059Have a config param that turns emails on or off
      KULRICE-1747Ordering of config parameters and nested configs is not respected
      KULRICE-4160update the rice\web\src\main\config\example-config\rice-config.xml
      KULRICE-2934Verify that Rice Config is as streamlined as possible and that the usage of common-config-defaults.xml is working properly
      KULRICE-1227Leverage port config property and push down through to all appropriate spots in config files for KEW, KSB, KEN, etc
    • Rice Module:
      Rice Core
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      If you have something like this in your xml config file:

      <param name="config.location">classpath:app-config.xml
      </param>

      Then the whitespace in there will result in the file not being found properly. Instead a String.trim() should be done on the config.location value to get around this problem. It's a fairly minor issue but can be hard for people to diagnose if they don't know what's going on.

        Issue Links

          Activity

          Hide
          Garey Taylor added a comment -

          updated the jaxbconfigimpl to trim the config.location. updated the unit test to check for this.

          Show
          Garey Taylor added a comment - updated the jaxbconfigimpl to trim the config.location. updated the unit test to check for this.

            People

            • Assignee:
              Garey Taylor
              Reporter:
              Eric Westfall
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Structure Helper Panel