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

Pulse badge not updated when using ExternalUserManager

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.4.7, 5.5
    • Component/s: None
    • Labels:
    • Release notes required:
      Yes
    • Sprint:
      Saigon 44
    • Story Points:
      8
    • Magnolia Release:
      5.4.7, 5.5

      Description

      Steps to reproduce

      Assuming we have 2 LDAP accounts: "mary" belongs to group "travel-demo-editors", and "john" belongs to "travel-demo-publishers".

      • Steps to reproduce for Messages app case:
        • Login as "superuser" on Firefox.
        • Login as "mary" on Chrome.
        • On "superuser" side, go to Messages app, send a message to "mary".

          => Expected: "mary" receives that message in Pulse.
          => Actual: She doesn't.
      • Steps to reproduce for publishing case:
        • Login as "mary" on Firefox.
        • Login as "john" on Chrome.
        • On "mary" side, go to Pages app, publish a page.

          => Expected: "john" receives that task in Pulse.
          => Actual: He doesn't.

      Description

      The pulse badge doesn't get updated, when using an ExternalUserManager e.g. LDAPUserManager .
      You have to explicitly click into the pulse-tab, to refresh the pulse. Even reload is not enough to refresh the badge.

      Normally, in MessagesManagerImpl#sendGroupMessage the message is pushed into the pulse of the User. The relevant users are identified by DelegatingUserManager#getUsersWithGroup - but within the method delegateUntilSupported - after the first UserManager returns users (SystemUserManager), the other UserManagers don't get called.

      Solution

      • Provide a default implementation for ExternalUserManager#getUsersWithGroup.
      • Add new implementation for DelegatingUserManager#getUsersWithGroup and DelegatingUserManager#getUsersWithRole to collect users from all user managers.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                oanh.thai Oanh Thai Hoang
                Reporter:
                fuchs.florian Florian Fuchs
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Date of First Response:

                  Time Tracking

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