contactsmanager shall limit number of results early

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
This commit is contained in:
Arthur Schiwon 2020-07-30 12:50:04 +02:00 committed by backportbot[bot]
parent 9d56c5142b
commit dc98aaa224
3 changed files with 23 additions and 9 deletions

View File

@ -73,11 +73,23 @@ class ContactsStore implements IContactsStore {
* @param string|null $filter
* @return IEntry[]
*/
public function getContacts(IUser $user, $filter) {
$allContacts = $this->contactsManager->search($filter ?: '', [
public function getContacts(IUser $user, $filter, ?int $limit = null, ?int $offset = null) {
$options = [];
if ($limit !== null) {
$options['limit'] = $limit;
}
if ($offset !== null) {
$options['offset'] = $offset;
}
$allContacts = $this->contactsManager->search(
$filter ?: '',
[
'FN',
'EMAIL'
]);
],
$options
);
$entries = array_map(function (array $contact) {
return $this->contactArrayToEntry($contact);

View File

@ -66,7 +66,7 @@ class Manager {
$minSearchStringLength = $this->config->getSystemValueInt('sharing.minSearchStringLength', 0);
$topEntries = [];
if (strlen($filter) >= $minSearchStringLength) {
$entries = $this->store->getContacts($user, $filter);
$entries = $this->store->getContacts($user, $filter, $maxAutocompleteResults);
$sortedEntries = $this->sortEntries($entries);
$topEntries = array_slice($sortedEntries, 0, $maxAutocompleteResults);

View File

@ -33,11 +33,13 @@ interface IContactsStore {
/**
* @param IUser $user
* @param $filter
* @param string $filter
* @param int $limit added 19.0.2
* @param int $offset added 19.0.2
* @return IEntry[]
* @since 13.0.0
*/
public function getContacts(IUser $user, $filter);
public function getContacts(IUser $user, $filter, ?int $limit = null, ?int $offset = null);
/**
* @brief finds a contact by specifying the property to search on ($shareType) and the value ($shareWith)