Uploaded image for project: 'Magnolia'
  1. Magnolia
  2. MAGNOLIA-6615

UserManager#getUsersWithGroup(groupName, transitive) is b0rked, never worked

    XMLWordPrintable

Details

    • Yes
    • Saigon 39
    • 8

    Description

      UserManager#getUsersWithGroup(groupName, transitive) is just b0rked and most likely never worked properly...

      Considering user/group setup from_templating-samples_, as follows:

      • (group) employees
        • (user) eve
        • (user) patrick
        • (group) developers
          • (user) david

      UserManager#getUsersWithGroup("employees", true) doesn't return david (wrong)
      whereas UserManager#getUser("david").getAllGroups() correctly returns developers + employees (correct)

      Concept: Implementing only part https://wiki.magnolia-cms.com/display/DEV/Concept+-+Improvements+in+the+security+package#Concept-Improvementsinthesecuritypackage-1a.super-groupsvs.sub-groups
      Solution: Provide new API to return all sub-groups MgnlGroupManager#getAllSubGroups and use that API to resolve all users transitively-assigned sub-groups correctly

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

                oanh.thai Oanh Thai Hoang
                mgeljic Mikaël Geljić
                Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Bug DoR
                    Task DoD

                    Time Tracking

                      Estimated:
                      Original Estimate - 4d Original Estimate - 4d
                      4d
                      Remaining:
                      Remaining Estimate - 0d
                      0d
                      Logged:
                      Time Spent - 5d 5h
                      5d 5h