fix bug with AC failing to ban because of EF issue.
This commit is contained in:
parent
16d7ccd590
commit
6071ad8653
@ -180,6 +180,10 @@ namespace IW4MAdmin
|
||||
{
|
||||
client.CurrentAlias = existingAlias;
|
||||
client.CurrentAliasId = existingAlias.AliasId;
|
||||
// we need to update their new ip and name to the virtual property
|
||||
client.Name = polledPlayer.Name;
|
||||
client.IPAddress = polledPlayer.IPAddress;
|
||||
|
||||
client = await Manager.GetClientService().Update(client);
|
||||
}
|
||||
|
||||
|
@ -152,7 +152,8 @@ namespace IW4MAdmin.Plugins.Stats.Cheat
|
||||
}
|
||||
|
||||
// ban
|
||||
if (currentStrain > Thresholds.MaxStrainBan)
|
||||
if (currentStrain > Thresholds.MaxStrainBan &&
|
||||
HitCount > 15)
|
||||
{
|
||||
result = new DetectionPenaltyResult()
|
||||
{
|
||||
|
@ -427,7 +427,8 @@ namespace IW4MAdmin.Plugins.Stats.Helpers
|
||||
{
|
||||
using (var ctx = new DatabaseContext())
|
||||
{
|
||||
foreach (var change in clientDetection.Tracker.GetChanges())
|
||||
// todo: why does this cause duplicate primary key
|
||||
foreach (var change in clientDetection.Tracker.GetChanges().Distinct())
|
||||
{
|
||||
ctx.Add(change);
|
||||
}
|
||||
@ -440,7 +441,6 @@ namespace IW4MAdmin.Plugins.Stats.Helpers
|
||||
case Penalty.PenaltyType.Ban:
|
||||
if (attacker.Level == Player.Permission.Banned)
|
||||
break;
|
||||
await saveLog();
|
||||
await attacker.Ban(Utilities.CurrentLocalization.LocalizationIndex["PLUGIN_STATS_CHEAT_DETECTED"], new Player()
|
||||
{
|
||||
ClientId = 1,
|
||||
@ -454,6 +454,7 @@ namespace IW4MAdmin.Plugins.Stats.Helpers
|
||||
}
|
||||
}
|
||||
});
|
||||
await saveLog();
|
||||
break;
|
||||
case Penalty.PenaltyType.Flag:
|
||||
if (attacker.Level != Player.Permission.User)
|
||||
|
Loading…
Reference in New Issue
Block a user