Converted BurnState Visualizer (#15661)

Content.Client/Smoking/BurnStateVisualizer.cs

@@ -1,49 +0,0 @@
-using Content.Shared.Smoking;
-using JetBrains.Annotations;
-using Robust.Client.GameObjects;
-using Robust.Shared.GameObjects;
-using Robust.Shared.IoC;
-using Robust.Shared.Serialization;
-using Robust.Shared.Serialization.Manager.Attributes;
-
-namespace Content.Client.Smoking
-{
-    [UsedImplicitly]
-    public sealed class BurnStateVisualizer : AppearanceVisualizer, ISerializationHooks
-    {
-        [Dependency] private readonly IEntityManager _entMan = default!;
-
-        [DataField("burntIcon")]
-        private string _burntIcon = "burnt-icon";
-        [DataField("litIcon")]
-        private string _litIcon = "lit-icon";
-        [DataField("unlitIcon")]
-        private string _unlitIcon = "icon";
-
-        void ISerializationHooks.AfterDeserialization()
-        {
-            IoCManager.InjectDependencies(this);
-        }
-
-        [Obsolete("Subscribe to AppearanceChangeEvent instead.")]
-        public override void OnChangeData(AppearanceComponent component)
-        {
-            base.OnChangeData(component);
-
-            if (!_entMan.TryGetComponent(component.Owner, out SpriteComponent? sprite))
-                return;
-
-            if (!component.TryGetData<SmokableState>(SmokingVisuals.Smoking, out var burnState))
-                return;
-
-            var state = burnState switch
-            {
-                SmokableState.Lit => _litIcon,
-                SmokableState.Burnt => _burntIcon,
-                _ => _unlitIcon
-            };
-
-            sprite.LayerSetState(0, state);
-        }
-    }
-}

Content.Client/Smoking/BurnStateVisualizerSystem.cs

@@ -0,0 +1,25 @@
+using Robust.Client.GameObjects;
+using Content.Shared.Smoking;
+
+namespace Content.Client.Smoking;
+
+public sealed class BurnStateVisualizerSystem : VisualizerSystem<BurnStateVisualsComponent>
+{
+    protected override void OnAppearanceChange(EntityUid uid, BurnStateVisualsComponent component, ref AppearanceChangeEvent args)
+    {
+        if (args.Sprite == null)
+            return;
+        if (!args.AppearanceData.TryGetValue(SmokingVisuals.Smoking, out var burnState))
+            return;
+
+        var state = burnState switch
+        {
+            SmokableState.Lit => component.LitIcon,
+            SmokableState.Burnt => component.BurntIcon,
+            _ => component.UnlitIcon
+        };
+
+        args.Sprite.LayerSetState(0, state);
+    }
+}
+

Content.Client/Smoking/BurnStateVisualsComponent.cs

@@ -0,0 +1,13 @@
+namespace Content.Client.Smoking;
+
+[RegisterComponent]
+public sealed class BurnStateVisualsComponent : Component
+{
+    [DataField("burntIcon")]
+    public string BurntIcon = "burnt-icon";
+    [DataField("litIcon")]
+    public string LitIcon = "lit-icon";
+    [DataField("unlitIcon")]
+    public string UnlitIcon = "icon";
+}
+

Content.Shared/Smoking/SmokableState.cs

@@ -1,4 +1,4 @@
-using Robust.Shared.Serialization;
+using Robust.Shared.Serialization;
 
 namespace Content.Shared.Smoking
 {

Resources/Prototypes/Entities/Objects/Consumable/Smokeables/Pipes/pipe.yml

@@ -16,8 +16,7 @@
     size: 3
     sprite: Objects/Consumable/Smokeables/Pipes/pipe.rsi
   - type: Appearance
-    visuals:
-      - type: BurnStateVisualizer
+  - type: BurnStateVisuals
     unlitIcon: unlit-icon
 
 - type: entity

Resources/Prototypes/Entities/Objects/Consumable/Smokeables/base_smokeables.yml

@@ -8,8 +8,7 @@
   - type: Sprite
     netsync: false
   - type: Appearance
-    visuals:
-      - type: BurnStateVisualizer
+  - type: BurnStateVisuals
   - type: Tag
     tags:
     - Trash

Resources/Prototypes/Entities/Objects/Tools/matches.yml

@@ -1,4 +1,4 @@
-- type: entity
+- type: entity
   id: SmallboxItem
   parent: BaseStorageItem
   abstract: true
@@ -37,8 +37,7 @@
     radius: 1.1
     color: darkorange
   - type: Appearance
-    visuals:
-      - type: BurnStateVisualizer
+  - type: BurnStateVisuals
     unlitIcon: match_unlit
     litIcon: match_lit
     burntIcon: match_burnt