]> git.mxchange.org Git - friendica.git/blobdiff - src/Module/BaseSearchModule.php
Merge pull request #7943 from MrPetovan/bug/7920-api-fix-followers-friends-ids
[friendica.git] / src / Module / BaseSearchModule.php
index 9430ec1b9145310674494859eb0fef5dc77688c5..9766c000cca8b4926d87de07b038b3a3df5f04fd 100644 (file)
@@ -2,14 +2,18 @@
 
 namespace Friendica\Module;
 
+use Friendica\App\Arguments;
 use Friendica\BaseModule;
 use Friendica\Content\ContactSelector;
 use Friendica\Content\Pager;
 use Friendica\Core\L10n;
 use Friendica\Core\Renderer;
-use Friendica\Object\Search\ContactResultList;
-use Friendica\Util\Proxy as ProxyUtils;
+use Friendica\Core\Search;
 use Friendica\Model;
+use Friendica\Network\HTTPException;
+use Friendica\Object\Search\ContactResult;
+use Friendica\Object\Search\ResultList;
+use Friendica\Util\Proxy as ProxyUtils;
 use Friendica\Util\Strings;
 
 /**
@@ -20,22 +24,23 @@ class BaseSearchModule extends BaseModule
        /**
         * Performs a search with an optional prefix
         *
+        * @param string $search Search query
         * @param string $prefix A optional prefix (e.g. @ or !) for searching
         *
         * @return string
-        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws HTTPException\InternalServerErrorException
         * @throws \ImagickException
         */
