X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FCore%2FSearch.php;h=41c75ad32b2b463c47af5d3c89bd5f69c7c9ecbf;hb=44291a465bb9b9c54b8781d6d6f13e1c3f317c1b;hp=208633272b3722be83e78837f69bcf10197ac121;hpb=e374aecc46c74f9c499a9abddf9d111fdd0b6984;p=friendica.git

diff --git a/src/Core/Search.php b/src/Core/Search.php
index 208633272b..41c75ad32b 100644
--- a/src/Core/Search.php
+++ b/src/Core/Search.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2020, Friendica
+ * @copyright Copyright (C) 2010-2022, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -21,15 +21,12 @@
 
 namespace Friendica\Core;
 
-use Friendica\Database\DBA;
 use Friendica\DI;
 use Friendica\Model\Contact;
-use Friendica\Model\GContact;
+use Friendica\Network\HTTPClient\Client\HttpClientAccept;
 use Friendica\Network\HTTPException;
-use Friendica\Network\Probe;
 use Friendica\Object\Search\ContactResult;
 use Friendica\Object\Search\ResultList;
-use Friendica\Protocol\PortableContact;
 use Friendica\Util\Network;
 use Friendica\Util\Strings;
 
@@ -64,8 +61,7 @@ class Search
 		if ((filter_var($user, FILTER_VALIDATE_EMAIL) && Network::isEmailDomainValid($user)) ||
 		    (substr(Strings::normaliseLink($user), 0, 7) == "http://")) {
 
-			/// @todo Possibly use "getIdForURL" instead?
-			$user_data = Probe::uri($user);
+			$user_data = Contact::getByURL($user);
 			if (empty($user_data)) {
 				return $emptyResultList;
 			}
@@ -74,10 +70,7 @@ class Search
 				return $emptyResultList;
 			}
 
-			// Ensure that we do have a contact entry
-			Contact::getIdForURL($user_data['url'] ?? '');
-
-			$contactDetails = Contact::getByURLForUser($user_data['url'] ?? '', local_user(), false);
+			$contactDetails = Contact::getByURLForUser($user_data['url'] ?? '', local_user());
 
 			$result = new ContactResult(
 				$user_data['name'] ?? '',
@@ -87,7 +80,7 @@ class Search
 				$user_data['photo'] ?? '',
 				$user_data['network'] ?? '',
 				$contactDetails['id'] ?? 0,
-				0,
+				$user_data['id'] ?? 0,
 				$user_data['tags'] ?? ''
 			);
 
@@ -111,7 +104,7 @@ class Search
 	 */
 	public static function getContactsFromGlobalDirectory($search, $type = self::TYPE_ALL, $page = 1)
 	{
-		$server = DI::config()->get('system', 'directory', self::DEFAULT_DIRECTORY);
+		$server = self::getGlobalDirectory();
 
 		$searchUrl = $server . '/search';
 
@@ -129,7 +122,7 @@ class Search
 			$searchUrl .= '&page=' . $page;
 		}
 
-		$resultJson = Network::fetchUrl($searchUrl, false, 0, 'application/json');
+		$resultJson = DI::httpClient()->fetch($searchUrl, HttpClientAccept::JSON);
 
 		$results = json_decode($resultJson, true);
 
