Apply code style and formatting

Author: RedBigz

TABGVR/Patches/AntiCheatBypass.cs

@@ -1,4 +1,3 @@
-using System.Reflection;
 using Epic.OnlineServices.AntiCheatClient;
 using HarmonyLib;
 using Landfall;
@@ -8,20 +7,23 @@ namespace TABGVR.Patches;
 public class AntiCheatBypass
 {
     /// <summary>
-    /// Bypasses the anticheat.<br/>
-    /// <b>WARNING: THIS KICKS YOU OFF OFFICIAL SERVERS!</b>
+    ///     Bypasses the anticheat.<br />
+    ///     <b>WARNING: THIS KICKS YOU OFF OFFICIAL SERVERS!</b>
     /// </summary>
     internal static void Bypass()
     {
         Harmony.CreateAndPatchAll(typeof(AntiCheatBypass), "com.redbigz.AntiCheatBypass");
-        
-        Plugin.Logger.LogInfo("Just so you know, bypassing the Anti-Cheat cuts you off from official servers. Matchmaking has been disabled for your safety.");
+
+        Plugin.Logger.LogInfo(
+            "Just so you know, bypassing the Anti-Cheat cuts you off from official servers. Matchmaking has been disabled for your safety.");
     }
-    
+
     /// <summary>
-    /// Patches <see cref="ACInterface"/> to disable AC.
+    ///     Patches <see cref="ACInterface" /> to disable AC.
     /// </summary>
-    /// <param name="ACInterface"><see cref="ACInterface"/></param>
+    /// <param name="ACInterface">
+    ///     <see cref="ACInterface" />
+    /// </param>
     [HarmonyPatch(typeof(Easy_AC_Client), nameof(Easy_AC_Client.SetACInterface))]
     [HarmonyPrefix]
     private static bool SetACInterface(AntiCheatClientInterface ACInterface)
@@ -31,13 +33,15 @@ private static bool SetACInterface(AntiCheatClientInterface ACInterface)
     }
 
     /// <summary>
-    /// Patches <see cref="MatchmakingHandler"/> to raise a notification if you try to matchmake, as well as blocking the matchmaking process.
+    ///     Patches <see cref="MatchmakingHandler" /> to raise a notification if you try to matchmake, as well as blocking the
+    ///     matchmaking process.
     /// </summary>
     [HarmonyPatch(typeof(MatchmakingHandler), nameof(MatchmakingHandler.StartSearch))]
     [HarmonyPrefix]
     private static bool StartSearch()
     {
-        GlobalCanvasSingleton.Instance.UIMessageBox.QueueMessage("Matchmaking disabled due to Anti-Cheat safety. Please use a community server when playing TABGVR.");
+        GlobalCanvasSingleton.Instance.UIMessageBox.QueueMessage(
+            "Matchmaking disabled due to Anti-Cheat safety. Please use a community server when playing TABGVR.");
         return false;
     }
 }

TABGVR/Patches/CameraPatch.cs

