]> git.smokeofanarchy.ru Git - space-station-14.git/commitdiff
DamageSpecifier Total => GetTotal (#24160)
authorLordCarve <27449516+LordCarve@users.noreply.github.com>
Mon, 22 Jan 2024 01:59:14 +0000 (02:59 +0100)
committerGitHub <noreply@github.com>
Mon, 22 Jan 2024 01:59:14 +0000 (18:59 -0700)
* DamageSpecifier [Obsolete] Total => GetTotal()

* Remove obsolete Total member from DamageSpecifier.

19 files changed:
Content.IntegrationTests/Tests/Construction/Interaction/WindowRepair.cs
Content.IntegrationTests/Tests/VendingMachineRestockTest.cs
Content.Server/Bed/Sleep/SleepingSystem.cs
Content.Server/Body/Systems/BloodstreamSystem.cs
Content.Server/Damage/ForceSay/DamageForceSaySystem.cs
Content.Server/Damage/Systems/DamageOtherOnHitSystem.cs
Content.Server/Damage/Systems/DamagePopupSystem.cs
Content.Server/KillTracking/KillTrackingSystem.cs
Content.Server/Light/EntitySystems/PoweredLightSystem.cs
Content.Server/Medical/HealingSystem.cs
Content.Server/NPC/Systems/NPCUtilitySystem.cs
Content.Server/Projectiles/ProjectileSystem.cs
Content.Server/Revenant/EntitySystems/RevenantSystem.cs
Content.Server/UserInterface/StatValuesCommand.cs
Content.Server/VendingMachines/VendingMachineSystem.cs
Content.Server/Weapons/Ranged/Systems/GunSystem.cs
Content.Shared/Damage/DamageSpecifier.cs
Content.Shared/Weapons/Melee/SharedMeleeWeaponSystem.cs
Content.Tests/Shared/DamageTest.cs

index 0bcc935fde3463397df9374a81b8076082d2fb2a..abd4bc265b30592156091d6888cb5c626f5bc479 100644 (file)
@@ -18,14 +18,14 @@ public sealed class WindowRepair : InteractionTest
         var comp = Comp<DamageableComponent>();
         var damageType = Server.ResolveDependency<IPrototypeManager>().Index<DamageTypePrototype>("Blunt");
         var damage = new DamageSpecifier(damageType, FixedPoint2.New(10));
-        Assert.That(comp.Damage.Total, Is.EqualTo(FixedPoint2.Zero));
+        Assert.That(comp.Damage.GetTotal(), Is.EqualTo(FixedPoint2.Zero));
         await Server.WaitPost(() => sys.TryChangeDamage(SEntMan.GetEntity(Target), damage, ignoreResistances: true));
         await RunTicks(5);
-        Assert.That(comp.Damage.Total, Is.GreaterThan(FixedPoint2.Zero));
+        Assert.That(comp.Damage.GetTotal(), Is.GreaterThan(FixedPoint2.Zero));
 
         // Repair the entity
         await Interact(Weld);
-        Assert.That(comp.Damage.Total, Is.EqualTo(FixedPoint2.Zero));
+        Assert.That(comp.Damage.GetTotal(), Is.EqualTo(FixedPoint2.Zero));
 
         // Validate that we can still deconstruct the entity (i.e., that welding deconstruction is not blocked).
         await Interact(
index 49fee976317fea2b93b3842092c030cc219aee3f..99481db70e71e78153fd8a5cc63deaae79fb706a 100644 (file)
@@ -298,7 +298,7 @@ namespace Content.IntegrationTests.Tests
                 Assert.That(damageResult, Is.Not.Null,
                     "Received null damageResult when attempting to damage restock box.");
 
-                Assert.That((int) damageResult!.Total, Is.GreaterThan(0),
+                Assert.That((int) damageResult!.GetTotal(), Is.GreaterThan(0),
                     "Box damage result was not greater than 0.");
 #pragma warning restore NUnit2045
             });
index 4a6874bcccdd9980bfdfa48d446cacca73908e4f..5d1def7cec11604609471bf776643ac5303d5c7a 100644 (file)
@@ -80,14 +80,14 @@ namespace Content.Server.Bed.Sleep
         }
 
         /// <summary>
-        /// Wake up if we take an instance of more than 2 damage.
+        /// Wake up on taking an instance of damage at least the value of WakeThreshold.
         /// </summary>
         private void OnDamageChanged(EntityUid uid, SleepingComponent component, DamageChangedEvent args)
         {
             if (!args.DamageIncreased || args.DamageDelta == null)
                 return;
 
-            if (args.DamageDelta.Total >= component.WakeThreshold)
+            if (args.DamageDelta.GetTotal() >= component.WakeThreshold)
                 TryWaking(uid, component);
         }
 
index 21dc2f4526c65e74f99789305895dfeabe18a86a..f6fdcfedff49f0c67475b17f4ed0112db3b09bdc 100644 (file)
@@ -187,7 +187,7 @@ public sealed class BloodstreamSystem : EntitySystem
 
         // Does the calculation of how much bleed rate should be added/removed, then applies it
         var oldBleedAmount = component.BleedAmount;
-        var total = bloodloss.Total;
+        var total = bloodloss.GetTotal();
         var totalFloat = total.Float();
         TryModifyBleedAmount(uid, totalFloat, component);
 
index 5bc30cce5daef698b1a692bddbd847c4f104f090..186fc91c46e9e973e124154cb8f8f881a655da2e 100644 (file)
@@ -1,4 +1,4 @@
-using Content.Shared.Damage;
+using Content.Shared.Damage;
 using Content.Shared.Damage.ForceSay;
 using Content.Shared.FixedPoint;
 using Content.Shared.Mobs;
@@ -90,7 +90,7 @@ public sealed class DamageForceSaySystem : EntitySystem
 
     private void OnDamageChanged(EntityUid uid, DamageForceSayComponent component, DamageChangedEvent args)
     {
-        if (args.DamageDelta == null || !args.DamageIncreased || args.DamageDelta.Total < component.DamageThreshold)
+        if (args.DamageDelta == null || !args.DamageIncreased || args.DamageDelta.GetTotal() < component.DamageThreshold)
             return;
 
         if (component.ValidDamageGroups != null)
index c7600ec2212f86c7a9c578aebded8d6929b576a0..0efa534981510ed33e19e8c87ca965b17325edf1 100644 (file)
@@ -36,7 +36,7 @@ namespace Content.Server.Damage.Systems
 
             // Log damage only for mobs. Useful for when people throw spears at each other, but also avoids log-spam when explosions send glass shards flying.
             if (dmg != null && HasComp<MobStateComponent>(args.Target))
-                _adminLogger.Add(LogType.ThrowHit, $"{ToPrettyString(args.Target):target} received {dmg.Total:damage} damage from collision");
+                _adminLogger.Add(LogType.ThrowHit, $"{ToPrettyString(args.Target):target} received {dmg.GetTotal():damage} damage from collision");
 
             if (dmg is { Empty: false })
             {
index 4cd8bf46261dca3e06d30dae215bbffd6da5238b..12fd894ac6f8980546a32a4716d075ca6ee4ebc5 100644 (file)
@@ -20,7 +20,7 @@ public sealed class DamagePopupSystem : EntitySystem
         if (args.DamageDelta != null)
         {
             var damageTotal = args.Damageable.TotalDamage;
-            var damageDelta = args.DamageDelta.Total;
+            var damageDelta = args.DamageDelta.GetTotal();
 
             var msg = component.Type switch
             {
index afb4283e82d2436da8d33897d729bd8a386f5acf..63627fd1b97198a47da18a2e87a793461ed79d0d 100644 (file)
@@ -1,4 +1,4 @@
-using Content.Server.NPC.HTN;
+using Content.Server.NPC.HTN;
 using Content.Shared.Damage;
 using Content.Shared.FixedPoint;
 using Content.Shared.Mobs;
@@ -27,7 +27,7 @@ public sealed class KillTrackingSystem : EntitySystem
         {
             foreach (var key in component.LifetimeDamage.Keys)
             {
-                component.LifetimeDamage[key] -= args.DamageDelta.Total;
+                component.LifetimeDamage[key] -= args.DamageDelta.GetTotal();
             }
 
             return;
@@ -35,7 +35,7 @@ public sealed class KillTrackingSystem : EntitySystem
 
         var source = GetKillSource(args.Origin);
         var damage = component.LifetimeDamage.GetValueOrDefault(source);
-        component.LifetimeDamage[source] = damage + args.DamageDelta.Total;
+        component.LifetimeDamage[source] = damage + args.DamageDelta.GetTotal();
     }
 
     private void OnMobStateChanged(EntityUid uid, KillTrackerComponent component, MobStateChangedEvent args)
index 46b8506b3dc6923a394c86e82006e3abe4794b34..e6cde4495cef82301ceecc696525ec57f299ccd8 100644 (file)
@@ -135,7 +135,7 @@ namespace Content.Server.Light.EntitySystems
 
                         var burnMsg = Loc.GetString("powered-light-component-burn-hand");
                         _popupSystem.PopupEntity(burnMsg, uid, userUid);
-                        _adminLogger.Add(LogType.Damaged, $"{ToPrettyString(args.User):user} burned their hand on {ToPrettyString(args.Target):target} and received {damage.Total:damage} damage");
+                        _adminLogger.Add(LogType.Damaged, $"{ToPrettyString(args.User):user} burned their hand on {ToPrettyString(args.Target):target} and received {damage.GetTotal():damage} damage");
                         _audio.PlayEntity(light.BurnHandSound, Filter.Pvs(uid), uid, true);
 
                         args.Handled = true;
index 69d3b04cc18189c83513ce79d2b184dca35a00ee..1b90d195c4e997adce8d90b339fc5255dad683c6 100644 (file)
@@ -84,7 +84,7 @@ public sealed class HealingSystem : EntitySystem
         if (healed == null && healing.BloodlossModifier != 0)
             return;
 
-        var total = healed?.Total ?? FixedPoint2.Zero;
+        var total = healed?.GetTotal() ?? FixedPoint2.Zero;
 
         // Re-verify that we can heal the damage.
 
index 76d0ebe83a4cca3044472f1376d374e1ac6881c0..fc483f68c3be72ecda6b2191939a9b0f09cab826 100644 (file)
@@ -331,7 +331,7 @@ public sealed class NPCUtilitySystem : EntitySystem
             {
                 if (TryComp<MeleeWeaponComponent>(targetUid, out var melee))
                 {
-                    return melee.Damage.Total.Float() * melee.AttackRate / 100f;
+                    return melee.Damage.GetTotal().Float() * melee.AttackRate / 100f;
                 }
 
                 return 0f;
index 85feaf2b8319b46ce2669e8566e0e38f142f94d7..15ea5936e13921d63c2a57ea1d57765dd0fa530e 100644 (file)
@@ -58,7 +58,7 @@ public sealed class ProjectileSystem : SharedProjectileSystem
 
             _adminLogger.Add(LogType.BulletHit,
                 HasComp<ActorComponent>(target) ? LogImpact.Extreme : LogImpact.High,
-                $"Projectile {ToPrettyString(uid):projectile} shot by {ToPrettyString(component.Shooter!.Value):user} hit {otherName:target} and dealt {modifiedDamage.Total:damage} damage");
+                $"Projectile {ToPrettyString(uid):projectile} shot by {ToPrettyString(component.Shooter!.Value):user} hit {otherName:target} and dealt {modifiedDamage.GetTotal():damage} damage");
         }
 
         if (!deleted)
index d5e1ae1af899b4d989139d372092b96b613f1895..b26cc4a320fb44a98222ef1438c43d5564d84732 100644 (file)
@@ -120,7 +120,7 @@ public sealed partial class RevenantSystem : EntitySystem
         if (!HasComp<CorporealComponent>(uid) || args.DamageDelta == null)
             return;
 
-        var essenceDamage = args.DamageDelta.Total.Float() * component.DamageToEssenceCoefficient * -1;
+        var essenceDamage = args.DamageDelta.GetTotal().Float() * component.DamageToEssenceCoefficient * -1;
         ChangeEssenceAmount(uid, essenceDamage, component);
     }
 
index 9ec38753d404f07025b341a184a08fdfc5d87bec..f0c4f531d0183ec2e8e3e16c18182637ab5df130 100644 (file)
@@ -190,9 +190,9 @@ public sealed class StatValuesCommand : IConsoleCommand
             values.Add(new[]
             {
                 proto.ID,
-                (comp.Damage.Total * comp.AttackRate).ToString(),
+                (comp.Damage.GetTotal() * comp.AttackRate).ToString(),
                 comp.AttackRate.ToString(CultureInfo.CurrentCulture),
-                comp.Damage.Total.ToString(),
+                comp.Damage.GetTotal().ToString(),
                 comp.Range.ToString(CultureInfo.CurrentCulture),
             });
         }
index da024888a50d2f1fbca7b436370db532ac1f644f..f2c7e6dd19b9ab4d7378b59f7a60859f79536f7e 100644 (file)
@@ -167,7 +167,7 @@ namespace Content.Server.VendingMachines
                 component.DispenseOnHitChance == null || args.DamageDelta == null)
                 return;
 
-            if (args.DamageIncreased && args.DamageDelta.Total >= component.DispenseOnHitThreshold &&
+            if (args.DamageIncreased && args.DamageDelta.GetTotal() >= component.DispenseOnHitThreshold &&
                 _random.Prob(component.DispenseOnHitChance.Value))
             {
                 if (component.DispenseOnHitCooldown > 0f)
index 4f236c0fa7498472feac556b2b4d9c4d57a089cc..4e4bdf78b101f4dd9dfa275e88185d2d5a61e4ce 100644 (file)
@@ -274,12 +274,12 @@ public sealed partial class GunSystem : SharedGunSystem
                             if (user != null)
                             {
                                 Logs.Add(LogType.HitScanHit,
-                                    $"{ToPrettyString(user.Value):user} hit {hitName:target} using hitscan and dealt {dmg.Total:damage} damage");
+                                    $"{ToPrettyString(user.Value):user} hit {hitName:target} using hitscan and dealt {dmg.GetTotal():damage} damage");
                             }
                             else
                             {
                                 Logs.Add(LogType.HitScanHit,
-                                    $"{hitName:target} hit by hitscan dealing {dmg.Total:damage} damage");
+                                    $"{hitName:target} hit by hitscan dealing {dmg.GetTotal():damage} damage");
                             }
                         }
                     }
@@ -371,7 +371,7 @@ public sealed partial class GunSystem : SharedGunSystem
         // 3. Nothing
         var playedSound = false;
 
-        if (!forceWeaponSound && modifiedDamage != null && modifiedDamage.Total > 0 && TryComp<RangedDamageSoundComponent>(otherEntity, out var rangedSound))
+        if (!forceWeaponSound && modifiedDamage != null && modifiedDamage.GetTotal() > 0 && TryComp<RangedDamageSoundComponent>(otherEntity, out var rangedSound))
         {
             var type = SharedMeleeWeaponSystem.GetHighestDamageSound(modifiedDamage, ProtoManager);
 
index 872bad05514481932614033623f5837aa339b420..8ab9116a975e0f3fb78d394622200ea8f31ffed1 100644 (file)
@@ -38,10 +38,6 @@ namespace Content.Shared.Damage
         [IncludeDataField(customTypeSerializer: typeof(DamageSpecifierDictionarySerializer), readOnly: true)]
         public Dictionary<string, FixedPoint2> DamageDict { get; set; } = new();
 
-        [JsonIgnore]
-        [Obsolete("Use GetTotal()")]
-        public FixedPoint2 Total => GetTotal();
-
         /// <summary>
         ///     Returns a sum of the damage values.
         /// </summary>
index 11228bba9ecce0a9b67e4c1420b5ddeaae85bb51..f7ac249f5f23ff8af15e0cac33ce129c37be09fc 100644 (file)
@@ -516,12 +516,12 @@ public abstract class SharedMeleeWeaponSystem : EntitySystem
             if (meleeUid == user)
             {
                 AdminLogger.Add(LogType.MeleeHit, LogImpact.Medium,
-                    $"{ToPrettyString(user):actor} melee attacked (light) {ToPrettyString(target.Value):subject} using their hands and dealt {damageResult.Total:damage} damage");
+                    $"{ToPrettyString(user):actor} melee attacked (light) {ToPrettyString(target.Value):subject} using their hands and dealt {damageResult.GetTotal():damage} damage");
             }
             else
             {
                 AdminLogger.Add(LogType.MeleeHit, LogImpact.Medium,
-                    $"{ToPrettyString(user):actor} melee attacked (light) {ToPrettyString(target.Value):subject} using {ToPrettyString(meleeUid):tool} and dealt {damageResult.Total:damage} damage");
+                    $"{ToPrettyString(user):actor} melee attacked (light) {ToPrettyString(target.Value):subject} using {ToPrettyString(meleeUid):tool} and dealt {damageResult.GetTotal():damage} damage");
             }
 
             PlayHitSound(target.Value, user, GetHighestDamageSound(modifiedDamage, _protoManager), hitEvent.HitSoundOverride, component.HitSound);
