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 Improvement
    • Status: Closed Closed
    • Priority: Major 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
    • Similar issues:
      KULRICE-4215Need a member "Inactivate" button that will automatically inactivate group/role/etc members
      KULRICE-4820ActionRequestServiceImpl.saveActionRequest does not allow for documents to route to inactive groups.
      KULRICE-12921UIDocumentService.loadRoleMembers yields IllegalArgumentException when all members are inactive
      KULRICE-12284Issue routing to a role which has an empty group member
      KULRICE-13968Reopening or reloading a role document can cause unmodified members to disappear
      KULRICE-4575IM Person doc should only allow editing of direct group memberships
      KULRICE-4852Kim Role search returns roles for "member" (group and principals) that are inactive
      KULRICE-8790Role lookup needs a way to search for all roles of which a given role is a member
      KULRICE-4153Inactive qualifiers cause problems on existing roles
      KULRICE-8456When the 'Member Identifier' field on Role document is left blank and the 'Type Code' is equal to Group or Role - receive incident report
    • 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

        Activity

        Eric Westfall made changes -
        Field Original Value New Value
        Summary If editing groups or roles, it validates the existing of all members before saving the doc, even those which are "inactive" If editing groups or roles, it validates the existence of all members before saving the doc, even those which are "inactive"
        Eric Westfall made changes -
        KTI Review Status Not Required Review Completed
        Jessica Coltrin (Inactive) made changes -
        Fix Version/s 2.1 [ 15812 ]
        Jessica Coltrin (Inactive) made changes -
        Start Date
        Fix Date 2012-05-07 [ set to sprint end date ]
        Jeremy Hanson made changes -
        Assignee Jeff Ruch [ jruch ]
        Jeff Ruch made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Jeff Ruch made changes -
        Status In Progress [ 3 ] Open [ 1 ]
        Jeff Ruch made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Jeff Ruch made changes -
        Status In Progress [ 3 ] Open [ 1 ]
        Hide
        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
        Jeff Ruch added a comment - The changes are complete. A review has been created. Once the review is complete I will commit the code.
        Jessica Coltrin (Inactive) made changes -
        Fix Version/s 2.1.0-m2 [ 16460 ]
        Jessica Coltrin (Inactive) made changes -
        Start Date
        Fix Date 2012-05-07 2012-04-02 [ set to sprint end date ]
        Hide
        Jeff Ruch added a comment -

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

        Show
        Jeff Ruch added a comment - Tested on sample app. test identityService and identityRemoteService unit tests. Committed changes.
        Jeff Ruch made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Jessica Coltrin (Inactive) made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Shem Patterson (Inactive) made changes -
        Workflow custom [ 134393 ] Copy of custom for rice [ 213965 ]
        Shem Patterson (Inactive) made changes -
        Workflow Copy of custom for rice [ 213965 ] custom [ 223713 ]
        Shem Patterson (Inactive) made changes -
        Workflow custom [ 223713 ] Rice Workflow [ 233461 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Structure Helper Panel