diff --git a/lib/private/Comments/Comment.php b/lib/private/Comments/Comment.php index 7190f252c82..90d4bbddd72 100644 --- a/lib/private/Comments/Comment.php +++ b/lib/private/Comments/Comment.php @@ -182,15 +182,7 @@ class Comment implements IComment { } /** - * returns an array containing mentions that are included in the comment - * - * @return array each mention provides a 'type' and an 'id', see example below - * @psalm-return list - * @since 30.0.2 Type 'email' is supported - * @since 29.0.0 Types 'federated_group', 'federated_team', 'team' and 'federated_user' are supported - * @since 23.0.0 Type 'group' is supported - * @since 17.0.0 Type 'guest' is supported - * @since 11.0.0 + * @inheritDoc */ public function getMentions(): array { $ok = preg_match_all("/\B(?getMessage(), $mentions); @@ -198,9 +190,6 @@ class Comment implements IComment { return []; } $mentionIds = array_unique($mentions[0]); - usort($mentionIds, static function ($mentionId1, $mentionId2) { - return mb_strlen($mentionId2) <=> mb_strlen($mentionId1); - }); $result = []; foreach ($mentionIds as $mentionId) { // Cut-off the @ and remove wrapping double-quotes @@ -237,6 +226,10 @@ class Comment implements IComment { $result[] = ['type' => 'user', 'id' => $cleanId]; } } + // Return sorted by id (descending length) + usort($result, static function ($mention1, $mention2) { + return mb_strlen($mention2['id']) <=> mb_strlen($mention1['id']); + }); return $result; }