Split up OStatusPlugin preg functions so they can be reused

This commit is contained in:
Mikael Nordfeldth 2017-04-22 10:58:14 +02:00
parent 54971842f2
commit bd6c93a811

View File

@ -256,6 +256,46 @@ class OStatusPlugin extends Plugin
return true; return true;
} }
/**
* Webfinger matches: @user@example.com or even @user--one.george_orwell@1984.biz
*
* @return array The matching IDs (without @ or acct:) and each respective position in the given string.
*/
static function extractWebfingerIds($text)
{
$wmatches = array();
$result = preg_match_all('/(?:^|\s+)@((?:\w+[\w\-\_\.]?)*(?:[\w\-\_\.]*\w+)@(?:(?!-)[A-Za-z0-9\-]{1,63}(?<!-)\.)+[A-Za-z]{2,10})/',
$text,
$wmatches,
PREG_OFFSET_CAPTURE);
if ($result === false) {
common_log(LOG_ERR, __METHOD__ . ': Error parsing webfinger IDs from text (preg_last_error=='.preg_last_error().').');
} else {
common_debug(sprintf('Found %i matches for WebFinger IDs: %s', count($wmatches), _ve($wmatches)));
}
return $wmatches[1];
}
/**
* Profile URL matches: @example.com/mublog/user
*
* @return array The matching URLs (without @ or acct:) and each respective position in the given string.
*/
static function extractUrlMentions($text)
{
$wmatches = array();
$result = preg_match_all('!(?:^|\s+)@((?:\w+\.)*\w+(?:\w+\-\w+)*\.\w+(?:/\w+)*)!',
$text,
$wmatches,
PREG_OFFSET_CAPTURE);
if ($result === false) {
common_log(LOG_ERR, __METHOD__ . ': Error parsing profile URL mentions from text (preg_last_error=='.preg_last_error().').');
} else {
common_debug(sprintf('Found %i matches for profile URL mentions: %s', count($wmatches), _ve($wmatches)));
}
return $wmatches[1];
}
/** /**
* Find any explicit remote mentions. Accepted forms: * Find any explicit remote mentions. Accepted forms:
* Webfinger: @user@example.com * Webfinger: @user@example.com
@ -269,13 +309,7 @@ class OStatusPlugin extends Plugin
{ {
$matches = array(); $matches = array();
$wmatches = array(); foreach (self::extractWebfingerIds($text) as $wmatch) {
// Webfinger matches: @user@example.com or even @user--one.george_orwell@1984.biz
if (preg_match_all('/(?:^|\s+)@((?:\w+[\w\-\_\.]?)*(?:[\w\-\_\.]*\w+)@(?:(?!-)[A-Za-z0-9\-]{1,63}(?<!-)\.)+[A-Za-z]{2,10})/',
$text,
$wmatches,
PREG_OFFSET_CAPTURE)) {
foreach ($wmatches[1] as $wmatch) {
list($target, $pos) = $wmatch; list($target, $pos) = $wmatch;
$this->log(LOG_INFO, "Checking webfinger '$target'"); $this->log(LOG_INFO, "Checking webfinger '$target'");
$profile = null; $profile = null;
@ -309,14 +343,8 @@ class OStatusPlugin extends Plugin
'length' => mb_strlen($target), 'length' => mb_strlen($target),
'url' => $url); 'url' => $url);
} }
}
// Profile matches: @example.com/mublog/user foreach (self::extractUrlMentions($text) as $wmatch) {
if (preg_match_all('!(?:^|\s+)@((?:\w+\.)*\w+(?:\w+\-\w+)*\.\w+(?:/\w+)*)!',
$text,
$wmatches,
PREG_OFFSET_CAPTURE)) {
foreach ($wmatches[1] as $wmatch) {
list($target, $pos) = $wmatch; list($target, $pos) = $wmatch;
$schemes = array('http', 'https'); $schemes = array('http', 'https');
foreach ($schemes as $scheme) { foreach ($schemes as $scheme) {
@ -341,7 +369,6 @@ class OStatusPlugin extends Plugin
} }
} }
} }
}
foreach ($mentions as $i => $other) { foreach ($mentions as $i => $other) {
// If we share a common prefix with a local user, override it! // If we share a common prefix with a local user, override it!