themaster1970sf/TShock
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

Merge pull request #1320 from DogooFalchion/fix_playerlogin

Browse source 
Ensure IPs are > 0 otherwise the Linq will fail.
This commit is contained in:
Lucas Nicodemus 2016-10-23 16:23:34 -06:00 • committed by GitHub
commit 3fdc89a576
1 changed files with 10 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

4
TShockAPI/TShock.cs
View file

@ -409,6 +409,8 @@ namespace TShockAPI
KnownIps = JsonConvert.DeserializeObject<List<String>>(args.Player.User.KnownIps); KnownIps = JsonConvert.DeserializeObject<List<String>>(args.Player.User.KnownIps);
} }
if (KnownIps.Count > 0)
{
bool last = KnownIps.Last() == args.Player.IP; bool last = KnownIps.Last() == args.Player.IP;
if (!last) if (!last)
{ {
@ -419,7 +421,7 @@ namespace TShockAPI
KnownIps.Add(args.Player.IP); KnownIps.Add(args.Player.IP);
} }
}
args.Player.User.KnownIps = JsonConvert.SerializeObject(KnownIps, Formatting.Indented); args.Player.User.KnownIps = JsonConvert.SerializeObject(KnownIps, Formatting.Indented);
Users.UpdateLogin(args.Player.User); Users.UpdateLogin(args.Player.User);
} }