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

If editing groups or roles, it validates the existence of all members before saving the doc, even those which are "inactive"

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.1.0-m2, 2.1
    • Component/s: User Interface
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Rice Module:
      KIM
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Review Completed

      Description

      The main problem here is similar in nature to KULRICE-3307. There are situations where someone's identity might drop out of the system (in the case of LDAP). In theory the historical entity cache ought to help with this, but that might not always be the case. The problem in these situations is that it's not possible to remove the offending principal from the group or member via the UI because it's validating that the membership represents a valid principal. Seems it should only be validating "active" membership.

      This was discussed at the KTI meeting on 02-29-2012: https://wiki.kuali.org/x/5nC1Eg

        Attachments

          Activity

          Hide
          jruch Jeff Ruch added a comment -

          The changes are complete. A review has been created. Once the review is complete I will commit the code.

          Show
          jruch Jeff Ruch added a comment - The changes are complete. A review has been created. Once the review is complete I will commit the code.
          Hide
          jruch Jeff Ruch added a comment -

          Tested on sample app. test identityService and identityRemoteService unit tests. Committed changes.

          Show
          jruch Jeff Ruch added a comment - Tested on sample app. test identityService and identityRemoteService unit tests. Committed changes.

            People

            • Assignee:
              jruch Jeff Ruch
              Reporter:
              ewestfal Eric Westfall
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: