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

KIM Group maintenance screens allow creation of circular group memberships

    Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0.3
    • Component/s: User Interface
    • Labels:
      None
    • Similar issues:
      KULRICE-97Add paging for large role/group memberships on KIM documents
      KULRICE-4210Create new Group link needs to go directly to Group creation screen.
      KULRICE-12285Group member with future "Active From Date" not shown on KIM screens
      KULRICE-14201REST-ful api for KIM Groups
      KULRICE-2357KIM Group Service Test
      KULRICE-7613New DB index for KIM group membership
      KULRICE-4575IM Person doc should only allow editing of direct group memberships
      KULRICE-8377Group KIM Type lookup returning too many results
      KULRICE-5175Implement master/detail grouping needed for KIM screens
      KULRICE-8712Kim group document returns exception when submitting group with empty namespace
    • Rice Module:
      KIM
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      KIM Group maintenance screens allow creation of circular group memberships.
      Need to add a check for this in GroupDocumentMemberRule.

      GroupUpdateServiceImpl properly checks to prevent this. Need the KIM UI screens to also prevent this.

        Activity

        Hide
        Daniel Seibert (Inactive) added a comment -

        Also need to ensure GroupServiceImpl getMember... methods handle a circular group membership gracefully if one should exist.

        Show
        Daniel Seibert (Inactive) added a comment - Also need to ensure GroupServiceImpl getMember... methods handle a circular group membership gracefully if one should exist.
        Hide
        Daniel Seibert (Inactive) added a comment -

        Modified GroupDocumentMemberRule to check for circular group membership, and return error if assignment would create one.
        Enhanced GroupServiceImpl to handle circular memberships should they exist.
        Added Unit Test to verify GroupService getMember... methods against a circular group membership scenario.

        Show
        Daniel Seibert (Inactive) added a comment - Modified GroupDocumentMemberRule to check for circular group membership, and return error if assignment would create one. Enhanced GroupServiceImpl to handle circular memberships should they exist. Added Unit Test to verify GroupService getMember... methods against a circular group membership scenario.

          People

          • Assignee:
            Daniel Seibert (Inactive)
            Reporter:
            Daniel Seibert (Inactive)
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 2 days
              2d
              Remaining:
              Remaining Estimate - 2 days
              2d
              Logged:
              Time Spent - Not Specified
              Not Specified

                Structure Helper Panel