use prim ctor and fixed warnings: TradeCallbacks, TraderCallbacks, WeatherCallbacks, WishlistCallbacks

Core/Callbacks/TradeCallbacks.cs

@@ -7,18 +7,8 @@ using Core.Models.Eft.Trade;
 namespace Core.Callbacks;
 
 [Injectable]
-public class TradeCallbacks
+public class TradeCallbacks(TradeController _tradeController)
 {
-    protected TradeController _tradeController;
-
-    public TradeCallbacks
-    (
-        TradeController tradeController
-    )
-    {
-        _tradeController = tradeController;
-    }
-
     /// <summary>
     /// Handle client/game/profile/items/moving TradingConfirm event
     /// </summary>

Core/Callbacks/TraderCallbacks.cs

@@ -2,9 +2,6 @@
 using Core.Controllers;
 using Core.DI;
 using Core.Models.Eft.Common;
-using Core.Models.Eft.Common.Tables;
-using Core.Models.Eft.HttpResponse;
-using Core.Models.Enums;
 using Core.Models.Spt.Config;
 using Core.Servers;
 using Core.Utils;
@@ -14,32 +11,23 @@ namespace Core.Callbacks;
 [Injectable(InjectableTypeOverride = typeof(OnLoad), TypePriority = OnLoadOrder.TraderCallbacks)]
 [Injectable(InjectableTypeOverride = typeof(OnUpdate), TypePriority = OnUpdateOrder.TraderCallbacks)]
 [Injectable(InjectableTypeOverride = typeof(TraderCallbacks))]
-public class TraderCallbacks : OnLoad, OnUpdate
+public class TraderCallbacks(
+    HttpResponseUtil _httpResponseUtil,
+    TraderController _traderController,
+    ConfigServer _configServer
+) : OnLoad, OnUpdate
 {
-    protected HttpResponseUtil _httpResponseUtil;
-    protected TraderController _traderController;
-    protected ConfigServer _configServer;
-
-    public TraderCallbacks
-    (
-        HttpResponseUtil httpResponseUtil,
-        TraderController traderController,
-        ConfigServer configServer
-    )
-    {
-        _httpResponseUtil = httpResponseUtil;
-        _traderController = traderController;
-        _configServer = configServer;
-    }
-
-    public async Task OnLoad()
+    private readonly TraderConfig _traderConfig = _configServer.GetConfig<TraderConfig>();
+    
+    public Task OnLoad()
     {
         _traderController.Load();
+        return Task.CompletedTask;
     }
 
-    public async Task<bool> OnUpdate(long _)
+    public Task<bool> OnUpdate(long _)
     {
-        return _traderController.Update();
+        return Task.FromResult(_traderController.Update());
     }
 
     public string GetRoute()
@@ -94,7 +82,6 @@ public class TraderCallbacks : OnLoad, OnUpdate
     /// <returns></returns>
     public string GetModdedTraderData(string url, EmptyRequestData info, string sessionID)
     {
-        var traderConfig = _configServer.GetConfig<TraderConfig>();
-        return _httpResponseUtil.NoBody(traderConfig.ModdedTraders);
+        return _httpResponseUtil.NoBody(_traderConfig.ModdedTraders);
     }
 }

Core/Callbacks/WeatherCallbacks.cs

@@ -1,29 +1,16 @@
 using Core.Annotations;
 using Core.Controllers;
 using Core.Models.Eft.Common;
-using Core.Models.Eft.HttpResponse;
-using Core.Models.Eft.Weather;
-using Core.Models.Spt.Weather;
 using Core.Utils;
 
 namespace Core.Callbacks;
 
 [Injectable]
-public class WeatherCallbacks
+public class WeatherCallbacks(
+    HttpResponseUtil _httpResponseUtil,
+    WeatherController _weatherController
+)
 {
-    protected HttpResponseUtil _httpResponseUtil;
-    protected WeatherController _weatherController;
-
-    public WeatherCallbacks
-    (
-        HttpResponseUtil httpResponseUtil,
-        WeatherController weatherController
-    )
-    {
-        _httpResponseUtil = httpResponseUtil;
-        _weatherController = weatherController;
-    }
-
     /// <summary>
     /// Handle client/weather
     /// </summary>

Core/Callbacks/WishlistCallbacks.cs

@@ -7,18 +7,8 @@ using Core.Models.Eft.Wishlist;
 namespace Core.Callbacks;
 
 [Injectable]
-public class WishlistCallbacks
+public class WishlistCallbacks(WishlistController _wishlistController)
 {
-    protected WishlistController _wishlistController;
-
-    public WishlistCallbacks
-    (
-        WishlistController wishlistController
-    )
-    {
-        _wishlistController = wishlistController;
-    }
-
     /// <summary>
     /// Handle AddToWishList event
     /// </summary>