Skip to content

[stable31] fix(unified-search): Remove hard-coded search result limit - #56344

Merged
sorbaugh merged 1 commit into
stable31from
backport/55434/stable31
Nov 13, 2025
Merged

[stable31] fix(unified-search): Remove hard-coded search result limit #56344
sorbaugh merged 1 commit into
stable31from
backport/55434/stable31

Conversation

@nfebe

Copy link
Copy Markdown
Contributor

Manual backport of #55434


A change added in #45317 introduced a hard stop (25) that prevents full search results from showing up.

If there are more than 25 search results for a query only 25 can be seen.

So two main issues:

  • Only 25 results can be seen in total no matter what.
  • Breaks web client pagination, which typically adds 5 results per request.

@nfebe
nfebe enabled auto-merge November 10, 2025 14:00
@nfebe
nfebe requested review from a team, icewind1991, leftybournes and yemkareems and removed request for a teamNovember 10, 2025 14:00
@nfebenfebe added 3. to review Waiting for reviews feature: search labels Nov 10, 2025
@luka-nextcloudluka-nextcloud mentioned this pull request Nov 10, 2025
14 tasks
Comment threadcore/Controller/UnifiedSearchController.php
A change added in #45317 introduced
a hard stop (25) that prevents full search results from showing up.
If there are more than 25 search results for a query only 25 can be seen.
So two main issues:
- Only 25 results can be seen in total no matter what.
- Breaks web client pagination, which typically adds 5 results per request.
Signed-off-by: nfebe <fenn25.fn@gmail.com>
@sorbaugh
sorbaugh disabled auto-merge November 13, 2025 16:07
@sorbaugh
sorbaugh merged commit 789045a into stable31Nov 13, 2025
153 of 154 checks passed
@sorbaugh
sorbaugh deleted the backport/55434/stable31 branch November 13, 2025 16:08
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsfeature: search

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@nfebe@blizzz@provokateurin@sorbaugh