Merge pull request #12222 from MrPetovan/bug/12219-hovercard-stay-local
Fix support for `/contact/123456/conversations` URLs in Module\Contact\Hovercard
This commit is contained in:
commit
d1a85f1fa4
|
@ -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'] ?? '';
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue
Block a user