chore: updated engine to use the new events

Engine

@@ -1 +1 @@
-Subproject commit b0f8b0dad6ec3b5c9b0ebff0bcf535ebf0901e30
+Subproject commit 61e276158001cac85dc5758546fa839e091c0299

Platforms/Desktop/KeyboardInputsBehaviour.cs

@@ -10,8 +10,11 @@ namespace Pong.Platforms.Desktop;
 
 public class KeyboardInputsBehaviour : Behaviour, IButtonInputs<Keys>
 {
-    private readonly Dictionary<Keys, IButtonInputs<Keys>.ButtonCallbackEventHandler> OnPressed = new(256);
+    public Event<IButtonInputs<Keys>, IButtonInputs<Keys>.ButtonCallbackArguments> OnAnyButtonPressed { get; } = new();
-    private readonly Dictionary<Keys, IButtonInputs<Keys>.ButtonCallbackEventHandler> OnReleased = new(256);
+    public Event<IButtonInputs<Keys>, IButtonInputs<Keys>.ButtonCallbackArguments> OnAnyButtonReleased { get; } = new();
+
+    private readonly Dictionary<Keys, Event<IButtonInputs<Keys>, IButtonInputs<Keys>.ButtonCallbackArguments>> OnPressed = new(256);
+    private readonly Dictionary<Keys, Event<IButtonInputs<Keys>, IButtonInputs<Keys>.ButtonCallbackArguments>> OnReleased = new(256);
 
     private int cachePressedCurrentlyCount = 0;
     private readonly Keys[] cachePressedCurrently = new Keys[256];
@@ -19,41 +22,38 @@ public class KeyboardInputsBehaviour : Behaviour, IButtonInputs<Keys>
     private int cachePressedPreviouslyCount = 0;
     private readonly Keys[] cachePressedPreviously = new Keys[256];
 
-    public event IButtonInputs<Keys>.ButtonCallbackEventHandler? OnAnyButtonPressed = null;
+    public void RegisterOnPress(Keys key, Event<IButtonInputs<Keys>, IButtonInputs<Keys>.ButtonCallbackArguments>.EventHandler callback)
-    public event IButtonInputs<Keys>.ButtonCallbackEventHandler? OnAnyButtonReleased = null;
-
-    public void RegisterOnPress(Keys key, IButtonInputs<Keys>.ButtonCallbackEventHandler callback)
     {
-        if (OnPressed.TryGetValue(key, out var action))
+        if (!OnPressed.TryGetValue(key, out Event<IButtonInputs<Keys>, IButtonInputs<Keys>.ButtonCallbackArguments>? delegateCallback))
         {
-            action += callback;
+            delegateCallback = new();
-            return;
+            OnPressed.Add(key, delegateCallback);
         }
 
-        OnPressed.Add(key, callback);
+        delegateCallback.AddListener(callback);
     }
 
-    public void UnregisterOnPress(Keys key, IButtonInputs<Keys>.ButtonCallbackEventHandler callback)
+    public void UnregisterOnPress(Keys key, Event<IButtonInputs<Keys>, IButtonInputs<Keys>.ButtonCallbackArguments>.EventHandler callback)
     {
-        if (OnPressed.TryGetValue(key, out var action))
+        if (OnPressed.TryGetValue(key, out Event<IButtonInputs<Keys>, IButtonInputs<Keys>.ButtonCallbackArguments>? delegateCallback))
-            action -= callback;
+            delegateCallback.RemoveListener(callback);
     }
 
-    public void RegisterOnRelease(Keys key, IButtonInputs<Keys>.ButtonCallbackEventHandler callback)
+    public void RegisterOnRelease(Keys key, Event<IButtonInputs<Keys>, IButtonInputs<Keys>.ButtonCallbackArguments>.EventHandler callback)
     {
-        if (OnReleased.TryGetValue(key, out var action))
+        if (!OnReleased.TryGetValue(key, out Event<IButtonInputs<Keys>, IButtonInputs<Keys>.ButtonCallbackArguments>? delegateCallback))
         {
-            action += callback;
+            delegateCallback = new();
-            return;
+            OnReleased.Add(key, delegateCallback);
         }
 
-        OnReleased.Add(key, callback);
+        delegateCallback.AddListener(callback);
     }
 
-    public void UnregisterOnRelease(Keys key, IButtonInputs<Keys>.ButtonCallbackEventHandler callback)
+    public void UnregisterOnRelease(Keys key, Event<IButtonInputs<Keys>, IButtonInputs<Keys>.ButtonCallbackArguments>.EventHandler callback)
     {
-        if (OnReleased.TryGetValue(key, out var action))
+        if (OnReleased.TryGetValue(key, out Event<IButtonInputs<Keys>, IButtonInputs<Keys>.ButtonCallbackArguments>? delegateCallback))
-            action -= callback;
+            delegateCallback.RemoveListener(callback);
     }
 
     protected override void OnUpdate()
@@ -66,28 +66,26 @@ public class KeyboardInputsBehaviour : Behaviour, IButtonInputs<Keys>
         {
             Keys currentlyPressedKey = cachePressedCurrently[i];
 
-            if (!OnPressed.TryGetValue(currentlyPressedKey, out var action))
-                continue;
-
             if (WasPressed(currentlyPressedKey))
                 continue;
 
-            action.Invoke(this, currentlyPressedKey);
+            if (OnPressed.TryGetValue(currentlyPressedKey, out Event<IButtonInputs<Keys>, IButtonInputs<Keys>.ButtonCallbackArguments>? callback))
-            OnAnyButtonPressed?.Invoke(this, currentlyPressedKey);
+                callback?.Invoke(this, new(currentlyPressedKey));
+
+            OnAnyButtonPressed?.Invoke(this, new(currentlyPressedKey));
         }
 
         for (int i = 0; i < cachePressedPreviouslyCount; i++)
         {
             Keys previouslyPressedKey = cachePressedPreviously[i];
 
-            if (!OnReleased.TryGetValue(previouslyPressedKey, out var action))
-                continue;
-
             if (IsPressed(previouslyPressedKey))
                 continue;
 
-            action.Invoke(this, previouslyPressedKey);
+            if (OnReleased.TryGetValue(previouslyPressedKey, out Event<IButtonInputs<Keys>, IButtonInputs<Keys>.ButtonCallbackArguments>? callback))
-            OnAnyButtonReleased?.Invoke(this, previouslyPressedKey);
+                callback?.Invoke(this, new(previouslyPressedKey));
+
+            OnAnyButtonReleased?.Invoke(this, new(previouslyPressedKey));
         }
 
         Array.Copy(cachePressedCurrently, cachePressedPreviously, cachePressedCurrentlyCount);
@@ -102,7 +100,7 @@ public class KeyboardInputsBehaviour : Behaviour, IButtonInputs<Keys>
         return false;
     }
 
