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

Services in KIM referencing KRAD dictionary validation service

    Details

    • Type: Bug Fix
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.1.2
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Rice Module:
      KNS
    • Application Requirement:
      KFS
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      Performed a review of dictionary validation service usage with Jonathan and found there are two references from KIM (still running in the KNS) that are using the KRAD dictionary validation service. These need to be changed to use the KNS dictionary validation service:

      org.kuali.rice.kns.datadictionary.validation.AttributeValidatingTypeServiceBase
      org.kuali.rice.kns.kim.type.DataDictionaryTypeServiceBase

      The goal is to not have any KNS code using the constraint logic of KRAD, so they should all be referencing the KNS dictionary validation service. After these two are cleaned up that separation will be achieved.

      Also, just one small cleanup item. KualiRuleServiceImpl has a dependency on DictionaryValidationService but is not actually using it. We should remove this to prevent confusion.

        Attachments

          Issue Links

            Activity

            Hide
            gilesp Peter Giles (Inactive) added a comment - - edited

            Hey Jerry, I've already Jira'd this one up I think (although you may have some additional work identified here too), see the linked issue. If you look at the Hierarchy for the KRAD DocumentRuleBase you'll see what I'm talking about. I had a brief thread with Jonathan about this where we had decided to change these over to the KNS DictionaryValidationService in 2.1.3 to give more time for testing w/ that change before the release (See this comment). Obviously, all the testing that KFS has done up to this point of KIM has been using the KRAD DictionaryValidationService. Do you feel that this is a low risk enough change to make pretty close to the release of 2.1.2?

            Show
            gilesp Peter Giles (Inactive) added a comment - - edited Hey Jerry, I've already Jira'd this one up I think (although you may have some additional work identified here too), see the linked issue. If you look at the Hierarchy for the KRAD DocumentRuleBase you'll see what I'm talking about. I had a brief thread with Jonathan about this where we had decided to change these over to the KNS DictionaryValidationService in 2.1.3 to give more time for testing w/ that change before the release (See this comment ). Obviously, all the testing that KFS has done up to this point of KIM has been using the KRAD DictionaryValidationService. Do you feel that this is a low risk enough change to make pretty close to the release of 2.1.2?
            Hide
            jkneal Jerry Neal (Inactive) added a comment -

            Hi Peter,

            My feeling is the validation of KIM attributes is probably not working correctly, or I don't see how it could be. So I would think we want to get it switched over. But I'll leave it up to the KFS as far as the timing and whether or not they want the change for 2.1.2.

            Jerry

            Show
            jkneal Jerry Neal (Inactive) added a comment - Hi Peter, My feeling is the validation of KIM attributes is probably not working correctly, or I don't see how it could be. So I would think we want to get it switched over. But I'll leave it up to the KFS as far as the timing and whether or not they want the change for 2.1.2. Jerry
            Hide
            jkeller Jonathan Keller added a comment -

            I don't think we are experiencing any blocker issues regarding this for KFS 5.0. So, I would say to, yes, make the fix, but make it on the main 2.1 branch.

            Show
            jkeller Jonathan Keller added a comment - I don't think we are experiencing any blocker issues regarding this for KFS 5.0. So, I would say to, yes, make the fix, but make it on the main 2.1 branch.
            Hide
            sdm86 Steve Manning (Inactive) added a comment -

            Fix for KFS-5.0 spring-rice-krad-overrides.xml file

            Show
            sdm86 Steve Manning (Inactive) added a comment - Fix for KFS-5 .0 spring-rice-krad-overrides.xml file
            Hide
            jcoltrin Jessica Coltrin (Inactive) added a comment -

            closing all 2.1.2 Jiras

            Show
            jcoltrin Jessica Coltrin (Inactive) added a comment - closing all 2.1.2 Jiras

              People

              • Assignee:
                sdm86 Steve Manning (Inactive)
                Reporter:
                jkneal Jerry Neal (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: