diff --git a/src/Module/Contact/Hovercard.php b/src/Module/Contact/Hovercard.php index 953d7e7a7..5c82fd2c4 100644 --- a/src/Module/Contact/Hovercard.php +++ b/src/Module/Contact/Hovercard.php @@ -44,11 +44,12 @@ class Hovercard extends BaseModule throw new HTTPException\ForbiddenException(); } - // If a contact is connected the url is internally changed to 'contact/redir/CID'. We need the pure url to search for - // the contact. So we strip out the contact id from the internal url and look in the contact table for - // the real url (nurl) - if (strpos($contact_url, 'contact/') === 0) { - $remote_contact = Contact::selectFirst(['nurl'], ['id' => intval(basename($contact_url))]); + /* Possible formats for relative URLs that need to be converted to the absolute contact URL: + * - contact/redir/123456 + * - contact/123456/conversations + */ + if (strpos($contact_url, 'contact/') === 0 && preg_match('/(\d+)/', $contact_url, $matches)) { + $remote_contact = Contact::selectFirst(['nurl'], ['id' => $matches[1]]); $contact_url = $remote_contact['nurl'] ?? ''; }