diff --git a/Content.Server/Access/Systems/PresetIdCardSystem.cs b/Content.Server/Access/Systems/PresetIdCardSystem.cs index f253d0e064..086c1143dd 100644 --- a/Content.Server/Access/Systems/PresetIdCardSystem.cs +++ b/Content.Server/Access/Systems/PresetIdCardSystem.cs @@ -60,7 +60,7 @@ namespace Content.Server.Access.Systems if (!_prototypeManager.TryIndex(id.JobName, out JobPrototype? job)) { - Logger.ErrorS("access", $"Invalid job id ({id.JobName}) for preset card"); + Log.Error($"Invalid job id ({id.JobName}) for preset card"); return; } diff --git a/Content.Server/Administration/Systems/BwoinkSystem.cs b/Content.Server/Administration/Systems/BwoinkSystem.cs index 822fbf18f1..3f602beed7 100644 --- a/Content.Server/Administration/Systems/BwoinkSystem.cs +++ b/Content.Server/Administration/Systems/BwoinkSystem.cs @@ -130,13 +130,13 @@ namespace Content.Server.Administration.Systems if (!match.Success) { // TODO: Ideally, CVar validation during setting should be better integrated - Logger.Warning("Webhook URL does not appear to be valid. Using anyways..."); + Log.Warning("Webhook URL does not appear to be valid. Using anyways..."); return; } if (match.Groups.Count <= 2) { - Logger.Error("Could not get webhook ID or token."); + Log.Error("Could not get webhook ID or token."); return; } diff --git a/Content.Server/AirlockPainter/AirlockPainterSystem.cs b/Content.Server/AirlockPainter/AirlockPainterSystem.cs index 59f0d6ae7b..2626c410fb 100644 --- a/Content.Server/AirlockPainter/AirlockPainterSystem.cs +++ b/Content.Server/AirlockPainter/AirlockPainterSystem.cs @@ -71,7 +71,7 @@ namespace Content.Server.AirlockPainter if (!_prototypeManager.TryIndex(airlock.Group, out var grp)) { - Logger.Error("Group not defined: %s", airlock.Group); + Log.Error("Group not defined: %s", airlock.Group); return; } diff --git a/Content.Server/Anomaly/Effects/ProjectileAnomalySystem.cs b/Content.Server/Anomaly/Effects/ProjectileAnomalySystem.cs index 8b03c425b8..d0081c6e46 100644 --- a/Content.Server/Anomaly/Effects/ProjectileAnomalySystem.cs +++ b/Content.Server/Anomaly/Effects/ProjectileAnomalySystem.cs @@ -54,10 +54,10 @@ public sealed class ProjectileAnomalySystem : EntitySystem priority.Add(entity); } - Logger.Debug($"shots: {projectileCount}"); + Log.Debug($"shots: {projectileCount}"); while (projectileCount > 0) { - Logger.Debug($"{projectileCount}"); + Log.Debug($"{projectileCount}"); var target = priority.Any() ? _random.PickAndTake(priority) : _random.Pick(inRange); diff --git a/Content.Server/Botany/Systems/MutationSystem.cs b/Content.Server/Botany/Systems/MutationSystem.cs index bc39e84eaa..5939019089 100644 --- a/Content.Server/Botany/Systems/MutationSystem.cs +++ b/Content.Server/Botany/Systems/MutationSystem.cs @@ -20,7 +20,7 @@ public sealed class MutationSystem : EntitySystem { if (!seed.Unique) { - Logger.Error($"Attempted to mutate a shared seed"); + Log.Error($"Attempted to mutate a shared seed"); return; } diff --git a/Content.Server/Chunking/ChunkingSystem.cs b/Content.Server/Chunking/ChunkingSystem.cs index be0e37971f..f04a720fc9 100644 --- a/Content.Server/Chunking/ChunkingSystem.cs +++ b/Content.Server/Chunking/ChunkingSystem.cs @@ -80,7 +80,7 @@ public sealed class ChunkingSystem : EntitySystem { if (!xformQuery.TryGetComponent(viewerUid, out var xform)) { - Logger.Error($"Player has deleted viewer entities? Viewers: {string.Join(", ", viewers.Select(x => ToPrettyString(x)))}"); + Log.Error($"Player has deleted viewer entities? Viewers: {string.Join(", ", viewers.Select(x => ToPrettyString(x)))}"); continue; } diff --git a/Content.Shared/Actions/SharedActionsSystem.cs b/Content.Shared/Actions/SharedActionsSystem.cs index a6c82c217e..35954464f2 100644 --- a/Content.Shared/Actions/SharedActionsSystem.cs +++ b/Content.Shared/Actions/SharedActionsSystem.cs @@ -127,7 +127,7 @@ public abstract class SharedActionsSystem : EntitySystem if (ev.EntityTarget is not { Valid: true } entityTarget) { - Logger.Error($"Attempted to perform an entity-targeted action without a target! Action: {entityAction.DisplayName}"); + Log.Error($"Attempted to perform an entity-targeted action without a target! Action: {entityAction.DisplayName}"); return; } @@ -138,11 +138,15 @@ public abstract class SharedActionsSystem : EntitySystem return; if (act.Provider == null) + { _adminLogger.Add(LogType.Action, - $"{ToPrettyString(user):user} is performing the {name:action} action targeted at {ToPrettyString(entityTarget):target}."); + $"{ToPrettyString(user):user} is performing the {name:action} action targeted at {ToPrettyString(entityTarget):target}."); + } else + { _adminLogger.Add(LogType.Action, - $"{ToPrettyString(user):user} is performing the {name:action} action (provided by {ToPrettyString(act.Provider.Value):provider}) targeted at {ToPrettyString(entityTarget):target}."); + $"{ToPrettyString(user):user} is performing the {name:action} action (provided by {ToPrettyString(act.Provider.Value):provider}) targeted at {ToPrettyString(entityTarget):target}."); + } if (entityAction.Event != null) { @@ -156,7 +160,7 @@ public abstract class SharedActionsSystem : EntitySystem if (ev.EntityCoordinatesTarget is not { } entityCoordinatesTarget) { - Logger.Error($"Attempted to perform a world-targeted action without a target! Action: {worldAction.DisplayName}"); + Log.Error($"Attempted to perform a world-targeted action without a target! Action: {worldAction.DisplayName}"); return; } @@ -166,11 +170,15 @@ public abstract class SharedActionsSystem : EntitySystem return; if (act.Provider == null) + { _adminLogger.Add(LogType.Action, $"{ToPrettyString(user):user} is performing the {name:action} action targeted at {entityCoordinatesTarget:target}."); + } else + { _adminLogger.Add(LogType.Action, $"{ToPrettyString(user):user} is performing the {name:action} action (provided by {ToPrettyString(act.Provider.Value):provider}) targeted at {entityCoordinatesTarget:target}."); + } if (worldAction.Event != null) { @@ -186,11 +194,15 @@ public abstract class SharedActionsSystem : EntitySystem return; if (act.Provider == null) + { _adminLogger.Add(LogType.Action, $"{ToPrettyString(user):user} is performing the {name:action} action."); + } else + { _adminLogger.Add(LogType.Action, $"{ToPrettyString(user):user} is performing the {name:action} action provided by {ToPrettyString(act.Provider.Value):provider}."); + } performEvent = instantAction.Event; break; @@ -325,7 +337,7 @@ public abstract class SharedActionsSystem : EntitySystem // Because action classes have state data, e.g. cooldowns and uses-remaining, people should not be adding prototypes directly if (action is IPrototype) { - Logger.Error("Attempted to directly add a prototype action. You need to clone a prototype in order to use it."); + Log.Error("Attempted to directly add a prototype action. You need to clone a prototype in order to use it."); return; } diff --git a/Content.Shared/Alert/AlertsSystem.cs b/Content.Shared/Alert/AlertsSystem.cs index 8f780af46e..37c49a46ce 100644 --- a/Content.Shared/Alert/AlertsSystem.cs +++ b/Content.Shared/Alert/AlertsSystem.cs @@ -42,7 +42,7 @@ public abstract class AlertsSystem : EntitySystem return alertsComponent.Alerts.ContainsKey(alert.AlertKey); } - Logger.DebugS("alert", "unknown alert type {0}", alertType); + Log.Debug("Unknown alert type {0}", alertType); return false; } @@ -101,7 +101,7 @@ public abstract class AlertsSystem : EntitySystem } else { - Logger.ErrorS("alert", "Unable to show alert {0}, please ensure this alertType has" + + Log.Error("Unable to show alert {0}, please ensure this alertType has" + " a corresponding YML alert prototype", alertType); } @@ -147,7 +147,7 @@ public abstract class AlertsSystem : EntitySystem } else { - Logger.ErrorS("alert", "unable to clear alert, unknown alertType {0}", alertType); + Log.Error("Unable to clear alert, unknown alertType {0}", alertType); } } @@ -206,9 +206,8 @@ public abstract class AlertsSystem : EntitySystem { if (!_typeToAlert.TryAdd(alert.AlertType, alert)) { - Logger.ErrorS("alert", - "Found alert with duplicate alertType {0} - all alerts must have" + - " a unique alerttype, this one will be skipped", alert.AlertType); + Log.Error("Found alert with duplicate alertType {0} - all alerts must have" + + " a unique alerttype, this one will be skipped", alert.AlertType); } } } @@ -230,7 +229,7 @@ public abstract class AlertsSystem : EntitySystem if (!IsShowingAlert(player.Value, msg.Type)) { - Logger.DebugS("alert", "user {0} attempted to" + + Log.Debug("User {0} attempted to" + " click alert {1} which is not currently showing for them", EntityManager.GetComponent(player.Value).EntityName, msg.Type); return; @@ -238,7 +237,7 @@ public abstract class AlertsSystem : EntitySystem if (!TryGet(msg.Type, out var alert)) { - Logger.WarningS("alert", "unrecognized encoded alert {0}", msg.Type); + Log.Warning("Unrecognized encoded alert {0}", msg.Type); return; } diff --git a/Content.Shared/Buckle/SharedBuckleSystem.Strap.cs b/Content.Shared/Buckle/SharedBuckleSystem.Strap.cs index 44670fd18f..62d89d0fe4 100644 --- a/Content.Shared/Buckle/SharedBuckleSystem.Strap.cs +++ b/Content.Shared/Buckle/SharedBuckleSystem.Strap.cs @@ -234,7 +234,7 @@ public abstract partial class SharedBuckleSystem if (!buckled.Buckled || buckled.LastEntityBuckledTo != uid) { - Logger.Error($"A moving strap entity {ToPrettyString(uid)} attempted to re-parent an entity that does not 'belong' to it {ToPrettyString(buckledEntity)}"); + Log.Error($"A moving strap entity {ToPrettyString(uid)} attempted to re-parent an entity that does not 'belong' to it {ToPrettyString(buckledEntity)}"); continue; } diff --git a/Content.Shared/Clothing/EntitySystems/ToggleableClothingSystem.cs b/Content.Shared/Clothing/EntitySystems/ToggleableClothingSystem.cs index 68648950d8..8f863e6831 100644 --- a/Content.Shared/Clothing/EntitySystems/ToggleableClothingSystem.cs +++ b/Content.Shared/Clothing/EntitySystems/ToggleableClothingSystem.cs @@ -26,7 +26,6 @@ public sealed class ToggleableClothingSystem : EntitySystem [Dependency] private readonly SharedDoAfterSystem _doAfter = default!; [Dependency] private readonly SharedStrippableSystem _strippable = default!; [Dependency] private readonly IPrototypeManager _proto = default!; - [Dependency] private readonly INetManager _net = default!; private Queue _toInsert = new(); diff --git a/Content.Shared/Containers/ContainerFillSystem.cs b/Content.Shared/Containers/ContainerFillSystem.cs index aad0acaa61..fb26d9b2cd 100644 --- a/Content.Shared/Containers/ContainerFillSystem.cs +++ b/Content.Shared/Containers/ContainerFillSystem.cs @@ -17,7 +17,7 @@ public sealed class ContainerFillSystem : EntitySystem { if (!TryComp(uid, out ContainerManagerComponent? containerComp)) { - Logger.Error($"Entity {ToPrettyString(uid)} with a {nameof(ContainerFillComponent)} has no {nameof(ContainerManagerComponent)}."); + Log.Error($"Entity {ToPrettyString(uid)} with a {nameof(ContainerFillComponent)} has no {nameof(ContainerManagerComponent)}."); return; } @@ -28,7 +28,7 @@ public sealed class ContainerFillSystem : EntitySystem { if (!_containerSystem.TryGetContainer(uid, contaienrId, out var container, containerComp)) { - Logger.Error($"Entity {ToPrettyString(uid)} with a {nameof(ContainerFillComponent)} is missing a container ({contaienrId})."); + Log.Error($"Entity {ToPrettyString(uid)} with a {nameof(ContainerFillComponent)} is missing a container ({contaienrId})."); continue; } @@ -37,7 +37,7 @@ public sealed class ContainerFillSystem : EntitySystem var ent = Spawn(proto, coords); if (!container.Insert(ent, EntityManager, null, xform)) { - Logger.Error($"Entity {ToPrettyString(uid)} with a {nameof(ContainerFillComponent)} failed to insert an entity: {ToPrettyString(ent)}."); + Log.Error($"Entity {ToPrettyString(uid)} with a {nameof(ContainerFillComponent)} failed to insert an entity: {ToPrettyString(ent)}."); Transform(ent).AttachToGridOrMap(); break; } diff --git a/Content.Shared/Damage/Systems/DamageableSystem.cs b/Content.Shared/Damage/Systems/DamageableSystem.cs index b902fe292e..53923207b2 100644 --- a/Content.Shared/Damage/Systems/DamageableSystem.cs +++ b/Content.Shared/Damage/Systems/DamageableSystem.cs @@ -153,7 +153,7 @@ namespace Content.Shared.Damage if (damage == null) { - Logger.Error("Null DamageSpecifier. Probably because a required yaml field was not given."); + Log.Error("Null DamageSpecifier. Probably because a required yaml field was not given."); return null; } diff --git a/Content.Shared/Dice/SharedDiceSystem.cs b/Content.Shared/Dice/SharedDiceSystem.cs index 7572ad7391..59a536e244 100644 --- a/Content.Shared/Dice/SharedDiceSystem.cs +++ b/Content.Shared/Dice/SharedDiceSystem.cs @@ -33,7 +33,8 @@ public abstract class SharedDiceSystem : EntitySystem private void OnUseInHand(EntityUid uid, DiceComponent component, UseInHandEvent args) { - if (args.Handled) return; + if (args.Handled) + return; args.Handled = true; Roll(uid, component); @@ -58,7 +59,7 @@ public abstract class SharedDiceSystem : EntitySystem if (side < 1 || side > die.Sides) { - Logger.Error($"Attempted to set die {ToPrettyString(uid)} to an invalid side ({side})."); + Log.Error($"Attempted to set die {ToPrettyString(uid)} to an invalid side ({side})."); return; } @@ -74,7 +75,7 @@ public abstract class SharedDiceSystem : EntitySystem if (value % die.Multiplier != 0 || value/ die.Multiplier + die.Offset < 1) { - Logger.Error($"Attempted to set die {ToPrettyString(uid)} to an invalid value ({value})."); + Log.Error($"Attempted to set die {ToPrettyString(uid)} to an invalid value ({value})."); return; } diff --git a/Content.Shared/DoAfter/SharedDoAfterSystem.cs b/Content.Shared/DoAfter/SharedDoAfterSystem.cs index a6e5ceb7f2..350f6e4b27 100644 --- a/Content.Shared/DoAfter/SharedDoAfterSystem.cs +++ b/Content.Shared/DoAfter/SharedDoAfterSystem.cs @@ -180,7 +180,7 @@ public abstract partial class SharedDoAfterSystem : EntitySystem if (!Resolve(args.User, ref comp)) { - Logger.Error($"Attempting to start a doAfter with invalid user: {ToPrettyString(args.User)}."); + Log.Error($"Attempting to start a doAfter with invalid user: {ToPrettyString(args.User)}."); id = null; return false; } @@ -317,7 +317,7 @@ public abstract partial class SharedDoAfterSystem : EntitySystem if (!comp.DoAfters.TryGetValue(id, out var doAfter)) { - Logger.Error($"Attempted to cancel do after with an invalid id ({id}) on entity {ToPrettyString(entity)}"); + Log.Error($"Attempted to cancel do after with an invalid id ({id}) on entity {ToPrettyString(entity)}"); return; } diff --git a/Content.Shared/Doors/Systems/SharedDoorSystem.cs b/Content.Shared/Doors/Systems/SharedDoorSystem.cs index d607319eb3..66499ba2bf 100644 --- a/Content.Shared/Doors/Systems/SharedDoorSystem.cs +++ b/Content.Shared/Doors/Systems/SharedDoorSystem.cs @@ -653,7 +653,7 @@ public abstract class SharedDoorSystem : EntitySystem case DoorState.Welded: // A welded door? This should never have been active in the first place. - Logger.Error($"Welded door was in the list of active doors. Door: {ToPrettyString(door.Owner)}"); + Log.Error($"Welded door was in the list of active doors. Door: {ToPrettyString(door.Owner)}"); break; } } diff --git a/Content.Shared/Friction/TileFrictionController.cs b/Content.Shared/Friction/TileFrictionController.cs index afd3290177..f3775d0f9d 100644 --- a/Content.Shared/Friction/TileFrictionController.cs +++ b/Content.Shared/Friction/TileFrictionController.cs @@ -77,7 +77,7 @@ namespace Content.Shared.Friction if (!xformQuery.TryGetComponent(uid, out var xform)) { - Logger.ErrorS("physics", $"Unable to get transform for {ToPrettyString(body.Owner)} in tilefrictioncontroller"); + Log.Error($"Unable to get transform for {ToPrettyString(body.Owner)} in tilefrictioncontroller"); continue; } diff --git a/Content.Shared/Hands/EntitySystems/SharedHandsSystem.Drop.cs b/Content.Shared/Hands/EntitySystems/SharedHandsSystem.Drop.cs index a1ddd17c13..0d794f8716 100644 --- a/Content.Shared/Hands/EntitySystems/SharedHandsSystem.Drop.cs +++ b/Content.Shared/Hands/EntitySystems/SharedHandsSystem.Drop.cs @@ -164,7 +164,7 @@ public abstract partial class SharedHandsSystem : EntitySystem if (!hand.Container.Remove(entity, EntityManager)) { - Logger.Error($"Failed to remove {ToPrettyString(entity)} from users hand container when dropping. User: {ToPrettyString(uid)}. Hand: {hand.Name}."); + Log.Error($"Failed to remove {ToPrettyString(entity)} from users hand container when dropping. User: {ToPrettyString(uid)}. Hand: {hand.Name}."); return; } diff --git a/Content.Shared/Hands/EntitySystems/SharedHandsSystem.Pickup.cs b/Content.Shared/Hands/EntitySystems/SharedHandsSystem.Pickup.cs index a5d4833fc4..c7706d492c 100644 --- a/Content.Shared/Hands/EntitySystems/SharedHandsSystem.Pickup.cs +++ b/Content.Shared/Hands/EntitySystems/SharedHandsSystem.Pickup.cs @@ -219,7 +219,7 @@ public abstract partial class SharedHandsSystem : EntitySystem if (!handContainer.Insert(entity, EntityManager)) { - Logger.Error($"Failed to insert {ToPrettyString(entity)} into users hand container when picking up. User: {ToPrettyString(uid)}. Hand: {hand.Name}."); + Log.Error($"Failed to insert {ToPrettyString(entity)} into users hand container when picking up. User: {ToPrettyString(uid)}. Hand: {hand.Name}."); return; } diff --git a/Content.Shared/Humanoid/NamingSystem.cs b/Content.Shared/Humanoid/NamingSystem.cs index bf19aeb57b..5e9ebdbf23 100644 --- a/Content.Shared/Humanoid/NamingSystem.cs +++ b/Content.Shared/Humanoid/NamingSystem.cs @@ -21,7 +21,7 @@ namespace Content.Shared.Humanoid if (!_prototypeManager.TryIndex(species, out SpeciesPrototype? speciesProto)) { speciesProto = _prototypeManager.Index("Human"); - Logger.Warning($"Unable to find species {species} for name, falling back to Human"); + Log.Warning($"Unable to find species {species} for name, falling back to Human"); } switch (speciesProto.Naming) diff --git a/Content.Shared/Interaction/RotateToFaceSystem.cs b/Content.Shared/Interaction/RotateToFaceSystem.cs index e788c861f2..e10f5df244 100644 --- a/Content.Shared/Interaction/RotateToFaceSystem.cs +++ b/Content.Shared/Interaction/RotateToFaceSystem.cs @@ -16,7 +16,6 @@ namespace Content.Shared.Interaction public sealed class RotateToFaceSystem : EntitySystem { [Dependency] private readonly ActionBlockerSystem _actionBlockerSystem = default!; - [Dependency] private readonly MobStateSystem _mobState = default!; [Dependency] private readonly SharedTransformSystem _transform = default!; /// diff --git a/Content.Shared/Inventory/InventorySystem.Slots.cs b/Content.Shared/Inventory/InventorySystem.Slots.cs index c84d24500e..49915f186b 100644 --- a/Content.Shared/Inventory/InventorySystem.Slots.cs +++ b/Content.Shared/Inventory/InventorySystem.Slots.cs @@ -48,7 +48,7 @@ public partial class InventorySystem : EntitySystem if (!containerComp.TryGetContainer(slotDefinition.Name, out var container)) { if (inventory.LifeStage >= ComponentLifeStage.Initialized) - Logger.Error($"Missing inventory container {slot} on entity {ToPrettyString(uid)}"); + Log.Error($"Missing inventory container {slot} on entity {ToPrettyString(uid)}"); return false; } diff --git a/Content.Shared/Maps/TurfSystem.cs b/Content.Shared/Maps/TurfSystem.cs index eac61cd778..360b865b60 100644 --- a/Content.Shared/Maps/TurfSystem.cs +++ b/Content.Shared/Maps/TurfSystem.cs @@ -10,7 +10,6 @@ namespace Content.Shared.Maps; /// public sealed class TurfSystem : EntitySystem { - [Dependency] private readonly IMapManager _mapMan = default!; [Dependency] private readonly EntityLookupSystem _entityLookup = default!; [Dependency] private readonly SharedTransformSystem _transform = default!; diff --git a/Content.Shared/Movement/Systems/SharedMoverController.Relay.cs b/Content.Shared/Movement/Systems/SharedMoverController.Relay.cs index 320bea0de4..846fbfaf7e 100644 --- a/Content.Shared/Movement/Systems/SharedMoverController.Relay.cs +++ b/Content.Shared/Movement/Systems/SharedMoverController.Relay.cs @@ -30,7 +30,7 @@ public abstract partial class SharedMoverController { if (uid == relayEntity) { - Logger.Error($"An entity attempted to relay movement to itself. Entity:{ToPrettyString(uid)}"); + Log.Error($"An entity attempted to relay movement to itself. Entity:{ToPrettyString(uid)}"); return; } diff --git a/Content.Shared/Pulling/Systems/SharedPullingStateManagementSystem.cs b/Content.Shared/Pulling/Systems/SharedPullingStateManagementSystem.cs index 409a1f75a0..799eced098 100644 --- a/Content.Shared/Pulling/Systems/SharedPullingStateManagementSystem.cs +++ b/Content.Shared/Pulling/Systems/SharedPullingStateManagementSystem.cs @@ -55,7 +55,7 @@ namespace Content.Shared.Pulling if (!TryComp(state.Puller.Value, out var comp)) { - Logger.Error($"Pullable state for entity {ToPrettyString(uid)} had invalid puller entity {ToPrettyString(state.Puller.Value)}"); + Log.Error($"Pullable state for entity {ToPrettyString(uid)} had invalid puller entity {ToPrettyString(state.Puller.Value)}"); // ensure it disconnects from any different puller, still ForceDisconnectPullable(component); return; diff --git a/Content.Shared/Pulling/Systems/SharedPullingSystem.Actions.cs b/Content.Shared/Pulling/Systems/SharedPullingSystem.Actions.cs index c0893a4079..5c43ce49a5 100644 --- a/Content.Shared/Pulling/Systems/SharedPullingSystem.Actions.cs +++ b/Content.Shared/Pulling/Systems/SharedPullingSystem.Actions.cs @@ -163,7 +163,7 @@ namespace Content.Shared.Pulling } else { - Logger.WarningS("c.go.c.pulling", "Well now you've done it, haven't you? Someone transferred pulling (onto {0}) while presently pulling something that has no Pullable component (on {1})!", pullable.Owner, oldPullable); + Log.Warning("Well now you've done it, haven't you? Someone transferred pulling (onto {0}) while presently pulling something that has no Pullable component (on {1})!", pullable.Owner, oldPullable); return false; } } diff --git a/Content.Shared/StatusEffect/StatusEffectsSystem.cs b/Content.Shared/StatusEffect/StatusEffectsSystem.cs index 23d18eb903..70822a05f4 100644 --- a/Content.Shared/StatusEffect/StatusEffectsSystem.cs +++ b/Content.Shared/StatusEffect/StatusEffectsSystem.cs @@ -16,7 +16,6 @@ namespace Content.Shared.StatusEffect [Dependency] private readonly IPrototypeManager _prototypeManager = default!; [Dependency] private readonly IComponentFactory _componentFactory = default!; [Dependency] private readonly IGameTiming _gameTiming = default!; - [Dependency] private readonly MobStateSystem _mobStateSystem = default!; [Dependency] private readonly AlertsSystem _alertsSystem = default!; public override void Initialize() diff --git a/Content.Shared/StepTrigger/Systems/StepTriggerSystem.cs b/Content.Shared/StepTrigger/Systems/StepTriggerSystem.cs index d4fcf1ce5c..3e1e10a07a 100644 --- a/Content.Shared/StepTrigger/Systems/StepTriggerSystem.cs +++ b/Content.Shared/StepTrigger/Systems/StepTriggerSystem.cs @@ -30,7 +30,7 @@ public sealed class StepTriggerSystem : EntitySystem return; if (!TryComp(uid, out FixturesComponent? fixtures) || fixtures.FixtureCount == 0) - Logger.Warning($"{ToPrettyString(uid)} has an active step trigger without any fixtures."); + Log.Warning($"{ToPrettyString(uid)} has an active step trigger without any fixtures."); #endif } diff --git a/Content.Shared/Storage/EntitySystems/BinSystem.cs b/Content.Shared/Storage/EntitySystems/BinSystem.cs index 587b58c865..897a20b45d 100644 --- a/Content.Shared/Storage/EntitySystems/BinSystem.cs +++ b/Content.Shared/Storage/EntitySystems/BinSystem.cs @@ -68,7 +68,7 @@ public sealed class BinSystem : EntitySystem var ent = Spawn(id, xform.Coordinates); if (!TryInsertIntoBin(uid, ent, component)) { - Logger.Error($"Entity {ToPrettyString(ent)} was unable to be initialized into bin {ToPrettyString(uid)}"); + Log.Error($"Entity {ToPrettyString(ent)} was unable to be initialized into bin {ToPrettyString(uid)}"); return; } } diff --git a/Content.Shared/Stunnable/SharedStunSystem.cs b/Content.Shared/Stunnable/SharedStunSystem.cs index 458d28dbf9..1cd2bcb557 100644 --- a/Content.Shared/Stunnable/SharedStunSystem.cs +++ b/Content.Shared/Stunnable/SharedStunSystem.cs @@ -32,7 +32,6 @@ namespace Content.Shared.Stunnable [Dependency] private readonly StatusEffectsSystem _statusEffectSystem = default!; [Dependency] private readonly MovementSpeedModifierSystem _movementSpeedModifierSystem = default!; [Dependency] private readonly SharedAudioSystem _audio = default!; - [Dependency] private readonly MobStateSystem _mobStateSystem = default!; [Dependency] private readonly ISharedAdminLogManager _adminLogger = default!; /// diff --git a/Content.Shared/Weapons/Melee/SharedMeleeWeaponSystem.cs b/Content.Shared/Weapons/Melee/SharedMeleeWeaponSystem.cs index e85be90dc3..ad61c39ad3 100644 --- a/Content.Shared/Weapons/Melee/SharedMeleeWeaponSystem.cs +++ b/Content.Shared/Weapons/Melee/SharedMeleeWeaponSystem.cs @@ -93,7 +93,7 @@ public abstract class SharedMeleeWeaponSystem : EntitySystem private void OnMapInit(EntityUid uid, MeleeWeaponComponent component, MapInitEvent args) { if (component.NextAttack > Timing.CurTime) - Logger.Warning($"Initializing a map that contains an entity that is on cooldown. Entity: {ToPrettyString(uid)}"); + Log.Warning($"Initializing a map that contains an entity that is on cooldown. Entity: {ToPrettyString(uid)}"); #endif } diff --git a/Content.Shared/Weapons/Ranged/Systems/SharedGunSystem.cs b/Content.Shared/Weapons/Ranged/Systems/SharedGunSystem.cs index b7002b1d99..503f98158d 100644 --- a/Content.Shared/Weapons/Ranged/Systems/SharedGunSystem.cs +++ b/Content.Shared/Weapons/Ranged/Systems/SharedGunSystem.cs @@ -98,7 +98,7 @@ public abstract partial class SharedGunSystem : EntitySystem private void OnMapInit(EntityUid uid, GunComponent component, MapInitEvent args) { if (component.NextFire > Timing.CurTime) - Logger.Warning($"Initializing a map that contains an entity that is on cooldown. Entity: {ToPrettyString(uid)}"); + Log.Warning($"Initializing a map that contains an entity that is on cooldown. Entity: {ToPrettyString(uid)}"); DebugTools.Assert((component.AvailableModes & component.SelectedMode) != 0x0); #endif