-    public bool WasPressed(Keys key)
+    private bool WasPressed(Keys key)
     {
         for (int i = 0; i < cachePressedPreviouslyCount; i++)
             if (cachePressedPreviously[i] == key)

Shared/Behaviours/BallBehaviour.cs

@@ -19,7 +19,7 @@ public class BallBehaviour : Behaviour2D, IPhysicsUpdate, INetworkEntity,
 
     protected override void OnFirstActiveFrame()
     {
-        BehaviourController.GetRequiredBehaviour<ICollider2D>().OnCollisionDetected += OnCollisionDetected;
+        BehaviourController.GetRequiredBehaviour<ICollider2D>().OnCollisionDetected.AddListener(OnCollisionDetected);
         physicsEngine2D = Universe.GetRequiredUniverseObject<IPhysicsEngine2D>();
         tweenManager = Universe.GetRequiredUniverseObject<ITweenManager>();
         RigidBody = BehaviourController.GetRequiredBehaviour<IRigidBody2D>();

Shared/Behaviours/CameraController.cs

@@ -41,7 +41,7 @@ public class CameraController : Behaviour
             cameraBehaviour.Transform.Rotation -= Universe.Time.DeltaTime * 45f;
     }
 
-    private void SwitchToFullScreen(IButtonInputs<Keys> inputs, Keys keys)
+    private void SwitchToFullScreen(IButtonInputs<Keys> sender, IButtonInputs<Keys>.ButtonCallbackArguments args)
     {
         if (cameraBehaviour.Graphics.IsFullScreen)
             return;
@@ -59,7 +59,7 @@ public class CameraController : Behaviour
         cameraBehaviour.Viewport = cameraBehaviour.Graphics.GraphicsDevice.Viewport;
     }
 
-    private void ResetCamera(IButtonInputs<Keys> inputs, Keys keys)
+    private void ResetCamera(IButtonInputs<Keys> sender, IButtonInputs<Keys>.ButtonCallbackArguments args)
     {
         cameraBehaviour.Zoom = defaultZoomLevel;
         cameraBehaviour.Transform.LocalPosition = Vector2D.Zero;

Shared/Behaviours/MovementBallBehaviour.cs

@@ -16,7 +16,7 @@ public class MovementBallBehaviour : Behaviour2D
         rigidBody = BehaviourController.GetRequiredBehaviour<IRigidBody2D>();
 
         rigidBody.Velocity = StartDirection * Speed;
-        BehaviourController.GetRequiredBehaviour<ICollider2D>().OnCollisionDetected += OnCollisionDetected;
+        BehaviourController.GetRequiredBehaviour<ICollider2D>().OnCollisionDetected.AddListener(OnCollisionDetected);
     }
 
     protected override void OnUpdate()

Shared/Behaviours/PaddleBehaviour.cs

@@ -6,7 +6,6 @@ using Syntriax.Engine.Core;
 using Syntriax.Engine.Network;
 using Syntriax.Engine.Physics2D;
 using Syntriax.Engine.Systems.Input;
-using Syntriax.Engine.Systems.Tween;
 
 namespace Pong.Behaviours;
 
@@ -57,10 +56,10 @@ public class PaddleBehaviour(Keys Up, Keys Down, float High, float Low, float Sp
         inputs.UnregisterOnRelease(Down, OnDownReleased);
     }
 
-    private void OnUpPressed(IButtonInputs<Keys> inputs, Keys keys) { isUpPressed = true; UpdateVelocity(); networkClient?.SendToServer(new PaddleKeyStatePacket(this)); }
+    private void OnUpPressed(IButtonInputs<Keys> sender, IButtonInputs<Keys>.ButtonCallbackArguments args) { isUpPressed = true; UpdateVelocity(); networkClient?.SendToServer(new PaddleKeyStatePacket(this)); }
-    private void OnUpReleased(IButtonInputs<Keys> inputs, Keys keys) { isUpPressed = false; UpdateVelocity(); networkClient?.SendToServer(new PaddleKeyStatePacket(this)); }
+    private void OnUpReleased(IButtonInputs<Keys> sender, IButtonInputs<Keys>.ButtonCallbackArguments args) { isUpPressed = false; UpdateVelocity(); networkClient?.SendToServer(new PaddleKeyStatePacket(this)); }
-    private void OnDownPressed(IButtonInputs<Keys> inputs, Keys keys) { isDownPressed = true; UpdateVelocity(); networkClient?.SendToServer(new PaddleKeyStatePacket(this)); }
+    private void OnDownPressed(IButtonInputs<Keys> sender, IButtonInputs<Keys>.ButtonCallbackArguments args) { isDownPressed = true; UpdateVelocity(); networkClient?.SendToServer(new PaddleKeyStatePacket(this)); }
-    private void OnDownReleased(IButtonInputs<Keys> inputs, Keys keys) { isDownPressed = false; UpdateVelocity(); networkClient?.SendToServer(new PaddleKeyStatePacket(this)); }
+    private void OnDownReleased(IButtonInputs<Keys> sender, IButtonInputs<Keys>.ButtonCallbackArguments args) { isDownPressed = false; UpdateVelocity(); networkClient?.SendToServer(new PaddleKeyStatePacket(this)); }
 
     private void UpdateVelocity()
     {

Shared/Behaviours/WallScoreBehaviour.cs

@@ -14,6 +14,6 @@ public class WallScoreBehaviour(Action OnCollision) : Behaviour2D
         if (!BehaviourController.TryGetBehaviour(out ICollider2D? collider2D))
             return;
 
-        collider2D.OnCollisionDetected += (_, _1) => OnCollision?.Invoke();
+        collider2D.OnCollisionDetected.AddListener((_, _1) => OnCollision?.Invoke());
     }
 }

