< Summary - Jellyfin

Information
Class: Jellyfin.Server.Implementations.Item.PeopleRepository
Assembly: Jellyfin.Server.Implementations
File(s): /srv/git/jellyfin/Jellyfin.Server.Implementations/Item/PeopleRepository.cs
Line coverage
0%
Covered lines: 1
Uncovered lines: 114
Coverable lines: 115
Total lines: 254
Line coverage: 0.8%
Branch coverage
0%
Covered branches: 0
Total branches: 54
Branch coverage: 0%
Method coverage

Feature is only available for sponsors

Upgrade to PRO version

Coverage history

Coverage history 0 25 50 75 100 11/7/2025 - 12:11:55 AM Line coverage: 0.8% (1/114) Branch coverage: 0% (0/50) Total lines: 2532/6/2026 - 12:13:21 AM Line coverage: 0.8% (1/115) Branch coverage: 0% (0/54) Total lines: 254 11/7/2025 - 12:11:55 AM Line coverage: 0.8% (1/114) Branch coverage: 0% (0/50) Total lines: 2532/6/2026 - 12:13:21 AM Line coverage: 0.8% (1/115) Branch coverage: 0% (0/54) Total lines: 254

Metrics

MethodBranch coverage Crap Score Cyclomatic complexity Line coverage
.ctor(...)100%11100%
GetPeople(...)0%2040%
GetPeopleNames(...)0%620%
UpdatePeople(...)0%210140%
Map(...)0%7280%
Map(...)100%210%
TranslateQuery(...)0%342180%
IsAlphaNumeric(...)0%7280%
IsValidPersonType(...)100%210%

File(s)

/srv/git/jellyfin/Jellyfin.Server.Implementations/Item/PeopleRepository.cs