@@ -13,9 +13,11 @@ namespace TABGVR.Patches;
 public class CameraPatch
 {
     /// <summary>
-    /// Creates VR Camera after <see cref="PlayerCamera"/> starts.
+    ///     Creates VR Camera after <see cref="PlayerCamera" /> starts.
     /// </summary>
-    /// <param name="__instance"><see cref="PlayerCamera"/></param>
+    /// <param name="__instance">
+    ///     <see cref="PlayerCamera" />
+    /// </param>
     [HarmonyPatch(nameof(PlayerCamera.Start))]
     [HarmonyPostfix]
     public static void Start(PlayerCamera __instance)
@@ -29,7 +31,7 @@ public static void Start(PlayerCamera __instance)
             transform =
             {
                 parent = playerManager.playerRoot.transform.Find("CameraMovement"),
-                position = __instance.transform.position,
+                position = __instance.transform.position
             },
             tag = "MainCamera",
             layer = __instance.gameObject.layer

TABGVR/Patches/KinematicsPatch.cs

@@ -8,16 +8,16 @@
 namespace TABGVR.Patches;
 
 [HarmonyPatch(typeof(Holding))]
-class KinematicsPatch
+internal class KinematicsPatch
 {
     internal static bool _gripAvailable;
     internal static bool _gripping;
 
     /// <summary>
-    /// Sets up hand connection for joint by removing properties that cause issues.
+    ///     Sets up hand connection for joint by removing properties that cause issues.
     /// </summary>
     /// <param name="joint">Hand Joint</param>
-    static void SetupConnection(Rigidbody joint)
+    private static void SetupConnection(Rigidbody joint)
     {
         // joint.GetComponentInChildren<Collider>().enabled = false;
 
@@ -38,11 +38,11 @@ static void SetupConnection(Rigidbody joint)
     }
 
     /// <summary>
-    /// Updates <paramref name="joint"/>'s position to <paramref name="controller"/>.
+    ///     Updates <paramref name="joint" />'s position to <paramref name="controller" />.
     /// </summary>
     /// <param name="joint">Hand Joint</param>
-    /// <param name="controller">VR Controller made by <see cref="Controllers"/></param>
-    static void UpdateConnection(Rigidbody joint, GameObject controller)
+    /// <param name="controller">VR Controller made by <see cref="Controllers" /></param>
+    private static void UpdateConnection(Rigidbody joint, GameObject controller)
     {
         var controllerRelativeToGameCamera = controller.transform.position - Controllers.Head.transform.position +
                                              Camera.current.transform.position;
@@ -54,34 +54,35 @@ static void UpdateConnection(Rigidbody joint, GameObject controller)
     }
 
     /// <summary>
-    /// Position left hand to left hand attachment point on gun (handguard) instead of the VR controller to avoid visual weirdness.
+    ///     Position left hand to left hand attachment point on gun (handguard) instead of the VR controller to avoid visual
+    ///     weirdness.
     /// </summary>
     /// <param name="holding">Holding Script</param>
-    static void PositionLeftHandToHandguard(Holding holding)
+    private static void PositionLeftHandToHandguard(Holding holding)
     {
         holding.leftHand.MovePosition(holding.leftHand.position + holding.heldObject.leftHandPos.position -
                                       holding.leftHand.transform.GetChild(0).position);
     }
 
     /// <summary>
-    /// Sets up hands for VR IK after Holding.Start.
+    ///     Sets up hands for VR IK after Holding.Start.
     /// </summary>
     /// <param name="__instance">Holding Script</param>
     [HarmonyPatch(nameof(Holding.Start))]
     [HarmonyPostfix]
-    static void StartPostfix(Holding __instance)
+    private static void StartPostfix(Holding __instance)
     {
         SetupConnection(__instance.rightHand);
         SetupConnection(__instance.leftHand);
     }
 
     /// <summary>
-    /// Runs aiming and positioning for arms and guns after Holding.Update.
+    ///     Runs aiming and positioning for arms and guns after Holding.Update.
     /// </summary>
     /// <param name="__instance">Holding Script</param>
     [HarmonyPatch(nameof(Holding.Update))]
     [HarmonyPostfix]
-    static void UpdatePostfix(Holding __instance)
+    private static void UpdatePostfix(Holding __instance)
     {
         if (!Controllers.LeftHand || !Controllers.RightHand || !Controllers.Head) return;
         if (__instance.player != global::Player.localPlayer) return;
@@ -107,7 +108,10 @@ static void UpdatePostfix(Holding __instance)
             {
                 if (_gripAvailable) _gripping = true;
             }
-            else _gripping = false;
+            else
+            {
+                _gripping = false;
+            }
 
             // look at left controller if gripping, or else just use the right controller rotation
             heldObject.gameObject.transform.rotation = _gripping
@@ -116,8 +120,8 @@ static void UpdatePostfix(Holding __instance)
                 : Controllers.RightHand.transform.rotation *
                   Quaternion.Euler(90f + rightHold.localRotation.x, rightHold.localRotation.y, 0f);
 
-            var toMove = (Controllers.RightHandFromGameCamera +
-                heldObject.gameObject.transform.position - rightHold.position);
+            var toMove = Controllers.RightHandFromGameCamera +
+                heldObject.gameObject.transform.position - rightHold.position;
 
             var rigidBody = heldObject.GetComponent<Rigidbody>();
 
@@ -128,21 +132,27 @@ static void UpdatePostfix(Holding __instance)
 
             heldObject.transform.position = toMove;
         }
-        else _gripping = false;
+        else
+        {
+            _gripping = false;
+        }
 
         if (_gripping) PositionLeftHandToHandguard(__instance);
         else UpdateConnection(__instance.leftHand, Controllers.LeftHand);
     }
 
     /// <summary>
-    /// Cancels Holding.ReachForPoint from running.
+    ///     Cancels Holding.ReachForPoint from running.
     /// </summary>
     [HarmonyPatch(nameof(Holding.ReachForPoint))]
     [HarmonyPrefix]
-    static bool ReachForPointCanceller() => false;
+    private static bool ReachForPointCanceller()
+    {
+        return false;
+    }
 
     /// <summary>
-    /// Method that does nothing.
+    ///     Method that does nothing.
     /// </summary>
     /// <returns>IEnumerator that breaks on start</returns>
     private static IEnumerator DoNothing()
@@ -151,9 +161,11 @@ private static IEnumerator DoNothing()
     }
 
     /// <summary>
-    /// Cancels Holding.HoldWeaponStill from running.
+    ///     Cancels Holding.HoldWeaponStill from running.
     /// </summary>
-    /// <param name="__result"><see cref="DoNothing"/></param>
+    /// <param name="__result">
+    ///     <see cref="DoNothing" />
+    /// </param>
     [HarmonyPatch(nameof(Holding.HoldweaponStill))]
     [HarmonyPrefix]
     private static bool HoldWeaponStillCanceller(ref IEnumerator __result)
@@ -163,9 +175,12 @@ private static bool HoldWeaponStillCanceller(ref IEnumerator __result)
     }
 
     /// <summary>
-    /// Cancels PlayerIKHandler.LateUpdate from running.
+    ///     Cancels PlayerIKHandler.LateUpdate from running.
     /// </summary>
     [HarmonyPatch(typeof(PlayerIKHandler), nameof(PlayerIKHandler.LateUpdate))]
     [HarmonyPrefix]
