avoid guest star/actor dupes
This commit is contained in:
parent
d792fa4359
commit
07d8649f97
|
@ -957,7 +957,7 @@ namespace MediaBrowser.Controller.Entities
|
||||||
{
|
{
|
||||||
if (person == null)
|
if (person == null)
|
||||||
{
|
{
|
||||||
throw new ArgumentNullException();
|
throw new ArgumentNullException("person");
|
||||||
}
|
}
|
||||||
|
|
||||||
if (string.IsNullOrWhiteSpace(person.Name))
|
if (string.IsNullOrWhiteSpace(person.Name))
|
||||||
|
@ -967,16 +967,39 @@ namespace MediaBrowser.Controller.Entities
|
||||||
|
|
||||||
if (People == null)
|
if (People == null)
|
||||||
{
|
{
|
||||||
People = new List<PersonInfo>();
|
People = new List<PersonInfo> { person };
|
||||||
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
// Check for dupes based on the combination of Name and Type
|
// If the type is GuestStar and there's already an Actor entry, then update it to avoid dupes
|
||||||
|
if (string.Equals(person.Type, PersonType.GuestStar, StringComparison.OrdinalIgnoreCase))
|
||||||
|
{
|
||||||
|
var existing = People.FirstOrDefault(p => p.Name.Equals(person.Name, StringComparison.OrdinalIgnoreCase) && p.Type.Equals(PersonType.Actor, StringComparison.OrdinalIgnoreCase));
|
||||||
|
|
||||||
|
if (existing != null)
|
||||||
|
{
|
||||||
|
existing.Type = PersonType.GuestStar;
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (string.Equals(person.Type, PersonType.Actor, StringComparison.OrdinalIgnoreCase))
|
||||||
|
{
|
||||||
|
// Only add actors if there isn't an existing one of type Actor or GuestStar
|
||||||
|
if (!People.Any(p => p.Name.Equals(person.Name, StringComparison.OrdinalIgnoreCase) && (p.Type.Equals(PersonType.Actor, StringComparison.OrdinalIgnoreCase) || p.Type.Equals(PersonType.GuestStar, StringComparison.OrdinalIgnoreCase))))
|
||||||
|
{
|
||||||
|
People.Add(person);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
// Check for dupes based on the combination of Name and Type
|
||||||
if (!People.Any(p => p.Name.Equals(person.Name, StringComparison.OrdinalIgnoreCase) && p.Type.Equals(person.Type, StringComparison.OrdinalIgnoreCase)))
|
if (!People.Any(p => p.Name.Equals(person.Name, StringComparison.OrdinalIgnoreCase) && p.Type.Equals(person.Type, StringComparison.OrdinalIgnoreCase)))
|
||||||
{
|
{
|
||||||
People.Add(person);
|
People.Add(person);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
/// <summary>
|
/// <summary>
|
||||||
/// Adds studios to the item
|
/// Adds studios to the item
|
||||||
|
|
Loading…
Reference in New Issue
Block a user