-       public static function performSearch($prefix = '')
+       public static function performSearch($search, $prefix = '')
        {
                $a      = self::getApp();
                $config = $a->getConfig();
 
-               $community = false;
+               $type = Search::TYPE_ALL;
 
                $localSearch = $config->get('system', 'poco_local_search');
 
-               $search = $prefix . Strings::escapeTags(trim(defaults($_REQUEST, 'search', '')));
+               $search = $prefix . $search;
 
                if (!$search) {
                        return '';
@@ -45,24 +50,29 @@ class BaseSearchModule extends BaseModule
 
                if (strpos($search, '@') === 0) {
                        $search  = substr($search, 1);
+                       $type    = Search::TYPE_PEOPLE;
                        $header  = L10n::t('People Search - %s', $search);
-                       $results = Model\Search::searchUser($search);
+
+                       if (strrpos($search, '@') > 0) {
+                               $results = Search::getContactsFromProbe($search);
+                       }
                }
 
                if (strpos($search, '!') === 0) {
-                       $search    = substr($search, 1);
-                       $community = true;
-                       $header    = L10n::t('Forum Search - %s', $search);
+                       $search = substr($search, 1);
+                       $type   = Search::TYPE_FORUM;
+                       $header = L10n::t('Forum Search - %s', $search);
                }
 
-               $pager = new Pager($a->query_string);
+               /** @var Arguments $args */
+               $args = self::getClass(Arguments::class);
+               $pager = new Pager($args->getQueryString());
 
                if ($localSearch && empty($results)) {
                        $pager->setItemsPerPage(80);
-                       $results = Model\Search::searchLocal($search, $pager->getStart(), $pager->getItemsPerPage(), $community);
-
+                       $results = Search::getContactsFromLocalDirectory($search, $type, $pager->getStart(), $pager->getItemsPerPage());
                } elseif (strlen($config->get('system', 'directory')) && empty($results)) {
-                       $results = Model\Search::searchDirectory($search, $pager->getPage());
+                       $results = Search::getContactsFromGlobalDirectory($search, $type, $pager->getPage());
                        $pager->setItemsPerPage($results->getItemsPage());
                }
 
@@ -72,79 +82,83 @@ class BaseSearchModule extends BaseModule
        /**
         * Prints a human readable search result
         *
-        * @param ContactResultList $results
-        * @param Pager             $pager
-        * @param string            $header
+        * @param ResultList $results
+        * @param Pager      $pager
+        * @param string     $header
         *
         * @return string The result
-        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws HTTPException\InternalServerErrorException
         * @throws \ImagickException
         */
-       protected static function printResult(ContactResultList $results, Pager $pager, $header = '')
+       protected static function printResult(ResultList $results, Pager $pager, $header = '')
        {
-               if (empty($results) || empty($results->getResults())) {
-                       info(L10n::t('No matches') . EOL);
+               if ($results->getTotal() == 0) {
+                       info(L10n::t('No matches'));
                        return '';
                }
 
                $a = self::getApp();
 
-               $id = 0;
+               $id      = 0;
                $entries = [];
                foreach ($results->getResults() as $result) {
 
-                       $alt_text    = '';
-                       $location    = '';
-                       $about       = '';
-                       $accountType = '';
-                       $photo_menu  = [];
-
-                       // If We already know this contact then don't show the "connect" button
-                       if ($result->getCid() > 0 || $result->getPCid() > 0) {
-                               $connLink = "";
-                               $connTxt = "";
-                               $contact = Model\Contact::getById(
-                                       ($result->getCid() > 0) ? $result->getCid() : $result->getPCid()
-                               );
-
-                               if (!empty($contact)) {
-                                       $photo_menu  = Model\Contact::photoMenu($contact);
-                                       $details     = Contact::getContactTemplateVars($contact);
-                                       $alt_text    = $details['alt_text'];
-                                       $location    = $contact['location'];
-                                       $about       = $contact['about'];
-                                       $accountType = Model\Contact::getAccountType($contact);
+                       // in case the result is a contact result, add a contact-specific entry
+                       if ($result instanceof ContactResult) {
+
+                               $alt_text    = '';
+                               $location    = '';
+                               $about       = '';
+                               $accountType = '';
+                               $photo_menu  = [];
+
+                               // If We already know this contact then don't show the "connect" button
+                               if ($result->getCid() > 0 || $result->getPCid() > 0) {
+                                       $connLink = "";
+                                       $connTxt  = "";
+                                       $contact  = Model\Contact::getById(
+                                               ($result->getCid() > 0) ? $result->getCid() : $result->getPCid()
+                                       );
+
+                                       if (!empty($contact)) {
+                                               $photo_menu  = Model\Contact::photoMenu($contact);
+                                               $details     = Contact::getContactTemplateVars($contact);
+                                               $alt_text    = $details['alt_text'];
+                                               $location    = $contact['location'];
+                                               $about       = $contact['about'];
+                                               $accountType = Model\Contact::getAccountType($contact);
+                                       } else {
+                                               $photo_menu = [];
+                                       }
                                } else {
-                                       $photo_menu = [];
+                                       $connLink = $a->getBaseURL() . '/follow/?url=' . $result->getUrl();
+                                       $connTxt  = L10n::t('Connect');
+
+                                       $photo_menu['profile'] = [L10n::t("View Profile"), Model\Contact::magicLink($result->getUrl())];
+                                       $photo_menu['follow']  = [L10n::t("Connect/Follow"), $connLink];
                                }
-                       } else {
-                               $connLink = $a->getBaseURL() . '/follow/?url=' . $result->getUrl();
-                               $connTxt = L10n::t('Connect');
 
-                               $photo_menu['profile'] = [L10n::t("View Profile"), Model\Contact::magicLink($result->getUrl())];
-                               $photo_menu['follow']  = [L10n::t("Connect/Follow"), $connLink];
+                               $photo = str_replace("http:///photo/", get_server() . "/photo/", $result->getPhoto());
+
+                               $entry     = [
+                                       'alt_text'     => $alt_text,
+                                       'url'          => Model\Contact::magicLink($result->getUrl()),
+                                       'itemurl'      => $result->getItem(),
+                                       'name'         => $result->getName(),
+                                       'thumb'        => ProxyUtils::proxifyUrl($photo, false, ProxyUtils::SIZE_THUMB),
+                                       'img_hover'    => $result->getTags(),
+                                       'conntxt'      => $connTxt,
+                                       'connlnk'      => $connLink,
+                                       'photo_menu'   => $photo_menu,
+                                       'details'      => $location,
+                                       'tags'         => $result->getTags(),
+                                       'about'        => $about,
+                                       'account_type' => $accountType,
+                                       'network'      => ContactSelector::networkToName($result->getNetwork(), $result->getUrl()),
+                                       'id'           => ++$id,
+                               ];
+                               $entries[] = $entry;
                        }
-
-                       $photo = str_replace("http:///photo/", get_server() . "/photo/", $result->getPhoto());
-
-                       $entry     = [
-                               'alt_text'     => $alt_text,
-                               'url'          => Model\Contact::magicLink($result->getUrl()),
-                               'itemurl'      => $result->getItem(),
-                               'name'         => $result->getName(),
-                               'thumb'        => ProxyUtils::proxifyUrl($photo, false, ProxyUtils::SIZE_THUMB),
-                               'img_hover'    => $result->getTags(),
-                               'conntxt'      => $connTxt,
-                               'connlnk'      => $connLink,
-                               'photo_menu'   => $photo_menu,
-                               'details'      => $location,
-                               'tags'         => $result->getTags(),
-                               'about'        => $about,
-                               'account_type' => $accountType,
-                               'network'      => ContactSelector::networkToName($result->getNetwork(), $result->getUrl()),
-                               'id'           => ++$id,
-                       ];
-                       $entries[] = $entry;
                }
 
                $tpl = Renderer::getMarkupTemplate('viewcontact_template.tpl');