[PR #15820] [MERGED] Fixed Multi Sort in New ActivityManager #14434

New Issue

OVERLORD · 2026-02-07T07:29:44+03:00

OVERLORD commented

2026-02-07 07:29:44 +03:00

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/15820
Author: @bjorntp
Created: 12/17/2025
Status: ✅ Merged
Merged: 12/20/2025
Merged by: @crobibero

Base: master ← Head: activity-multi-sort

📝 Commits (4)

1760f56 Fixed bug where sorting was only applied by the last query
8e28bb1 contributor
6d4c9cb Added MaxDate endpoint
71d1364 indendations and removed unnecessary statement

📊 Changes

4 files changed (+27 additions, -3 deletions)

View changed files

📝 CONTRIBUTORS.md (+1 -0)
📝 Jellyfin.Api/Controllers/ActivityLogController.cs (+3 -0)
📝 Jellyfin.Data/Queries/ActivityLogQuery.cs (+5 -0)
📝 Jellyfin.Server.Implementations/Activity/ActivityManager.cs (+18 -3)

📄 Description

Changes
Fixed Multi Sort by adding condition that sets THEN BY to the queries following the first one instead of ORDER BY on every one as ORDER BY overwrites.
Added MaxDate endpoint to allow for range selector among dates as that might come useful among logs and the MRT-table used in the frontend has support for it.

Issues
Multi sort did not work as intended #15583.
jellyfin/jellyfin-web/issues/6221

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/15820 **Author:** [@bjorntp](https://github.com/bjorntp) **Created:** 12/17/2025 **Status:** ✅ Merged **Merged:** 12/20/2025 **Merged by:** [@crobibero](https://github.com/crobibero) **Base:** `master` ← **Head:** `activity-multi-sort` --- ### 📝 Commits (4) - [`1760f56`](https://github.com/jellyfin/jellyfin/commit/1760f56df12e422ac71c56bd93a627d682cb7473) Fixed bug where sorting was only applied by the last query - [`8e28bb1`](https://github.com/jellyfin/jellyfin/commit/8e28bb12d6dd9c9e19545c2ff45ec22e2e91b8c6) contributor - [`6d4c9cb`](https://github.com/jellyfin/jellyfin/commit/6d4c9cb43477ec0df4a1d52b068ac97b8aed7ca8) Added MaxDate endpoint - [`71d1364`](https://github.com/jellyfin/jellyfin/commit/71d136403954eac7df12c4f368e9889fac77ccee) indendations and removed unnecessary statement ### 📊 Changes **4 files changed** (+27 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `CONTRIBUTORS.md` (+1 -0) 📝 `Jellyfin.Api/Controllers/ActivityLogController.cs` (+3 -0) 📝 `Jellyfin.Data/Queries/ActivityLogQuery.cs` (+5 -0) 📝 `Jellyfin.Server.Implementations/Activity/ActivityManager.cs` (+18 -3) </details> ### 📄 Description  **Changes** Fixed Multi Sort by adding condition that sets `THEN BY` to the queries following the first one instead of `ORDER BY` on every one as `ORDER BY` overwrites. Added MaxDate endpoint to allow for range selector among dates as that might come useful among logs and the MRT-table used in the frontend has support for it. **Issues** Multi sort did not work as intended #15583. jellyfin/jellyfin-web/issues/6221 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2026-02-07 07:29:44 +03:00

OVERLORD closed this issue

2026-02-07 07:29:45 +03:00

Sign in to join this conversation.

Branches Tags

master

renovate/major-skiasharp-monorepo

renovate/sharpcompress-0.x

renovate/svg.skia-5.x

renovate/ghcr.io-devcontainers-features-docker-in-docker-4.x

release-10.11.z

standards-fix

release-10.9.z

openapi-cache

release-10.10.z

explicit-load

explicit-load-v2

feature/entity-framework

only-count-folder-item

fix/ci-workflow-refactor

release-10.8.z

release-10.7.z

EraYaN-add-management-interface

release-10.6.z

release-10.5.z

release-10.4.z

release-10.3.z

release-10.2.z

release-10.1.0

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/jellyfin#14434