warrentyexpired/ModernUO
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: Fixes guild roster crash (#906)

Browse source
This commit is contained in:
Kamron Batman 2021-12-29 10:18:27 -08:00 • committed by GitHub
parent c1134c526f
commit 752407d6f4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

8
Projects/UOContent/Gumps/Guilds/New Guild System/BaseGuildListGump.cs
View file

@ -31,7 +31,7 @@ namespace Server.Guilds
m_List = list; m_List = list;
} }
public virtual bool WillFilter => m_Filter.Length >= 0; public virtual bool WillFilter => m_Filter.Length > 0;
public override void PopulateGump() public override void PopulateGump()
{ {
@ -55,7 +55,7 @@ namespace Server.Guilds
} }
m_List.Sort(m_Comparer); m_List.Sort(m_Comparer);
m_StartNumber = Math.Clamp(m_StartNumber, 0, m_List.Count - 1); m_StartNumber = Math.Max(Math.Min(m_StartNumber, m_List.Count - 1), 0);
AddBackground(130, 75, 385, 30, 0xBB8); AddBackground(130, 75, 385, 30, 0xBB8);
AddTextEntry(135, 80, 375, 30, 0x481, 1, m_Filter); AddTextEntry(135, 80, 375, 30, 0x481, 1, m_Filter);
@ -109,8 +109,8 @@ namespace Server.Guilds
else // descending, go from bottom of list to the top else // descending, go from bottom of list to the top
{ {
for (var i = m_List.Count - 1 - m_StartNumber; for (var i = m_List.Count - 1 - m_StartNumber;
i >= 0 && i >= m_List.Count - itemsPerPage - m_StartNumber; i >= 0 && i >= m_List.Count - itemsPerPage - m_StartNumber;
i--) i--)
{ {
DrawEntry(m_List[i], i, itemNumber++); DrawEntry(m_List[i], i, itemNumber++);
} }