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.

[github-actions]

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)