feat(UserPlugin): Include teams in group search

[pymnh]

• (Partly) Resolves: Autocompletion for fellow team members and known users #58465

Summary

• Extend UserPlugin: If autocompletion is enabled for groups and the teams interface is available, also search in teams.

Changes:

• Update the private getTeamsForUser() function to return a dict (user id => display name) instead of only user id
• Add the updated getTeamsForUser() and the preexisting hasTeamSupport() to the public ITeamManager class
• Previously there was only one usage of the private getTeamsForUser() -> Adjust it to the new return value (dict instead of array), use the public ITeamInterface instead of TeamInterface and adjust the mocks for testing.

TODO

• There does not seem to be unit tests for UserPlugin. I am not sure if I should add one to test the behaviour, if so I would need a bit of guidance

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• [-] Screenshots before/after for front-end changes
• [-] Documentation (manuals or wiki) has been updated or is not required
• [-] Backports requested where applicable (ex: critical bugfixes)
• Labels added where applicable (ex: bug/enhancement, 3. to review, feature component)
• [-] Milestone added for target branch/version (ex: 32.x for stable32)

AI

I used AI to adjust the test mocks and comments 🙂

[pymnh]

Hi @mejo- , maybe you can have a look at my changes, since this also addresses the usability of the collectives app?

[pymnh]

Had some trouble setting up cs-fixer and rector with latest Fedora (PHP 8.4), should pass the CI now, I believe. 🙈

[mejo-]

@pymnh can you squash the two commits so "Block unconventional commits" doesn't fail?

[pymnh]

@pymnh can you squash the two commits so "Block unconventional commits" doesn't fail?

done

[mejo-]

Nice work, thanks for the contribution @pymnh 💙

Tested and works as expected.

[mejo-]

@come-nc @CarlSchwan needs a force-merge as some CI jobs didn't run due to the PR being made from a fork.

[come-nc]

I do not have the powers to force-merge, but I second that.