@@ -542,7 +542,7 @@ public abstract class SharedMeleeWeaponSystem : EntitySystem
             }
         }
 
-        if (damageResult?.Total > FixedPoint2.Zero)
+        if (damageResult?.GetTotal() > FixedPoint2.Zero)
         {
             DoDamageEffect(targets, user, targetXform);
         }
@@ -670,7 +670,7 @@ public abstract class SharedMeleeWeaponSystem : EntitySystem
                 else
                 {
                     AdminLogger.Add(LogType.MeleeHit, LogImpact.Medium,
-                        $"{ToPrettyString(user):actor} melee attacked (heavy) {ToPrettyString(entity):subject} using {ToPrettyString(meleeUid):tool} and dealt {damageResult.Total:damage} damage");
+                        $"{ToPrettyString(user):actor} melee attacked (heavy) {ToPrettyString(entity):subject} using {ToPrettyString(meleeUid):tool} and dealt {damageResult.GetTotal():damage} damage");
                 }
             }
         }
index 66f85cf1cce11433485692c871c9ca3207d6baba..b06b8368a900dfb4ea76ffb0758e5ec6eb62a356 100644 (file)
@@ -61,7 +61,7 @@ namespace Content.Tests.Shared
 
             // Check that it properly split up the groups into types
             FixedPoint2 damage;
-            Assert.That(damageSpec.Total, Is.EqualTo(FixedPoint2.New(8)));
+            Assert.That(damageSpec.GetTotal(), Is.EqualTo(FixedPoint2.New(8)));
             Assert.That(damageSpec.DamageDict.TryGetValue("Blunt", out damage));
             Assert.That(damage, Is.EqualTo(FixedPoint2.New(2)));
             Assert.That(damageSpec.DamageDict.TryGetValue("Piercing", out damage));
@@ -73,7 +73,7 @@ namespace Content.Tests.Shared
 
             // check that integer multiplication works
             damageSpec = damageSpec * 2;
-            Assert.That(damageSpec.Total, Is.EqualTo(FixedPoint2.New(16)));
+            Assert.That(damageSpec.GetTotal(), Is.EqualTo(FixedPoint2.New(16)));
             Assert.That(damageSpec.DamageDict.TryGetValue("Blunt", out damage));
             Assert.That(damage, Is.EqualTo(FixedPoint2.New(4)));
             Assert.That(damageSpec.DamageDict.TryGetValue("Piercing", out damage));
@@ -93,7 +93,7 @@ namespace Content.Tests.Shared
             Assert.That(damage, Is.EqualTo(FixedPoint2.New(4.4)));
             Assert.That(damageSpec.DamageDict.TryGetValue("Radiation", out damage));
             Assert.That(damage, Is.EqualTo(FixedPoint2.New(13.2)));
-            Assert.That(damageSpec.Total, Is.EqualTo(FixedPoint2.New(8.8 + 8.8 + 4.4 + 13.2)));
+            Assert.That(damageSpec.GetTotal(), Is.EqualTo(FixedPoint2.New(8.8 + 8.8 + 4.4 + 13.2)));
 
             // check that integer division works
             damageSpec = damageSpec / 2;