From 57c1978212e246d554ecde05f1e00b11bb4a5e44 Mon Sep 17 00:00:00 2001
From: AeonLucid <aeonlucid@outlook.com>
Date: Sat, 24 Oct 2020 23:10:07 +0200
Subject: [PATCH] Remove scoping for event listeners

---
 src/Impostor.Server/Events/EventManager.cs | 12 ++----------
 1 file changed, 2 insertions(+), 10 deletions(-)

diff --git a/src/Impostor.Server/Events/EventManager.cs b/src/Impostor.Server/Events/EventManager.cs
index 6d0a47f..6147e2f 100644
--- a/src/Impostor.Server/Events/EventManager.cs
+++ b/src/Impostor.Server/Events/EventManager.cs
@@ -59,8 +59,6 @@ namespace Impostor.Server.Events
         public bool IsRegistered<TEvent>()
             where TEvent : IEvent
         {
-            using var scope = _serviceProvider.CreateScope();
-
             return GetHandlers<TEvent>(_serviceProvider).Any();
         }
 
@@ -68,24 +66,18 @@ namespace Impostor.Server.Events
         public async ValueTask CallAsync<T>(T @event)
             where T : IEvent
         {
-            var scope = _serviceProvider.CreateScope();
-
             try
             {
-                foreach (var (handler, eventListener) in GetHandlers<T>(scope.ServiceProvider)
+                foreach (var (handler, eventListener) in GetHandlers<T>(_serviceProvider)
                     .OrderByDescending(e => e.Listener.Priority))
                 {
-                    await eventListener.InvokeAsync(handler, @event, scope.ServiceProvider);
+                    await eventListener.InvokeAsync(handler, @event, _serviceProvider);
                 }
             }
             catch (Exception e)
             {
                 _logger.LogError(e, "Invocation of event {0} threw an exception.", @event.GetType().Name);
             }
-            finally
-            {
-                scope.Dispose();
-            }
         }
 
         /// <summary>
-- 
2.39.5