ToxicMushroom/Penumbra
1
0
Fork 0
mirror of https://github.com/xivdev/Penumbra.git synced 2025-12-12 18:27:24 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix redundancy check for Your Character.

Browse source
This commit is contained in:
Ottermandias 2023-04-29 14:21:08 +02:00
parent c2fe0d6ed1
commit d831b61c02
1 changed files with 27 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

27
Penumbra/Collections/Manager/ActiveCollections.cs
View file

@ -433,6 +433,33 @@ public class ActiveCollections : ISavable, IDisposable
switch (type)
{
// Yourself is redundant if
case CollectionType.Yourself:
var yourself = ByType(CollectionType.Yourself);
if (yourself == null)
return string.Empty;
var @base = ByType(CollectionType.Default);
var male = ByType(CollectionType.MalePlayerCharacter);
var female = ByType(CollectionType.FemalePlayerCharacter);
if (male == yourself && female == yourself)
return
"Assignment is redundant due to overwriting Male Players and Female Players with an identical collection.\nYou can remove it.";
if (male == null)
{
if (female == null && @base == yourself)
return "Assignment is redundant due to overwriting Base with an identical collection.\nYou can remove it.";
if (female == yourself && @base == yourself)
return
"Assignment is redundant due to overwriting Base and Female Players with an identical collection.\nYou can remove it.";
}
else if (male == yourself && female == null && @base == yourself)
{
return "Assignment is redundant due to overwriting Base and Male Players with an identical collection.\nYou can remove it.";
}
break;
// Check individual assignments. We can only be sure of redundancy for world-overlap or ownership overlap.
case CollectionType.Individual:
switch (id.Type)