Fix null dereference

Dummy/Actions/Interaction/Actions/ExecuteCommandAction.cs

@@ -1,5 +1,4 @@
-extern alias JetBrainsAnnotations;
-using Dummy.API;
+using Dummy.API;
 using Dummy.Users;
 using OpenMod.API.Commands;
 using System;

Dummy/Commands/CommandDummyCopy.cs

@@ -41,7 +41,7 @@ protected override async Task OnExecuteAsync()
             {
                 settings = m_Configuration.GetSection("default").Get<ConfigurationSettings>();
 
-                settings.CharacterName = name!;
+                settings!.CharacterName = name!;
                 settings.NickName = name!;
                 settings.PlayerName = name!;
             }

Dummy/Commands/CommandDummyCreate.cs

@@ -41,7 +41,7 @@ protected override async Task OnExecuteAsync()
             {
                 settings = m_Configuration.GetSection("default").Get<ConfigurationSettings>();
 
-                settings.CharacterName = name!;
+                settings!.CharacterName = name!;
                 settings.NickName = name!;
                 settings.PlayerName = name!;
             }

Dummy/Commands/CommandDummySpawn.cs

@@ -27,7 +27,7 @@ public CommandDummySpawn(IServiceProvider serviceProvider, IDummyProvider dummyP
 
         protected override async Task OnExecuteAsync()
         {
-            var settings = m_Configuration.Get<Configuration>().Default;
+            var settings = m_Configuration.Get<Configuration>()!.Default;
             var name = CommandDummy.s_NameArgument.GetArgument(Context.Parameters);
             if (!string.IsNullOrEmpty(name))
             {

Dummy/ConfigurationEx/NullConfiguration.cs

@@ -29,10 +29,10 @@ public IChangeToken GetReloadToken()
             return NullChangeToken.Singleton;
         }
 
-        public string this[string key]
+        public string? this[string key]
         {
             get => default!;
-            set {}
+            set { }
         }
     }
 }

Dummy/Events/DummyDeadEvent.cs

@@ -67,12 +67,14 @@ private static async UniTaskVoid Revive(UnturnedPlayer player)
                 return;
             }
 
+            // todo rewrite to revive and then teleport it back(?)
+
             var life = player.Player.life;
             var transform = life.transform;
             life.sendRevive();
 
             s_SendRevive.InvokeAndLoopback(life.GetNetId(), ENetReliability.Reliable,
-                Provider.EnumerateClients_Remote(), transform.position,
+                Provider.GatherRemoteClientConnections(), transform.position,
                 MeasurementTool.angleToByte(transform.rotation.eulerAngles.y));
         }
     }

Dummy/NetTransports/DummyTransportConnection.cs

@@ -18,7 +18,7 @@ public DummyTransportConnection(IPluginAccessor<Dummy> pluginAccessor)
             var random = new System.Random();
             var config = pluginAccessor.Instance!.Configuration.Get<Configuration>();
 
-            m_IP = config.Connection.RandomIp || config.Default.IP == null
+            m_IP = config!.Connection.RandomIp || config.Default.IP == null
                 ? $"{random.Next(1, 256)}.{random.Next(256)}.{random.Next(256)}.{random.Next(256)}"
                 : (m_Address = config.Default.IP).ToString();
             m_Address ??= IPAddress.Parse(m_IP);
@@ -75,7 +75,7 @@ public void Send(byte[] buffer, long size, ENetReliability sendType)
             if (num >= NetReflection.clientMethodsLength)
                 return;
 
-            var method = NetReflection.clientMethods[num];
+            var method = NetReflection.clientMethods[(int)num];
 
             if (method.declaringType == typeof(PlayerInput)
                 && method.debugName.Contains("ReceiveSimulateMispredictedInputs"))

Dummy/Permissions/DummyPermissionCheckProvider.cs

@@ -26,7 +26,7 @@ public DummyPermissionCheckProvider(IPluginAccessor<Dummy> pluginAccessor) : bas
             }
 
             var options = pluginAccessor.Instance!.Configuration.GetValue<ConfigurationOptions>("options");
-            return options.CanExecuteCommands || options.IsAdmin;
+            return options!.CanExecuteCommands || options.IsAdmin;
         })
         {
         }

Dummy/Players/DummyPlayer.cs

@@ -26,7 +26,7 @@ public bool Equals(DummyUser other)
             return other.SteamID.Equals(SteamId);
         }
 
-        public override bool Equals(object obj)
+        public override bool Equals(object? obj)
         {
             if (obj is DummyPlayer other) return Equals(other);
             else return false;

Dummy/Services/DummyUserProvider.cs

@@ -262,7 +262,7 @@ public async Task<DummyUser> AddDummyAsync(CSteamID? id, HashSet<CSteamID>? owne
 
                 pending = new(GetTransportConnection(), playerID, userSteamPlayer.isPro,
                     userSteamPlayer.face, userSteamPlayer.hair, userSteamPlayer.beard, userSteamPlayer.skin,
-                    userSteamPlayer.color, userSteamPlayer.markerColor, userSteamPlayer.hand,
+                    userSteamPlayer.color, userSteamPlayer.markerColor, userSteamPlayer.IsLeftHanded,
                     (ulong)userSteamPlayer.shirtItem, (ulong)userSteamPlayer.pantsItem, (ulong)userSteamPlayer.hatItem,
                     (ulong)userSteamPlayer.backpackItem, (ulong)userSteamPlayer.vestItem,
                     (ulong)userSteamPlayer.maskItem, (ulong)userSteamPlayer.glassesItem, Array.Empty<ulong>(),
@@ -275,7 +275,7 @@ public async Task<DummyUser> AddDummyAsync(CSteamID? id, HashSet<CSteamID>? owne
             }
             else
             {
-                settings ??= config.Default;
+                settings ??= config!.Default;
                 if (settings is null)
                 {
                     throw new ArgumentNullException(nameof(settings));
@@ -315,7 +315,7 @@ public async Task<DummyUser> AddDummyAsync(CSteamID? id, HashSet<CSteamID>? owne
 
             var logJoinLeave = configuration.GetSection("logs:enableJoinLeaveLog").Get<bool>();
 
-            await PreAddDummyAsync(pending, config.Events!, logJoinLeave);
+            await PreAddDummyAsync(pending, config!.Events!, logJoinLeave);
             try
             {
                 await UniTask.SwitchToMainThread();
@@ -474,7 +474,7 @@ private void ValidateSpawn(CSteamID id)
                 throw new DummyContainsException(localizer, id.m_SteamID); // or id is taken
             }
 
-            var amountDummiesConfig = configuration.Get<Configuration>().Options?.AmountDummies ?? 0;
+            var amountDummiesConfig = configuration.Get<Configuration>()!.Options?.AmountDummies ?? 0;
             if (amountDummiesConfig != 0 && DummyUsers.Count + 1 > amountDummiesConfig)
             {
                 throw new DummyOverflowsException(localizer, (byte)DummyUsers.Count, amountDummiesConfig);

Dummy/Threads/DummyUserSimulationThread.Equipment.cs

@@ -12,7 +12,7 @@ public partial class DummyUserSimulationThread
 
     private void SimulateEquipment()
     {
-        if (Player.equipment.isSelected)
+        if (Player.equipment.HasValidUseable)
         {
             // todo: simulate useable
             return;