Skip to content

Fix filtered albums#3403

Merged
artonge merged 4 commits intomasterfrom
fix/fix-filtered-albums
Feb 6, 2026
Merged

Fix filtered albums#3403
artonge merged 4 commits intomasterfrom
fix/fix-filtered-albums

Conversation

@come-nc
Copy link
Contributor

@come-nc come-nc commented Feb 3, 2026

Problems fixed:

  • += does a union of arrays and was ignoring some of the filters. Replaced by array_push and spread operator.
  • The wrong user was used to query configuration.
  • fetchAll()[0] could trigger a PHP error when there is no match, and getOwner was called on null when a file id was not found in the albums.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc come-nc requested a review from artonge February 3, 2026 15:01
@come-nc come-nc self-assigned this Feb 3, 2026
@come-nc come-nc added the 3. to review Waiting for reviews label Feb 3, 2026
@come-nc
Copy link
Contributor Author

come-nc commented Feb 3, 2026

/backport to stable33

@come-nc
Copy link
Contributor Author

come-nc commented Feb 3, 2026

/backport to stable32

@backportbot backportbot bot added the backport-request Pending backport by the backport-bot label Feb 3, 2026
@come-nc
Copy link
Contributor Author

come-nc commented Feb 5, 2026

CI is broken, this needs a force merge @artonge

Signed-off-by: Louis Chmn <louis@chmn.me>
@codecov
Copy link

codecov bot commented Feb 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@artonge artonge merged commit 9ef6960 into master Feb 6, 2026
51 of 53 checks passed
@artonge artonge deleted the fix/fix-filtered-albums branch February 6, 2026 11:55
@backportbot backportbot bot removed the backport-request Pending backport by the backport-bot label Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants