Be more strict about PersonType assignments (#15872)

This commit is contained in:
Tim Eisele
2026-01-17 03:03:13 +01:00
committed by GitHub
parent 093cfc3f3b
commit b56de6493f
5 changed files with 8 additions and 15 deletions

View File

@@ -303,9 +303,7 @@ namespace MediaBrowser.Providers.Plugins.Tmdb.Movies
CrewMember = crewMember, CrewMember = crewMember,
PersonType = TmdbUtils.MapCrewToPersonType(crewMember) PersonType = TmdbUtils.MapCrewToPersonType(crewMember)
}) })
.Where(entry => .Where(entry => TmdbUtils.WantedCrewKinds.Contains(entry.PersonType));
TmdbUtils.WantedCrewKinds.Contains(entry.PersonType) ||
TmdbUtils.WantedCrewTypes.Contains(entry.CrewMember.Job ?? string.Empty, StringComparison.OrdinalIgnoreCase));
if (config.HideMissingCrewMembers) if (config.HideMissingCrewMembers)
{ {

View File

@@ -275,9 +275,7 @@ namespace MediaBrowser.Providers.Plugins.Tmdb.TV
CrewMember = crewMember, CrewMember = crewMember,
PersonType = TmdbUtils.MapCrewToPersonType(crewMember) PersonType = TmdbUtils.MapCrewToPersonType(crewMember)
}) })
.Where(entry => .Where(entry => TmdbUtils.WantedCrewKinds.Contains(entry.PersonType));
TmdbUtils.WantedCrewKinds.Contains(entry.PersonType) ||
TmdbUtils.WantedCrewTypes.Contains(entry.CrewMember.Job ?? string.Empty, StringComparison.OrdinalIgnoreCase));
if (config.HideMissingCrewMembers) if (config.HideMissingCrewMembers)
{ {

View File

@@ -120,9 +120,7 @@ namespace MediaBrowser.Providers.Plugins.Tmdb.TV
CrewMember = crewMember, CrewMember = crewMember,
PersonType = TmdbUtils.MapCrewToPersonType(crewMember) PersonType = TmdbUtils.MapCrewToPersonType(crewMember)
}) })
.Where(entry => .Where(entry => TmdbUtils.WantedCrewKinds.Contains(entry.PersonType));
TmdbUtils.WantedCrewKinds.Contains(entry.PersonType) ||
TmdbUtils.WantedCrewTypes.Contains(entry.CrewMember.Job ?? string.Empty, StringComparison.OrdinalIgnoreCase));
if (config.HideMissingCrewMembers) if (config.HideMissingCrewMembers)
{ {

View File

@@ -367,9 +367,7 @@ namespace MediaBrowser.Providers.Plugins.Tmdb.TV
CrewMember = crewMember, CrewMember = crewMember,
PersonType = TmdbUtils.MapCrewToPersonType(crewMember) PersonType = TmdbUtils.MapCrewToPersonType(crewMember)
}) })
.Where(entry => .Where(entry => TmdbUtils.WantedCrewKinds.Contains(entry.PersonType));
TmdbUtils.WantedCrewKinds.Contains(entry.PersonType) ||
TmdbUtils.WantedCrewTypes.Contains(entry.CrewMember.Job ?? string.Empty, StringComparison.OrdinalIgnoreCase));
if (config.HideMissingCrewMembers) if (config.HideMissingCrewMembers)
{ {

View File

@@ -70,18 +70,19 @@ namespace MediaBrowser.Providers.Plugins.Tmdb
public static PersonKind MapCrewToPersonType(Crew crew) public static PersonKind MapCrewToPersonType(Crew crew)
{ {
if (crew.Department.Equals("production", StringComparison.OrdinalIgnoreCase) if (crew.Department.Equals("production", StringComparison.OrdinalIgnoreCase)
&& crew.Job.Contains("director", StringComparison.OrdinalIgnoreCase)) && crew.Job.Equals("director", StringComparison.OrdinalIgnoreCase))
{ {
return PersonKind.Director; return PersonKind.Director;
} }
if (crew.Department.Equals("production", StringComparison.OrdinalIgnoreCase) if (crew.Department.Equals("production", StringComparison.OrdinalIgnoreCase)
&& crew.Job.Contains("producer", StringComparison.OrdinalIgnoreCase)) && crew.Job.Equals("producer", StringComparison.OrdinalIgnoreCase))
{ {
return PersonKind.Producer; return PersonKind.Producer;
} }
if (crew.Department.Equals("writing", StringComparison.OrdinalIgnoreCase)) if (crew.Department.Equals("writing", StringComparison.OrdinalIgnoreCase)
&& crew.Job.Equals("writer", StringComparison.OrdinalIgnoreCase))
{ {
return PersonKind.Writer; return PersonKind.Writer;
} }