Skip to content

[stable20] Handle limit offset and sorting in files search #26258

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rullzer merged 11 commits into from
Mar 30, 2021
Merged

[stable20] Handle limit offset and sorting in files search #26258

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
60551ce
use "newer" node search api directly in unified search
icewind1991 Mar 18, 2021
a624243
unify handling of Folder::search methods into "new" query objects
icewind1991 Mar 18, 2021
4655f6f
only require user to be set in a query that handles tags
icewind1991 Mar 18, 2021
6a639ec
handle limit and offset in folder file search
icewind1991 Mar 18, 2021
dbd8b0a
format
icewind1991 Mar 18, 2021
4e60b9f
cleanup fileinfo creation
icewind1991 Mar 19, 2021
671b992
folder filtering in sql
icewind1991 Mar 19, 2021
055af29
handle ordering in folder search
icewind1991 Mar 19, 2021
10bf93a
pass order from unified search to file search
icewind1991 Mar 19, 2021
4ede9db
use node search api for legacy file search endpoint
icewind1991 Mar 19, 2021
4b65a1d
fix type hints
icewind1991 Mar 24, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fix type hints 
Signed-off-by: Robin Appelman <[email protected]>
  • Loading branch information
@icewind1991
icewind1991 committed Mar 24, 2021
commit 4b65a1dcd7192c0a32b2fb8b3184c06f788574c7
4 changes: 4 additions & 0 deletions lib/private/Files/Cache/Cache.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -811,6 +811,10 @@ public function searchByMime($mimetype) {
}, $files);
}

/**
* @param ISearchQuery $searchQuery
* @return CacheEntry[]
*/
public function searchQuery(ISearchQuery $searchQuery) {
$builder = $this->getQueryBuilder();

Expand Down
6 changes: 3 additions & 3 deletions lib/private/Files/Node/Folder.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,14 +32,14 @@
namespace OC\Files\Node;

use OC\DB\QueryBuilder\Literal;
use OC\Files\Cache\CacheEntry;
use OC\Files\Search\SearchBinaryOperator;
use OC\Files\Search\SearchComparison;
use OC\Files\Search\SearchQuery;
use OC\Files\Storage\Wrapper\Jail;
use OC\Files\Storage\Storage;
use OCA\Files_Sharing\SharedStorage;
use OCP\DB\QueryBuilder\IQueryBuilder;
use OCP\Files\Cache\ICacheEntry;
use OCP\Files\Config\ICachedMountInfo;
use OCP\Files\FileInfo;
use OCP\Files\Mount\IMountPoint;
Expand Down Expand Up @@ -323,7 +323,7 @@ public function search($query) {
}, $files);
}

private function cacheEntryToFileInfo(IMountPoint $mount, string $appendRoot, string $trimRoot, ICacheEntry $cacheEntry): FileInfo {
private function cacheEntryToFileInfo(IMountPoint $mount, string $appendRoot, string $trimRoot, CacheEntry $cacheEntry): FileInfo {
$trimLength = strlen($trimRoot);
$cacheEntry['internalPath'] = $cacheEntry['path'];
$cacheEntry['path'] = $appendRoot . substr($cacheEntry['path'], $trimLength);
Expand Down Expand Up @@ -364,7 +364,7 @@ public function searchByTag($tag, $userId) {
public function getById($id) {
$mountCache = $this->root->getUserMountCache();
if (strpos($this->getPath(), '/', 1) > 0) {
list(, $user) = explode('/', $this->getPath());
[, $user] = explode('/', $this->getPath());
} else {
$user = null;
}
Expand Down