Renamed a variable opPlayer to player

reason: no longer uses Optional<T>

Author: Measurity

NitroxClient/Communication/Packets/Processors/PlayerHeldItemChangedProcessor.cs

@@ -28,12 +28,11 @@ public PlayerHeldItemChangedProcessor(PlayerManager playerManager)
 
     public Task Process(ClientProcessorContext context, PlayerHeldItemChanged packet)
     {
-        if (!playerManager.TryFind(packet.SessionId, out RemotePlayer opPlayer))
+        if (!playerManager.TryFind(packet.SessionId, out RemotePlayer player))
         {
             Log.Warn($"[{nameof(PlayerHeldItemChangedProcessor)}] Could not find player with session id: {packet.SessionId}.");
             return Task.CompletedTask;
         }
-
         if (!NitroxEntity.TryGetObjectFrom(packet.ItemId, out GameObject item))
         {
             Log.Warn($"[{nameof(PlayerHeldItemChangedProcessor)}] Could not find entity with id: {packet.ItemId}.");
@@ -45,15 +44,15 @@ public Task Process(ClientProcessorContext context, PlayerHeldItemChanged packet
 
         Validate.NotNull(pickupable.inventoryItem);
 
-        ItemsContainer inventory = opPlayer.Inventory;
+        ItemsContainer inventory = player.Inventory;
         PlayerTool tool = item.GetComponent<PlayerTool>();
 
         // Copied from QuickSlots
         switch (packet.Type)
         {
             case PlayerHeldItemChanged.ChangeType.DRAW_AS_TOOL:
                 Validate.IsTrue(tool);
-                ModelPlug.PlugIntoSocket(tool, opPlayer.ItemAttachPoint);
+                ModelPlug.PlugIntoSocket(tool, player.ItemAttachPoint);
                 Utils.SetLayerRecursively(item, viewModelLayer);
                 foreach (Animator componentsInChild in tool.GetComponentsInChildren<Animator>())
                 {
@@ -70,8 +69,8 @@ public Task Process(ClientProcessorContext context, PlayerHeldItemChanged packet
                 }
                 item.SetActive(true);
                 tool.SetHandIKTargetsEnabled(true);
-                SafeAnimator.SetBool(opPlayer.ArmsController.GetComponent<Animator>(), $"holding_{tool.animToolName}", true);
-                opPlayer.AnimationController["using_tool_first"] = packet.IsFirstTime != null;
+                SafeAnimator.SetBool(player.ArmsController.GetComponent<Animator>(), $"holding_{tool.animToolName}", true);
+                player.AnimationController["using_tool_first"] = packet.IsFirstTime != null;
 
                 if (item.TryGetComponent(out FPModel fpModelDraw)) //FPModel needs to be updated
                 {
@@ -97,8 +96,8 @@ public Task Process(ClientProcessorContext context, PlayerHeldItemChanged packet
                 {
                     componentsInChild.cullingMode = AnimatorCullingMode.CullUpdateTransforms;
                 }
-                SafeAnimator.SetBool(opPlayer.ArmsController.GetComponent<Animator>(), $"holding_{tool.animToolName}", false);
-                opPlayer.AnimationController["using_tool_first"] = false;
+                SafeAnimator.SetBool(player.ArmsController.GetComponent<Animator>(), $"holding_{tool.animToolName}", false);
+                player.AnimationController["using_tool_first"] = false;
 
                 if (item.TryGetComponent(out FPModel fpModelHolster)) //FPModel needs to be updated
                 {
@@ -108,7 +107,7 @@ public Task Process(ClientProcessorContext context, PlayerHeldItemChanged packet
                 break;
 
             case PlayerHeldItemChanged.ChangeType.DRAW_AS_ITEM:
-                pickupable.inventoryItem.item.Reparent(opPlayer.ItemAttachPoint);
+                pickupable.inventoryItem.item.Reparent(player.ItemAttachPoint);
                 pickupable.inventoryItem.item.SetVisible(true);
                 Utils.SetLayerRecursively(pickupable.inventoryItem.item.gameObject, viewModelLayer);
                 break;