Cleanup warnings in SmesSystem (#37399)
* Cleanup warnings in SmesSystem * Access modifier for SmesVisualLayers
This commit is contained in:
@@ -6,6 +6,8 @@ namespace Content.Client.Power.SMES;
|
|||||||
|
|
||||||
public sealed class SmesVisualizerSystem : VisualizerSystem<SmesComponent>
|
public sealed class SmesVisualizerSystem : VisualizerSystem<SmesComponent>
|
||||||
{
|
{
|
||||||
|
[Dependency] private readonly SpriteSystem _sprite = default!;
|
||||||
|
|
||||||
protected override void OnAppearanceChange(EntityUid uid, SmesComponent comp, ref AppearanceChangeEvent args)
|
protected override void OnAppearanceChange(EntityUid uid, SmesComponent comp, ref AppearanceChangeEvent args)
|
||||||
{
|
{
|
||||||
if (args.Sprite == null)
|
if (args.Sprite == null)
|
||||||
@@ -13,12 +15,12 @@ public sealed class SmesVisualizerSystem : VisualizerSystem<SmesComponent>
|
|||||||
|
|
||||||
if (!AppearanceSystem.TryGetData<int>(uid, SmesVisuals.LastChargeLevel, out var level, args.Component) || level == 0)
|
if (!AppearanceSystem.TryGetData<int>(uid, SmesVisuals.LastChargeLevel, out var level, args.Component) || level == 0)
|
||||||
{
|
{
|
||||||
args.Sprite.LayerSetVisible(SmesVisualLayers.Charge, false);
|
_sprite.LayerSetVisible((uid, args.Sprite), SmesVisualLayers.Charge, false);
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
args.Sprite.LayerSetVisible(SmesVisualLayers.Charge, true);
|
_sprite.LayerSetVisible((uid, args.Sprite), SmesVisualLayers.Charge, true);
|
||||||
args.Sprite.LayerSetState(SmesVisualLayers.Charge, $"{comp.ChargeOverlayPrefix}{level}");
|
_sprite.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Charge, $"{comp.ChargeOverlayPrefix}{level}");
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!AppearanceSystem.TryGetData<ChargeState>(uid, SmesVisuals.LastChargeState, out var state, args.Component))
|
if (!AppearanceSystem.TryGetData<ChargeState>(uid, SmesVisuals.LastChargeState, out var state, args.Component))
|
||||||
@@ -27,22 +29,22 @@ public sealed class SmesVisualizerSystem : VisualizerSystem<SmesComponent>
|
|||||||
switch (state)
|
switch (state)
|
||||||
{
|
{
|
||||||
case ChargeState.Still:
|
case ChargeState.Still:
|
||||||
args.Sprite.LayerSetState(SmesVisualLayers.Input, $"{comp.InputOverlayPrefix}0");
|
_sprite.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Input, $"{comp.InputOverlayPrefix}0");
|
||||||
args.Sprite.LayerSetState(SmesVisualLayers.Output, $"{comp.OutputOverlayPrefix}1");
|
_sprite.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Output, $"{comp.OutputOverlayPrefix}1");
|
||||||
break;
|
break;
|
||||||
case ChargeState.Charging:
|
case ChargeState.Charging:
|
||||||
args.Sprite.LayerSetState(SmesVisualLayers.Input, $"{comp.InputOverlayPrefix}1");
|
_sprite.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Input, $"{comp.InputOverlayPrefix}1");
|
||||||
args.Sprite.LayerSetState(SmesVisualLayers.Output, $"{comp.OutputOverlayPrefix}1");
|
_sprite.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Output, $"{comp.OutputOverlayPrefix}1");
|
||||||
break;
|
break;
|
||||||
case ChargeState.Discharging:
|
case ChargeState.Discharging:
|
||||||
args.Sprite.LayerSetState(SmesVisualLayers.Input, $"{comp.InputOverlayPrefix}0");
|
_sprite.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Input, $"{comp.InputOverlayPrefix}0");
|
||||||
args.Sprite.LayerSetState(SmesVisualLayers.Output, $"{comp.OutputOverlayPrefix}2");
|
_sprite.LayerSetRsiState((uid, args.Sprite), SmesVisualLayers.Output, $"{comp.OutputOverlayPrefix}2");
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
enum SmesVisualLayers : byte
|
public enum SmesVisualLayers : byte
|
||||||
{
|
{
|
||||||
Input,
|
Input,
|
||||||
Charge,
|
Charge,
|
||||||
|
|||||||
Reference in New Issue
Block a user