Merge pull request #11025 from MrPetovan/task/11022-improve-connector-hooks

Improve connector hooks

doc/Addons.md

@@ -477,7 +477,18 @@ Hook data:
 - **uri** (input): the profile URI.
 - **network** (input): the target network (can be empty for auto-detection).
 - **uid** (input): the user to return the contact data for (can be empty for public contacts).
-- **result** (output): Set by the hook function to indicate a successful detection.
+- **result** (output): Leave null if address isn't relevant to the connector, set to contact array if probe is successful, false otherwise.
+
+### item_by_link
+
+Called when trying to probe an item from a given URI.
+If any registered hook function sets the `item_id` key of the hook data array, it will be returned immediately.
+Hook functions should also return immediately if the hook data contains an existing `item_id`.
+
+Hook data:
+- **uri** (input): the item URI.
+- **uid** (input): the user to return the item data for (can be empty for public contacts).
+- **item_id** (output): Leave null if URI isn't relevant to the connector, set to created item array if probe is successful, false otherwise.
 
 ### support_follow
 

src/Model/Item.php

@@ -3262,6 +3262,18 @@ class Item
 			return $item_id;
 		}
 
+		$hookData = [
+			'uri'     => $uri,
+			'uid'     => $uid,
+			'item_id' => null,
+		];
+
+		Hook::callAll('item_by_link', $hookData);
+
+		if (isset($hookData['item_id'])) {
+			return is_numeric($hookData['item_id']) ? $hookData['item_id'] : 0;
+		}
+
 		if ($fetched_uri = ActivityPub\Processor::fetchMissingActivity($uri)) {
 			$item_id = self::searchByLink($fetched_uri, $uid);
 		} else {

src/Network/Probe.php

@@ -682,26 +682,18 @@ class Probe
 			'uri'     => $uri,
 			'network' => $network,
 			'uid'     => $uid,
-			'result'  => [],
+			'result'  => null,
 		];
 
 		Hook::callAll('probe_detect', $hookData);
 
-		if ($hookData['result']) {
-			if (!is_array($hookData['result'])) {
-				return [];
-			} else {
-				return $hookData['result'];
-			}
+		if (isset($hookData['result'])) {
+			return is_array($hookData['result']) ? $hookData['result'] : [];
 		}
 
 		$parts = parse_url($uri);
 
-		if (!empty($parts['scheme']) && !empty($parts['host'])) {
-			if (in_array($parts['host'], ['twitter.com', 'mobile.twitter.com'])) {
-				return self::twitter($uri);
-			}
-		} elseif (strstr($uri, '@')) {
+		if (empty($parts['scheme']) || !empty($parts['host']) && strstr($uri, '@')) {
 			// If the URI starts with "mailto:" then jump directly to the mail detection
 			if (strpos($uri, 'mailto:') !== false) {
 				$uri = str_replace('mailto:', '', $uri);
@@ -711,12 +703,6 @@ class Probe
 			if ($network == Protocol::MAIL) {
 				return self::mail($uri, $uid);
 			}
-
-			if (Strings::endsWith($uri, '@twitter.com')
-				|| Strings::endsWith($uri, '@mobile.twitter.com')
-			) {
-				return self::twitter($uri);
-			}
 		} else {
 			Logger::info('URI was not detectable', ['uri' => $uri]);
 			return [];
@@ -1742,33 +1728,6 @@ class Probe
 		return $data;
 	}
 
-	/**
-	 * Check for twitter contact
-	 *
-	 * @param string $uri
-	 *
-	 * @return array twitter data
-	 */
-	private static function twitter($uri)
-	{
-		if (preg_match('=([^@]+)@(?:mobile\.)?twitter\.com$=i', $uri, $matches)) {
-			$nick = $matches[1];
-		} elseif (preg_match('=^https?://(?:mobile\.)?twitter\.com/(.+)=i', $uri, $matches)) {
-			$nick = $matches[1];
-		} else {
-			return [];
-		}
-
-		$data = [];
-		$data['url'] = 'https://twitter.com/' . $nick;
-		$data['addr'] = $nick . '@twitter.com';
-		$data['nick'] = $data['name'] = $nick;
-		$data['network'] = Protocol::TWITTER;
-		$data['baseurl'] = 'https://twitter.com';
-
-		return $data;
-	}
-
 	/**
 	 * Checks HTML page for RSS feed link
 	 *