Uploaded image for project: 'Kuali Rice Development'
  1. Kuali Rice Development
  2. KULRICE-1763

SpringLoader doesn't support kew.additionalSpringFiles param in rice-config.xml

    Details

    • Rice Module:
      KEW
    • Application Requirement:
      Rice

      Description

      The current implementation of edu.iu.uis.eden.SpringLoader does not support the loading of extra Spring Beans files via the kew.additionalSpringFiles parameter set in a rice-config.xml file.

      Essentially, the path of the existing KEW spring beans file is being concatenated with the additional path. SpringLoader is then looking for a file that will not exist and will throw an exception, halting the application.

      This prevents anyone interested from overriding existing services in a standalone rice server installation (as an example).

        Attachments

          Activity

          Hide
          enricopulatzo Josh Peters (Inactive) added a comment -

          This is a patch I've written to address the issue. It's in the unix diff format.

          Show
          enricopulatzo Josh Peters (Inactive) added a comment - This is a patch I've written to address the issue. It's in the unix diff format.
          Hide
          agodert Aaron Godert (Inactive) added a comment -

          Thanks Josh. We'll have someone take a look at this.

          Show
          agodert Aaron Godert (Inactive) added a comment - Thanks Josh. We'll have someone take a look at this.
          Hide
          agodert Aaron Godert (Inactive) added a comment -

          Seems like one for you to analyze Eric and decide who should get it. Thanks.

          Show
          agodert Aaron Godert (Inactive) added a comment - Seems like one for you to analyze Eric and decide who should get it. Thanks.
          Hide
          ewestfal Eric Westfall added a comment -

          Thanks for the patch! Assigning this one to Craig.

          Craig, please apply this patch to 0.9.3 and write a unit test to verify.

          Show
          ewestfal Eric Westfall added a comment - Thanks for the patch! Assigning this one to Craig. Craig, please apply this patch to 0.9.3 and write a unit test to verify.
          Hide
          cdang Chi-Thanh Dang (Inactive) added a comment -

          Applied the code in SpringLoader.patch.

          Show
          cdang Chi-Thanh Dang (Inactive) added a comment - Applied the code in SpringLoader.patch.

            People

            • Assignee:
              cdang Chi-Thanh Dang (Inactive)
              Reporter:
              enricopulatzo Josh Peters (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 30 minutes
                30m
                Remaining:
                Remaining Estimate - 30 minutes
                30m
                Logged:
                Time Spent - Not Specified
                Not Specified