tommy/tbd-station-14
2
1
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix Loadout Requirements Only Displaying One Requirement (#28994)

Browse Source 
Collect all group effects failed instead of only first one failed
This commit is contained in:
Thomas
2024-06-14 15:32:39 -05:00
committed by GitHub
parent 2711ef7f48
commit 9ebfb1d64f
1 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
Content.Shared/Preferences/Loadouts/Effects/GroupLoadoutEffect.cs
View File

@@ -17,13 +17,16 @@ public sealed partial class GroupLoadoutEffect : LoadoutEffect
{ {
var effectsProto = collection.Resolve<IPrototypeManager>().Index(Proto); var effectsProto = collection.Resolve<IPrototypeManager>().Index(Proto);
var reasons = new List<string>();
foreach (var effect in effectsProto.Effects) foreach (var effect in effectsProto.Effects)
{ {
if (!effect.Validate(profile, loadout, session, collection, out reason)) if (effect.Validate(profile, loadout, session, collection, out reason))
return false; continue;
reasons.Add(reason.ToMarkup());
} }
reason = null; reason = reasons.Count == 0 ? null : FormattedMessage.FromMarkup(string.Join('\n', reasons));
return true; return reason == null;
} }
} }