Include chat metadata in unified search entry

[thisIsTheFoxe]

Is your feature request related to a problem? Please describe.
Implementing unified search on clients as it is now requires parsing the { "entries": [ { "resourceUrl": ... to get the relevant data. This is a very unclean solution.

Describe the solution you'd like
As was mentioned in 2nd the linked issue, the files provider uses the attributes key to provide additional metadata. A similar approach could be used here, where roomToken would be the most important, but other info could be useful as well.
Example: > http://localhost/ocs/v2.php/search/providers/talk-message/search?format=json&term=test

"entries": [
  {
    "thumbnailUrl": "http://localhost/avatar/admin/64",
    "title": "admin in Test",
    "subline": "test message",
    "resourceUrl": "http://localhost/call/abc123#message_42",
    "icon": "icon-talk",
    "rounded": true,
    "attributes": {
      "roomName": "Test" 
      "roomToken": "abc123",
      "messageId": 42,
      "sentBy": "admin",
      "timestamp": "2021-12-17"
    }
  }
]

Describe alternatives you've considered
As said, the only other solution I see is parsing the url manually.

Additional context

• Deeper integration of unified search results ios#1571
• Include file attributes on fulltextsearch result files_fulltextsearch#143

[nickvergessen]

Interesting, I was not aware of the attributes thing as it was not used in the core at the time of implementation:
https://github.com/nextcloud/server/blob/0447b53bda9fe95ea0cbed765aa332584605d652/lib/public/Search/SearchResultEntry.php#L115-L126

Code to adjust for Messages search is:

spreed/lib/Search/MessageSearch.php

Lines 209 to 220 in 52eb2fd

	return new SearchResultEntry(
	$iconUrl,
	str_replace(
	['{user}', '{conversation}'],
	[$message->getActorDisplayName(), $room->getDisplayName($user->getUID())],
	$subline
	),
	$messageStr,
	$this->url->linkToRouteAbsolute('spreed.Page.showCall', ['token' => $room->getToken()]) . '#message_' . $comment->getId(),
	'icon-talk', // $iconClass,
	true
	);

In order to not have too many unused attributes exposed which we can never remove/modify, let's only add:

{
    "conversation": "token",
    "messageId": 42
}

For now?

For conversation search we can of course only expose the conversation attribute:

spreed/lib/Search/ConversationSearch.php

Lines 144 to 151 in 52eb2fd

	$result[] = new SearchResultEntry(
	$icon,
	$room->getDisplayName($user->getUID()),
	'',
	$this->url->linkToRouteAbsolute('spreed.Page.showCall', ['token' => $room->getToken()]),
	$iconClass,
	true
	);

[thisIsTheFoxe]

@nickvergessen sounds great, thanks for the fast response!