Shared/GamePong.cs

@@ -48,11 +48,12 @@ public class GamePong : Game
     {
         // TODO: Add your initialization logic here
         universe = new();
-        displayableCollector = new(universe);
-        displayableShapeCollector = new(universe);
 
         universe.Initialize();
 
+        displayableCollector = new(universe);
+        displayableShapeCollector = new(universe);
+
         base.Initialize();
     }
 

Shared/Network/LiteNetLib/LiteNetLibServer.cs

@@ -60,17 +60,17 @@ public class LiteNetLibServer : LiteNetLibCommunicatorBase, INetworkCommunicator
         return this;
     }
 
-    private void PollEvents(IUniverse sender, UniverseTime engineTime) => Manager.PollEvents();
+    private void PollEvents(IUniverse sender, IUniverse.UpdateArguments args) => Manager.PollEvents();
 
     protected override void OnEnteringUniverse(IUniverse universe)
     {
         base.OnEnteringUniverse(universe);
-        universe.OnPostUpdate += PollEvents;
+        universe.OnPostUpdate.AddListener(PollEvents);
     }
 
     protected override void OnExitingUniverse(IUniverse universe)
     {
         base.OnExitingUniverse(universe);
-        universe.OnPostUpdate -= PollEvents;
+        universe.OnPostUpdate.RemoveListener(PollEvents);
     }
 }

