ExtinguishOnTrigger and TriggerOnInteractHand (#39537)

* simplely one commit

* simplelly two commit

* requested changes

---------

Co-authored-by: iaada <iaada@users.noreply.github.com>

Content.Server/Trigger/Systems/FireStackOnTriggerSystem.cs

@@ -0,0 +1,53 @@
+using Content.Server.Atmos.EntitySystems;
+using Content.Shared.Trigger;
+using Content.Shared.Trigger.Components.Effects;
+
+namespace Content.Server.Trigger.Systems;
+
+/// <summary>
+/// Trigger system for adding or removing fire stacks from an entity with <see cref="FlammableComponent"/>.
+/// </summary>
+/// <seealso cref="IgniteOnTriggerSystem"/>
+public sealed class FireStackOnTriggerSystem : EntitySystem
+{
+    [Dependency] private readonly FlammableSystem _flame = default!;
+
+    /// <inheritdoc/>
+    public override void Initialize()
+    {
+        base.Initialize();
+
+        SubscribeLocalEvent<FireStackOnTriggerComponent, TriggerEvent>(OnTriggerFlame);
+        SubscribeLocalEvent<ExtinguishOnTriggerComponent, TriggerEvent>(OnTriggerExtinguish);
+    }
+
+    private void OnTriggerFlame(Entity<FireStackOnTriggerComponent> ent, ref TriggerEvent args)
+    {
+        if (args.Key != null && !ent.Comp.KeysIn.Contains(args.Key))
+            return;
+
+        var target = ent.Comp.TargetUser ? args.User : ent.Owner;
+
+        if (target == null)
+            return;
+
+        _flame.AdjustFireStacks(target.Value, ent.Comp.FireStacks, ignite: ent.Comp.DoIgnite);
+
+        args.Handled = true;
+    }
+
+    private void OnTriggerExtinguish(Entity<ExtinguishOnTriggerComponent> ent, ref TriggerEvent args)
+    {
+        if (args.Key != null && !ent.Comp.KeysIn.Contains(args.Key))
+            return;
+
+        var target = ent.Comp.TargetUser ? args.User : ent.Owner;
+
+        if (target == null)
+            return;
+
+        _flame.Extinguish(target.Value);
+
+        args.Handled = true;
+    }
+}

Content.Server/Trigger/Systems/FlameStackOnTriggerSystem.cs

@@ -1,37 +0,0 @@
-using Content.Server.Atmos.EntitySystems;
-using Content.Shared.Trigger;
-using Content.Shared.Trigger.Components.Effects;
-
-namespace Content.Server.Trigger.Systems;
-
-/// <summary>
-/// Trigger system for setting something on fire.
-/// </summary>
-/// <seealso cref="IgniteOnTriggerSystem"/>
-public sealed class FlameStackOnTriggerSystem : EntitySystem
-{
-    [Dependency] private readonly FlammableSystem _flame = default!;
-
-    /// <inheritdoc/>
-    public override void Initialize()
-    {
-        base.Initialize();
-
-        SubscribeLocalEvent<FlameStackOnTriggerComponent, TriggerEvent>(OnTrigger);
-    }
-
-    private void OnTrigger(Entity<FlameStackOnTriggerComponent> ent, ref TriggerEvent args)
-    {
-        if (args.Key != null && !ent.Comp.KeysIn.Contains(args.Key))
-            return;
-
-        var target = ent.Comp.TargetUser ? args.User : ent.Owner;
-
-        if (target == null)
-            return;
-
-        _flame.AdjustFireStacks(target.Value, ent.Comp.FireStacks, ignite: ent.Comp.DoIgnite);
-
-        args.Handled = true;
-    }
-}

Content.Server/Trigger/Systems/IgniteOnTriggerSystem.cs

@@ -8,7 +8,7 @@ namespace Content.Server.Trigger.Systems;
 /// <summary>
 /// Handles igniting when triggered and stopping ignition after the delay.
 /// </summary>
-/// <seealso cref="FlameStackOnTriggerSystem"/>
+/// <seealso cref="FireStackOnTriggerSystem"/>
 public sealed class IgniteOnTriggerSystem : EntitySystem
 {
     [Dependency] private readonly IGameTiming _timing = default!;

Content.Shared/Trigger/Components/Effects/ExtinguishOnTriggerComponent.cs

@@ -0,0 +1,10 @@
+using Robust.Shared.GameStates;
+
+namespace Content.Shared.Trigger.Components.Effects;
+
+/// <summary>
+/// This trigger removes all the fire stacks on a target with <see cref="FlammableComponent"/>.
+/// If TargetUser is true, the entity that caused this trigger will be extinguished instead.
+/// </summary>
+[RegisterComponent, NetworkedComponent, AutoGenerateComponentState]
+public sealed partial class ExtinguishOnTriggerComponent : BaseXOnTriggerComponent;

Content.Shared/Trigger/Components/Effects/FireStackOnTriggerComponent.cs

@@ -9,7 +9,7 @@ namespace Content.Shared.Trigger.Components.Effects;
 /// </summary>
 /// <seealso cref="IgniteOnTriggerComponent"/>
 [RegisterComponent, NetworkedComponent, AutoGenerateComponentState]
-public sealed partial class FlameStackOnTriggerComponent : BaseXOnTriggerComponent
+public sealed partial class FireStackOnTriggerComponent : BaseXOnTriggerComponent
 {
     /// <summary>
     /// How many fire stacks to add or remove.

Content.Shared/Trigger/Components/Effects/IgniteOnTriggerComponent.cs

@@ -8,7 +8,7 @@ namespace Content.Shared.Trigger.Components.Effects;
 /// Requires <see cref="IgnitionSourceComponent"/> along with triggering components.
 /// The if TargetUser is true they will be ignited instead (they need IgnitionSourceComponent as well).
 /// </summary>
-/// <seealso cref="FlameStackOnTriggerComponent"/>
+/// <seealso cref="FireStackOnTriggerComponent"/>
 [RegisterComponent, NetworkedComponent]
 [AutoGenerateComponentState, AutoGenerateComponentPause]
 public sealed partial class IgniteOnTriggerComponent : BaseXOnTriggerComponent

Content.Shared/Trigger/Components/Triggers/TriggerOnInteractHandComponent.cs

@@ -0,0 +1,11 @@
+using Content.Shared.Interaction;
+using Robust.Shared.GameStates;
+
+namespace Content.Shared.Trigger.Components.Triggers;
+
+/// <summary>
+/// Trigger on <see cref="InteractHandEvent"/>, aka clicking on an entity with an empty hand.
+/// User is the player with the hand doing the clicking.
+/// </summary>
+[RegisterComponent, NetworkedComponent, AutoGenerateComponentState]
+public sealed partial class TriggerOnInteractHandComponent : BaseTriggerOnXComponent;

Content.Shared/Trigger/Systems/TriggerSystem.Interaction.cs

@@ -12,6 +12,7 @@ public sealed partial class TriggerSystem
     {
         SubscribeLocalEvent<TriggerOnActivateComponent, ActivateInWorldEvent>(OnActivate);
         SubscribeLocalEvent<TriggerOnUseComponent, UseInHandEvent>(OnUse);
+        SubscribeLocalEvent<TriggerOnInteractHandComponent, InteractHandEvent>(OnInteractHand);
 
         SubscribeLocalEvent<ItemToggleOnTriggerComponent, TriggerEvent>(HandleItemToggleOnTrigger);
         SubscribeLocalEvent<AnchorOnTriggerComponent, TriggerEvent>(HandleAnchorOnTrigger);
@@ -39,6 +40,15 @@ public sealed partial class TriggerSystem
         args.Handled = true;
     }
 
+    private void OnInteractHand(Entity<TriggerOnInteractHandComponent> ent, ref InteractHandEvent args)
+    {
+        if (args.Handled)
+            return;
+
+        Trigger(ent.Owner, args.User, ent.Comp.KeyOut);
+        args.Handled = true;
+    }
+
     private void HandleItemToggleOnTrigger(Entity<ItemToggleOnTriggerComponent> ent, ref TriggerEvent args)
     {
         if (args.Key != null && !ent.Comp.KeysIn.Contains(args.Key))