@@ -142,8 +135,8 @@ class Search
 		$profiles = $results['profiles'] ?? [];
 
 		foreach ($profiles as $profile) {
-			$profile_url = $profile['url'] ?? '';
-			$contactDetails = Contact::getByURLForUser($profile_url, local_user(), false);
+			$profile_url = $profile['profile_url'] ?? '';
+			$contactDetails = Contact::getByURLForUser($profile_url, local_user());
 
 			$result = new ContactResult(
 				$profile['name'] ?? '',
@@ -176,68 +169,13 @@ class Search
 	 */
 	public static function getContactsFromLocalDirectory($search, $type = self::TYPE_ALL, $start = 0, $itemPage = 80)
 	{
-		$config = DI::config();
-
-		$diaspora = $config->get('system', 'diaspora_enabled') ? Protocol::DIASPORA : Protocol::DFRN;
-		$ostatus  = !$config->get('system', 'ostatus_disabled') ? Protocol::OSTATUS : Protocol::DFRN;
-
-		$wildcard = Strings::escapeHtml('%' . $search . '%');
-
-		$count = DBA::count('gcontact', [
-			'NOT `hide`
-			AND `network` IN (?, ?, ?, ?)
-			AND ((`last_contact` >= `last_failure`) OR (`updated` >= `last_failure`))
-			AND (`url` LIKE ? OR `name` LIKE ? OR `location` LIKE ? 
-				OR `addr` LIKE ? OR `about` LIKE ? OR `keywords` LIKE ?)
-			AND `community` = ?',
-			Protocol::ACTIVITYPUB, Protocol::DFRN, $ostatus, $diaspora,
-			$wildcard, $wildcard, $wildcard,
-			$wildcard, $wildcard, $wildcard,
-			($type === self::TYPE_FORUM),
-		]);
-
-		$resultList = new ResultList($start, $itemPage, $count);
-
-		if (empty($count)) {
-			return $resultList;
-		}
+		Logger::info('Searching', ['search' => $search, 'type' => $type, 'start' => $start, 'itempage' => $itemPage]);
 
-		$data = DBA::select('gcontact', ['nurl'], [
-			'NOT `hide`
-			AND `network` IN (?, ?, ?, ?)
-			AND ((`last_contact` >= `last_failure`) OR (`updated` >= `last_failure`))
-			AND (`url` LIKE ? OR `name` LIKE ? OR `location` LIKE ? 
-				OR `addr` LIKE ? OR `about` LIKE ? OR `keywords` LIKE ?)
-			AND `community` = ?',
-			Protocol::ACTIVITYPUB, Protocol::DFRN, $ostatus, $diaspora,
-			$wildcard, $wildcard, $wildcard,
-			$wildcard, $wildcard, $wildcard,
-			($type === self::TYPE_FORUM),
-		], [
-			'group_by' => ['nurl', 'updated'],
-			'limit'    => [$start, $itemPage],
-			'order'    => ['updated' => 'DESC']
-		]);
-
-		if (!DBA::isResult($data)) {
-			return $resultList;
-		}
-
-		while ($row = DBA::fetch($data)) {
-			$urlParts = parse_url($row["nurl"]);
+		$contacts = Contact::searchByName($search, $type == self::TYPE_FORUM ? 'community' : '');
 
-			// Ignore results that look strange.
-			// For historic reasons the gcontact table does contain some garbage.
-			if (!empty($urlParts['query']) || !empty($urlParts['fragment'])) {
-				continue;
-			}
-
-			$contact = Contact::getByURLForUser($row["nurl"], local_user(), false);
-
-			if ($contact["name"] == "") {
-				$contact["name"] = end(explode("/", $urlParts["path"]));
-			}
+		$resultList = new ResultList($start, $itemPage, count($contacts));
 
+		foreach ($contacts as $contact) {
 			$result = new ContactResult(
 				$contact["name"],
 				$contact["addr"],
@@ -245,16 +183,14 @@ class Search
 				$contact["url"],
 				$contact["photo"],
 				$contact["network"],
-				$contact["cid"],
-				$contact["zid"],
+				$contact["cid"] ?? 0,
+				$contact["zid"] ?? 0,
 				$contact["keywords"]
 			);
 
 			$resultList->addResult($result);
 		}
 
-		DBA::close($data);
-
 		// Add found profiles from the global directory to the local directory
 		Worker::add(PRIORITY_LOW, 'SearchDirectory', $search);
 
@@ -262,7 +198,7 @@ class Search
 	}
 
 	/**
-	 * Searching for global contacts for autocompletion
+	 * Searching for contacts for autocompletion
 	 *
 	 * @param string $search Name or part of a name or nick
 	 * @param string $mode   Search mode (e.g. "community")
@@ -270,8 +206,10 @@ class Search
 	 * @return array with the search results
 	 * @throws HTTPException\InternalServerErrorException
 	 */
-	public static function searchGlobalContact($search, $mode, int $page = 1)
+	public static function searchContact($search, $mode, int $page = 1)
 	{
+		Logger::info('Searching', ['search' => $search, 'mode' => $mode, 'page' => $page]);
+
 		if (DI::config()->get('system', 'block_public') && !Session::isAuthenticated()) {
 			return [];
 		}
@@ -287,10 +225,10 @@ class Search
 
 		// check if searching in the local global contact table is enabled
 		if (DI::config()->get('system', 'poco_local_search')) {
-			$return = GContact::searchByName($search, $mode);
+			$return = Contact::searchByName($search, $mode);
 		} else {
 			$p = $page > 1 ? 'p=' . $page : '';
-			$curlResult = Network::curl(self::getGlobalDirectory() . '/search/people?' . $p . '&q=' . urlencode($search), false, ['accept_content' => 'application/json']);
+			$curlResult = DI::httpClient()->get(self::getGlobalDirectory() . '/search/people?' . $p . '&q=' . urlencode($search), HttpClientAccept::JSON);
 			if ($curlResult->isSuccess()) {
 				$searchResult = json_decode($curlResult->getBody(), true);
 				if (!empty($searchResult['profiles'])) {