-    static bool IKCanceller() => false;
+    private static bool IKCanceller()
+    {
+        return false;
+    }
 }

TABGVR/Patches/PlayerPatch.cs

@@ -11,16 +11,16 @@ public class PlayerPatch
     private static Transform _rotationTarget;
     private static PlayerManager _playerManager;
 
-    
+
     /// <summary>
-    /// Adds <see cref="VRControls"/> and <see cref="RotationTargetDriver"/> on local <see cref="Player"/>.
+    ///     Adds <see cref="VRControls" /> and <see cref="RotationTargetDriver" /> on local <see cref="Player" />.
     /// </summary>
     /// <param name="__instance"></param>
     [HarmonyPatch(nameof(global::Player.Start))]
     [HarmonyPostfix]
     public static void Start(global::Player __instance)
     {
-        _playerManager = new(__instance.gameObject);
+        _playerManager = new PlayerManager(__instance.gameObject);
 
         if (!_playerManager.playerIsClient) return; // run the rest if we are the player
 

TABGVR/Player/Controllers.cs

@@ -2,34 +2,33 @@
 using UnityEngine;
 using UnityEngine.SpatialTracking;
 using UnityEngine.XR;
-using UnityEngine.XR.Interaction.Toolkit;
 
 namespace TABGVR.Player;
 
 public static class Controllers
 {
     public static GameObject LeftHand, RightHand, Head;
 
+
+    /// <summary>
+    ///     <see cref="InputDevice" />s of each hand.
+    /// </summary>
+    public static InputDevice LeftHandXR, RightHandXR;
+
     /// <summary>
-    /// Left hand Vector3 in relative space.
+    ///     Left hand Vector3 in relative space.
     /// </summary>
     public static Vector3 LeftHandFromGameCamera =>
         LeftHand.transform.position - Head.transform.position + Camera.current.transform.position;
 
     /// <summary>
-    /// Right hand Vector3 in relative space.
+    ///     Right hand Vector3 in relative space.
     /// </summary>
     public static Vector3 RightHandFromGameCamera =>
         RightHand.transform.position - Head.transform.position + Camera.current.transform.position;
 
-    
-    /// <summary>
-    /// <see cref="InputDevice"/>s of each hand.
-    /// </summary>
-    public static InputDevice LeftHandXR, RightHandXR;
-
     /// <summary>
-    /// Sets up VR Controllers.
+    ///     Sets up VR Controllers.
     /// </summary>
     public static void Setup()
     {

TABGVR/Player/Mundanities/Grenades.cs

@@ -5,7 +5,7 @@ namespace TABGVR.Player.Mundanities;
 public class Grenades
 {
     /// <summary>
-    /// Selected Grenade.
+    ///     Selected Grenade.
     /// </summary>
     [CanBeNull]
     public static Grenade SelectedGrenade =>

TABGVR/Player/PlayerManager.cs

@@ -4,42 +4,46 @@ namespace TABGVR.Player;
 
 public class PlayerManager
 {
-    public GameObject playerRoot;
-    public global::Player player;
+    public static PlayerManager LocalPlayer;
+    public Camera cameraComponent;
 
     public GameObject cameraObject;
-    public Camera cameraComponent;
+    public global::Player player;
 
     public bool playerIsClient;
-    
-    public static PlayerManager LocalPlayer;
+    public GameObject playerRoot;
 
     public PlayerManager(GameObject root)
     {
         playerRoot = root;
-        
+
         player = root.GetComponent<global::Player>();
         playerIsClient = player == global::Player.localPlayer; // i think this works
-        
+
         cameraObject = playerRoot.transform.FindChildRecursive("Main Camera").gameObject;
         cameraComponent = cameraObject.GetComponent<Camera>();
 
         if (playerIsClient) LocalPlayer = this;
     }
 
     /// <summary>
-    /// Creates PlayerManager from game camera.
+    ///     Creates PlayerManager from game camera.
     /// </summary>
-    /// <param name="camera">Player's <see cref="Camera"/></param>
-    /// <returns><see cref="PlayerManager"/></returns>
-    public static PlayerManager FromCamera(Camera camera) => new(camera.transform
-        .parent // Pivot
-        .parent // PositionShake
-        .parent // RotationShake
-        .parent // CameraPhysics
-        .parent // CameraRotationY
-        .parent // CameraRotationX
-        .parent // CameraMovement
-        .parent // Player
-        .gameObject);
+    /// <param name="camera">Player's <see cref="Camera" /></param>
+    /// <returns>
+    ///     <see cref="PlayerManager" />
+    /// </returns>
+    public static PlayerManager FromCamera(Camera camera)
+    {
+        return new PlayerManager(camera.transform
+            .parent // Pivot
+            .parent // PositionShake
+            .parent // RotationShake
+            .parent // CameraPhysics
+            .parent // CameraRotationY
+            .parent // CameraRotationX
+            .parent // CameraMovement
+            .parent // Player
+            .gameObject);
+    }
 }