Convert sus events to eventbus (#5044)

Content.Server/Mind/Mind.cs

@@ -166,8 +166,8 @@ namespace Content.Server.Mind
             _roles.Add(role);
             role.Greet();
 
-            var message = new RoleAddedMessage(role);
+            var message = new RoleAddedEvent(role);
-            OwnedEntity?.SendMessage(OwnedComponent, message);
+            OwnedEntity?.EntityManager.EventBus.RaiseLocalEvent(OwnedEntity.Uid, message);
 
             return role;
         }
@@ -188,8 +188,8 @@ namespace Content.Server.Mind
 
             _roles.Remove(role);
 
-            var message = new RoleRemovedMessage(role);
+            var message = new RoleRemovedEvent(role);
-            OwnedEntity?.SendMessage(OwnedComponent, message);
+            OwnedEntity?.EntityManager.EventBus.RaiseLocalEvent(OwnedEntity.Uid, message);
         }
 
         public bool HasRole<T>() where T : Role
@@ -278,7 +278,7 @@ namespace Content.Server.Mind
             if (IsVisitingEntity
                 && (ghostCheckOverride // to force mind transfer, for example from ControlMobVerb
                 || !VisitingEntity!.TryGetComponent(out GhostComponent? ghostComponent) // visiting entity is not a Ghost
-                || !ghostComponent.CanReturnToBody))  // it is a ghost, but cannot return to body anyway, so it's okay                
+                || !ghostComponent.CanReturnToBody))  // it is a ghost, but cannot return to body anyway, so it's okay
             {
                 VisitingEntity = null;
             }

Content.Server/Roles/RoleAddedEvent.cs

@@ -0,0 +1,7 @@
+namespace Content.Server.Roles
+{
+    public sealed class RoleAddedEvent : RoleEvent
+    {
+        public RoleAddedEvent(Role role) : base(role) { }
+    }
+}

Content.Server/Roles/RoleAddedMessage.cs

@@ -1,7 +0,0 @@
-namespace Content.Server.Roles
-{
-    public class RoleAddedMessage : RoleMessage
-    {
-        public RoleAddedMessage(Role role) : base(role) { }
-    }
-}

Content.Server/Roles/RoleEvent.cs

@@ -2,11 +2,11 @@
 
 namespace Content.Server.Roles
 {
-    public class RoleMessage : ComponentMessage
+    public class RoleEvent : EntityEventArgs
     {
         public readonly Role Role;
 
-        public RoleMessage(Role role)
+        public RoleEvent(Role role)
         {
             Role = role;
         }

Content.Server/Roles/RoleRemovedEvent.cs

@@ -0,0 +1,7 @@
+namespace Content.Server.Roles
+{
+    public sealed class RoleRemovedEvent : RoleEvent
+    {
+        public RoleRemovedEvent(Role role) : base(role) { }
+    }
+}

Content.Server/Roles/RoleRemovedMessage.cs

@@ -1,7 +0,0 @@
-namespace Content.Server.Roles
-{
-    public class RoleRemovedMessage : RoleMessage
-    {
-        public RoleRemovedMessage(Role role) : base(role) { }
-    }
-}

Content.Server/Suspicion/EntitySystems/SuspicionRoleSystem.cs

@@ -1,4 +1,6 @@
 using System.Collections.Generic;
+using Content.Server.Roles;
+using Content.Server.Suspicion.Roles;
 using Content.Shared.GameTicking;
 using JetBrains.Annotations;
 using Robust.Server.GameObjects;
@@ -20,20 +22,34 @@ namespace Content.Server.Suspicion.EntitySystems
             base.Initialize();
 
             SubscribeLocalEvent<RoundRestartCleanupEvent>(Reset);
-            SubscribeLocalEvent<SuspicionRoleComponent, PlayerAttachedEvent>((HandlePlayerAttached));
+            SubscribeLocalEvent<SuspicionRoleComponent, PlayerAttachedEvent>(OnPlayerAttached);
-            SubscribeLocalEvent<SuspicionRoleComponent, PlayerDetachedEvent>((HandlePlayerDetached));
+            SubscribeLocalEvent<SuspicionRoleComponent, PlayerDetachedEvent>(OnPlayerDetached);
+            SubscribeLocalEvent<SuspicionRoleComponent, RoleAddedEvent>(OnRoleAdded);
+            SubscribeLocalEvent<SuspicionRoleComponent, RoleRemovedEvent>(OnRoleRemoved);
         }
 
-        private void HandlePlayerDetached(EntityUid uid, SuspicionRoleComponent component, PlayerDetachedEvent args)
+        private void OnPlayerDetached(EntityUid uid, SuspicionRoleComponent component, PlayerDetachedEvent args)
         {
             component.SyncRoles();
         }
 
-        private void HandlePlayerAttached(EntityUid uid, SuspicionRoleComponent component, PlayerAttachedEvent args)
+        private void OnPlayerAttached(EntityUid uid, SuspicionRoleComponent component, PlayerAttachedEvent args)
         {
             component.SyncRoles();
         }
 
+        private void OnRoleAdded(EntityUid uid, SuspicionRoleComponent component, RoleAddedEvent args)
+        {
+            if (args.Role is not SuspicionRole role) return;
+            component.Role = role;
+        }
+
+        private void OnRoleRemoved(EntityUid uid, SuspicionRoleComponent component, RoleRemovedEvent args)
+        {
+            if (args.Role is not SuspicionRole) return;
+            component.Role = null;
+        }
+
         #endregion
 
         public void AddTraitor(SuspicionRoleComponent role)

Content.Server/Suspicion/SuspicionRoleComponent.cs

@@ -162,20 +162,5 @@ namespace Content.Server.Suspicion
 
             return new SuspicionRoleComponentState(Role?.Name, Role?.Antagonist, allies.ToArray());
         }
-
-        public override void HandleMessage(ComponentMessage message, IComponent? component)
-        {
-            base.HandleMessage(message, component);
-
-            switch (message)
-            {
-                case RoleAddedMessage {Role: SuspicionRole role}:
-                    Role = role;
-                    break;
-                case RoleRemovedMessage {Role: SuspicionRole}:
-                    Role = null;
-                    break;
-            }
-        }
     }
 }