[PR #14634] [MERGED] Fix GetBaseItemDto to return related item counts via SQL count #14056

New Issue

OVERLORD · 2026-02-07T07:23:38+03:00

OVERLORD commented

2026-02-07 07:23:38 +03:00

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/14634
Author: @crobibero
Created: 8/12/2025
Status: ✅ Merged
Merged: 8/13/2025
Merged by: @crobibero

Base: master ← Head: itemname-counts

📝 Commits (3)

0a4ff3f Fix GetBaseItemDto to return related item counts via SQL count
5eef85f move new TaggedItemCounts to its own file
beca405 Refactor to pull item counts in a single query

📊 Changes

6 files changed (+224 additions, -37 deletions)

View changed files

📝 Emby.Server.Implementations/Dto/DtoService.cs (+132 -37)
📝 Emby.Server.Implementations/Library/LibraryManager.cs (+19 -0)
📝 Jellyfin.Server.Implementations/Item/BaseItemRepository.cs (+60 -0)
📝 MediaBrowser.Controller/Library/ILibraryManager.cs (+2 -0)
📝 MediaBrowser.Controller/Persistence/IItemRepository.cs (+2 -0)
📝 MediaBrowser.Model/Dto/ItemCounts.cs (+9 -0)

📄 Description

Builds upon https://github.com/jellyfin/jellyfin/pull/14610 with a refactor to pull item counts in a single query

_{🔄 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/14634 **Author:** [@crobibero](https://github.com/crobibero) **Created:** 8/12/2025 **Status:** ✅ Merged **Merged:** 8/13/2025 **Merged by:** [@crobibero](https://github.com/crobibero) **Base:** `master` ← **Head:** `itemname-counts` --- ### 📝 Commits (3) - [`0a4ff3f`](https://github.com/jellyfin/jellyfin/commit/0a4ff3f3c0592cb32b7fb98bfc9f423386ecb84c) Fix GetBaseItemDto to return related item counts via SQL count - [`5eef85f`](https://github.com/jellyfin/jellyfin/commit/5eef85f027194491796ceb693057239858ac017c) move new TaggedItemCounts to its own file - [`beca405`](https://github.com/jellyfin/jellyfin/commit/beca405ad4c771f9cfdd5b410b4a4ba9f1c2d6c8) Refactor to pull item counts in a single query ### 📊 Changes **6 files changed** (+224 additions, -37 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Server.Implementations/Dto/DtoService.cs` (+132 -37) 📝 `Emby.Server.Implementations/Library/LibraryManager.cs` (+19 -0) 📝 `Jellyfin.Server.Implementations/Item/BaseItemRepository.cs` (+60 -0) 📝 `MediaBrowser.Controller/Library/ILibraryManager.cs` (+2 -0) 📝 `MediaBrowser.Controller/Persistence/IItemRepository.cs` (+2 -0) 📝 `MediaBrowser.Model/Dto/ItemCounts.cs` (+9 -0) </details> ### 📄 Description Builds upon https://github.com/jellyfin/jellyfin/pull/14610 with a refactor to pull item counts in a single query --- <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:23:38 +03:00

OVERLORD closed this issue

2026-02-07 07:23:39 +03:00

Sign in to join this conversation.

Branches Tags

master

release-10.11.z

renovate/skiasharp-monorepo

JPVenson-patch-1

renovate/morestachio-5.x

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#14056