Mark various removed implementations as obsolete

Libraries/SPTarkov.Server.Core/DI/ServiceLocator.cs

@@ -6,8 +6,10 @@
 ///
 /// This should not be used at all when having direct access to DI.
 /// </summary>
+[Obsolete("This will be removed in the next version of SPT in favor of DI injecting patches")]
 public static class ServiceLocator
 {
+    [Obsolete("This will be removed in the next version of SPT in favor of DI injecting patches")]
     public static IServiceProvider ServiceProvider { get; private set; }
 
     internal static void SetServiceProvider(IServiceProvider provider)

Libraries/SPTarkov.Server.Core/Servers/SaveServer.cs

@@ -31,6 +31,7 @@ public class SaveServer(
     protected const string profileFilepath = "user/profiles/";
 
     // onLoad = require("../bindings/SaveLoad");
+    [Obsolete("This will be removed in the next version of SPT")]
     protected readonly Dictionary<string, Func<SptProfile, SptProfile>> onBeforeSaveCallbacks = new();
 
     protected readonly ConcurrentDictionary<MongoId, SptProfile> profiles = new();
@@ -42,6 +43,7 @@ public class SaveServer(
     /// </summary>
     /// <param name="id"> ID for the save callback </param>
     /// <param name="callback"> Callback to execute prior to running SaveServer.saveProfile() </param>
+    [Obsolete("This will be removed in the next version of SPT")]
     public void AddBeforeSaveCallback(string id, Func<SptProfile, SptProfile> callback)
     {
         onBeforeSaveCallbacks[id] = callback;
@@ -51,6 +53,7 @@ public class SaveServer(
     ///     Remove a callback from being executed prior to saving profile in SaveServer.saveProfile()
     /// </summary>
     /// <param name="id"> ID of Callback to remove </param>
+    [Obsolete("This will be removed in the next version of SPT")]
     public void RemoveBeforeSaveCallback(string id)
     {
         onBeforeSaveCallbacks.Remove(id);