A reference to the character who triggered a signal is passed to all the recipients of the signal. Allows more accurate server logging: using doors, docking ports or turrets is now properly logged.

Subsurface/Source/Items/Components/Machines/Controller.cs

@@ -158,8 +158,8 @@ namespace Barotrauma.Items.Components
                 return false;
             }
 
-            item.SendSignal(0, "1", "trigger_out");
-
+            item.SendSignal(0, "1", "trigger_out", character);
+            
             ApplyStatusEffects(ActionType.OnUse, 1.0f, activator);
             
             return true;
@@ -194,7 +194,7 @@ namespace Barotrauma.Items.Components
 
             if (focusTarget == null)
             {
-                item.SendSignal(0, ToolBox.Vector2ToString(character.CursorWorldPosition), "position_out");
+                item.SendSignal(0, ToolBox.Vector2ToString(character.CursorWorldPosition), "position_out", character);
                 return;
             }
 
@@ -209,13 +209,13 @@ namespace Barotrauma.Items.Components
             
             if (!character.IsRemotePlayer || character.ViewTarget == focusTarget)
             {
-                item.SendSignal(0, ToolBox.Vector2ToString(character.CursorWorldPosition), "position_out");
+                item.SendSignal(0, ToolBox.Vector2ToString(character.CursorWorldPosition), "position_out", character);
             }
         }
 
         public override bool Pick(Character picker)
         {
-            item.SendSignal(0, "1", "signal_out");
+            item.SendSignal(0, "1", "signal_out", picker);
 
             PlaySound(ActionType.OnUse, item.WorldPosition);
 
@@ -261,7 +261,7 @@ namespace Barotrauma.Items.Components
                 IsActive = true;
             }
 
-            item.SendSignal(0, "1", "signal_out");
+            item.SendSignal(0, "1", "signal_out", character);
             return true;
         }
 

Subsurface/Source/Items/Components/Machines/Engine.cs

@@ -131,9 +131,9 @@ namespace Barotrauma.Items.Components
             force = MathHelper.Lerp(force, 0.0f, 0.1f);
         }
 
-        public override void ReceiveSignal(int stepsTaken, string signal, Connection connection, Item sender, float power=0.0f)
+        public override void ReceiveSignal(int stepsTaken, string signal, Connection connection, Item source, Character sender, float power=0.0f)
         {
-            base.ReceiveSignal(stepsTaken, signal, connection, sender, power);
+            base.ReceiveSignal(stepsTaken, signal, connection, source, sender, power);
 
             if (connection.Name == "set_force")
             {

Subsurface/Source/Items/Components/Machines/MiniMap.cs

@@ -194,9 +194,9 @@ namespace Barotrauma.Items.Components
             }
         }
 
-        public override void ReceiveSignal(int stepsTaken, string signal, Connection connection, Item sender, float power = 0)
+        public override void ReceiveSignal(int stepsTaken, string signal, Connection connection, Item source, Character sender, float power = 0)
         {
-            base.ReceiveSignal(stepsTaken, signal, connection, sender, power);
+            base.ReceiveSignal(stepsTaken, signal, connection, source, sender, power);
 
             if (sender == null || sender.CurrentHull == null) return;
 
@@ -213,7 +213,7 @@ namespace Barotrauma.Items.Components
             {
                 case "water_data_in":
                     //cheating a bit because water detectors don't actually send the water level
-                    if (sender.GetComponent<WaterDetector>() == null)
+                    if (source.GetComponent<WaterDetector>() == null)
                     {
                         hullData.Water = Rand.Range(0.0f, 1.0f);
                     }

Subsurface/Source/Items/Components/Machines/Pump.cs

@@ -198,9 +198,9 @@ namespace Barotrauma.Items.Components
             GuiFrame.Update(1.0f / 60.0f);
         }
 
-        public override void ReceiveSignal(int stepsTaken, string signal, Connection connection, Item sender, float power=0.0f)
+        public override void ReceiveSignal(int stepsTaken, string signal, Connection connection, Item source, Character sender, float power=0.0f)
         {
-            base.ReceiveSignal(stepsTaken, signal, connection, sender, power);
+            base.ReceiveSignal(stepsTaken, signal, connection, source, sender, power);
             
             if (connection.Name == "toggle")
             {

Subsurface/Source/Items/Components/Machines/Reactor.cs

@@ -308,7 +308,7 @@ namespace Barotrauma.Items.Components
             ExtraCooling = 0.0f;
             AvailableFuel = 0.0f;
 
-            item.SendSignal(0, ((int)temperature).ToString(), "temperature_out");
+            item.SendSignal(0, ((int)temperature).ToString(), "temperature_out", null);
               
             sendUpdateTimer = Math.Max(sendUpdateTimer - deltaTime, 0.0f);
 
@@ -532,7 +532,7 @@ namespace Barotrauma.Items.Components
             GUI.DrawLine(spriteBatch, prevPoint, lastPoint, color);
         }
 
-        public override void ReceiveSignal(int stepsTaken, string signal, Connection connection, Item sender, float power)
+        public override void ReceiveSignal(int stepsTaken, string signal, Connection connection, Item source, Character sender, float power)
         {
             switch (connection.Name)
             {

Subsurface/Source/Items/Components/Machines/Steering.cs

@@ -165,13 +165,13 @@ namespace Barotrauma.Items.Components
                 UpdateAutoPilot(deltaTime);
             }
 
-            item.SendSignal(0, targetVelocity.X.ToString(CultureInfo.InvariantCulture), "velocity_x_out");
+            item.SendSignal(0, targetVelocity.X.ToString(CultureInfo.InvariantCulture), "velocity_x_out", null);
 
             float targetLevel = -targetVelocity.Y;
 
             targetLevel += (neutralBallastLevel - 0.5f) * 100.0f;
 
-            item.SendSignal(0, targetLevel.ToString(CultureInfo.InvariantCulture), "velocity_y_out");
+            item.SendSignal(0, targetLevel.ToString(CultureInfo.InvariantCulture), "velocity_y_out", null);
 
 
             voltage -= deltaTime;
@@ -476,7 +476,7 @@ namespace Barotrauma.Items.Components
             return true;
         }
 
-        public override void ReceiveSignal(int stepsTaken, string signal, Connection connection, Item sender, float power=0.0f)
+        public override void ReceiveSignal(int stepsTaken, string signal, Connection connection, Item source, Character sender, float power=0.0f)
         {
             if (connection.Name == "velocity_in")
             {
@@ -484,7 +484,7 @@ namespace Barotrauma.Items.Components
             }
             else
             {
-                base.ReceiveSignal(stepsTaken, signal, connection, sender, power);
+                base.ReceiveSignal(stepsTaken, signal, connection, source, sender, power);
             }
         }