Skip to content

fix(search): All elements are included regardless of their keys #57761

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

Jump to bottom
Open
solracsf wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix(search): All elements are included regardless of their keys #57761

solracsf wants to merge 1 commit into from
+29 −24

Conversation

@solracsf
Copy link
Member

@solracsf solracsf commented Jan 23, 2026 •
edited
Loading

Summary

This ensures all elements are included regardless of their original keys.

Checklist

@solracsf solracsf added this to the Nextcloud 34 milestone Jan 23, 2026
@solracsf solracsf requested a review from a team as a code owner January 23, 2026 15:14
@solracsf solracsf added bug 3. to review Waiting for reviews labels Jan 23, 2026
@solracsf solracsf removed the request for review from a team January 23, 2026 15:14
@solracsf
Copy link
Member Author

solracsf commented Jan 23, 2026

/backport to stable33

@solracsf
Copy link
Member Author

solracsf commented Jan 23, 2026

/backport to stable32

@solracsf
Copy link
Member Author

solracsf commented Jan 23, 2026

/backport to stable31

@solracsf solracsf enabled auto-merge January 23, 2026 15:15
@solracsf solracsf self-assigned this Jan 23, 2026
@solracsf
fix(search): All elements are included regardless of their keys
7352aaf 
Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
@solracsf solracsf force-pushed the searchTagsAllElementsIncluded branch from 9b1ea3b to 7352aaf Compare January 24, 2026 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@icewind1991 icewind1991 Awaiting requested review from icewind1991 icewind1991 is a code owner automatically assigned from nextcloud/server-backend

@leftybournes leftybournes Awaiting requested review from leftybournes leftybournes is a code owner automatically assigned from nextcloud/server-backend

@provokateurin provokateurin Awaiting requested review from provokateurin provokateurin is a code owner automatically assigned from nextcloud/server-backend

@come-nc come-nc Awaiting requested review from come-nc come-nc is a code owner automatically assigned from nextcloud/server-backend

At least 2 approving reviews are required to merge this pull request.

Assignees

@solracsf solracsf

Labels

3. to review Waiting for reviews backport-request bug feature: search feature: tags

Projects

None yet

Milestone

Nextcloud 34

Development

Successfully merging this pull request may close these issues.

Bug: SystemTags search missing tags with low IDs due to array key collision

2 participants

@solracsf