#LineLine coverage
 1using System;
 2using System.Collections.Generic;
 3using System.Collections.Immutable;
 4using System.Linq;
 5using Jellyfin.Data.Enums;
 6using Jellyfin.Database.Implementations;
 7using Jellyfin.Database.Implementations.Entities;
 8using Jellyfin.Database.Implementations.Entities.Libraries;
 9using Jellyfin.Extensions;
 10using MediaBrowser.Controller.Entities;
 11using MediaBrowser.Controller.Persistence;
 12using Microsoft.EntityFrameworkCore;
 13
 14namespace Jellyfin.Server.Implementations.Item;
 15#pragma warning disable RS0030 // Do not use banned APIs
 16#pragma warning disable CA1304 // Specify CultureInfo
 17#pragma warning disable CA1311 // Specify a culture or use an invariant version
 18#pragma warning disable CA1862 // Use the 'StringComparison' method overloads to perform case-insensitive string compari
 19
 20/// <summary>
 21/// Manager for handling people.
 22/// </summary>
 23/// <param name="dbProvider">Efcore Factory.</param>
 24/// <param name="itemTypeLookup">Items lookup service.</param>
 25/// <remarks>
 26/// Initializes a new instance of the <see cref="PeopleRepository"/> class.
 27/// </remarks>
 28public class PeopleRepository(IDbContextFactory<JellyfinDbContext> dbProvider, IItemTypeLookup itemTypeLookup) : IPeople
 29{
 2130    private readonly IDbContextFactory<JellyfinDbContext> _dbProvider = dbProvider;
 31
 32    /// <inheritdoc/>
 33    public IReadOnlyList<PersonInfo> GetPeople(InternalPeopleQuery filter)
 34    {
 035        using var context = _dbProvider.CreateDbContext();
 036        var dbQuery = TranslateQuery(context.Peoples.AsNoTracking(), context, filter);
 37
 38        // Include PeopleBaseItemMap
 039        if (!filter.ItemId.IsEmpty())
 40        {
 041            dbQuery = dbQuery.Include(p => p.BaseItems!.Where(m => m.ItemId == filter.ItemId))
 042                .OrderBy(e => e.BaseItems!.First(e => e.ItemId == filter.ItemId).ListOrder)
 043                .ThenBy(e => e.PersonType)
 044                .ThenBy(e => e.Name);
 45        }
 46        else
 47        {
 048            dbQuery = dbQuery.OrderBy(e => e.Name);
 49        }
 50
 051        if (filter.Limit > 0)
 52        {
 053            dbQuery = dbQuery.Take(filter.Limit);
 54        }
 55
 056        return dbQuery.AsEnumerable().Select(Map).ToArray();
 057    }
 58
 59    /// <inheritdoc/>
 60    public IReadOnlyList<string> GetPeopleNames(InternalPeopleQuery filter)
 61    {
 062        using var context = _dbProvider.CreateDbContext();
 063        var dbQuery = TranslateQuery(context.Peoples.AsNoTracking(), context, filter).Select(e => e.Name).Distinct();
 64
 65        // dbQuery = dbQuery.OrderBy(e => e.ListOrder);
 066        if (filter.Limit > 0)
 67        {
 068            dbQuery = dbQuery.Take(filter.Limit);
 69        }
 70
 071        return dbQuery.ToArray();
 072    }
 73
 74    /// <inheritdoc />
 75    public void UpdatePeople(Guid itemId, IReadOnlyList<PersonInfo> people)
 76    {
 077        foreach (var person in people)
 78        {
 079            person.Name = person.Name.Trim();
 080            person.Role = person.Role?.Trim() ?? string.Empty;
 81        }
 82
 83        // multiple metadata providers can provide the _same_ person
 084        people = people.DistinctBy(e => e.Name + "-" + e.Type).ToArray();
 085        var personKeys = people.Select(e => e.Name + "-" + e.Type).ToArray();
 86
 087        using var context = _dbProvider.CreateDbContext();
 088        using var transaction = context.Database.BeginTransaction();
 089        var existingPersons = context.Peoples.Select(e => new
 090            {
 091                item = e,
 092                SelectionKey = e.Name + "-" + e.PersonType
 093            })
 094            .Where(p => personKeys.Contains(p.SelectionKey))
 095            .Select(f => f.item)
 096            .ToArray();
 97
 098        var toAdd = people
 099            .Where(e => e.Type is not PersonKind.Artist && e.Type is not PersonKind.AlbumArtist)
 0100            .Where(e => !existingPersons.Any(f => f.Name == e.Name && f.PersonType == e.Type.ToString()))
 0101            .Select(Map);
 0102        context.Peoples.AddRange(toAdd);
 0103        context.SaveChanges();
 104
 0105        var personsEntities = toAdd.Concat(existingPersons).ToArray();
 106
 0107        var existingMaps = context.PeopleBaseItemMap.Include(e => e.People).Where(e => e.ItemId == itemId).ToList();
 108
 0109        var listOrder = 0;
 110
 0111        foreach (var person in people)
 112        {
 0113            if (person.Type == PersonKind.Artist || person.Type == PersonKind.AlbumArtist)
 114            {
 115                continue;
 116            }
 117
 0118            var entityPerson = personsEntities.First(e => e.Name == person.Name && e.PersonType == person.Type.ToString(
 0119            var existingMap = existingMaps.FirstOrDefault(e => e.People.Name == person.Name && e.People.PersonType == pe
 0120            if (existingMap is null)
 121            {
 0122                context.PeopleBaseItemMap.Add(new PeopleBaseItemMap()
 0123                {
 0124                    Item = null!,
 0125                    ItemId = itemId,
 0126                    People = null!,
 0127                    PeopleId = entityPerson.Id,
 0128                    ListOrder = listOrder,
 0129                    SortOrder = person.SortOrder,
 0130                    Role = person.Role
 0131                });
 132            }
 133            else
 134            {
 135                // Update the order for existing mappings
 0136                existingMap.ListOrder = listOrder;
 0137                existingMap.SortOrder = person.SortOrder;
 138                // person mapping already exists so remove from list
 0139                existingMaps.Remove(existingMap);
 140            }
 141
 0142            listOrder++;
 143        }
 144
 0145        context.PeopleBaseItemMap.RemoveRange(existingMaps);
 146
 0147        context.SaveChanges();
 0148        transaction.Commit();
 0149    }
 150
 151    private PersonInfo Map(People people)
 152    {
 0153        var mapping = people.BaseItems?.FirstOrDefault();
 0154        var personInfo = new PersonInfo()
 0155        {
 0156            Id = people.Id,
 0157            Name = people.Name,
 0158            Role = mapping?.Role,
 0159            SortOrder = mapping?.SortOrder
 0160        };
 0161        if (Enum.TryParse<PersonKind>(people.PersonType, out var kind))
 162        {
 0163            personInfo.Type = kind;
 164        }
 165
 0166        return personInfo;
 167    }
 168
 169    private People Map(PersonInfo people)
 170    {
 0171        var personInfo = new People()
 0172        {
 0173            Name = people.Name,
 0174            PersonType = people.Type.ToString(),
 0175            Id = people.Id,
 0176        };
 177
 0178        return personInfo;
 179    }
 180
 181    private IQueryable<People> TranslateQuery(IQueryable<People> query, JellyfinDbContext context, InternalPeopleQuery f
 182    {
 0183        if (filter.User is not null && filter.IsFavorite.HasValue)
 184        {
 0185            var personType = itemTypeLookup.BaseItemKindNames[BaseItemKind.Person];
 0186            var oldQuery = query;
 187
 0188            query = context.UserData
 0189                .Where(u => u.Item!.Type == personType && u.IsFavorite == filter.IsFavorite && u.UserId.Equals(filter.Us
 0190                .Join(oldQuery, e => e.Item!.Name, e => e.Name, (item, person) => person)
 0191                .Distinct()
 0192                .AsNoTracking();
 193        }
 194
 0195        if (!filter.ItemId.IsEmpty())
 196        {
 0197            query = query.Where(e => e.BaseItems!.Any(w => w.ItemId.Equals(filter.ItemId)));
 198        }
 199
 0200        if (!filter.AppearsInItemId.IsEmpty())
 201        {
 0202            query = query.Where(e => e.BaseItems!.Any(w => w.ItemId.Equals(filter.AppearsInItemId)));
 203        }
 204
 0205        var queryPersonTypes = filter.PersonTypes.Where(IsValidPersonType).ToList();
 0206        if (queryPersonTypes.Count > 0)
 207        {
 0208            query = query.Where(e => queryPersonTypes.Contains(e.PersonType));
 209        }
 210
 0211        var queryExcludePersonTypes = filter.ExcludePersonTypes.Where(IsValidPersonType).ToList();
 212
 0213        if (queryExcludePersonTypes.Count > 0)
 214        {
 0215            query = query.Where(e => !queryPersonTypes.Contains(e.PersonType));
 216        }
 217
 0218        if (filter.MaxListOrder.HasValue && !filter.ItemId.IsEmpty())
 219        {
 0220            query = query.Where(e => e.BaseItems!.First(w => w.ItemId == filter.ItemId).ListOrder <= filter.MaxListOrder
 221        }
 222
 0223        if (!string.IsNullOrWhiteSpace(filter.NameContains))
 224        {
 0225            var nameContainsUpper = filter.NameContains.ToUpper();
 0226            query = query.Where(e => e.Name.ToUpper().Contains(nameContainsUpper));
 227        }
 228
 0229        return query;
 230    }
 231
 232    private bool IsAlphaNumeric(string str)
 233    {
 0234        if (string.IsNullOrWhiteSpace(str))
 235        {
 0236            return false;
 237        }
 238
 0239        for (int i = 0; i < str.Length; i++)
 240        {
 0241            if (!char.IsLetter(str[i]) && !char.IsNumber(str[i]))
 242            {
 0243                return false;
 244            }
 245        }
 246
 0247        return true;
 248    }
 249
 250    private bool IsValidPersonType(string value)
 251    {
 0252        return IsAlphaNumeric(value);
 253    }
 254}