Shared/Network/NetworkManager.cs

@@ -41,10 +41,9 @@ public class NetworkManager : UniverseObject, INetworkManager
     {
         CacheDelegates();
 
-        _networkEntityCollector.OnCollected += OnCollected;
+        _networkEntityCollector.OnCollected.AddListener(OnCollected);
-        _networkEntityCollector.OnRemoved += OnRemoved;
+        _networkEntityCollector.OnRemoved.AddListener(OnRemoved);
     }
-
     private void CacheDelegates()
     {
         // Find network packets implementing INetworkPacket
@@ -121,12 +120,14 @@ public class NetworkManager : UniverseObject, INetworkManager
                     serverListenerData.ListenerMethod.Invoke(serverListenerData.Object, [entityDataPacket, fromClientId]);
     }
 
-    private void OnCollected(IBehaviourCollector<INetworkEntity> sender, INetworkEntity behaviourCollected)
+    private void OnCollected(IBehaviourCollector<INetworkEntity> sender, IBehaviourCollector<INetworkEntity>.BehaviourCollectedArguments args)
     {
-        if (!_networkEntities.TryAdd(behaviourCollected.Id, behaviourCollected))
+        INetworkEntity collectedBehaviour = args.BehaviourCollected;
-            throw new($"Unable to add {behaviourCollected.Id} to {nameof(NetworkManager)}");
 
-        foreach (Type clientListenerType in behaviourCollected.GetType().GetInterfaces().Where(i => i.IsGenericType && i.GetGenericTypeDefinition() == typeof(IPacketListenerClient<>)))
+        if (!_networkEntities.TryAdd(collectedBehaviour.Id, collectedBehaviour))
+            throw new($"Unable to add {collectedBehaviour.Id} to {nameof(NetworkManager)}");
+
+        foreach (Type clientListenerType in collectedBehaviour.GetType().GetInterfaces().Where(i => i.IsGenericType && i.GetGenericTypeDefinition() == typeof(IPacketListenerClient<>)))
         {
             Type clientListenerParameterType = clientListenerType.GetGenericArguments().First();
             MethodInfo clientListenerReceiveMethod = clientListenerType.GetMethods().First(m => m.Name == nameof(IPacketListenerClient<INetworkEntity>.OnClientPacketArrived));
@@ -137,10 +138,10 @@ public class NetworkManager : UniverseObject, INetworkManager
                 clientPacketListeners.Add(clientListenerParameterType, clientListeners);
             }
 
-            clientListeners.Add(behaviourCollected.Id, new(behaviourCollected, clientListenerReceiveMethod));
+            clientListeners.Add(collectedBehaviour.Id, new(collectedBehaviour, clientListenerReceiveMethod));
         }
 
-        foreach (Type serverListenerType in behaviourCollected.GetType().GetInterfaces().Where(i => i.IsGenericType && i.GetGenericTypeDefinition() == typeof(IPacketListenerServer<>)))
+        foreach (Type serverListenerType in collectedBehaviour.GetType().GetInterfaces().Where(i => i.IsGenericType && i.GetGenericTypeDefinition() == typeof(IPacketListenerServer<>)))
         {
             Type serverListenerParameterType = serverListenerType.GetGenericArguments().First();
             MethodInfo serverListenerReceiveMethod = serverListenerType.GetMethods().First(m => m.Name == nameof(IPacketListenerServer<INetworkEntity>.OnServerPacketArrived));
@@ -151,13 +152,13 @@ public class NetworkManager : UniverseObject, INetworkManager
                 serverPacketListeners.Add(serverListenerParameterType, serverListeners);
             }
 
-            serverListeners.Add(behaviourCollected.Id, new(behaviourCollected, serverListenerReceiveMethod));
+            serverListeners.Add(collectedBehaviour.Id, new(collectedBehaviour, serverListenerReceiveMethod));
         }
     }
 
-    private void OnRemoved(IBehaviourCollector<INetworkEntity> sender, INetworkEntity behaviourRemoved)
+    private void OnRemoved(IBehaviourCollector<INetworkEntity> sender, IBehaviourCollector<INetworkEntity>.BehaviourRemovedArguments args)
     {
-        _networkEntities.Remove(behaviourRemoved.Id);
+        _networkEntities.Remove(args.BehaviourRemoved.Id);
     }
 
     protected override void OnExitingUniverse(IUniverse universe) => _networkEntityCollector.Unassign();