From bd6645b021f7f1d6e160aaba71cb6e2439600015 Mon Sep 17 00:00:00 2001 From: Tayrtahn Date: Fri, 23 May 2025 14:12:20 -0400 Subject: [PATCH] Cleanup redundant `SpriteSystem` dependencies in `VisualizerSystem`s (#37758) Cleanup redundant SpriteSystem dependencies in VisualizerSystems --- .../EntitySystems/FireVisualizerSystem.cs | 19 ++++---- .../Monitor/AtmosAlarmableVisualsSystem.cs | 14 +++--- .../PortableScrubberVisualsSystem.cs | 8 ++-- .../Botany/PlantHolderVisualizerSystem.cs | 12 +++-- .../TypingIndicatorVisualizerSystem.cs | 17 ++++--- .../Visualizers/FoamVisualizerSystem.cs | 3 +- .../SolutionContainerVisualsSystem.cs | 39 ++++++++-------- Content.Client/Damage/DamageVisualsSystem.cs | 45 +++++++++---------- .../DamageStateVisualizerSystem.cs | 16 +++---- .../ElectrocutionHUDVisualizerSystem.cs | 7 ++- .../EntitySystems/EmergencyLightSystem.cs | 10 ++--- .../EntitySystems/ExpendableLightSystem.cs | 19 ++++---- .../Light/EntitySystems/LightBulbSystem.cs | 10 ++--- .../PoweredLightVisualizerSystem.cs | 9 ++-- .../Lock/Visualizers/LockVisualizerSystem.cs | 8 ++-- Content.Client/PDA/PdaVisualizerSystem.cs | 7 ++- Content.Client/Paper/EnvelopeSystem.cs | 8 ++-- .../Paper/UI/PaperVisualizerSystem.cs | 10 ++--- ...ParticleAcceleratorPartVisualizerSystem.cs | 10 ++--- .../Power/APC/ApcVisualizerSystem.cs | 21 +++++---- Content.Client/Power/SMES/SmesSystem.cs | 20 ++++----- .../PowerCell/PowerChargerVisualizerSystem.cs | 12 +++-- Content.Client/Shuttles/ThrusterSystem.cs | 12 +++-- .../Systems/StorageContainerVisualsSystem.cs | 10 ++--- .../EntityStorageVisualizerSystem.cs | 26 +++++------ Content.Client/TextScreen/TextScreenSystem.cs | 27 ++++++----- .../ToggleableLightVisualsSystem.cs | 7 ++- .../Visualizers/WiresVisualizerSystem.cs | 8 ++-- .../RandomArtifactSpriteSystem.cs | 18 ++++---- 29 files changed, 193 insertions(+), 239 deletions(-) diff --git a/Content.Client/Atmos/EntitySystems/FireVisualizerSystem.cs b/Content.Client/Atmos/EntitySystems/FireVisualizerSystem.cs index bd221eb4bf..431a598678 100644 --- a/Content.Client/Atmos/EntitySystems/FireVisualizerSystem.cs +++ b/Content.Client/Atmos/EntitySystems/FireVisualizerSystem.cs @@ -11,7 +11,6 @@ namespace Content.Client.Atmos.EntitySystems; /// public sealed class FireVisualizerSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; [Dependency] private readonly PointLightSystem _lights = default!; public override void Initialize() @@ -33,9 +32,9 @@ public sealed class FireVisualizerSystem : VisualizerSystem(uid, out var sprite) && - _sprite.LayerMapTryGet((uid, sprite), FireVisualLayers.Fire, out var layer, false)) + SpriteSystem.LayerMapTryGet((uid, sprite), FireVisualLayers.Fire, out var layer, false)) { - _sprite.RemoveLayer((uid, sprite), layer); + SpriteSystem.RemoveLayer((uid, sprite), layer); } } @@ -44,11 +43,11 @@ public sealed class FireVisualizerSystem : VisualizerSystem(uid, out var sprite) || !TryComp(uid, out AppearanceComponent? appearance)) return; - _sprite.LayerMapReserve((uid, sprite), FireVisualLayers.Fire); - _sprite.LayerSetVisible((uid, sprite), FireVisualLayers.Fire, false); + SpriteSystem.LayerMapReserve((uid, sprite), FireVisualLayers.Fire); + SpriteSystem.LayerSetVisible((uid, sprite), FireVisualLayers.Fire, false); sprite.LayerSetShader(FireVisualLayers.Fire, "unshaded"); if (component.Sprite != null) - _sprite.LayerSetRsi((uid, sprite), FireVisualLayers.Fire, new ResPath(component.Sprite)); + SpriteSystem.LayerSetRsi((uid, sprite), FireVisualLayers.Fire, new ResPath(component.Sprite)); UpdateAppearance(uid, component, sprite, appearance); } @@ -61,12 +60,12 @@ public sealed class FireVisualizerSystem : VisualizerSystem(uid, FireVisuals.OnFire, out var onFire, appearance); AppearanceSystem.TryGetData(uid, FireVisuals.FireStacks, out var fireStacks, appearance); - _sprite.LayerSetVisible((uid, sprite), index, onFire); + SpriteSystem.LayerSetVisible((uid, sprite), index, onFire); if (!onFire) { @@ -80,9 +79,9 @@ public sealed class FireVisualizerSystem : VisualizerSystem component.FireStackAlternateState && !string.IsNullOrEmpty(component.AlternateState)) - _sprite.LayerSetRsiState((uid, sprite), index, component.AlternateState); + SpriteSystem.LayerSetRsiState((uid, sprite), index, component.AlternateState); else - _sprite.LayerSetRsiState((uid, sprite), index, component.NormalState); + SpriteSystem.LayerSetRsiState((uid, sprite), index, component.NormalState); component.LightEntity ??= Spawn(null, new EntityCoordinates(uid, default)); var light = EnsureComp(component.LightEntity.Value); diff --git a/Content.Client/Atmos/Monitor/AtmosAlarmableVisualsSystem.cs b/Content.Client/Atmos/Monitor/AtmosAlarmableVisualsSystem.cs index 6d3a373759..472e1e3d34 100644 --- a/Content.Client/Atmos/Monitor/AtmosAlarmableVisualsSystem.cs +++ b/Content.Client/Atmos/Monitor/AtmosAlarmableVisualsSystem.cs @@ -7,11 +7,9 @@ namespace Content.Client.Atmos.Monitor; public sealed class AtmosAlarmableVisualsSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - protected override void OnAppearanceChange(EntityUid uid, AtmosAlarmableVisualsComponent component, ref AppearanceChangeEvent args) { - if (args.Sprite == null || !_sprite.LayerMapTryGet((uid, args.Sprite), component.LayerMap, out var layer, false)) + if (args.Sprite == null || !SpriteSystem.LayerMapTryGet((uid, args.Sprite), component.LayerMap, out var layer, false)) return; if (!args.AppearanceData.TryGetValue(PowerDeviceVisuals.Powered, out var poweredObject) || @@ -24,8 +22,8 @@ public sealed class AtmosAlarmableVisualsSystem : VisualizerSystem public sealed class PortableScrubberSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - protected override void OnAppearanceChange(EntityUid uid, PortableScrubberVisualsComponent component, ref AppearanceChangeEvent args) { if (args.Sprite == null) @@ -20,15 +18,15 @@ public sealed class PortableScrubberSystem : VisualizerSystem(uid, PortableScrubberVisuals.IsRunning, out var isRunning, args.Component)) { var runningState = isRunning ? component.RunningState : component.IdleState; - _sprite.LayerSetRsiState((uid, args.Sprite), PortableScrubberVisualLayers.IsRunning, runningState); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), PortableScrubberVisualLayers.IsRunning, runningState); var fullState = isFull ? component.FullState : component.ReadyState; - _sprite.LayerSetRsiState((uid, args.Sprite), PowerDeviceVisualLayers.Powered, fullState); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), PowerDeviceVisualLayers.Powered, fullState); } if (AppearanceSystem.TryGetData(uid, PortableScrubberVisuals.IsDraining, out var isDraining, args.Component)) { - _sprite.LayerSetVisible((uid, args.Sprite), PortableScrubberVisualLayers.IsDraining, isDraining); + SpriteSystem.LayerSetVisible((uid, args.Sprite), PortableScrubberVisualLayers.IsDraining, isDraining); } } } diff --git a/Content.Client/Botany/PlantHolderVisualizerSystem.cs b/Content.Client/Botany/PlantHolderVisualizerSystem.cs index 7050abb39f..bd5b5e56dc 100644 --- a/Content.Client/Botany/PlantHolderVisualizerSystem.cs +++ b/Content.Client/Botany/PlantHolderVisualizerSystem.cs @@ -7,8 +7,6 @@ namespace Content.Client.Botany; public sealed class PlantHolderVisualizerSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - public override void Initialize() { base.Initialize(); @@ -20,8 +18,8 @@ public sealed class PlantHolderVisualizerSystem : VisualizerSystem(uid, out var sprite)) return; - _sprite.LayerMapReserve((uid, sprite), PlantHolderLayers.Plant); - _sprite.LayerSetVisible((uid, sprite), PlantHolderLayers.Plant, false); + SpriteSystem.LayerMapReserve((uid, sprite), PlantHolderLayers.Plant); + SpriteSystem.LayerSetVisible((uid, sprite), PlantHolderLayers.Plant, false); } protected override void OnAppearanceChange(EntityUid uid, PlantHolderVisualsComponent component, ref AppearanceChangeEvent args) @@ -34,12 +32,12 @@ public sealed class PlantHolderVisualizerSystem : VisualizerSystem(uid, TypingIndicatorVisuals.State, out var state); - _sprite.LayerSetVisible((uid, args.Sprite), layer, state != TypingIndicatorState.None); + SpriteSystem.LayerSetVisible((uid, args.Sprite), layer, state != TypingIndicatorState.None); switch (state) { case TypingIndicatorState.Idle: - _sprite.LayerSetRsiState((uid, args.Sprite), layer, proto.IdleState); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), layer, proto.IdleState); break; case TypingIndicatorState.Typing: - _sprite.LayerSetRsiState((uid, args.Sprite), layer, proto.TypingState); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), layer, proto.TypingState); break; } } diff --git a/Content.Client/Chemistry/Visualizers/FoamVisualizerSystem.cs b/Content.Client/Chemistry/Visualizers/FoamVisualizerSystem.cs index 00a29563c1..06bc790696 100644 --- a/Content.Client/Chemistry/Visualizers/FoamVisualizerSystem.cs +++ b/Content.Client/Chemistry/Visualizers/FoamVisualizerSystem.cs @@ -11,7 +11,6 @@ namespace Content.Client.Chemistry.Visualizers; public sealed class FoamVisualizerSystem : VisualizerSystem { [Dependency] private readonly IGameTiming _timing = default!; - [Dependency] private readonly SpriteSystem _sprite = default!; public override void Initialize() { @@ -72,7 +71,7 @@ public sealed class FoamVisualizerSystem : VisualizerSystem(uid, out var sprite)) - _sprite.SetVisible((uid, sprite), false); + SpriteSystem.SetVisible((uid, sprite), false); } } diff --git a/Content.Client/Chemistry/Visualizers/SolutionContainerVisualsSystem.cs b/Content.Client/Chemistry/Visualizers/SolutionContainerVisualsSystem.cs index 2baef6a535..6fc07785b2 100644 --- a/Content.Client/Chemistry/Visualizers/SolutionContainerVisualsSystem.cs +++ b/Content.Client/Chemistry/Visualizers/SolutionContainerVisualsSystem.cs @@ -16,7 +16,6 @@ public sealed class SolutionContainerVisualsSystem : VisualizerSystem(uid, SolutionContainerVisuals.BaseOverride, out var baseOverride, @@ -80,35 +79,35 @@ public sealed class SolutionContainerVisualsSystem : VisualizerSystem 0) { - _sprite.LayerSetVisible((uid, args.Sprite), fillLayer, true); + SpriteSystem.LayerSetVisible((uid, args.Sprite), fillLayer, true); maxFillLevels = reagentProto.MetamorphicMaxFillLevels; fillBaseName = reagentProto.MetamorphicFillBaseName; changeColor = reagentProto.MetamorphicChangeColor; fillSprite = sprite; } else - _sprite.LayerSetVisible((uid, args.Sprite), fillLayer, false); + SpriteSystem.LayerSetVisible((uid, args.Sprite), fillLayer, false); if (hasOverlay) - _sprite.LayerSetVisible((uid, args.Sprite), overlayLayer, false); + SpriteSystem.LayerSetVisible((uid, args.Sprite), overlayLayer, false); } else { - _sprite.LayerSetVisible((uid, args.Sprite), fillLayer, true); + SpriteSystem.LayerSetVisible((uid, args.Sprite), fillLayer, true); if (hasOverlay) - _sprite.LayerSetVisible((uid, args.Sprite), overlayLayer, true); + SpriteSystem.LayerSetVisible((uid, args.Sprite), overlayLayer, true); if (component.MetamorphicDefaultSprite != null) - _sprite.LayerSetSprite((uid, args.Sprite), baseLayer, component.MetamorphicDefaultSprite); + SpriteSystem.LayerSetSprite((uid, args.Sprite), baseLayer, component.MetamorphicDefaultSprite); } } } } else { - _sprite.LayerSetVisible((uid, args.Sprite), fillLayer, true); + SpriteSystem.LayerSetVisible((uid, args.Sprite), fillLayer, true); } var closestFillSprite = ContentHelpers.RoundToLevels(fraction, 1, maxFillLevels + 1); @@ -120,25 +119,25 @@ public sealed class SolutionContainerVisualsSystem : VisualizerSystem(uid, SolutionContainerVisuals.Color, out var color, args.Component)) - _sprite.LayerSetColor((uid, args.Sprite), fillLayer, color); + SpriteSystem.LayerSetColor((uid, args.Sprite), fillLayer, color); else - _sprite.LayerSetColor((uid, args.Sprite), fillLayer, Color.White); + SpriteSystem.LayerSetColor((uid, args.Sprite), fillLayer, Color.White); } else { if (component.EmptySpriteName == null) - _sprite.LayerSetVisible((uid, args.Sprite), fillLayer, false); + SpriteSystem.LayerSetVisible((uid, args.Sprite), fillLayer, false); else { - _sprite.LayerSetRsiState((uid, args.Sprite), fillLayer, component.EmptySpriteName); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), fillLayer, component.EmptySpriteName); if (changeColor) - _sprite.LayerSetColor((uid, args.Sprite), fillLayer, component.EmptySpriteColor); + SpriteSystem.LayerSetColor((uid, args.Sprite), fillLayer, component.EmptySpriteColor); else - _sprite.LayerSetColor((uid, args.Sprite), fillLayer, Color.White); + SpriteSystem.LayerSetColor((uid, args.Sprite), fillLayer, Color.White); } } diff --git a/Content.Client/Damage/DamageVisualsSystem.cs b/Content.Client/Damage/DamageVisualsSystem.cs index b38e70f6cc..de866ca9a4 100644 --- a/Content.Client/Damage/DamageVisualsSystem.cs +++ b/Content.Client/Damage/DamageVisualsSystem.cs @@ -27,7 +27,6 @@ namespace Content.Client.Damage; public sealed class DamageVisualsSystem : VisualizerSystem { [Dependency] private readonly IPrototypeManager _prototypeManager = default!; - [Dependency] private readonly SpriteSystem _sprite = default!; public override void Initialize() { @@ -229,7 +228,7 @@ public sealed class DamageVisualsSystem : VisualizerSystem private void AddDamageLayerToSprite(Entity spriteEnt, DamageVisualizerSprite sprite, string state, string mapKey, int? index = null) { - var newLayer = _sprite.AddLayer( + var newLayer = SpriteSystem.AddLayer( spriteEnt, new SpriteSpecifier.Rsi( new(sprite.Sprite), state ), index ); - _sprite.LayerMapSet(spriteEnt, mapKey, newLayer); + SpriteSystem.LayerMapSet(spriteEnt, mapKey, newLayer); if (sprite.Color != null) - _sprite.LayerSetColor(spriteEnt, newLayer, Color.FromHex(sprite.Color)); - _sprite.LayerSetVisible(spriteEnt, newLayer, false); + SpriteSystem.LayerSetColor(spriteEnt, newLayer, Color.FromHex(sprite.Color)); + SpriteSystem.LayerSetVisible(spriteEnt, newLayer, false); } protected override void OnAppearanceChange(EntityUid uid, DamageVisualsComponent damageVisComp, ref AppearanceChangeEvent args) @@ -410,7 +409,7 @@ public sealed class DamageVisualsSystem : VisualizerSystem spriteEnt, DamageVisualsComponent damageVisComp, DamageVisualizerSprite sprite, string key, string statePrefix, FixedPoint2 threshold) { - _sprite.LayerMapTryGet(spriteEnt.AsNullable(), key, out var spriteLayer, false); + SpriteSystem.LayerMapTryGet(spriteEnt.AsNullable(), key, out var spriteLayer, false); var visibility = spriteEnt.Comp[spriteLayer].Visible; - _sprite.RemoveLayer(spriteEnt.AsNullable(), spriteLayer); + SpriteSystem.RemoveLayer(spriteEnt.AsNullable(), spriteLayer); if (threshold == FixedPoint2.Zero) // these should automatically be invisible threshold = damageVisComp.Thresholds[1]; - spriteLayer = _sprite.AddLayer( + spriteLayer = SpriteSystem.AddLayer( spriteEnt.AsNullable(), new SpriteSpecifier.Rsi( new(sprite.Sprite), $"{statePrefix}_{threshold}" ), spriteLayer); - _sprite.LayerMapSet(spriteEnt.AsNullable(), key, spriteLayer); - _sprite.LayerSetVisible(spriteEnt.AsNullable(), spriteLayer, visibility); + SpriteSystem.LayerMapSet(spriteEnt.AsNullable(), key, spriteLayer); + SpriteSystem.LayerSetVisible(spriteEnt.AsNullable(), spriteLayer, visibility); // this is somewhat iffy since it constantly reallocates damageVisComp.TopMostLayerKey = key; } @@ -610,7 +609,7 @@ public sealed class DamageVisualsSystem : VisualizerSystem private void UpdateOverlay(Entity spriteEnt, FixedPoint2 threshold) { - _sprite.LayerMapTryGet(spriteEnt.AsNullable(), $"DamageOverlay", out var spriteLayer, false); + SpriteSystem.LayerMapTryGet(spriteEnt.AsNullable(), $"DamageOverlay", out var spriteLayer, false); UpdateDamageLayerState(spriteEnt, spriteLayer, @@ -690,7 +689,7 @@ public sealed class DamageVisualsSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - protected override void OnAppearanceChange(EntityUid uid, DamageStateVisualsComponent component, ref AppearanceChangeEvent args) { var sprite = args.Sprite; @@ -25,18 +23,18 @@ public sealed class DamageStateVisualizerSystem : VisualizerSystem (int)DrawDepth.DeadMobs) { component.OriginalDrawDepth = sprite.DrawDepth; - _sprite.SetDrawDepth((uid, sprite), (int)DrawDepth.DeadMobs); + SpriteSystem.SetDrawDepth((uid, sprite), (int)DrawDepth.DeadMobs); } } else if (component.OriginalDrawDepth != null) { - _sprite.SetDrawDepth((uid, sprite), component.OriginalDrawDepth.Value); + SpriteSystem.SetDrawDepth((uid, sprite), component.OriginalDrawDepth.Value); component.OriginalDrawDepth = null; } } diff --git a/Content.Client/Electrocution/ElectrocutionHUDVisualizerSystem.cs b/Content.Client/Electrocution/ElectrocutionHUDVisualizerSystem.cs index 7fc81219f7..65b70727c0 100644 --- a/Content.Client/Electrocution/ElectrocutionHUDVisualizerSystem.cs +++ b/Content.Client/Electrocution/ElectrocutionHUDVisualizerSystem.cs @@ -11,7 +11,6 @@ namespace Content.Client.Electrocution; public sealed class ElectrocutionHUDVisualizerSystem : VisualizerSystem { [Dependency] private readonly IPlayerManager _playerMan = default!; - [Dependency] private readonly SpriteSystem _sprite = default!; public override void Initialize() { @@ -59,7 +58,7 @@ public sealed class ElectrocutionHUDVisualizerSystem : VisualizerSystem(uid, ElectrifiedVisuals.IsElectrified, out var electrified, appearanceComp)) continue; - _sprite.LayerSetVisible((uid, spriteComp), ElectrifiedLayers.HUD, electrified); + SpriteSystem.LayerSetVisible((uid, spriteComp), ElectrifiedLayers.HUD, electrified); } } @@ -70,7 +69,7 @@ public sealed class ElectrocutionHUDVisualizerSystem : VisualizerSystem(); while (electrifiedQuery.MoveNext(out var uid, out _, out _, out var spriteComp)) { - _sprite.LayerSetVisible((uid, spriteComp), ElectrifiedLayers.HUD, false); + SpriteSystem.LayerSetVisible((uid, spriteComp), ElectrifiedLayers.HUD, false); } } @@ -84,6 +83,6 @@ public sealed class ElectrocutionHUDVisualizerSystem : VisualizerSystem(player)); + SpriteSystem.LayerSetVisible((uid, args.Sprite), ElectrifiedLayers.HUD, electrified && HasComp(player)); } } diff --git a/Content.Client/Light/EntitySystems/EmergencyLightSystem.cs b/Content.Client/Light/EntitySystems/EmergencyLightSystem.cs index 11ccc86550..fcbba97a9c 100644 --- a/Content.Client/Light/EntitySystems/EmergencyLightSystem.cs +++ b/Content.Client/Light/EntitySystems/EmergencyLightSystem.cs @@ -6,8 +6,6 @@ namespace Content.Client.Light.EntitySystems; public sealed class EmergencyLightSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - protected override void OnAppearanceChange(EntityUid uid, EmergencyLightComponent comp, ref AppearanceChangeEvent args) { if (args.Sprite == null) @@ -16,13 +14,13 @@ public sealed class EmergencyLightSystem : VisualizerSystem(uid, EmergencyLightVisuals.On, out var on, args.Component)) on = false; - _sprite.LayerSetVisible((uid, args.Sprite), EmergencyLightVisualLayers.LightOff, !on); - _sprite.LayerSetVisible((uid, args.Sprite), EmergencyLightVisualLayers.LightOn, on); + SpriteSystem.LayerSetVisible((uid, args.Sprite), EmergencyLightVisualLayers.LightOff, !on); + SpriteSystem.LayerSetVisible((uid, args.Sprite), EmergencyLightVisualLayers.LightOn, on); if (AppearanceSystem.TryGetData(uid, EmergencyLightVisuals.Color, out var color, args.Component)) { - _sprite.LayerSetColor((uid, args.Sprite), EmergencyLightVisualLayers.LightOn, color); - _sprite.LayerSetColor((uid, args.Sprite), EmergencyLightVisualLayers.LightOff, color); + SpriteSystem.LayerSetColor((uid, args.Sprite), EmergencyLightVisualLayers.LightOn, color); + SpriteSystem.LayerSetColor((uid, args.Sprite), EmergencyLightVisualLayers.LightOff, color); } } } diff --git a/Content.Client/Light/EntitySystems/ExpendableLightSystem.cs b/Content.Client/Light/EntitySystems/ExpendableLightSystem.cs index bc4c85cca7..e47430677d 100644 --- a/Content.Client/Light/EntitySystems/ExpendableLightSystem.cs +++ b/Content.Client/Light/EntitySystems/ExpendableLightSystem.cs @@ -10,7 +10,6 @@ public sealed class ExpendableLightSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - protected override void OnAppearanceChange(EntityUid uid, LightBulbComponent comp, ref AppearanceChangeEvent args) { if (args.Sprite == null) @@ -18,13 +16,13 @@ public sealed class LightBulbSystem : VisualizerSystem switch (state) { case LightBulbState.Normal: - _sprite.LayerSetRsiState((uid, args.Sprite), LightBulbVisualLayers.Base, comp.NormalSpriteState); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), LightBulbVisualLayers.Base, comp.NormalSpriteState); break; case LightBulbState.Broken: - _sprite.LayerSetRsiState((uid, args.Sprite), LightBulbVisualLayers.Base, comp.BrokenSpriteState); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), LightBulbVisualLayers.Base, comp.BrokenSpriteState); break; case LightBulbState.Burned: - _sprite.LayerSetRsiState((uid, args.Sprite), LightBulbVisualLayers.Base, comp.BurnedSpriteState); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), LightBulbVisualLayers.Base, comp.BurnedSpriteState); break; } } @@ -32,7 +30,7 @@ public sealed class LightBulbSystem : VisualizerSystem // also update sprites color if (AppearanceSystem.TryGetData(uid, LightBulbVisuals.Color, out var color, args.Component)) { - _sprite.SetColor((uid, args.Sprite), color); + SpriteSystem.SetColor((uid, args.Sprite), color); } } } diff --git a/Content.Client/Light/Visualizers/PoweredLightVisualizerSystem.cs b/Content.Client/Light/Visualizers/PoweredLightVisualizerSystem.cs index ba83e69af7..7ed1fb2dde 100644 --- a/Content.Client/Light/Visualizers/PoweredLightVisualizerSystem.cs +++ b/Content.Client/Light/Visualizers/PoweredLightVisualizerSystem.cs @@ -11,7 +11,6 @@ public sealed class PoweredLightVisualizerSystem : VisualizerSystem(uid, out var light)) { - _sprite.LayerSetColor((uid, args.Sprite), PoweredLightLayers.Glow, light.Color); + SpriteSystem.LayerSetColor((uid, args.Sprite), PoweredLightLayers.Glow, light.Color); } - _sprite.LayerSetVisible((uid, args.Sprite), PoweredLightLayers.Glow, state == PoweredLightState.On); + SpriteSystem.LayerSetVisible((uid, args.Sprite), PoweredLightLayers.Glow, state == PoweredLightState.On); } SetBlinkingAnimation( diff --git a/Content.Client/Lock/Visualizers/LockVisualizerSystem.cs b/Content.Client/Lock/Visualizers/LockVisualizerSystem.cs index 76385ffd90..b69a7cb379 100644 --- a/Content.Client/Lock/Visualizers/LockVisualizerSystem.cs +++ b/Content.Client/Lock/Visualizers/LockVisualizerSystem.cs @@ -6,8 +6,6 @@ namespace Content.Client.Lock.Visualizers; public sealed class LockVisualizerSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - protected override void OnAppearanceChange(EntityUid uid, LockVisualsComponent comp, ref AppearanceChangeEvent args) { if (args.Sprite == null @@ -22,14 +20,14 @@ public sealed class LockVisualizerSystem : VisualizerSystem(uid, StorageVisuals.Open, out var open, args.Component)) { - _sprite.LayerSetVisible((uid, args.Sprite), LockVisualLayers.Lock, !open); + SpriteSystem.LayerSetVisible((uid, args.Sprite), LockVisualLayers.Lock, !open); } else if (!(bool)unlockedStateExist!) - _sprite.LayerSetVisible((uid, args.Sprite), LockVisualLayers.Lock, locked); + SpriteSystem.LayerSetVisible((uid, args.Sprite), LockVisualLayers.Lock, locked); if (!open && (bool)unlockedStateExist!) { - _sprite.LayerSetRsiState((uid, args.Sprite), LockVisualLayers.Lock, locked ? comp.StateLocked : comp.StateUnlocked); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), LockVisualLayers.Lock, locked ? comp.StateLocked : comp.StateUnlocked); } } } diff --git a/Content.Client/PDA/PdaVisualizerSystem.cs b/Content.Client/PDA/PdaVisualizerSystem.cs index 1c132de586..0d7685d6ae 100644 --- a/Content.Client/PDA/PdaVisualizerSystem.cs +++ b/Content.Client/PDA/PdaVisualizerSystem.cs @@ -6,20 +6,19 @@ namespace Content.Client.PDA; public sealed class PdaVisualizerSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; protected override void OnAppearanceChange(EntityUid uid, PdaVisualsComponent comp, ref AppearanceChangeEvent args) { if (args.Sprite == null) return; if (AppearanceSystem.TryGetData(uid, PdaVisuals.PdaType, out var pdaType, args.Component)) - _sprite.LayerSetRsiState((uid, args.Sprite), PdaVisualLayers.Base, pdaType); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), PdaVisualLayers.Base, pdaType); if (AppearanceSystem.TryGetData(uid, UnpoweredFlashlightVisuals.LightOn, out var isFlashlightOn, args.Component)) - _sprite.LayerSetVisible((uid, args.Sprite), PdaVisualLayers.Flashlight, isFlashlightOn); + SpriteSystem.LayerSetVisible((uid, args.Sprite), PdaVisualLayers.Flashlight, isFlashlightOn); if (AppearanceSystem.TryGetData(uid, PdaVisuals.IdCardInserted, out var isCardInserted, args.Component)) - _sprite.LayerSetVisible((uid, args.Sprite), PdaVisualLayers.IdLight, isCardInserted); + SpriteSystem.LayerSetVisible((uid, args.Sprite), PdaVisualLayers.IdLight, isCardInserted); } public enum PdaVisualLayers : byte diff --git a/Content.Client/Paper/EnvelopeSystem.cs b/Content.Client/Paper/EnvelopeSystem.cs index 49bd36349c..0e9dac6415 100644 --- a/Content.Client/Paper/EnvelopeSystem.cs +++ b/Content.Client/Paper/EnvelopeSystem.cs @@ -5,8 +5,6 @@ namespace Content.Client.Paper; public sealed class EnvelopeSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - public override void Initialize() { base.Initialize(); @@ -23,9 +21,9 @@ public sealed class EnvelopeSystem : VisualizerSystem if (!Resolve(ent.Owner, ref sprite)) return; - _sprite.LayerSetVisible((ent.Owner, sprite), EnvelopeVisualLayers.Open, ent.Comp.State == EnvelopeComponent.EnvelopeState.Open); - _sprite.LayerSetVisible((ent.Owner, sprite), EnvelopeVisualLayers.Sealed, ent.Comp.State == EnvelopeComponent.EnvelopeState.Sealed); - _sprite.LayerSetVisible((ent.Owner, sprite), EnvelopeVisualLayers.Torn, ent.Comp.State == EnvelopeComponent.EnvelopeState.Torn); + SpriteSystem.LayerSetVisible((ent.Owner, sprite), EnvelopeVisualLayers.Open, ent.Comp.State == EnvelopeComponent.EnvelopeState.Open); + SpriteSystem.LayerSetVisible((ent.Owner, sprite), EnvelopeVisualLayers.Sealed, ent.Comp.State == EnvelopeComponent.EnvelopeState.Sealed); + SpriteSystem.LayerSetVisible((ent.Owner, sprite), EnvelopeVisualLayers.Torn, ent.Comp.State == EnvelopeComponent.EnvelopeState.Torn); } public enum EnvelopeVisualLayers : byte diff --git a/Content.Client/Paper/UI/PaperVisualizerSystem.cs b/Content.Client/Paper/UI/PaperVisualizerSystem.cs index b8e429b64b..eb8489b7cb 100644 --- a/Content.Client/Paper/UI/PaperVisualizerSystem.cs +++ b/Content.Client/Paper/UI/PaperVisualizerSystem.cs @@ -6,26 +6,24 @@ namespace Content.Client.Paper.UI; public sealed class PaperVisualizerSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - protected override void OnAppearanceChange(EntityUid uid, PaperVisualsComponent component, ref AppearanceChangeEvent args) { if (args.Sprite == null) return; if (AppearanceSystem.TryGetData(uid, PaperVisuals.Status, out var writingStatus, args.Component)) - _sprite.LayerSetVisible((uid, args.Sprite), PaperVisualLayers.Writing, writingStatus == PaperStatus.Written); + SpriteSystem.LayerSetVisible((uid, args.Sprite), PaperVisualLayers.Writing, writingStatus == PaperStatus.Written); if (AppearanceSystem.TryGetData(uid, PaperVisuals.Stamp, out var stampState, args.Component)) { if (stampState != string.Empty) { - _sprite.LayerSetRsiState((uid, args.Sprite), PaperVisualLayers.Stamp, stampState); - _sprite.LayerSetVisible((uid, args.Sprite), PaperVisualLayers.Stamp, true); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), PaperVisualLayers.Stamp, stampState); + SpriteSystem.LayerSetVisible((uid, args.Sprite), PaperVisualLayers.Stamp, true); } else { - _sprite.LayerSetVisible((uid, args.Sprite), PaperVisualLayers.Stamp, false); + SpriteSystem.LayerSetVisible((uid, args.Sprite), PaperVisualLayers.Stamp, false); } } diff --git a/Content.Client/ParticleAccelerator/ParticleAcceleratorPartVisualizerSystem.cs b/Content.Client/ParticleAccelerator/ParticleAcceleratorPartVisualizerSystem.cs index 0f27da360c..0e98c1a292 100644 --- a/Content.Client/ParticleAccelerator/ParticleAcceleratorPartVisualizerSystem.cs +++ b/Content.Client/ParticleAccelerator/ParticleAcceleratorPartVisualizerSystem.cs @@ -5,14 +5,12 @@ namespace Content.Client.ParticleAccelerator; public sealed class ParticleAcceleratorPartVisualizerSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - protected override void OnAppearanceChange(EntityUid uid, ParticleAcceleratorPartVisualsComponent comp, ref AppearanceChangeEvent args) { if (args.Sprite == null) return; - if (!_sprite.LayerMapTryGet((uid, args.Sprite), ParticleAcceleratorVisualLayers.Unlit, out var index, false)) + if (!SpriteSystem.LayerMapTryGet((uid, args.Sprite), ParticleAcceleratorVisualLayers.Unlit, out var index, false)) return; if (!AppearanceSystem.TryGetData(uid, ParticleAcceleratorVisuals.VisualState, out var state, args.Component)) @@ -22,12 +20,12 @@ public sealed class ParticleAcceleratorPartVisualizerSystem : VisualizerSystem

{ [Dependency] private readonly SharedPointLightSystem _lights = default!; - [Dependency] private readonly SpriteSystem _sprite = default!; protected override void OnAppearanceChange(EntityUid uid, ApcVisualsComponent comp, ref AppearanceChangeEvent args) { @@ -14,8 +13,8 @@ public sealed class ApcVisualizerSystem : VisualizerSystem return; // get the mapped layer index of the first lock layer and the first channel layer - var lockIndicatorOverlayStart = _sprite.LayerMapGet((uid, args.Sprite), ApcVisualLayers.InterfaceLock); - var channelIndicatorOverlayStart = _sprite.LayerMapGet((uid, args.Sprite), ApcVisualLayers.Equipment); + var lockIndicatorOverlayStart = SpriteSystem.LayerMapGet((uid, args.Sprite), ApcVisualLayers.InterfaceLock); + var channelIndicatorOverlayStart = SpriteSystem.LayerMapGet((uid, args.Sprite), ApcVisualLayers.Equipment); // Handle APC screen overlay: if (!AppearanceSystem.TryGetData(uid, ApcVisuals.ChargeState, out var chargeState, args.Component)) @@ -23,7 +22,7 @@ public sealed class ApcVisualizerSystem : VisualizerSystem if (chargeState >= 0 && chargeState < ApcChargeState.NumStates) { - _sprite.LayerSetRsiState((uid, args.Sprite), ApcVisualLayers.ChargeState, $"{comp.ScreenPrefix}-{comp.ScreenSuffixes[(sbyte)chargeState]}"); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), ApcVisualLayers.ChargeState, $"{comp.ScreenPrefix}-{comp.ScreenSuffixes[(sbyte)chargeState]}"); // LockState does nothing currently. The backend doesn't exist. if (AppearanceSystem.TryGetData(uid, ApcVisuals.LockState, out var lockStates, args.Component)) @@ -32,8 +31,8 @@ public sealed class ApcVisualizerSystem : VisualizerSystem { var layer = (byte)lockIndicatorOverlayStart + i; var lockState = (sbyte)((lockStates >> (i << (sbyte)ApcLockState.LogWidth)) & (sbyte)ApcLockState.All); - _sprite.LayerSetRsiState((uid, args.Sprite), layer, $"{comp.LockPrefix}{i}-{comp.LockSuffixes[lockState]}"); - _sprite.LayerSetVisible((uid, args.Sprite), layer, true); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), layer, $"{comp.LockPrefix}{i}-{comp.LockSuffixes[lockState]}"); + SpriteSystem.LayerSetVisible((uid, args.Sprite), layer, true); } } @@ -44,8 +43,8 @@ public sealed class ApcVisualizerSystem : VisualizerSystem { var layer = (byte)channelIndicatorOverlayStart + i; var channelState = (sbyte)((channelStates >> (i << (sbyte)ApcChannelState.LogWidth)) & (sbyte)ApcChannelState.All); - _sprite.LayerSetRsiState((uid, args.Sprite), layer, $"{comp.ChannelPrefix}{i}-{comp.ChannelSuffixes[channelState]}"); - _sprite.LayerSetVisible((uid, args.Sprite), layer, true); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), layer, $"{comp.ChannelPrefix}{i}-{comp.ChannelSuffixes[channelState]}"); + SpriteSystem.LayerSetVisible((uid, args.Sprite), layer, true); } } @@ -57,16 +56,16 @@ public sealed class ApcVisualizerSystem : VisualizerSystem else { /// Overrides all of the lock and channel indicators. - _sprite.LayerSetRsiState((uid, args.Sprite), ApcVisualLayers.ChargeState, comp.EmaggedScreenState); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), ApcVisualLayers.ChargeState, comp.EmaggedScreenState); for (var i = 0; i < comp.LockIndicators; ++i) { var layer = (byte)lockIndicatorOverlayStart + i; - _sprite.LayerSetVisible((uid, args.Sprite), layer, false); + SpriteSystem.LayerSetVisible((uid, args.Sprite), layer, false); } for (var i = 0; i < comp.ChannelIndicators; ++i) { var layer = (byte)channelIndicatorOverlayStart + i; - _sprite.LayerSetVisible((uid, args.Sprite), layer, false); + SpriteSystem.LayerSetVisible((uid, args.Sprite), layer, false); } if (TryComp(uid, out var light)) diff --git a/Content.Client/Power/SMES/SmesSystem.cs b/Content.Client/Power/SMES/SmesSystem.cs index ac4b51547e..de0a5363cb 100644 --- a/Content.Client/Power/SMES/SmesSystem.cs +++ b/Content.Client/Power/SMES/SmesSystem.cs @@ -6,8 +6,6 @@ namespace Content.Client.Power.SMES; public sealed class SmesVisualizerSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - protected override void OnAppearanceChange(EntityUid uid, SmesComponent comp, ref AppearanceChangeEvent args) { if (args.Sprite == null) @@ -15,12 +13,12 @@ public sealed class SmesVisualizerSystem : VisualizerSystem if (!AppearanceSystem.TryGetData(uid, SmesVisuals.LastChargeLevel, out var level, args.Component) || level == 0) { - _sprite.LayerSetVisible((uid, args.Sprite), SmesVisualLayers.Charge, false); + SpriteSystem.LayerSetVisible((uid, args.Sprite), SmesVisualLayers.Charge, false); } else { - _sprite.LayerSetVisible((uid, args.Sprite), SmesVisualLayers.Charge, true); - _sprite.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Charge, $"{comp.ChargeOverlayPrefix}{level}"); + SpriteSystem.LayerSetVisible((uid, args.Sprite), SmesVisualLayers.Charge, true); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Charge, $"{comp.ChargeOverlayPrefix}{level}"); } if (!AppearanceSystem.TryGetData(uid, SmesVisuals.LastChargeState, out var state, args.Component)) @@ -29,16 +27,16 @@ public sealed class SmesVisualizerSystem : VisualizerSystem switch (state) { case ChargeState.Still: - _sprite.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Input, $"{comp.InputOverlayPrefix}0"); - _sprite.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Output, $"{comp.OutputOverlayPrefix}1"); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Input, $"{comp.InputOverlayPrefix}0"); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Output, $"{comp.OutputOverlayPrefix}1"); break; case ChargeState.Charging: - _sprite.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Input, $"{comp.InputOverlayPrefix}1"); - _sprite.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Output, $"{comp.OutputOverlayPrefix}1"); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Input, $"{comp.InputOverlayPrefix}1"); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Output, $"{comp.OutputOverlayPrefix}1"); break; case ChargeState.Discharging: - _sprite.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Input, $"{comp.InputOverlayPrefix}0"); - _sprite.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Output, $"{comp.OutputOverlayPrefix}2"); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Input, $"{comp.InputOverlayPrefix}0"); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Output, $"{comp.OutputOverlayPrefix}2"); break; } } diff --git a/Content.Client/PowerCell/PowerChargerVisualizerSystem.cs b/Content.Client/PowerCell/PowerChargerVisualizerSystem.cs index 852cd61905..c76e25b7af 100644 --- a/Content.Client/PowerCell/PowerChargerVisualizerSystem.cs +++ b/Content.Client/PowerCell/PowerChargerVisualizerSystem.cs @@ -5,8 +5,6 @@ namespace Content.Client.PowerCell; public sealed class PowerChargerVisualizerSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - protected override void OnAppearanceChange(EntityUid uid, PowerChargerVisualsComponent comp, ref AppearanceChangeEvent args) { if (args.Sprite == null) @@ -16,22 +14,22 @@ public sealed class PowerChargerVisualizerSystem : VisualizerSystem(uid, CellVisual.Occupied, out var occupied, args.Component) && occupied) { // TODO: don't throw if it doesn't have a full state - _sprite.LayerSetRsiState((uid, args.Sprite), PowerChargerVisualLayers.Base, comp.OccupiedState); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), PowerChargerVisualLayers.Base, comp.OccupiedState); } else { - _sprite.LayerSetRsiState((uid, args.Sprite), PowerChargerVisualLayers.Base, comp.EmptyState); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), PowerChargerVisualLayers.Base, comp.EmptyState); } // Update lighting if (AppearanceSystem.TryGetData(uid, CellVisual.Light, out var status, args.Component) && comp.LightStates.TryGetValue(status, out var lightState)) { - _sprite.LayerSetRsiState((uid, args.Sprite), PowerChargerVisualLayers.Light, lightState); - _sprite.LayerSetVisible((uid, args.Sprite), PowerChargerVisualLayers.Light, true); + SpriteSystem.LayerSetRsiState((uid, args.Sprite), PowerChargerVisualLayers.Light, lightState); + SpriteSystem.LayerSetVisible((uid, args.Sprite), PowerChargerVisualLayers.Light, true); } else - _sprite.LayerSetVisible((uid, args.Sprite), PowerChargerVisualLayers.Light, false); + SpriteSystem.LayerSetVisible((uid, args.Sprite), PowerChargerVisualLayers.Light, false); } } diff --git a/Content.Client/Shuttles/ThrusterSystem.cs b/Content.Client/Shuttles/ThrusterSystem.cs index 4205879b16..5ec5476258 100644 --- a/Content.Client/Shuttles/ThrusterSystem.cs +++ b/Content.Client/Shuttles/ThrusterSystem.cs @@ -8,8 +8,6 @@ namespace Content.Client.Shuttles; /// public sealed class ThrusterSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - ///

/// Updates whether or not the thruster is visibly active/thrusting. /// @@ -19,7 +17,7 @@ public sealed class ThrusterSystem : VisualizerSystem || !AppearanceSystem.TryGetData(uid, ThrusterVisualState.State, out var state, args.Component)) return; - _sprite.LayerSetVisible((uid, args.Sprite), ThrusterVisualLayers.ThrustOn, state); + SpriteSystem.LayerSetVisible((uid, args.Sprite), ThrusterVisualLayers.ThrustOn, state); SetThrusting( uid, state && AppearanceSystem.TryGetData(uid, ThrusterVisualState.Thrusting, out var thrusting, args.Component) && thrusting, @@ -32,14 +30,14 @@ public sealed class ThrusterSystem : VisualizerSystem /// private void SetThrusting(EntityUid uid, bool value, SpriteComponent sprite) { - if (_sprite.LayerMapTryGet((uid, sprite), ThrusterVisualLayers.Thrusting, out var thrustingLayer, false)) + if (SpriteSystem.LayerMapTryGet((uid, sprite), ThrusterVisualLayers.Thrusting, out var thrustingLayer, false)) { - _sprite.LayerSetVisible((uid, sprite), thrustingLayer, value); + SpriteSystem.LayerSetVisible((uid, sprite), thrustingLayer, value); } - if (_sprite.LayerMapTryGet((uid, sprite), ThrusterVisualLayers.ThrustingUnshaded, out var unshadedLayer, false)) + if (SpriteSystem.LayerMapTryGet((uid, sprite), ThrusterVisualLayers.ThrustingUnshaded, out var unshadedLayer, false)) { - _sprite.LayerSetVisible((uid, sprite), unshadedLayer, value); + SpriteSystem.LayerSetVisible((uid, sprite), unshadedLayer, value); } } } diff --git a/Content.Client/Storage/Systems/StorageContainerVisualsSystem.cs b/Content.Client/Storage/Systems/StorageContainerVisualsSystem.cs index 4753b308aa..301de2bf04 100644 --- a/Content.Client/Storage/Systems/StorageContainerVisualsSystem.cs +++ b/Content.Client/Storage/Systems/StorageContainerVisualsSystem.cs @@ -8,8 +8,6 @@ namespace Content.Client.Storage.Systems; /// public sealed class StorageContainerVisualsSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - protected override void OnAppearanceChange(EntityUid uid, StorageContainerVisualsComponent component, ref AppearanceChangeEvent args) { if (args.Sprite == null) @@ -23,7 +21,7 @@ public sealed class StorageContainerVisualsSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - public override void Initialize() { base.Initialize(); @@ -25,7 +23,7 @@ public sealed class EntityStorageVisualizerSystem : VisualizerSystem(uid, out var sprite)) return; - _sprite.LayerSetRsiState((uid, sprite), StorageVisualLayers.Base, comp.StateBaseClosed); + SpriteSystem.LayerSetRsiState((uid, sprite), StorageVisualLayers.Base, comp.StateBaseClosed); } protected override void OnAppearanceChange(EntityUid uid, EntityStorageVisualsComponent comp, ref AppearanceChangeEvent args) @@ -35,41 +33,41 @@ public sealed class EntityStorageVisualizerSystem : VisualizerSystem { [Dependency] private readonly IGameTiming _gameTiming = default!; - [Dependency] private readonly SpriteSystem _sprite = default!; /// /// Contains char/state Key/Value pairs.
@@ -90,11 +89,11 @@ public sealed class TextScreenSystem : VisualizerSystem(uid); @@ -190,7 +189,7 @@ public sealed class TextScreenSystem : VisualizerSystem pairs.Value != null)) - _sprite.LayerSetRsiState((uid, sprite), key, state); + SpriteSystem.LayerSetRsiState((uid, sprite), key, state); } public override void Update(float frameTime) diff --git a/Content.Client/Toggleable/ToggleableLightVisualsSystem.cs b/Content.Client/Toggleable/ToggleableLightVisualsSystem.cs index 28144486c3..ce940225bf 100644 --- a/Content.Client/Toggleable/ToggleableLightVisualsSystem.cs +++ b/Content.Client/Toggleable/ToggleableLightVisualsSystem.cs @@ -15,7 +15,6 @@ public sealed class ToggleableLightVisualsSystem : VisualizerSystem(uid, ToggleableLightVisuals.Color, out var color, args.Component); // Update the item's sprite - if (args.Sprite != null && component.SpriteLayer != null && _sprite.LayerMapTryGet((uid, args.Sprite), component.SpriteLayer, out var layer, false)) + if (args.Sprite != null && component.SpriteLayer != null && SpriteSystem.LayerMapTryGet((uid, args.Sprite), component.SpriteLayer, out var layer, false)) { - _sprite.LayerSetVisible((uid, args.Sprite), layer, enabled); + SpriteSystem.LayerSetVisible((uid, args.Sprite), layer, enabled); if (modulate) - _sprite.LayerSetColor((uid, args.Sprite), layer, color); + SpriteSystem.LayerSetColor((uid, args.Sprite), layer, color); } // Update any point-lights diff --git a/Content.Client/Wires/Visualizers/WiresVisualizerSystem.cs b/Content.Client/Wires/Visualizers/WiresVisualizerSystem.cs index e107dd220a..477848ba09 100644 --- a/Content.Client/Wires/Visualizers/WiresVisualizerSystem.cs +++ b/Content.Client/Wires/Visualizers/WiresVisualizerSystem.cs @@ -5,24 +5,22 @@ namespace Content.Client.Wires.Visualizers { public sealed class WiresVisualizerSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - protected override void OnAppearanceChange(EntityUid uid, WiresVisualsComponent component, ref AppearanceChangeEvent args) { if (args.Sprite == null) return; - var layer = _sprite.LayerMapReserve((uid, args.Sprite), WiresVisualLayers.MaintenancePanel); + var layer = SpriteSystem.LayerMapReserve((uid, args.Sprite), WiresVisualLayers.MaintenancePanel); if (args.AppearanceData.TryGetValue(WiresVisuals.MaintenancePanelState, out var panelStateObject) && panelStateObject is bool panelState) { - _sprite.LayerSetVisible((uid, args.Sprite), layer, panelState); + SpriteSystem.LayerSetVisible((uid, args.Sprite), layer, panelState); } else { //Mainly for spawn window - _sprite.LayerSetVisible((uid, args.Sprite), layer, false); + SpriteSystem.LayerSetVisible((uid, args.Sprite), layer, false); } } } diff --git a/Content.Client/Xenoarchaeology/XenoArtifacts/RandomArtifactSpriteSystem.cs b/Content.Client/Xenoarchaeology/XenoArtifacts/RandomArtifactSpriteSystem.cs index 030d1a364b..5e60086987 100644 --- a/Content.Client/Xenoarchaeology/XenoArtifacts/RandomArtifactSpriteSystem.cs +++ b/Content.Client/Xenoarchaeology/XenoArtifacts/RandomArtifactSpriteSystem.cs @@ -5,8 +5,6 @@ namespace Content.Client.Xenoarchaeology.XenoArtifacts; public sealed class RandomArtifactSpriteSystem : VisualizerSystem { - [Dependency] private readonly SpriteSystem _sprite = default!; - protected override void OnAppearanceChange(EntityUid uid, RandomArtifactSpriteComponent component, ref AppearanceChangeEvent args) { if (args.Sprite == null) @@ -25,24 +23,24 @@ public sealed class RandomArtifactSpriteSystem : VisualizerSystem