Raise for all teams: use display name cache

[nickvergessen]

• Activity providers
• Notification providers
• spreed/lib/Chat/Parser/UserMention.php 94
• comments mentions
• apps/user_status/lib/Dashboard/UserStatusWidget.php

[nickvergessen]

IUserManager::getDisplayName is a heavily cached method. When only the display name of a user is required. It reduces roughly 1-2 query per existing user backend database queries.

The method is new in 25, so when the minimum version of the app allows it please use this. Prime examples as per above are rendering activities and notifications and similar things.

Please raise for your teams and apps:

• Make use of \OCP\IUserManager::getDisplayName groupware#53 Groupware @ChristophWurst
• Office @juliushaertl
  • Use displayname cache for fetching session data text#2805
  • Move to display name cache deck#4001
  • Move to display name cache richdocuments#2406
  • Collectives: no usage
  • Use user display name cache files_lock#87
• Talk @nickvergessen Use display name cache spreed#7769
• Server
  • Activity providers Use user name cache in activity providers #33615
  • Comment mentions Use user displayname cache for comment mentions #33614

[PVince81]

does LazyUser also count as display name cache ? because I saw a bunch of PRs from @icewind1991 for using that as well

[nickvergessen]

Yes, LazyUser uses the cache iirc

[nickvergessen]

@PVince81 shall we close as all teams reacted? You can create a follow up for server if you think it makes sense, but a generic ticket for server is probably also not helpful and fixes must be applied once a spot is found.